Add trove classifiers for all supported Python versions

I frequently use the PyPI trove classifiers to check if a 3rd party package is usable by my project. Documenting all supported versions makes this easier.

So codecov received the information: https://codecov.io/gh/shazow/urllib3/commit/bb16fc295121aa757033708157ba3692b06a41a4 It's just not updating it's status here.

Also, I can't exactly merge this manually because of our required status checks.

Yeah, we either need to fix codecov or get @shazow to put the repo in an org so someone can get permissions to remove the requirement

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1202   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files          21      21
  Lines        1975    1975
======================================
  Hits         1975    1975

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 f9b661d...77d76a9. Read the comment docs.

I did a git commit --amend --reset-author and pushed. codecov looks to have gone through now. I'll do the same on my other PR.


pipenv seemingly not respecting conditional install_requires

Issue description
When trying to install ruamel.yaml==0.15.0 using Python 3.6, pipenv tries to build ruamel.ordereddict even though this package is only a dependency for Python 2.7. Interestingly though, pipenv does not attempt to build ruamel.ordereddict when installing ruamel.yaml==0.15.1, which changed the way the dependency is defined:
ruamel.yaml 0.15.0 - __init__.py
install_requires=dict(
    any=[],
    py27=['ruamel.ordereddict'],
),

ruamel.yaml 0.15.1 - __init__.py
extras_require={':platform_python_implementation=="CPython" and python_version<="2.7"': [
    'ruamel.ordereddict',
    ]},

I'm not terribly familiar with python dependencies, but I've never seen them defined as in ruamel.yaml 0.15.0, so I'm thinking that may be the issue.
Expected result
pipenv not to attempt to build ruamel.ordereddict when running Python 3.6.
Actual result
pipenv tries to build ruamel.ordereddict, which is an issue since the ordereddict package is not only not required when running Python 3.6, but it doesn't even build :-(.
Steps to replicate
pipenv --python 3.6
pipenv install ruamel.yaml==0.15.0

or just run pipenv install with pipfile included below.

$ pipenv --support
Pipenv version: '2018.7.1'
Pipenv location: '/usr/local/lib/python3.7/site-packages/pipenv'
Python location: '/usr/local/opt/python/bin/python3.7'
Other Python installations in PATH:

2.7: /usr/local/bin/python2.7

2.7: /usr/local/bin/python2.7

2.7: /usr/bin/python2.7

3.7: /usr/local/bin/python3.7m

3.7: /usr/local/bin/python3.7

2.7.15: /usr/local/bin/python

2.7.10: /usr/bin/python

2.7.15: /usr/local/bin/python2

3.7.0: /usr/local/bin/python3

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.7.0',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '17.6.0',
 'platform_system': 'Darwin',
 'platform_version': 'Darwin Kernel Version 17.6.0: Tue May  8 15:22:16 PDT '
                     '2018; root:xnu-4570.61.1~1/RELEASE_X86_64',
 'python_full_version': '3.7.0',
 'python_version': '3.7',
 'sys_platform': 'darwin'}

System environment variables:

LC_ALL
NVM_DIR
LANG
XPC_FLAGS
HISTIGNORE
HISTCONTROL
HISTTIMEFORMAT
PYENV_ROOT
OLDPWD
TERM_SESSION_ID
EDITOR
COLORTERM
NVM_CD_FLAGS
__CF_USER_TEXT_ENCODING
ITERM_SESSION_ID
PYTHONIOENCODING
SSH_AUTH_SOCK
NODE_REPL_HISTORY_SIZE
USER
LSCOLORS
PWD
MANPAGER
HOME
TERM_PROGRAM
TERM_PROGRAM_VERSION
Apple_PubSub_Socket_Render
ITERM_PROFILE
LESS_TERMCAP_md
LESS_TERMCAP_me
TMPDIR
XPC_SERVICE_NAME
TERM
SHELL
NVM_BIN
SHLVL
COLORFGBG
PROMPT_COMMAND
LOGNAME
PATH
PS1
PS2
PS4
HISTSIZE
HISTFILESIZE
_
PYTHONDONTWRITEBYTECODE
PIP_PYTHON_PATH

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: /Users/newtonne/.nvm/versions/node/v8.11.3/bin:/Users/newtonne/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin
SHELL: /usr/local/bin/bash
EDITOR: vim
LANG: en_US
PWD: /Users/newtonne/Downloads/test

Contents of Pipfile ('/Users/newtonne/Downloads/test/Pipfile'):
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]
"ruamel.yaml" = "==0.15.0"

[dev-packages]

[requires]
python_version = "3.6"

Contents of Pipfile.lock ('/Users/newtonne/Downloads/test/Pipfile.lock'):
{
    "_meta": {
        "hash": {
            "sha256": "e2776e1e940389b95536ebe856f3f35f4ded3371aaba60d62ad69dfe2ffbfb4e"
        },
        "pipfile-spec": 6,
        "requires": {
            "python_version": "3.6"
        },
        "sources": [
            {
                "name": "pypi",
                "url": "https://pypi.org/simple",
                "verify_ssl": true
            }
        ]
    },
    "default": {
        "ruamel.ordereddict": {
            "hashes": [
                "sha256:08b4b19fe518d32251a5338e039c4dc9eb0876f2919f94c9b8d2f9446ea80806",
                "sha256:150ce8e6c514a2a2b62753622a75874962561f8e5eeec81a3172ab952807bf0b",
                "sha256:45541836cbfdde630033cae7bbbe35acbac87a0ceec79f944b7a3bedd940fe78",
                "sha256:aee2fa23e884249b4284b728888c553d551e5bfd4de2731f10153fd7813ec55f",
                "sha256:bf0a198c8ce5d973c24e5dba12d3abc254996788ca6ad8448eabc6aa710db149"
            ],
            "version": "==0.4.13"
        },
        "ruamel.yaml": {
            "hashes": [
                "sha256:16520c1da7a02b02df711791dfd72e9a6709f842ecab0ffcb5f0781586fad665",
                "sha256:20d239c8e61905debb85979acb0773b261651fdb2422a796e61d25ba9852ad03",
                "sha256:95371af26123c5c3a2f0a121444cf5197d3e6dac83ba76a2351499c328694471",
                "sha256:ab2dd4a86816d14027bb11ad2e41a2e59114bc48c9f9f27994b3598ccb00683e",
                "sha256:b03ac00d603e30f7a030eb2ac69787313051f6fa0c84ca09ea461fb5f67b749b",
                "sha256:b0bcd8d11bd06fc041a72e95f7bb0769483f20b3fdbefc252a3643d0b74e0bd3",
                "sha256:f9f4a50422acf4a6ab55b568ae879b3ceb62e8e62a60022cb88816c17e40782e"
            ],
            "index": "pypi",
            "version": "==0.15.0"
        }
    },
    "develop": {}
}

Hi, sorry for the inconvenience. Unfortunately Pipenv only supports the latest environment marker spec, and requires the library to specify conditions in install_requires, not extras_require. There are several reasons behind this decision, but ultimately the install_requires syntax is the most recommended approach, and libraries are strongly encouraged to adapt it as soon as possible.
Please help push ruamel.yaml to revert to the 0.15.0 syntax instead.
Closing as a duplicate to… many others. #1229 is one.

Isn't https://github.com/pypa/pipenv/issues/1229 the opposite of this issue?
What I described above is that ruamel.yaml 0.15.0, the version that uses install_requires, does not respect the python2.7 condition. If pipenv requires the library to specify conditions in install_requires, then shouldn't 0.15.0 be the version that builds correctly, and not 0.15.1?

Err, sorry, I misunderstood what ruamel.yaml did in 0.15.0. Let me try to get this straight. So there are mainly three ways packages do to specify conditional dependencies:

With code, e.g. if version < (2, 7): install_requires.append(...)
With extras_require
With install_requires and environment markers

Pipenv only supports the last option. ruamel.yaml 0.15.0 was using option 1 (I thought it was using 3, hence the confusion), and after 0.15.1 it is using option 2, which is not supported. For Pipenv to work correctly, it needs to upgrade to use option 3.
So no, it should revert back to the old way. But it should also need to upgrade to the new, recommended way.

That makes more sense. Thanks.

Thanks, this was pretty helpful! I ran into this same issue while trying to migrate to pipenv. Since I'm locked into an older version of ruamel.yaml, my workaround was to use PEP 508 specifiers from the pipenv docs to mirror the py27 requirement in the Pipfile.
Pipfile snippet:
[packages]
"ruamel.yaml" = "==0.15.0"
"ruamel.ordereddict" = { version = "==0.4.13", markers = "python_version <= '2.7'" }

[requires]
python_version = "3.6"

pipenv install output:
(test) bash-3.2$ pipenv install
Pipfile.lock not found, creating…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
✔ Success!
Updated Pipfile.lock (d2113f)!
Installing dependencies from Pipfile.lock (d2113f)…
Ignoring ruamel.ordereddict: markers 'python_version <= "2.7"' don't match your environment


Support for defining location of validation for each schema param.

Is all fine? Should I do something more to finalise this PR?

You have tests that check for using in within the schema for check(). I would also have a test that checks if in works within a specific check like checkBody.

Ok, doc fixed, tests added. I added simple tests to check if other check() method works in a specific location. Should be enough :)

Can you please merge in latest master?

Here you go :)


"max_age" QS param for "/{tenant}/oauth2/v2.0/authorize" is not documented

The /{tenant}/oauth2/v2.0/authorize endpoint supports a max_age query-string parameter that appears to control whether or not SSO can be applied to a particular authorization request. In our testing, if we set this to 300 -- for example -- then the user does not have to provide a password and MFA code if the last time they authenticated with Azure was within five minutes ago. This seems to strike a nice balance between prompt=none and prompt=login for sites that wish to re-authenticate users when accessing confidential information.
Interestingly, if we set max_age to 0 it does not behave the same as if we pass prompt=login -- the user can still choose to use a remembered username and only have to enter a password. This is usually the approach we desire for our systems because it cuts down on extra typing when we really only want to validate that the user who was there before is still the same user.
We learned of the undocumented QS param from this third-party blog post:
https://hajekj.net/2017/03/06/forcing-reauthentication-with-azure-ad/

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

ID: 628db02d-345a-5dc4-17f0-3d6ff76d7fe5
Version Independent ID: 524b460d-47b7-72c9-e994-ea94b3a9dd2b
Content: OpenID Connect protocol - Microsoft identity platform
Content Source: articles/active-directory/develop/v2-protocols-oidc.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@GuyPaddock Thank you for the feedback. We will review and update accordingly.

@rwike77, Can you please take a look at this.

Another question here -- why does specifying a max_age cause the "Don't ask again for 7 days" option to become greyed out if an MFA token is required during the re-auth?

Any updates @souravmishra-msft or @rwike77? We have gotten a few questions about the MFA side effect here and I don't have docs to fallback on.

@rwike77 can you assign this to me? We don't officially support this claim (by virtue of lack of docs) so I'll need to collect design and support statements internally before we doc it and support it.

#reassign:@hpsin  Thanks

@GuyPaddock @hpsin Moving this stale issue to Azure DevOps work item backlog for review in recurring sprint triage. We'll move on it as appropriate and resolve when possible.

ID
Title
Iteration Path
Assigned To

1308686
[ISSUE][51307] "max_age" QS param for "/{tenant}/oauth2/v2.0/authorize" is not documented
Technical Content\Backlog
Hirsch Singhal

#please-close


Return promises form api

Return promises form api instead of objects representing not yet resolved results.

Task linked: SALEOR-1308 Non-React SDK does not offer a way to wait until results are ready


use structs for BitCircuit

Instead of the more opaque array encoding, try to make the BitCircuits data structure more readable using structs which are now supported by CUDA.jl: https://cuda.juliagpu.org/dev/tutorials/custom_structs/

Done in v0.4


More updates for CloudEvents

Signed-off-by: Doug Davis<EMAIL_ADDRESS>Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you added your SVG to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[x] ~30 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

LGTM
but I can't seem to get it to show our OpenSSF status properly

@AndreyKozlov1984  any idea on the OpenSFF question/

To be clear, I'm wondering about the OpenSSF badge here:

where it says "No OpenSSF Best Practices". I do have:
url_for_bestpractices: https://bestpractices.coreinfrastructure.org/en/projects/6770

in our metadata


[Validator Badge] Masternode24.de 0x95aE59515915D6c493a846aE022F93726…

Hello. I got 83 out of 120 points in the audit and I'm definitely entitled to a Master Validator Badge. Expressed in%, this is 69,2% of 100%.

Hello. I got 83 out of 120 points in the audit and that entitles me to the Master Validator badge. Expressed in%, this is 69,2% of 100%, and if you count that back on points, you get 83 points. Please inform yourself first before closing this PR!

Hey @alex-m24 -- you're correct and my apologies for mixing up the numbers!


X,Y,Z seems to be changing when only rotation angle is changed

Depends on what point is the rotation is happening about. Even if its a little off than the object center, its going to be off


Please add two endpoints with an ability to calculate Plutus script execution units

Input: Plutus Script
Output: CBOR representation of compiled Plutus Script + metadata
Input: Id of Plutus Script
Output: CBOR representation of compiled Plutus Script + metadata
It would be highly appreciated if a developer could pass only id of script and script could be looked up by blockfrost in cardano-db. This will make it easier for clients because they will not need to know the whole body of Plutus Script.

It would be highly appreciated if a developer could pass only id of script and script could be looked up by blockfrost in cardano-db. This will make it easier for clients because they will not need to know the whole body of Plutus Script.

In order to calculate ex units properly, the whole transaction context needs to be taken into consideration as well. So you need the script, datums, redeemers and the context of the tx, like inputs, ouputs, minting etc.

It would be highly appreciated if a developer could pass only id of script and script could be looked up by blockfrost in cardano-db. This will make it easier for clients because they will not need to know the whole body of Plutus Script.

In order to calculate ex units properly, the whole transaction context needs to be taken into consideration as well. So you need the script, datums, redeemers and the context of the tx, like inputs, ouputs, minting etc.

@satya004 where you aware of it?

That's right @matiwinnetou .
Probably,
Endpoint-1 : To compile a plutus script to cbor format.
Endpoint -2 :
Input > Either raw txn in cbor or txn in Json format.
Output > Exec Unit in json

I just wanted to comment to say that I also support this idea, hopefully it gets implemented!

@mmahut Cardano Fans could sponsor this work if amount in ADA is reasonable. Please contact us on<EMAIL_ADDRESS>Since we are developers ourselves we do understand that infrastructure doesn't magically appear out of nowhere, that devs need to be paid.

Sounds good to me, there are few complications:

Compiling Plutus scripts depends on the version of the Plutus compiler and the accompanying toolchain - different versions will produce different Plutus Core and the resulting CBOR, which means the script address will change as well. I think that the recommended approach is using Nix and pinned dependencies and updating the toolchain only when the script and its address is allowed to change (and accounted for with the rest of the application).
Plutus script API changes as well so if we would update the toolchain behind the endpoint it could cause the old scripts to fail compilation.

Script evaluation needs all of the script inputs, at least a Datum and Redeemer but I think @alessandrokonrad is right and the raw transaction is needed since the third script parameter is a ScriptContext containing additional transaction data.

It would be highly appreciated if a developer could pass only id of script and script could be looked up by blockfrost in cardano-db. This will make it easier for clients because they will not need to know the whole body of Plutus Script.

This is already possible for scripts executed at least once using scripts/{script_hash}/cbor - https://docs.blockfrost.io/#tag/Cardano-Scripts/paths/~1scripts~1{script_hash}~1cbor/get

I think this is something that could also be worked out right? I see this is more complicated so will be harder to maintain but it is soo important. Perhaps we could have multiple versions supported, a path param /v1/ /v2, etc which binds to specific Plutus version?

Sounds good to me, there are few complications:

Compiling Plutus scripts depends on the version of the Plutus compiler and the accompanying toolchain - different versions will produce different Plutus Core and the resulting CBOR, which means the script address will change as well. I think that the recommended approach is using Nix and pinned dependencies and updating the toolchain only when the script and its address is allowed to change (and accounted for with the rest of the application).
Plutus script API changes as well so if we would update the toolchain behind the endpoint it could cause the old scripts to fail compilation.
Script evaluation needs all of the script inputs, at least a Datum and Redeemer but I think @alessandrokonrad is right and the raw transaction is needed since the third script parameter is a ScriptContext containing additional transaction data.

It would be highly appreciated if a developer could pass only id of script and script could be looked up by blockfrost in cardano-db. This will make it easier for clients because they will not need to know the whole body of Plutus Script.

This is already possible for scripts executed at least once using scripts/{script_hash}/cbor - https://docs.blockfrost.io/#tag/Cardano-Scripts/paths/~1scripts~1{script_hash}~1cbor/get

Entry point to ExUnits's eval function:
https://github.com/input-output-hk/plutus/blob/master/plutus-ledger-api/src/Plutus/V1/Ledger/Api.hs#L238

I think this is something that could also be worked out right? I see this is more complicated so will be harder to maintain but it is soo important. Perhaps we could have multiple versions supported, a path param /v1/ /v2, etc which binds to specific Plutus version?

Yes, this is doable - we could follow plutus-starter-devcontainer releases

Entry point to ExUnits's eval function: https://github.com/input-output-hk/plutus/blob/master/plutus-ledger-api/src/Plutus/V1/Ledger/Api.hs#L238

Note the [PLC.Data] argument of that function which requires supplying Datum, Redeemer and ScriptContext. The first two are not a problem but reconstructing ScriptContext is not trivial as it contains info about pending transaction. That said I need to test how the function behaves when you e.g. only supply Datum but the validator works with Redeemer as well - my guess is the evaluation in that case won't produce a correct value (i.e. this could work with partial inputs depending on what data the actual validator works with).

The first two are not a problem but reconstructing ScriptContext is not trivial as it contains info about pending transaction

ScriptContext is constructed from the tx you build locally, so it does not depend on the blockchain state, however the data structure can be defined arbitrarily and it's not obvious how it looks like or it's kinda hard to extract it from the tx.
But I checked the cardano-ledger source code and found how it's done on the ledger:
https://github.com/input-output-hk/cardano-ledger/blob/5527a510306c46c51dae7106eb23d4a0c8009063/eras/alonzo/impl/src/Cardano/Ledger/Alonzo/PlutusScriptApi.hs#L148
With the function collectTwoPhaseScriptInputs  we can extract all relevant data directly from a tx. The data are returned in an array, because there could be multiple scripts.
Then we run evaluateScriptCounting  for all data in the array and get the ex units for each redeemer.
The evaluateScriptCounting function is here: https://github.com/input-output-hk/plutus/blob/master/plutus-ledger-api/src/Plutus/V1/Ledger/Api.hs#L238

Is there any update on this? I think @alessandrokonrad and @sorki have some working solution now on Blockfrost or?

@matiwinnetou we have a possible solution, but sadly still in our backlog.

This has proven little bit challenging, but after some time, the /utils/tx/evalute endpoint is out!


markdown with mathjax

Can you help me in making these two work together? Here is what I am trying to do -
{{#mathjax}}{{#markdown}}{{some mark-down text mixed with latex formula}}{{/markdown}}{{/mathjax}}
I am using shwdown markdown for markdown template.

@kaushik1979 @kaushik1979
The recommended way to use markdown along with mathjax is by a transform option, e.g.
var converter = new Showdown.converter();
var helper = new MeteorMathJax.Helper({
  transform : function (x) {
    return converter.makeHtml(x);
  },
});
Template.registerHelper('mathjax', helper.getTemplate());

Now you can skip the nested markdown helper.


Update dependencies

Github complains that the Django we use has a security vulnerability.

Closing in favour of https://github.com/hoover/snoop2/pull/153


Pure python 3Di implementation

mini3Di is a pure python 3Di implementation (GPL). That would avoid the awkward Google drive distribution of Foldseek binaries. mini3Di was made by a community member and not by us.
Alternatively, you can download Foldseek binaries from either the latest commit or from previous commits through:
mmseqs.com/archive/COMMIT_HASH (e.g. https://mmseqs.com/archive/fc40ba1afdcb20602116907fdc73a47fd4c78615/).

Hi, thank you for the recommendations!
To ensure users could run our examples without unexpected errors we provide a fixed version of Foldseek binary file. Users can also manually install Foldseek on their own and run the examples.


Help needed: Not able to remove tags from remote

@ajoberstar
With the following code iam able to remove the tag form local but not from remote. kindly help me out.
grgit.tag.remove(names: [tagName])

task removeTag <<{
    def tagName = getVersion()
    println 'removing tag '+tagName
    grgit.tag.remove(names: [tagName])
    grgit.push(tags: true)
}

I think you have to do this similar to how you might on command line Git:
CLI Git:
git push origin :tagName

Grgit:
grgit.push(refsOrSpecs: [':tagName'])

I tried it, and still its the same...
classpath 'org.ajoberstar:gradle-git:1.4.0'
task removeTag <<{
    def tagName = getVersion()
    println 'removing tag '+tagName
    grgit.tag.remove(names: [tagName])
    grgit.push(refsOrSpecs: [':tagName'])
}
Here follows the info:
Tasks to be executed: [task ':removeTag']
:removeTag (Thread[main,5,main]) started.
:removeTag
Executing task ':removeTag' (up-to-date check took 0.001 secs) due to:
  Task has not declared any outputs.
Starting process 'command 'git''. Working directory: /home/murali/GITHUB/gradle-gitpush-upload-release Command: git describe --tags
Successfully started process 'command 'git''
removing tag 9.9.2
The following authentication options are allowed (though they may not be available): [PAGEANT,` `HARDCODED,` `SSHAGENT, `INTERACTIVE]`
`using hardcoded credentials` `from system properties`
`:removeTag `(Thread[main,5,main])` `completed. Took 3.324 secs.`

What do you mean by the draft?

I mean the release draft, where we added the release notes an upload the war to it.

OK, that would be Github specific. Right now, I don't have anything integrating with Github.


Add difficulty knob.

Allows setting difficulty level for the server via an environment variable.

Oh yeah, that's a great knob to have. Thanks.
Do you mind also adding a little snippet in README?

There you go. :)

Sorry, I hadn't noticed before -- how about a default of normal or easy? It looks like the stock default is easy, which I think is the same for the client.

Sorry for the delay on merging. It's built and pushed to Docker hub -- image ID 2bca89e353c5.


Shared-memory with containers

When running zenohd, a ROS 2 publisher and a ROS 2 echo in a container, shared memory is seemingly used, with little traffic on the loopback interface.
However, with the ROS 2 echo subscriber in a different container, I see hundreds of megabytes of traffic on the loopback interface, implying shared memory is not being used. This is despite the containers being configured with --ipc=host to share the host's IPC namespace, and /dev/shm/ being bind-mounted into both.
Is there some heuristic that determines if both processes are running on the same physical host in order to trigger shared memory usage that's being confused by the face they're in containers? There was a similar issue with FastDDS where they used the network interface names to determine if the processes were in the same host, which was the case with --net=host, but that triggered shared-memory to be used, which wouldn't of course work without --ipc=host, a less common option to have enabled.
I have shared-memory configured in both the router and session JSON configs, and am using unprivileged rootless Podman containers. Is there a way to explicit logs as to what type of IPC is being used and why?

SHM is disabled by default in rmw_zenoh due to some known issues See https://github.com/ros2/rmw_zenoh/issues/199
Regarding your observations, I'd try to reproduce the same results with the zenoh-c examples and open a ticket upstream if you see the same behavior.

Issue #199 says it's disabled by default in session config, however I manually enabled it there as well as the router config. It seems to work correctly when in the same container, it's just the inter-container use-case that it wouldn't work.

Is there a way to get more verbose information on the IPC methods chosen by rmw_zenoh in runtime?
Do you know what by what heuristic shared-memory is used or not? Does it attempt and fall back to the network stack?
Does it make sense to pursue this now, or wait until https://github.com/eclipse-zenoh/zenoh-c/pull/405 gets merged into rmw_zenoh?

Regarding your observations, I'd try to reproduce the same results with the zenoh-c examples and open a ticket upstream if you see the same behavior.

The zenoh-c examples (z_pub_shm.c & z_sub.c) seem to work fine intra and inter-container, with shared memory segments opened by them in /dev/shm/. However I can't rule out that they might be using the loopback interface, any advice in ensuring that's not the case? Is there a way to ensure they use shared-memory exclusively? Because running the example in a separate container with no --ipc=host flag set also works, so I assume that must have fallen back to the loopback interface?

The issue was that the /tmp/ directory wasn't shared, and rmw_zenoh uses that to coordinate the shared-memory negotiation on the same host, bind mounting it into both containers fixed the first issue and allowed shared-memory communication between the nodes in different containers.
However, when the subscriber shuts down, even gracefully, the publisher crashes with:
what():  failed to publish message: Failed to allocate a SHM buffer, even after GCing, at /opt/ros/akara_ws/src/rmw_zenoh/rmw_zenoh_cpp/src/rmw_zenoh.cpp:949, at ./src/rcl/publisher.c:284

@ciandonovan , thanks for publishing your findings.  Haven't migrated to zenoh yet, but this'll be something to watch for.


Add libs to gem and fix permissions

currently the libs are not included in gem
there was also a problem with permissions after gem was installed, investigate further

not an issue when using along with github


[Event Request] Table 25 - Vendor Ledger Entry - RecalculateAmounts

Could you please create an Integration event in Table 25 Vendor Ledger Entry, in function RecalculateAmounts ?
[External] RecalculateAmounts(FromCurrencyCode : Code[10];ToCurrencyCode : Code[10];PostingDate : Date)
IF ToCurrencyCode = FromCurrencyCode THEN
  EXIT;

"Remaining Amount" :=
  CurrExchRate.ExchangeAmount("Remaining Amount",FromCurrencyCode,ToCurrencyCode,PostingDate);
"Remaining Pmt. Disc. Possible" :=
  CurrExchRate.ExchangeAmount("Remaining Pmt. Disc. Possible",FromCurrencyCode,ToCurrencyCode,PostingDate);
"Accepted Payment Tolerance" :=
  CurrExchRate.ExchangeAmount("Accepted Payment Tolerance",FromCurrencyCode,ToCurrencyCode,PostingDate);
"Amount to Apply" :=
  CurrExchRate.ExchangeAmount("Amount to Apply",FromCurrencyCode,ToCurrencyCode,PostingDate);

// >>>>
OnAfterRecalculateAmounts(Rec, FromCurrencyCode, ToCurrencyCode, PostingDate);
// <<<<

Event signature:
LOCAL [IntegrationEvent] OnAfterRecalculateAmounts(VAR Rec : Record "Vendor Ledger Entry";FromCurrencyCode : Code[10];ToCurrencyCode : Code[10];PostingDate : Date)

For our extension, we need to recalculate our own amount fields.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Compilation error in errors.rs

I get these errors when trying to compile using the latest nightly rustc:
src\errors.rs:30:62: 30:83 error: the trait `core::fmt::Display` is not implemented for the type `S` [E0277]
src\errors.rs:30                                Some(name) => format!("'{}'", Format::Warning(name)),
                                                                              ^~~~~~~~~~~~~~~~~~~~~
<std macros>:2:26: 2:57 note: in this expansion of format_args!
src\errors.rs:30:46: 30:84 note: in this expansion of format! (defined in <std macros>)
<std macros>:2:26: 2:57 note: in this expansion of format_args!
src\errors.rs:24:20: 34:51 note: in this expansion of format! (defined in <std macros>)
src\errors.rs:30:62: 30:83 help: run `rustc --explain E0277` to see a detailed explanation
src\errors.rs:30:62: 30:83 note: `S` cannot be formatted with the default formatter; try using `:?` instead if you are using a format string
src\errors.rs:30:62: 30:83 note: required by `core::fmt::Display::fmt`
src\errors.rs:101:28: 101:48 error: the trait `core::fmt::Display` is not implemented for the type `S` [E0277]
src\errors.rs:101                            Format::Warning(arg),
                                             ^~~~~~~~~~~~~~~~~~~~
<std macros>:2:26: 2:57 note: in this expansion of format_args!
src\errors.rs:97:20: 108:51 note: in this expansion of format! (defined in <std macros>)
src\errors.rs:101:28: 101:48 help: run `rustc --explain E0277` to see a detailed explanation
src\errors.rs:101:28: 101:48 note: `S` cannot be formatted with the default formatter; try using `:?` instead if you are using a format string
src\errors.rs:101:28: 101:48 note: required by `core::fmt::Display::fmt`
src\errors.rs:161:28: 161:49 error: the trait `core::fmt::Display` is not implemented for the type `S` [E0277]
src\errors.rs:161                            Format::Warning(name),
                                             ^~~~~~~~~~~~~~~~~~~~~
<std macros>:2:26: 2:57 note: in this expansion of format_args!
src\errors.rs:157:20: 163:51 note: in this expansion of format! (defined in <std macros>)
src\errors.rs:161:28: 161:49 help: run `rustc --explain E0277` to see a detailed explanation
src\errors.rs:161:28: 161:49 note: `S` cannot be formatted with the default formatter; try using `:?` instead if you are using a format string
src\errors.rs:161:28: 161:49 note: required by `core::fmt::Display::fmt`
src\errors.rs:194:28: 194:48 error: the trait `core::fmt::Display` is not implemented for the type `S` [E0277]
src\errors.rs:194                            Format::Warning(val),
                                             ^~~~~~~~~~~~~~~~~~~~
<std macros>:2:26: 2:57 note: in this expansion of format_args!
src\errors.rs:189:20: 197:51 note: in this expansion of format! (defined in <std macros>)
src\errors.rs:194:28: 194:48 help: run `rustc --explain E0277` to see a detailed explanation
src\errors.rs:194:28: 194:48 note: `S` cannot be formatted with the default formatter; try using `:?` instead if you are using a format string
src\errors.rs:194:28: 194:48 note: required by `core::fmt::Display::fmt`
src\errors.rs:270:28: 270:48 error: the trait `core::fmt::Display` is not implemented for the type `S` [E0277]
src\errors.rs:270                            Format::Warning(arg),
                                             ^~~~~~~~~~~~~~~~~~~~
<std macros>:2:26: 2:57 note: in this expansion of format_args!
src\errors.rs:266:20: 273:51 note: in this expansion of format! (defined in <std macros>)
src\errors.rs:270:28: 270:48 help: run `rustc --explain E0277` to see a detailed explanation
src\errors.rs:270:28: 270:48 note: `S` cannot be formatted with the default formatter; try using `:?` instead if you are using a format string
src\errors.rs:270:28: 270:48 note: required by `core::fmt::Display::fmt`
src\errors.rs:288:28: 288:48 error: the trait `core::fmt::Display` is not implemented for the type `S` [E0277]
src\errors.rs:288                            Format::Warning(arg),
                                             ^~~~~~~~~~~~~~~~~~~~
<std macros>:2:26: 2:57 note: in this expansion of format_args!
src\errors.rs:283:20: 290:51 note: in this expansion of format! (defined in <std macros>)
src\errors.rs:288:28: 288:48 help: run `rustc --explain E0277` to see a detailed explanation
src\errors.rs:288:28: 288:48 note: `S` cannot be formatted with the default formatter; try using `:?` instead if you are using a format string
src\errors.rs:288:28: 288:48 note: required by `core::fmt::Display::fmt`
error: aborting due to 6 previous errors

rustc version: rustc 1.6.0-nightly (bdfb13518 2015-11-14)
Platform: Windows 10 x64

Thanks for reporting this! I'm updating nightly rust as we speak and will look into it. I'm assuming you're not getting the errors on beta or stable, correct? I'm not primarily a Windows guy, which is why I'm asking.

I just confirmed it and think I know what the fix is - once it's patched I'll update crates.io to 1.5.2 :+1:

Thanks for your quick response! FYI, the same error happens on stable (1.4.0)

#346 fixes this, once it merges I'll put out the new version on crates.io

@ennis this is fixed, and 1.5.2 is out on crates.io. Let us know if you have any other issues :smile:


[Spike] investigate what we need to update in response to CVE-2023-44487 HTTP2 rapid reset

investigate what we need to update in response to CVE-2023-44487 HTTP2 rapid reset
potentially includes updating base images and updating libraries that contain http servers
Acceptance criteria:

creation of new card(s) that list out what we need to fix

Snyk has some suggestions:
https://app.snyk.io/org/red-hat-openshift-data-science-rhods/reporting?context[page]=issues-detail&issue_status=Open&issue_by=Severity&table_issues_detail_cols=SCORE%257CCVE%257CCWE%257CPROJECT%257CEXPLOIT%2520MATURITY%257CAUTO%2520FIXABLE%257CINTRODUCED%257CSNYK%2520PRODUCT&table_issues_detail_sort=%2520CVE%2520DESC&cve=%257B%2522CVE%2522%253A%255B%2522CVE-2023-44487%2522%255D%257D


Shortcuts.ShowErrorDialog -> Prompt.Error

Compiled mod for testing: TMPE.zip

Replaced 4 calls to Shortcuts.ShowErrorDialog() with calls to Prompt.Error().

Does same thing but with some extra safeguards if used in-game

Removed Shortcuts.ShowErrorDialog() method as no longer used.
Update Prompt methods so they queue the dialog action in editor as well

Sorry that was my bad. didn't see the prompt thing.


turn off typeAhead feature flag

remove the non-type ahead variable dropdown from the variable bar
a) don't remove it from the code base, it is still used in the tooltip section of the query builder (the variable subtab)

b) keep the tests in, too (the unit test)

2). remove the flag from the tests
a) fix remaining old-style non-typeahead tests to typeahead

remove from idpe after it is in production
remove from configcat after removal from idpe

removed in the code (ui)
removed in idpe
removed in oss (influxdb)
and yesterday, removed from configcat.
so all done.


is there a way to detect if the page is already scrolling ?

for example lets say the user scrolled a page twice so fast before the scroll has finished "like using a keyboard" atm the package will scroll the first time then a second time which produce a flicker.
so how to avoid that ?
using a smaller duration fix it to some level, but the flicker is still there 😢

Can you create a reproduction jsfiddle? The library should abort scrolling (not force animation if the user interrupts)...

that would be hard, but i can show u a gif ?
or maybe i can get you the timing between each click, which i think could be used as a debounce value internally ?

using console.log(new Date().getMilliseconds())

first: 138
second: 320
diff: 182

setting duration: 182
fix the flickering.

I can't reproduce any flickering, if I click on an element that triggers a scroll, and then tap any key on my keyboard, the scrolling is stopped. That's the intended behavior.
The events that will abort scrolling are defined here: https://github.com/rigor789/vue-scrollto/blob/master/src/scrollTo.js#L5-L12

am using keys to navigate from one item to another, not normal scroll then canceling with key event.

Can you show me a video / gif of the issue? I still don't know what the problem is

sure https://drive.google.com/open?id=0B1q9LR33Y0lNdDQ3M1dhQ1RabDg

I can see it flickering now, but not sure what is actually happening.
The page that opens is listening for keyboard events, and scrolls to different items in the list?
If it's listening for keydown, then you will want to throttle / debounce it on your end...

nop, nothing is hijacking the keyboard event except this list.
but for the debounce, is there a way other than using lodash _debounce ?

A quick search lead me to this: https://gist.github.com/nmsdvid/8807205

thanx, but honestly not sure how exactly to use it, maybe
this.debounce(() => {
    return this.$scrollTo(item)
}, 180)

if so , it doesnt work.

If you paste the relevant code from your component (the method where you are calling this.$scrollTo, and where it gets called), I can help. Usually you need to wrap the function with debounce and then call it (it returns a new function)

here is the full comp
<script>
export default {
    name: 'rev-comp',
    data() {
        return {
            selected: null,
            list : []
        }
    },
    methods: {
        debounce(func, wait, immediate) {
            let timeout

            return () => {
                let context = this,
                    args = arguments

                clearTimeout(timeout)

                timeout = setTimeout(() => {
                    timeout = null
                    if (!immediate) func.apply(context, args)
                }, wait)

                if (immediate && !timeout) {
                    func.apply(context, args)
                }
            }
        },
        navigation() {
            // key nav
        },
        toggleRev(item = null) {
            if (!item) {
                this.list = []
                this.selected = null
                $('html').unbind()
                return EventHub.fire('rev-hide')
            }

            EventHub.fire('rev-show')
            this.updateRev(item)
            this.$nextTick(() => {
                this.goTo(`#${item}`)
                this.navigation()
            })
        },
        goTo(item) {
            return this.debounce(() => {
                this.$scrollTo(item)
            }, 180)
        },
        updateRev(item) {
            this.selected = item
        }
    },
    render () {}
}
</script>

I've altered your code, and added comments, this hopefully should set you on the right track!
This has not been tested, so keep that in mind.
<script>
// normally this would go into it's own utility file, since we might reuse it later on
function debounce(func, wait, immediate) {
    let timeout

    return () => {
        let context = this,
            args = arguments

        clearTimeout(timeout)

        timeout = setTimeout(() => {
            timeout = null
            if (!immediate) func.apply(context, args)
        }, wait)

        if (immediate && !timeout) {
            func.apply(context, args)
        }
    }
}

export default {
    name: 'rev-comp',
    data() {
        return {
            selected: null,
            list : []
        }
    },
    methods: {
        navigation() {
            // key nav
        },
        toggleRev(item = null) {
            if (!item) {
                this.list = []
                this.selected = null
                $('html').unbind()
                return EventHub.fire('rev-hide')
            }

            EventHub.fire('rev-show')
            this.updateRev(item)
            this.$nextTick(() => {
                this.goTo(`#${item}`)
                this.navigation()
            })
        },
        // The idea is to assign a debounced version of the method
        // this has not been tested, so it might have binding issues (arrow function vs regular function)
        // and I'm not sure if an argument can be passed to a debounced function without testing this
        // if the debounced func can't accept arguments, your best bet is to store it in data / in the vm
        goTo: debounce((item) => {
            return this.debounce(() => {
                this.$scrollTo(item)
            }, 180)
        }, 200),
        updateRev(item) {
            this.selected = item
        }
    },
    render () {}
}
</script>

np, thanx for ur effort :1st_place_medal:


Jakarta JSON Upgrades for Jakarta EE 10

https://issues.redhat.com/browse/WFLY-16263
https://issues.redhat.com/browse/WFLY-16264
https://issues.redhat.com/browse/WFLY-16200
https://issues.redhat.com/browse/WFLY-16262

Upgrade Jakarta JSON Processing to 2.1.0 and migrate to using Eclipse Parsson. Also upgrade Jakarta JSON Binding to 3.0.0 and Yasson to 3.0.0-RC1.

Thanks @jamezp


Typescript error when running project

I did a quick test run of the project and immediately ran into a roadblock with these errors (I'm on TS version 3.1.6, MacOS). I'm new to Typescript so appreciate any and all help 😀
Error dump below:
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:4030:14 - error TS2687: All declarations of 'fullscreen' must have identical modifiers.
4030     readonly fullscreen: boolean;
~~~~~~~~~~
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:5152:14 - error TS2687: All declarations of 'angularAcceleration' must have identical modifiers.
5152     readonly angularAcceleration: Float32Array | null;
~~~~~~~~~~~~~~~~~~~
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:5153:14 - error TS2687: All declarations of 'angularVelocity' must have identical modifiers.
5153     readonly angularVelocity: Float32Array | null;
~~~~~~~~~~~~~~~
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:5154:14 - error TS2687: All declarations of 'hasOrientation' must have identical modifiers.
5154     readonly hasOrientation: boolean;
~~~~~~~~~~~~~~
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:5155:14 - error TS2687: All declarations of 'hasPosition' must have identical modifiers.
5155     readonly hasPosition: boolean;
~~~~~~~~~~~
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:5156:14 - error TS2687: All declarations of 'linearAcceleration' must have identical modifiers.
5156     readonly linearAcceleration: Float32Array | null;
~~~~~~~~~~~~~~~~~~
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:5157:14 - error TS2687: All declarations of 'linearVelocity' must have identical modifiers.
5157     readonly linearVelocity: Float32Array | null;
~~~~~~~~~~~~~~
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:5158:14 - error TS2687: All declarations of 'orientation' must have identical modifiers.
5158     readonly orientation: Float32Array | null;
~~~~~~~~~~~
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:5159:14 - error TS2687: All declarations of 'position' must have identical modifiers.
5159     readonly position: Float32Array | null;
~~~~~~~~
../../../.npm-packages/lib/node_modules/typescript/lib/lib.dom.d.ts:13881:14 - error TS2687: All declarations of 'orientation' must have identical modifiers.
13881     readonly orientation: ScreenOrientation;
~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:82:5 - error TS2687: All declarations of 'fullscreen' must have identical modifiers.
82     fullscreen: boolean;
~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:122:5 - error TS2717: Subsequent property declarations must have the same type.  Property 'getVRDisplays' must be of type '() => Promise<VRDisplay[]>', but here has type '() => any'.
122     getVRDisplays: () => any;
~~~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:137:5 - error TS2687: All declarations of 'orientation' must have identical modifiers.
137     orientation: string;
~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:137:5 - error TS2717: Subsequent property declarations must have the same type.  Property 'orientation' must be of type 'ScreenOrientation', but here has type 'string'.
137     orientation: string;
~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:166:5 - error TS2687: All declarations of 'hasOrientation' must have identical modifiers.
166     hasOrientation: boolean;
~~~~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:167:5 - error TS2687: All declarations of 'hasPosition' must have identical modifiers.
167     hasPosition: boolean;
~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:168:5 - error TS2687: All declarations of 'position' must have identical modifiers.
168     position?: Float32Array;
~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:169:5 - error TS2687: All declarations of 'linearVelocity' must have identical modifiers.
169     linearVelocity?: Float32Array;
~~~~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:170:5 - error TS2687: All declarations of 'linearAcceleration' must have identical modifiers.
170     linearAcceleration?: Float32Array;
~~~~~~~~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:171:5 - error TS2687: All declarations of 'orientation' must have identical modifiers.
171     orientation?: Float32Array;
~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:172:5 - error TS2687: All declarations of 'angularVelocity' must have identical modifiers.
172     angularVelocity?: Float32Array;
~~~~~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:173:5 - error TS2687: All declarations of 'angularAcceleration' must have identical modifiers.
173     angularAcceleration?: Float32Array;
~~~~~~~~~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:17013:108 - error TS2304: Cannot find name 'MSBaseReader'.
17013         static ReadFileAsDataURL(fileToLoad: Blob, callback: (data: any) => void, progressCallback: (this: MSBaseReader, ev: ProgressEvent) => any): void;
~~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:17014:100 - error TS2304: Cannot find name 'MSBaseReader'.
17014         static ReadFile(fileToLoad: File, callback: (data: any) => void, progressCallBack?: (this: MSBaseReader, ev: ProgressEvent) => any, useArrayBuffer?: boolean): void;
~~~~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:18024:13 - error TS2403: Subsequent variable declarations must have the same type.  Variable 'VRDisplay' must be of type '{ new (): VRDisplay; prototype: VRDisplay; }', but here has type 'any'.
18024 declare var VRDisplay: any;
~~~~~~~~~
node_modules/babylonjs/dist/preview release/babylon.d.ts:18025:13 - error TS2403: Subsequent variable declarations must have the same type.  Variable 'VRFrameData' must be of type '{ new (): VRFrameData; prototype: VRFrameData; }', but here has type 'any'.
18025 declare var VRFrameData: any;`

@vjk3 I separately run npm run start in client, and server folder, then it works for me on develop mode.

I updated all the packages and fixed the errors. The project should be up and running again. Sorry for the half year delay ; )


Entered text can get overwritten while typing when wrapped

Windows Terminal version
1.21.1772.0
Windows build number
10.0.19045.0
Other Software
oh-my-posh
Steps to reproduce

Type some text until it gets wrapped (resize for a quicker repro)
Or past a long command and start editing it in the middle

Expected Behavior
Text should re-wrapped while editing
Actual Behavior
Sometimes the caret stays on the previously entered text, causing overwrite.
Note: apparently this doesn't affect the committed text when hitting enter so it's different from what is rendered.

Are you using an IME to input text/?

Thanks for following up! Since it's an issue with Git bash we'll close this. Thanks!


Add fields to filter out templates appearing in the add new item dialog for non UAP projects.

ViewModel template should still be available in Windows Desktop projects, because it's for creating a Windows Runtime class and it's not specifically for UWP XAML (although it seems to include UWP XAML headers).

This does not seem to work properly. I still see C++/WinRT XAML templates in a WinUI 3 project.


Update UI.

Coverage increased (+0.005%) to 16.014% when pulling 0401edcbd70ff3a81e92a78bb275f07a95e3faab on ui-update into 31cb96d99c4f89c741640a085627de6b7e7c98b2 on master.


119 react tests

Increased test coverage to 23%.
Further increase requires solving npm test deadlock on react-router / apollo provider tests

@vilkazz Do you need a review from someone on these tests?

Unit test in enzyme does not render child components, while integration test runs for the whole component tree down.
So even if i am testing the same thing with integration and unit test the result is not necessarily the same. The integration test was the one that actually caught a few leftover bugs in date and checkbox components.
I do admit i was too lazy to test the entire clicking flow of the checkbox, but as Material-ui has close to 100% test coverage, I skipped checking that for the checkbox component.
Re snapshots ... whoooops... nice catch..


Log analytics workspaces was not found

Is there an existing issue for this?

[X] I have searched the existing issues

Greenfield/Brownfield provisioning
greenfield
Terraform Version
1.4.6
Module Version
7.2.0
AzureRM Provider Version
3.62.1
Affected Resource(s)/Data Source(s)
azurerm_log_analytics_workspace
Terraform Configuration Files
terraform {
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = ">=3.0.0"
    }
  }
}

variable "subscription_id_1" {}
variable "subscription_id_2" {}

provider "azurerm" {
  features {}

  subscription_id = var.subscription_id_1
}

provider "azurerm" {
  alias = "log"

  features {}

  subscription_id = var.subscription_id_2
}

resource "azurerm_resource_group" "log" {
  name     = "log-347347347"
  location = "West Europe"

  provider = azurerm.log
}

resource "azurerm_log_analytics_workspace" "log" {
  name                = "log-856785685468"
  location            = azurerm_resource_group.log.location
  resource_group_name = azurerm_resource_group.log.name

  provider = azurerm.log
}

resource "azurerm_resource_group" "aks" {
  name     = "aks-34574357457"
  location = "West Europe"
}

module "aks" {
  source                          = "Azure/aks/azurerm"
  version                         = "7.2.0"
  resource_group_name             = azurerm_resource_group.aks.name
  prefix                          = "425867245986"
  rbac_aad                        = false
  log_analytics_workspace_enabled = true
  log_analytics_workspace = {
    id   = azurerm_log_analytics_workspace.log.id
    name = azurerm_log_analytics_workspace.log.name
  }

  depends_on = [azurerm_resource_group.aks]
}

tfvars variables values
subscription_id_1 = ""
subscription_id_2 = ""

Debug Output/Panic Output
╷
│ Error: log analytics workspaces "log-856785685468" (Resource Group "log-347347347") was not found
│
│   with module.aks.data.azurerm_log_analytics_workspace.main[0],
│   on .terraform/modules/aks/main.tf line 725, in data "azurerm_log_analytics_workspace" "main":
│  725: data "azurerm_log_analytics_workspace" "main" {
│
╵

Expected Behaviour
Provisioning aks cluster by providing an existing log_analytics_workspace from another subscription should succeed.
Actual Behaviour
Provisioning aks cluster fails.
Steps to Reproduce
No response
Important Factoids
No response
References
No response

Hi @dkronhardt-juris thanks for opening this pr, unfortunately this module is designed for single subscription use for now, so in the module's code we didn't support multiple subscriptions.
I'll keep this issue open and try to figure out a solution if I can, but I can't promise a solution. Apology for your inconvenience.

@lonegunmanb but it was working in version 6.7.0 of the module. Now we cannot upgrade the module anymore.

I see, apology for your inconvenience @dkronhardt-juris, I'll see what I can to to mitigate this issue for you.


Subscribe Section

Replace font awesome icon with subscribe-icon.svg.
Fix input and button styling

I would like to do it please assign me the task


Spelling

I'm not sure what your requirements are for PRs, I can rearrange these/fold as requested...

Hi @jsoref, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Can you update the PR with latest merge from master, to resolve conflicts so that I can go ahead to merge

@mvvsubbu : done...

@mvvsubbu - what is going on with this PR?  it now has conflicts again.  take or close

@bryanmacfarlane, @mvvsubbu : updated...

Hi @jsoref, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


move to "Require a pull request before merging"

I plan to enable "Require a pull request before merging" o this repo similar to what we have elsewhere.
This enables collaboration and feedback with all builds passing as doing it on direct commits is just not possible.
For now I don't add required checks / approvals. Just a pull  request so its more visible.

@linux-china , works for you ?

Yes, please use it as we seem to break tests/builds often and with this we avoid getting that into main.
I’ve enabled required pullrequest now for main with the only check being that builds passes. If there are issues let me know and we can adjust as needed.


Update ExampleStack.ts

The use of VITE_APP_AUTH0_DOMAIN in the frontend assumes that https:// is not already a part of the VITE_APP_AUTH0_DOMAIN. But the backend expects it, otherwise aws gives the error ""Invalid issuer: Issuer is not a valid URL for JWT Authorizer"
getAccessTokenSilently({
        audience: `https://${import.meta.env.VITE_APP_AUTH0_DOMAIN}/api/v2/`,
        scope: "read:current_user",
      });)

Ah thanks. These files are generated. Can you make the change here instead and submit a new PR? https://github.com/sst/sst/blob/master/packages/create-sst/bin/presets/examples/api-auth-jwt-auth0/templates/stacks/ExampleStack.ts


⚠ Bofrost.lu has degraded performance

In 67c0ea3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1561 ms

Resolved: Bofrost.lu performance has improved in d345420 after 6 minutes.


Add Detail for Matching Context

re-enable matching contexts with an optional detail editor.

please be aware that we currently have not implemented this within the detail (because details are only available for is and seeks graphs, the context however is a "toplevel" element of the need


All maas plugins broken in Liberty after upgrade from Kilo

What version (commit) do you encounter this issue on?
r12.1.1
What type of issue is this (bug, feature, documentation, regression)?
Bug
What is the expected behavior?
It Works
What is actually happening?
All maas plugins broken in Liberty after upgrading from Kilo
/usr/lib/rackspace-monitoring-agent/plugins# ./hp_monitoring.py
Traceback (most recent call last):
File "./hp_monitoring.py", line 17, in
import maas_common
File "/usr/lib/rackspace-monitoring-agent/plugins/maas_common.py", line 51, in
from cinderclient import client as c_client
File "/usr/local/lib/python2.7/dist-packages/cinderclient/client.py", line 46, in
from eventlet import sleep
File "/usr/local/lib/python2.7/dist-packages/eventlet/init.py", line 10, in
from eventlet import convenience
File "/usr/local/lib/python2.7/dist-packages/eventlet/convenience.py", line 3, in
from eventlet import greenio
File "/usr/local/lib/python2.7/dist-packages/eventlet/greenio/init.py", line 3, in
from eventlet.greenio.base import *  # noqa
File "/usr/local/lib/python2.7/dist-packages/eventlet/greenio/base.py", line 440, in
from OpenSSL import SSL
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/init.py", line 8, in
from OpenSSL import rand, crypto, SSL
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/rand.py", line 11, in
from OpenSSL._util import (
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/_util.py", line 6, in
from cryptography.hazmat.bindings.openssl.binding import Binding
File "/usr/local/lib/python2.7/dist-packages/cryptography/hazmat/bindings/openssl/binding.py", line 68, in
error=-1)
File "/usr/local/lib/python2.7/dist-packages/cryptography/hazmat/bindings/openssl/binding.py", line 57, in wrapper
ffi.def_extern(name=name, **kwargs)(func)
AttributeError: 'CompiledFFI' object has no attribute 'def_extern'
How can we reproduce this issue?
N/A
How can we verify your issue is addressed?
N/A
If this is a large issue, what can be done immediately to alleviate the issue while working on a longer-term solution?
Prio 1
What is your evaluation on the importance of this issue, and why?
Customer impacting
What branches / milestones do you believe this issue should target?
The list below is what is currently supported for backports
Remove branches (lines) you do not wish to target.

[ ] Master
[ ] liberty-12.2
[ ] liberty-12.1
[ ] kilo

So I fix this temporarily with
pip install --force-reinstall nova
The issue is that none of the MAAS plugin stuff runs inside a VENS and nova was still 2015.1.4 while the liberty version is now 12.x, so a normal pip install would never upgrade the package due to the "downgrade of versioning"
So all maas python plugins have to implement the OSA venvs

@BjoernT
How can we reproduce this issue? N/A
How can we verify your issue is addressed? N/A
If this is a large issue, what can be done immediately to alleviate the issue while working on a longer-term solution? Prio 1
This is not very helpful - in order to help fix these issues please include al of this information. Understanding the environment in which you saw this as well as how to reproduce it makes it much easier for us to address.

@mancdaz I can only answer what I can and that includes how to reproduce it. Just upgrade a Kilo to Liberty and go from there. That's all I know.

@BjoernT that's my point - nowhere in your bug report did you mention that this is encountered on an upgrade. That's crucial information that makes a difference.

Yes that is fine, just let me know if you need info and I usually provide it.

/me is currently working through an upgrade

Looks like reinstalling cryptography fixed it in my aio test.
On first glance it did not even update/downgrade packages doing that:
Installing collected packages: setuptools, enum34, ipaddress, pyasn1, six, idna, pycparser, cffi, cryptography
Found existing installation: setuptools 21.0.0
Uninstalling setuptools-21.0.0:
Successfully uninstalled setuptools-21.0.0
Found existing installation: enum34 1.1.2
Uninstalling enum34-1.1.2:
Successfully uninstalled enum34-1.1.2
Found existing installation: ipaddress 1.0.16
Uninstalling ipaddress-1.0.16:
Successfully uninstalled ipaddress-1.0.16
Found existing installation: pyasn1 0.1.9
Uninstalling pyasn1-0.1.9:
Successfully uninstalled pyasn1-0.1.9
Found existing installation: six 1.10.0
Uninstalling six-1.10.0:
Successfully uninstalled six-1.10.0
Found existing installation: idna 2.0
Uninstalling idna-2.0:
Successfully uninstalled idna-2.0
Found existing installation: pycparser 2.14
Uninstalling pycparser-2.14:
Successfully uninstalled pycparser-2.14
Found existing installation: cffi 1.3.1
Uninstalling cffi-1.3.1:
Successfully uninstalled cffi-1.3.1
Found existing installation: cryptography 1.2.2
Uninstalling cryptography-1.2.2:
Successfully uninstalled cryptography-1.2.2
Successfully installed cffi-1.3.1 cryptography-1.1.2 enum34-1.1.1 idna-2.0 ipaddress-1.0.15 pyasn1-0.1.9 pycparser-2.14 setuptools-21.0.0 six-1.10.0

So I just did an upgrade to r12.1.2 and couldn't reproduce this. Granted it was an AIO and not a multinode. But every plugin I tried worked perfectly fine after running setup-maas.yml to ugprade the maas bits.

@sigmavirus24  Did you try the upgrade with  echo 'pip_install_options: "--force-reinstall"' >> /etc/openstack_deploy/user_deleteme_post_upgrade_variables.yml ?

@BjoernT I ran the upgrade with that today and again couldn't reproduce it when trying to upgrade to r12.1.2.

Wait, I may have spoken too soon. 💃

Yep, I did speak too soon. I can reproduce this and I have a fix that's hilariously simple I think

@BjoernT check out #1102

Can we make it part of a pip package (rackspace-monitoring or similar) so that pip installs will trigger it rather than the playbooks.

Uh, I don't think so. The monitoring agent is a lua package that we put on our mirrors for people to use. I guess we could put work into making the plugins into a pip package that has those dependencies, but I'm not sure how we like that. #1102 is at least a stop-gap solution for this particular problem.
I'm also fairly certain that we manually put the plugin scripts into a very particular location on disk so that the monitoring agent will find them. It would be a lot of shenanigans to get them into the right place through python packaging (although probably possible).

needs backports


Adding endpoint for upgrade-risks-prediction

Description
Adding a new endpoint for the upgrade-risks-prediction feature.
Fixes #CCXDEV-9834
Type of change

New feature (non-breaking change which adds functionality)
Unit tests (no changes in the code)
Configuration update

Testing steps
Please describe how the change was tested locally. If, for some reason, the testing was not done or not done fully, please describe what are the testing steps.
Checklist

[x] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Base: 60.50% // Head: 60.91% // Increases project coverage by +0.40% :tada:

Coverage data is based on head (cb29523) compared to base (31ca995).
Patch coverage: 79.66% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1007      +/-   ##
==========================================
+ Coverage   60.50%   60.91%   +0.40%
==========================================
  Files          23       24       +1
  Lines        3226     3285      +59
==========================================
+ Hits         1952     2001      +49
- Misses       1053     1058       +5
- Partials      221      226       +5

Impacted Files
Coverage Δ

server/upgrade_risks_prediction.go
78.18% <78.18%> (ø)

amsclient/amsclient.go
74.45% <100.00%> (+0.57%)
:arrow_up:

server/endpoints_v2.go
100.00% <100.00%> (ø)

server/errors.go
64.10% <0.00%> (+5.12%)
:arrow_up:

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

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


PubSubPublisher : High CPU usage

We use PubSub in a fairly conventional way :
                                                Database X
                                              /
                                             /
Pubsub (Topic A) --> Worker (Subscription A)
                                             \
                                              \
                                                Database Y

Our infrastructure is on google cloud (and we use k8s). For a steady ~3000 messages / second, 6 pods with the following config are enough and work at around 80% CPU
  "resources": {
    "requests": {
      "cpu": "500m",
      "memory": "2Gi"
    },
    "limits": {
      "cpu": "1500m",
      "memory": "3Gi"
    }
  }

We would like to use the message in different ways and move towards something like :
                                                 Database X
                                              /
                                             /
Pubsub (Topic A) --> Worker (Subscription A) -- Database Y
                                             \
                                              \
                                                Pubsub (Topic B)

The only modification we made to the code is creating a new PubSubPublisher and publishing messages to the topic. Every message from Topic A / Subscription A ends up in Topic B so we are also effectively publishing ~3000 messages / second.
After adding that operation, CPU usage for the pods went through the roof. What 6 pods were able to handle @ 80% CPU could not be handled by 20 pods @ 100% CPU with the config noted above (we rolled the code back as the workers weren't able to keep up).
The PubSubPublisher is created with very standard settings (I believe the retry settings are the default ones) :
private void startPublisher() {
RetrySettings retrySettings = RetrySettings.newBuilder()
    .setTotalTimeout(Duration.ofSeconds(10))
    .setInitialRetryDelay(Duration.ofMillis(5))
    .setRetryDelayMultiplier(2.0)
    .setMaxRetryDelay(Duration.ofMillis(Long.MAX_VALUE))
    .setInitialRpcTimeout(Duration.ofSeconds(10))
    .setRpcTimeoutMultiplier(2)
    .setMaxRpcTimeout(Duration.ofSeconds(10))
    .build();

publisher = Publisher.newBuilder(topic)
    .setRetrySettings(retrySettings)
    .setCredentialsProvider(FixedCredentialsProvider.create(
            ServiceAccountCredentials.fromStream(new FileInputStream(Configurations.getGoogleCloudCredentials())))
    )
    .build();
}

The actual publish method is :
private void publish(byte[] message) {
    try {
        PubsubMessage pubsubMessage = PubsubMessage.newBuilder().setData(ByteString.copyFrom(message)).build();
        ApiFuture<String> future = publisher.publish(pubsubMessage);

        ApiFutures.addCallback(future, new ApiFutureCallback<String>() {
            @Override
            public void onFailure(Throwable throwable) {
                if (throwable instanceof ApiException) {
                    ApiException apiException = ((ApiException) throwable);
                    logger.debug(new LogEntry().setMessage(String.format("PubSubException : ApiException. Code %s. IsRetryable ? %s",
                            apiException.getStatusCode().getCode(),
                            apiException.isRetryable())));
                }
                logger.warn(new LogEntry().setMessage("Error publishing message : " + Helper.getStackTrace(throwable)));
            }

            @Override
            public void onSuccess(String messageId) {

            }
        });
    }
    catch(Exception ex) {
        logger.error(new LogEntry().setMessage("An error occured publishing a message to pubsub : " + Helper.getStackTrace(ex)));
    }
}

Converting our objects to byte arrays is simple serialization :
org.apache.commons.lang3.SerializationUtils.serialize(message);

Is such a high CPU usage to be expected ? Should queueing into Pubsub need that much processing power ? What are we doing wrong for our publishers to consume so much ?
We've seen this with different versions. We got these results with 0.39.0-beta on a box running centos 7. We use gRPC 1.10.0 in the project.

Huge thanks to @pongad, the solution offered in #3149 worked for us : updating google-auth-library-oauth2-http to >= 0.9.1.
About the numbers I gave, after updating the auth lib, what  20 pods @ 100% could not handle is handled perfectly well by 7 of those pods that hover around 100%.

@hmigneron I'm glad that worked for you. Are more actions required to resolve this issue?

No, that should be it !  Thanks again


"Table output unavailable" for command "az network public-ip list -o table" on Mac OS.

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
Command az network public-ip list -o table returns an error message as indicated below (tested on both Mac OS Big Sur or Monterey). Command works as expected on Linux systems (same Azure CLI version, 2.30.0, tested on different Linux versions).
Errors:
Table output unavailable. Use the --query option to specify an appropriate query. Use --debug for more info.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az network public-ip list -o table

Expected Behavior
Display a table-formatted list of public IP addresses available in the subscription.
Environment Summary
macOS-11.6.1-x86_64-i386-64bit, Darwin 20.6.0
Python 3.10.0
Installer: HOMEBREW

azure-cli 2.30.0

Additional Context

Table output unavailable

It looks like any command with table output fails.
E.g: az account list -o table.
See also the diagnostics here on #17744 17744

Indeed, I have just confirmed that “-o table” isn’t accepted as a command-line option for any command on Mac OS.
Panos Tsapralis
Head of Networks
(M)<PHONE_NUMBER>3
(direct)<PHONE_NUMBER>05
4 Imperial Place, Maxwell Road, Borehamwood, Hertfordshire, WD6 1JN
<EMAIL_ADDRESS>[signature_1980114370]   [signature_218181445]     @.***
From: Juarez Rudsatz @.>
Reply to: Azure/azure-cli @.>
Date: Friday, 19 November 2021 at 15:36
To: Azure/azure-cli @.>
Cc: Panos Tsapralis @.>, Author @.***>
Subject: Re: [Azure/azure-cli] "Table output unavailable" for command "az network public-ip list -o table" on Mac OS. (Issue #20440)
It looks like any command with table output fails.
E.g: az account list -o table.
See also the diagnostics herehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F17744%23issuecomment-974163422&data=04|01|panos.tsapralis%40buyagift.com|6e2dc6e6227c405779e908d9ab724862|1691bca2aa30424db58761d9d74d6c21|0|0|637729329628752120|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=umDoUDYZyxDBP6Ujxkt0NqBwNl%2FgvdcGW0vGj8iuWM4%3D&reserved=0 on #17744https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F17744&data=04|01|panos.tsapralis%40buyagift.com|6e2dc6e6227c405779e908d9ab724862|1691bca2aa30424db58761d9d74d6c21|0|0|637729329628762114|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=5bL20nWR69zJSph5ck4E4CalZtAXYCVz3dYBajkyx30%3D&reserved=0 17744
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F20440%23issuecomment-974174857&data=04|01|panos.tsapralis%40buyagift.com|6e2dc6e6227c405779e908d9ab724862|1691bca2aa30424db58761d9d74d6c21|0|0|637729329628762114|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=8CssDWhUEYUKqvUsnUg4K4QQu6k%2BLt7MsBL5ReWP2FI%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADLRXZOC3ZBT3YNQJLQQR7DUMZVFZANCNFSM5IL45DKA&data=04|01|panos.tsapralis%40buyagift.com|6e2dc6e6227c405779e908d9ab724862|1691bca2aa30424db58761d9d74d6c21|0|0|637729329628772105|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=%2Fq7CT8B6uJoCgpNl9HP6Ehriz5CCjDacrYFrZIrsBN8%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|panos.tsapralis%40buyagift.com|6e2dc6e6227c405779e908d9ab724862|1691bca2aa30424db58761d9d74d6c21|0|0|637729329628782110|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=oTeU4Dwf%2BQwFMrV%2By7mGg5vlIiGv97c8Bk4GMZs8hFE%3D&reserved=0 or Androidhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|panos.tsapralis%40buyagift.com|6e2dc6e6227c405779e908d9ab724862|1691bca2aa30424db58761d9d74d6c21|0|0|637729329628782110|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=l4iIQOHoUAT3y5VrN4Az6UFuu3%2FiXrAa5u1maGuApS4%3D&reserved=0.

The issue seems to be related to python 3.10
A workaround until the problem is fixed was for me installing the Azure CLI with pip:
pip install az-cli
After that the azure cli was using Python 3.9 again. And table as output format worked again.
az network vnet  list -o table
Name     ResourceGroup    Location    NumSubnets    Prefixes     DnsServers    DDOSProtection
-------  ---------------  ----------  ------------  -----------  ------------  ----------------
vnetde1  AZ-104           westeurope  1             <IP_ADDRESS>/16                False
vnetus1  AZ-104           centralus   1             <IP_ADDRESS>/16                False

Duplicate of #20348

@ralfkrause, the pip command should be
pip install azure-cli

(not az-cli)

@jiasli That didn't work for me at the first try. But installing it with  pip install az-cli which installs https://github.com/MarkWarneke/Az.Cli worked immediately.
When I tried pip install azure-cli the first time, it was still using python 3.10 which resulted in the problem.
Maybe there was still some information kept which pointed to Python 3.10.
I removed uninstalled now the azure cli from homebrew with brew uninstall azure-cli and uninstalled also az which I installed previously pip uninstall az.
I installed it now with pip install azure-cli. This time it was using Python 3.9

az.cli (https://pypi.org/project/az.cli/) is not azure-cli at all. It is a Python interface to call Azure CLI. It depends on azure-cli directly, without any version specification: https://github.com/MarkWarneke/Az.Cli/blob/dev/requirements.txt. So I don't think it will solve the problem. Something should be missing.
az (https://pypi.org/project/az/): I don't know what this is.

I installed it now with pip install azure-cli. This time it was using Python 3.9

Yes. This should be the correct solution for now.

@jiasli I know that az.cli isn't azure cli and just a python library for Azure CLI.
I also don't understand why installing az.cli corrected which Python version Azure CLI is using. That is completely out of my understanding.

@ralfkrause, I don't understand either. Anyway, you may use azure-cli with Python 3.9 for now.

Wait, looks like homebrew hasn't take Python 3.9.9 yet (https://github.com/Azure/azure-cli/issues/20348#issuecomment-973803211). We can only wait for Homebrew to take Python 3.9.9 and install with pip.

Hi.
Thanks for the hint on un-installing / re-installing "az-cli" component. Not that simple in my case, though, as I had to remove “sfctl” 9Service Fabric Control command-line utility) because of versions’ conflicts. Anyway, I now have Azure CLI working properly on the command line on the Mac - hopefully, there will soon be a fix to re-align versions / dependencies etc.
Regards,
Panos Tsapralis
Head of Networks
(M)<PHONE_NUMBER>3
(direct)<PHONE_NUMBER>05
4 Imperial Place, Maxwell Road, Borehamwood, Hertfordshire, WD6 1JN
<EMAIL_ADDRESS>@.***   @.*** @.***
On 22 Nov 2021, at 09:58, Jiashuo Li<EMAIL_ADDRESS>wrote:
Wait, looks like homebrew hasn't take Python 3.9.9 yet (#20348 (comment)https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F20348%23issuecomment-973803211&data=04|01|panos.tsapralis%40buyagift.com|f45d1173200641e5805a08d9ad9e9b75|1691bca2aa30424db58761d9d74d6c21|0|0|637731718997601240|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Bdw%2FLjOnuTHtzQmxmmHfsto4PZrzOGobQUqIl366GzA%3D&reserved=0). We can only wait for Homebrew to take Python 3.9.9 and install with pip.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F20440%23issuecomment-975349084&data=04|01|panos.tsapralis%40buyagift.com|f45d1173200641e5805a08d9ad9e9b75|1691bca2aa30424db58761d9d74d6c21|0|0|637731718997611233|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=dUiBxCOht7lFKFp4ULLBRzagxQRsDH46j8OTnZApDpk%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADLRXZKKTU3IUGNNX2YWMELUNIH3RANCNFSM5IL45DKA&data=04|01|panos.tsapralis%40buyagift.com|f45d1173200641e5805a08d9ad9e9b75|1691bca2aa30424db58761d9d74d6c21|0|0|637731718997621228|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=rwvW8%2FtrIEFC9bCFzIKBHo%2B326HfVgraqCc26LN6GIk%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|panos.tsapralis%40buyagift.com|f45d1173200641e5805a08d9ad9e9b75|1691bca2aa30424db58761d9d74d6c21|0|0|637731718997621228|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=FA9L4pSkQC%2FGa8of6y3npl2ZCT1HcqEzcgFmhslGNqY%3D&reserved=0 or Androidhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|panos.tsapralis%40buyagift.com|f45d1173200641e5805a08d9ad9e9b75|1691bca2aa30424db58761d9d74d6c21|0|0|637731718997631218|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=tV1z9EIf5OeaQrr067rOJvpuOYHvqt4B1R4PepY4bMM%3D&reserved=0.


utterly-nord-plasma: init at 2.1

Description of changes
Add Utterly Nord Plasma Global Theme
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2852

Result of nixpkgs-review pr 263786 run on x86_64-linux 1

1 package built:

utterly-nord-plasma


[MINOR-UPDATE]: Set Brotli codec jar and test to occur only on Linux amd64.

[MINOR-UPDATE]: Set Brotli codec jar and test to occur only on Linux amd64.
Description
Because it is not fully cross platform, com.rdblue.brotli-codec is moved to test scope on Linux amd64 only.  TestParquetWriter#testTPCHReadWriteBrotli is set to execute on Linux amd64 only.
Documentation
None.
Testing
Verify that the distribution archive still does not include brotli-codec, which breaks startup on unsupported platforms.
Obtain clean CI runs on amd64 and aarch64 with TestParquetWriter#testTPCHReadWriteBrotli included only on amd64.

This PR has been merged into #2471.


Request transformation -- cannot have both Remove and Add for same body field

Summary
POST an Add rule when a Remove rule already exists: Conflict message from OpenRest
Steps To Reproduce

Create a request transformation / remove rule on an API:

curl -X POST --url ADMINURL/apis/THEAPI --data 'name=request-transformer' --data 'config.remove.body=key'

OK

Try to create a request transformation / add rule on the same API:

curl -X POST --url ADMINURL/apis/THEAPI --data 'name=request-transformer' --data 'config.add.body=key:NEWVALUE'

Get this error message:

<title>409 Conflict</title>

409 Conflict

openresty/<IP_ADDRESS>

The documentation seems to say multiple rules are OK -- see https://getkong.org/plugins/request-transformer/:

Order of execution

Plugin performs the response transformation in following order

remove --> replace --> add --> append

That suggests you can have multiple rules -- why else worry about the order?
(The doco says "response" but the page is about requests -- a typo)
Additional Details & Logs

Kong version ($ kong version) -- 0.9.1
Operating System -- Ubuntu

I think you can't do it in two requests, you have to do it in one go:
curl -X POST --url ADMINURL/apis/THEAPI --data 'name=request-transformer' --data 'config.remove.body=key' --data 'config.add.body=key:NEWVALUE'

Does that work?
The request-transformer plugin can AFAIK only exist once on the API (that's the 409 you get), and has to configured as a whole.

Thanks for reporting!

Confirmed


swipedirection to left without stack not working

Is it possible to swipe left in normal mode without stack.
swipeDirection="left" is not working.

same issue

Have you found any solution


body with GET/HEAD

What should happen if the user supplies a body with a GET/HEAD request?  I think it should be an option. See https://github.com/sindresorhus/got/issues/1074 https://github.com/sindresorhus/got/issues/966

Hm, that was quite a contentious. I guess we better allow it to avoid a repeat of that. Close? @mcollina?

I mean it can be disabled by default, but there should be an option to enable this.

let’s allow it


Export more types

I wish to do something like this:
import type { SitemapUrlInput } from "@nuxtjs/sitemap/dist/runtime/types";

export default defineCachedEventHandler(getSitemapEntries, {
  // ...
});

async function getSitemapEntries(): Promise<SitemapUrlInput[]> {
  // ...
}

But on the import line, I get TS2307: Cannot find module @nuxtjs/sitemap/dist/runtime/types or its corresponding type declarations.. It seems that I can only import types from @nuxtjs/sitemap directly, which does not export the types I need.
Would it be possible to export more types?

Workaround for now:
import type { SitemapUrlInput } from "~/node_modules/@nuxtjs/sitemap/dist/runtime/types";

Another workaround:
type SitemapEntryInput = Exclude<Exclude<ModuleOptions['urls'], Promise<any> | CallableFunction | undefined>[number], string>;

Good idea, I've introduced an alias #sitemap which points to the runtime types.
You should be able to do this in 5.1.1.
import type { SitemapUrlInput } from "#sitemap";


loose-matching is suddenly required since version 1.6.0 (not documented in README)

this is not compatible with previous releases and breaks all our PR checks
would appreciate if we can in the meanwhile refer v1 tag back to v1.5.0 🙏

We're looking into it as we speak. Bear with us for a few hours, we'll come up with a solution soon

Solved in #506


Make skizze-cli a real program; Support address as argument

looks great :D
Merge once travis has passed

Cheers!


Media quality control

🚀 Feature Request: Media Quality Control
Description
Provide a way for the user to change the quality of the video to a different resolution. It should read all the available qualities from the manifest (HLS, DASH, or MSS). It should also provide an Auto option if available.
Acceptance criteria

[x] The settings menu should have the option to change the quality
[x] Qualities available should be fetched from the manifest
[x] Qualities list should have an Auto option if available
[x] Should be default on Auto

Blocked by

[x] https://github.com/profusion/android-enhanced-video-player/issues/60

https://demo-streaming.gcdn.co/videos/676_YJHUNNocCsrjiCDx/master.m3u8

Deppends on #60

S05: 05
S06: 01


Custom named lua symbols .so librlua instead of liblua

I'm trying to build this on my target system but liblua.so has been renamed with the package I'm using. Instead I need to link librlua.so for the base lua symbols.
with a makefile this is pretty straightforward. I just add a -lrlua and make sure it's in my -L path. How the heck do I do it with luke?

Why are you linking against liblua at all?  Missing symbols will be resolved when the lua binary loads the posix module using the symbols it already has loaded from liblua.
Here's a sample module linked by travis before passing all the self tests:
cc -O2 -fPIC -shared   -D_XOPEN_SOURCE=700 -DVERSION='"git-1"' -DNDEBUG -D_FORTIFY_SOURCE=2 -D_POSIX_C_SOURCE=200809L -DPACKAGE='"luaposix"' -Iext/include -I/home/travis/build/luaposix/luaposix/here/include ext/posix/sys/stat.c -o linux/posix/sys/stat.so


Support several database connections

From @ThomasJaspers on January 27, 2012 12:53
It would be good to have the possibility to have operate the tests on more
than one database connection in the same test.
There should be keywords to open additional connections and to switch
between the available connections.
Copied from original issue: ThomasJaspers/robotframework-dblibrary#3

From @tisto on July 4, 2014 9:57
+1


v1.12 Backports 2023-08-28

[x] #27644 (@brb)
[x] #27391 (@brlbil)

:warning: merge conflicts due to missing workflow files in v1.12 branch:

.github/workflows/build-images-docs-builder.yaml
.github/workflows/ci-images-cache-cleaner.yaml
.github/workflows/ci-images-garbage-collect.yaml
.github/workflows/close-stale-issues.yaml
.github/workflows/conformance-e2e.yaml
.github/workflows/conformance-gateway-api.yaml
.github/workflows/conformance-ginkgo.yaml
.github/workflows/conformance-ipsec-e2e.yaml
.github/workflows/conformance-k8s-kind-network-policies.yaml
.github/workflows/conformance-k8s-kind.yaml
.github/workflows/conformance-k8s-network-policies.yaml
.github/workflows/conformance-kind-proxy-daemonset.yaml
.github/workflows/conformance-multi-pool.yaml
.github/workflows/conformance-runtime.yaml
.github/workflows/container-scan.yaml
.github/workflows/integration-test.yaml
.github/workflows/lint-codeowners.yaml
.github/workflows/lint-workflows.yaml
.github/workflows/push-chart-ci.yaml
.github/workflows/tests-cifuzz.yaml
.github/workflows/tests-datapath-verifier.yaml

PRs skipped due to conflicts:

#27346 (@qmonnet)
#27646 (@giorio94)

Once this PR is merged, you can update the PR labels via:
for pr in 27644 27391; do contrib/backporting/set-labels.py $pr done 1.12; done

or with
make add-labels BRANCH=v1.12 ISSUES=27644,27391

PRs skipped due to conflicts:
https://github.com/cilium/cilium/pull/27646 (@giorio94)

I mistakenly assumed that the Chart CI Push existed in v1.12, but it appears to not be present. I've dropped the associated needs-backport label from https://github.com/cilium/cilium/pull/27646. Sorry for the noise.

Rebased to fix conflicts with target branch.

/test-backport-1.12

/test-1.16-4.9

/test-1.16-4.9


HeliumLogger enhancements

[x] Print only filename, not the full path
[x] Be able to specify a custom format string for log output.
For example, if the user specifies log outputs as "%t - (%m)". It would print "04-12-2016 08:51 - (Message here)"
[ ] Filter based on levels.
Such as only "info" and above would print all errors, warnings, and info. But would exclude verbose.

Looks like we only have filtering based on logging-levels left.

The last item was handled by a PR.