Bluetooth: GATT: Fix not handling duplicated CCC storage

Some backends may actually contain the same key multiple times so the
code needs to check if there is already a ccc_cfg for an address before
attempting to use one that is unallocated.
Fixes #11409
Signed-off-by: Luiz Augusto von Dentz<EMAIL_ADDRESS>

@pdunaj @Olivier-ProGlove @Qbicz please give this a try.

In my opinion, it does not make sense to have key-value based settings and being able to have multiple time the same key name present in the settings with different values.
Is there any documentation mentioning this designed decision?
The issue related to this behavior and the 'workaround' in this PR shows it might not have been the best design decision.

@Olivier-ProGlove Afaik FCB (Flash Circular Buffer) works like that, any attempt to de-duplicate the data may incur in extra memory used to cached the last value read so I guess the memory penalty is the reason not to attempt such a thing.

@Vudentz My PR adds an extra byte to each entry to specify if it is a valid (last entry) or invalid (older entry or deleted key): https://github.com/zephyrproject-rtos/zephyr/pull/11574/files#diff-fda54055afbcaebc1652ba00a5d42563R329
We might lose one byte per entry but we save RAM + the necessity to create a new entry when we want to delete a key + the required workaround to handle old key-value pair.

@Qbicz Well okay, it wasn't intended to fix that one but it seems the original problem was the same. Anyway it is good to fix the clearing as well.

@Vudentz at first I didn't notice you created 2 very similar PRs and added a similar comment to both of them - so I didn't test this PR, only resetting CCC.

@Vudentz Is it correct with your change we restore the old CCC value stored in setting and not the latest stored CCC value? Is it expected? Are they always the same value?

@Olivier-ProGlove all this PR does is that it ensures duplicate entries are not created (or false-positive "we're out of memory") if the FCB history contains multiple entries for the same CCC. The FCB history is always loaded chronologically, so the code after this PR should always end up with the latest value.

Effectively, you are right, I missed the last line cfg->value = load->entry->value; :+1:


Send authentication header for restore when authentication provided

This PR adds the sending of authentication headers for the restore endpoint in case they are provided. This is important for partial restore. Otherwise the restore encounters a 401 error.
Changes

The restore endpoint adds basic authentication headers, depending on if auth in class NetHSM is set.

Checklist
Make sure to run make check and make fix before creating a PR, otherwise the CI will fail.

[x] tested with Python3.9
[x] signed commits
[ ] updated documentation (e.g. parameter description, inline doc, docs.nitrokey)
[ ] added labels

Can you please add a note to the changelog?  It would be great to also have a test case for this scenario but we can also add that in a separate PR.

I have added the note to the changelog and a test case. The integration tests fail on the newly added test right now. This is due to the bug that the NetHSM becomes inaccessible after a partial restore, returning an unauthorized error. I would suggest we wait for a fix there and afterwards merge this one to make sure everything works.

Okay.  Otherwise we can also merge this with a reduced test case for the time being if you need it.

No, this isn't urgent. I would prefer to get the problems on the NetHSM fixed first, and then revise this before the 1.1 release.

I’ve tried to re-run the checks with the fixed container.  Now it looks like the unlock after restore is missing and the update tests fail due to downgrade protection.  Should I look into fixing the test cases or will you take care of it as part of the release tests?

Thanks, I will take care.


OpenVINO on Arc A770 exception Check 'written_size == size'

c.c @fakezeta
LocalAI version:
quay.io/go-skynet/local-ai@sha256:4e4e427433285b056f32bfaa313ec0e75aeacb5b5c8c273953f9d2242fb55a60
Environment, CPU architecture, OS, and Version:

Linux dell 6.5.0-1020-oem #21-Ubuntu SMP PREEMPT_DYNAMIC Wed Apr  3 14:54:32 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Describe the bug

When I restrict a Pod to my A770 I get an exception, more details on the setup in #2208
Logs

10:36AM DBG GRPC(fakezeta/llama-3-8b-instruct-ov-int8-<IP_ADDRESS>:33205): stderr Error: Exception from src/inference/src/cpp/core.cpp:106:
10:36AM DBG GRPC(fakezeta/llama-3-8b-instruct-ov-int8-<IP_ADDRESS>:33205): stderr Check 'written_size == size' failed at src/plugins/intel_gpu/include/intel_gpu/graph/serialization/binary_buffer.hpp:25:
10:36AM DBG GRPC(fakezeta/llama-3-8b-instruct-ov-int8-<IP_ADDRESS>:33205): stderr [GPU] Failed to write 58720256 bytes to stream! Wrote 2727448
10:36AM DBG GRPC(fakezeta/llama-3-8b-instruct-ov-int8-<IP_ADDRESS>:33205): stderr
10:36AM DBG GRPC(fakezeta/llama-3-8b-instruct-ov-int8-<IP_ADDRESS>:33205): stderr
10:36AM ERR Server error error="could not load model (no success): Unexpected err=RuntimeError(\"Exception from src/inference/src/cpp/core.cpp:106:\\nCheck 'written_size == size' failed at src/plugins/intel_gpu/include/intel_gpu/graph/serialization/binary_buffer.hpp:25:\\n[GPU] Failed to write 58720256 bytes to stream! Wrote 2727448\\n\\n\"), type(err)=<class 'RuntimeError'>" ip=<IP_ADDRESS> latency=20.027449511s method=POST status=500 url=/v1/chat/completions

Additional context

I haven't got much to work with the A770 yet.

Hi @richiejp,
I replicated the error:
RuntimeError: Exception from src/inference/src/cpp/core.cpp:106:
Check 'written_size == size' failed at src/plugins/intel_gpu/include/intel_gpu/graph/serialization/binary_buffer.hpp:25:
[GPU] Failed to write 9437184 bytes to stream! Wrote 5800777

It triggers when there is not enough disk space when compiling the model for the selected device.
Have you checked that there is enough disk space?
I know that it's like asking to someone if he has gasoline in the car when it is not starting, and I feel stupid to ask but anyway it's worth to check. 😄


Bootstrap 4 support

Does this work with the latest bootstrap 4 beta?

It will work, but it probably won't fit precisely with the others form controls. Icons won't be available neither. The component has several classes you may customize to try to fit your needs, but I haven't tested them by myself in order to play nicely with bootstrap 4.

@Hesesses I added the css below and it doesn't look so bad now. At least the height is same as other inputs.
// ng2-iq-select2 fixes for bootstrap 4
.select2-selection-container {
  min-height: 38px !important;
}

<EMAIL_ADDRESS>supports bootstrap 4 now


Crawler outputs damaged videos with parallel ffmpeg calls

The Crawler/Kinetics/download.py script fails when using num_jobs > 1. The videos are downloaded correctly but the clipping with ffmpeg produces a damaged video that doesn't play fully. My solution was to set prefer='threads' in the call to Parallel. The job is still calling subprocess_check_output for downloads and ffmpeg calls so will likely be ran in parallel. I'm posting in case anybody else encounters this frustrating issue. To devs, please reply if you want me to make a pull request with this fix.

Hi! I am having a similar issue. The videos download but in most cases I cannot open them nor can OpenCV.
Thanks for the proposed solution! I have a question, though. To be clear, you mean making a change in https://github.com/activitynet/ActivityNet/blob/82304fa9f4a7219421df13dae6347a445c4cf5bc/Crawler/Kinetics/download.py#L191 resulting in
    status_lst = Parallel(n_jobs=num_jobs, prefer='threads')(delayed(download_clip_wrapper)(

Is that correct? Thanks in advance!


Improved description of the Fizz Buzz

Improved description of the Fizz Buzz and added solution url

Hi @maridilip! Thanks for this contribution! It seems like @Jeff-Duke beat you to the Pull Request by a few minutes. A lot of these solutions are really clever, but I think they are more complicated than what I would like to show developers looking at their very first whiteboard challenge. If you want to make improvements on the current code base, please feel free to contribute!


Add standard error to mean lines

Add shaded area of width +/- std/sqrt(n) around mean lines like in this figure

Resolved with #145


cast index string does not work

Component
Cast
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (da2392e 2023-08-02T09:00:14.068523319Z)
What command(s) is the bug in?
cast index
Operating System
Linux
Describe the bug
cast index error if the key_type is a string. (https://book.getfoundry.sh/reference/cast/cast-index)
The cast index uses abi.encode to calculate slots for the mapping type (https://github.com/foundry-rs/foundry/blob/master/crates/cast/src/lib.rs#L1664C75-L1664C75), but it doesn't work for string .
abi.encodePakced works instead of abi.encode for string.
The contract code: https://arbiscan.io/address/0xA32E7eA512525B89171F3b48C7F01bD933d17c59#readContract
root@user:/home/user# cast index string "hello" 1
0x3556fc8e3c702d4479a1ab7928dd05d87508462a12f53307b5407c969223d1f8
root@user:/home/user# cast storage 0xA32E7eA512525B89171F3b48C7F01bD933d17c59 0x3556fc8e3c702d4479a1ab7928dd05d87508462a12f53307b5407c969223d1f8 -r https://arbitrum-one.publicnode.com
0x0000000000000000000000000000000000000000000000000000000000000000
root@user:/home/user# chisel
Welcome to Chisel! Type `!help` to show available commands.
➜ keccak256(abi.encodePacked("hello",uint256(1)))
Type: bytes32
└ Data: 0x8404bb4d805e9ca2bd5dd5c43a107e935c8ec393caa7851b353b3192cd5379ae
➜
(To exit, press Ctrl+C again)
➜
root@user:/home/user# cast storage 0xA32E7eA512525B89171F3b48C7F01bD933d17c59 0x8404bb4d805e9ca2bd5dd5c43a107e935c8ec393caa7851b353b3192cd5379ae -r https://arbitrum-one.publi
cnode.com
0x776f726c6400000000000000000000000000000000000000000000000000000a

This issue applies to bytes types as well as strings.

thoughts @DaniPopes ?

When is this gonna fixed? I'm even surprised that another person from my company already reported this like two months ago.
Also bytes is not handled in a proper way.

@DaniPopes mind taking a look at this?
I think strings/bytes <32 are packed? https://docs.soliditylang.org/en/latest/internals/layout_in_storage.html#bytes-and-string


[pdal] Revise and update

What does your PR fix?
Make pdal build on non-Windows platforms (fixes #14182).
Update pdal to 2.3.0 (fixes #17671, fixes #13946), and pdal-c to v2.1.0.
Switch pdal to dynamic linkage. (Upstream doesn't support static linkage, cf. https://github.com/PDAL/PDAL/issues/3621. So there is also no need to carry transitive usage requirements.)
Add supports fields and remove ci baseline entries.
Provide plugins via features (fixes #8478).
Get rid of absolute paths in config and RPATH.
Add usage, based on CMake variables as upstream does.

Which triplets are supported/not supported? Have you updated the CI baseline?
windows/linux/osx (depending on features), yes
The current version doesn't build with mingw, but PDAL 2.4 will have fixes.

Does your PR follow the maintainer guide?
yes

Successful build on osx and linux, finally. However, I don't know yet if forcing PDAL [plugins] into static linkage result in working binaries.
I also plan to disable the debug build of the pdal tool.

TODO from #21723:

[ ] Remove pc files (need gdal.pc)
[ ] Replace absolute paths in includes.
[ ] Add usage file.

Can you please resolve the file conflict?

Resolved.

Updated.

Looks good to me, but other guys may have some suggestions.

Thanks @dg0yt :)


Augment development and usage documentation and utilities

This pull request includes a number of small updates to the documentation for running the project and developing contributions.
It also introduces (optional) utilities for installing, configuring, building, testing, running things.

I've created this as a "draft" PR to open things up for discussion. There's still a bit to do (specifically to DEVELOPMENT.md and dev) before it's ready to be considered for merging.
I'm particularly interested in discussion around the suggested additions of the package management tools (e.g., Homebrew and asdf) as well as the bash "CLI" utilities.

Codecov Report

Merging #233 (fc1b1cc) into main (6c46945) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #233   +/-   ##
=======================================
  Coverage   91.49%   91.49%
=======================================
  Files          45       45
  Lines        1776     1776
=======================================
  Hits         1625     1625
  Misses        101      101
  Partials       50       50

Impacted Files
Coverage Δ

cmd/orchestrator/orchestrator.go
82.53% <ø> (ø)

pkg/hawksupport/hawk_support.go
100.00% <ø> (ø)

cmd/demo/demo.go
82.05% <100.00%> (ø)

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 6c46945...fc1b1cc. Read the comment docs.

Ready to review. This PR introduces a number of workspace/project "ergonomics" that would be good to look at with others before deciding they're worthy or desirable.


[enkits] New port

Add new port enkits : https://github.com/dougbinks/enkiTS

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

Note: I will be converting your PR to draft status. When you're ready, please revert to "ready for review".

Please provide the usage file.

It should be good to go.

The name case and location issues of the config file:
1> [CMake] CMake Error at G:/enkits/scripts/buildsystems/vcpkg.cmake:859 (_find_package):
1> [CMake]   Could not find a package configuration file provided by "unofficial-enkits"
1> [CMake]   with any of the following names:
1> [CMake]
1> [CMake]     unofficial-enkitsConfig.cmake
1> [CMake]     unofficial-enkits-config.cmake

The usage test passed on x64-windows (header files found):
enkits provides CMake targets:

    find_package(unofficial-enkits CONFIG REQUIRED)
    target_link_libraries(main PRIVATE unofficial::enkiTS::enkiTS)

I think this was done by https://github.com/microsoft/vcpkg/pull/34386 ?

Yes, it's the same with minor differences due to inconsistent reviews. I'm closing it.

Duplicate of #34386.


No license

There is no LICENSE In this repository. It'd be great to know if the repo is meant to be proprietary, open source, or something else. Ie, https://choosealicense.com/

thanks, resolved


Update anticheat-compatibility.md

Added some anticheats that were missing in my original list

GrimAC isn’t paid yet. It’s free until GrimAC will be fully released.

Thanks!


feat: port LinearAlgebra.AffineSpace.MidpointZero

bors r+

Looks like there will be a merge conflict.
bors r-

This can be merged after the other PR is in master and the conflict is fixed.
bors d+

bors merge


Too many font sizes

Hello!
Browsing through the CSS, I felt like everything had its own font-size, so I dug a little deeper to find out that it was nearly true. I found all these rules:
font-size: 2.5em;
font-size: 1.8em;
font-size: 1.5em;
font-size: 1.4em;
font-size: 1.3em;
font-size: 1.2em;
font-size: 1.1em;
font-size: 1.0em;
font-size: 0.95em;
font-size: 0.9em;
font-size: 0.85em;
font-size: 0.8em;
font-size: 0.75em;
font-size: 0.7em;
font-size: 0.6em;
font-size: 0.5em;
font-size: 150%;
font-size: 100%;
font-size: 80%;

I think that this is detrimental to the homogeneity of the application. I think that 5 font sizes would be OK, namely:

App title 1.5em
Main title 1.6em
Section title 1.2em
Normal font 1em
Smaller font 0.8em

EDIT: this does not count the font size for smaller screens (which only changes the main font-size).
What do you think?

Agreed, what about using a css class for each font-size?

Well, there are HTML tags that will cover most of the use cases (h1, h2, h3 and small). For other cases (like tables or the project overview where the numbers on top are displayed with a bigger font), we should have the rule directly in the CSS.
The problem is not to have to many font-size rules, but to have to many font-size values IMHO.

In dev, the number of font-size values have been reduced to 6 and that can be changed easily since I converted all vanilla CSS to SASS.


Make model parameter optional for Backbone.sync

I encountered a situation in which I needed to call Backbone.sync directly, w/o a model, which works perfectly except it will throw when trying to trigger a 'request' event on model. Obviously I could easily pass a trigger:noop object or something, but thought this was cleaner.
Test included btw.

Sorry but -- what's the use case? Every extra conditional is one more potential bug.

It's a bit hard to explain, but basically it comes down to this: I have a number of things that are automatically attached to all Backbone.sync requests (e.g. a unique ID for each request, centralised error handling et cetera), which is why I'd like to use Backbone.sync for a multipart/formdata RPC request, to benefit from the extras, but all the data it's sending is massaged and collected, i.e. there's no specific model that I need to use. As I said, I can easily solve it, but I assumed that model was in fact meant to be optional since on this line it's truthiness is being checked as well, i.e. why bother checking if it's required anyway?
If you dig through the function you'll see there's no real dependency on model except to use as data if none is provided and to send out a 'request' notification, which struck me as it being optional. But no problem if it's too much a hassle.

To be honest, I'm not sure why that's there (but it's been there for years), along with the model.trigger. I think wrapping sync with a dummy object is going to be your best bet here, as yours is probably not a use case we want to be supporting. I'll leave this issue open a little longer in case any other contributors want to chime in.

We should either add this, or remove the && model guard.

I was curious about this guard and for posterity it was introduced by 4c1bdb46a8aeba6a6fc9803614b074dbca2ec050. Seems safe to remove it.

Back then the guard made totally sense, since model was optional... Which I think TBH it still should be 😁

or remove the && model guard.

I thought of that (removing it doesn't fail any tests), but what's the harm in keeping it?
In this case it's probably fine to remove, but I think recent history has shown that we tend to get bit by edge cases we think are safe to remove.


"If" statement or more operations needed

I'd like to set up a less template that could have variables substituted in order to create resulting style sheets.  The immediate questions in my mind are 1) how to easily turn on/off certain sections based on single variable values, and 2) how to "flip" color values based on lightness / darkness.  In other words, if I change my background color variable to a dark color, I want the text color to be lighter, and obviously, if I change my background color to a dark color, the reverse should happen.
As far as I can tell, this can't happen with the current set of operations.  Is there any chance that LESS might get some more sophisticated expressions?  If statements would solve that, along with some other types of expressions / functions.
Thanks.

+1
example:
.rounded(@radius: 5px, @topleft: true, @topright: true, @bottomleft: true, @bottomright: true)
{
  -moz-border-radius: @radius;
  -webkit-border-radius: @radius;
  border-radius: @radius;

  @if(@topleft == false){
    -moz-border-radius-topleft: 0px;
    -webkit-border-top-left-radius: 0px;
    border-radius-topleft: 0px;
  }

  @if(@topright == false){
    -moz-border-radius-topright: 0px;
    -webkit-border-top-right-radius: 0px;
    border-radius-topright: 0px;
  }

  @if(@topright == false){
    -moz-border-radius-bottomright: 0px;
    -webkit-border-bottom-right-radius: 0px;
    border-radius-bottomright: 0px;
  }

  @if(@topright == false){
    -moz-border-radius-bottomleft: 0px;
    -webkit-border-bottom-left-radius: 0px;
    border-radius-bottomleft: 0px;
  }
}

Yeah, don't know why it was closed either without comment.
"If" statements seem like such a basic necessity.
Another "nice to have" would be more integrated media query support, but that's a separate issue.

Here's how Sass does it:
@mixin link-colors($normal, $hover: false, $active: false, $visited: false, $focus: false) {
  color: $normal;
  @if $visited {
    &:visited {
      color: $visited; } }
  @if $focus {
    &:focus {
      color: $focus; } }
  @if $hover {
    &:hover {
      color: $hover; } }
  @if $active {
    &:active {
      color: $active; } } }


Switch away from gulp-server-livereload

gulp-server-livereload hasn't been updated in four years and is the source of quite a lot of vulnerabilities reported by npm audit. It also doesn't run on node.js >= 14 because of this bug in one of its outdated dependencies: https://github.com/yuanchuan/node-watch/issues/94. I'll try looking at some alternatives, but I'm not familiar with the gulp ecosystem, so any tips or recommendations are appreciated.

@dpeukert - I dont think this is still valid - i couldn't find references to gulp-server-reload in the ferdi repo. Please close if this is correct.

@vraravam Yep, thanks for the heads-up, closing.


Cleanup .rvmrc to remove redundancy

It didn't seem as if the second part of the gemset really served any purpose.

Thanks man!  :beer:


feat: symmetric monoidal structure on graded objects

In this PR, we construct the braiding for the monoidal structure on graded objects. We show it is symmetric if the original category is symmetric.

(Refactor of #6379)
This is a draft.

Indeed, it is now ready for review.


Can someone provide a reliable source for LATEST_NET_G.PTH file?

The automated download link provided by this install:
_https://sfu.ca/~yagiz/CVPR21/latest_net_G.pth_
fails.
It fails when I put it into my browser manually or without the http as well. Is there a more reliable source? Is that file hosted here & I don't see it?

Yes the server seems to be intermittent, I'm not sure licensing allows for me to mirror it.
I believe there was another post here mentioning an unofficial mirror, I will update this post when I find it..

https://wormhole.app/5q4A1#fDPs4Ki9IFRnZouW7vqaRg

SFU.CA still timing out for me. D/ling from the fielshare site now, thanx.

Unfortunately:
loading the model from ./models/pix2pix\latest_net_G.pth
unexpected EOF, expected 3377425 more bytes. The file might be corrupted.
All done.
** re d/l'd --- must've had a corrupt save initially now it's doing work

Hello! Can I get a link for the latest_net_G.pth file? Download just doesn't start from this link https://sfu.ca/~yagiz/CVPR21/latest_net_G.pth

https://wormhole.app/xjA5P#9ilCfgOWUC39xO_N1x5LUA
We really should look into making a mirror somehow ..


find _dlopen and _dlsym

Apple's review was rejected，Does this matter ？

Could you please add some details?


add suggestions in failure message in install script

Adds suggestions to where you can install docker in the case
that the install script fails to install because of the architecture
not being officially supported.
Signed-off-by: Christopher Jones<EMAIL_ADDRESS>

Can this error message include a URL where a more detailed explanation might be found? That way you'd have some idea both of what problems people are running into as well as not requiring that the error message be complete (as the URL could be more easily edited to accumulate the detail of questions that people have).

This is the case of docker not having deb/rpms in their repository for the platform you are trying to install on, in which case the install script exits immediately with this. I can change the wording around to be more verbose, but I can't think of anywhere to point to that would be more helpful.

I can see two places this might point to, to documents that might or might not exist:

How to port Docker to a new platform.
How to build Docker from source code using dind.

The first might be very long and complex, but the second should be something that already exists, or if it doesn't exist then a PR to create it should be welcomed.

I've changed the wording to hopefully be less confusing.
I don't think this is the right place to add advice on porting Docker to a new platform. A link to building Docker from source would be good here, but I think the suggestions here should be fine.
@thaJeztah what are you thoughts on this?

LGTM
ping @thaJeztah @vdemeester


RDR: exporting client info

Handles exporting the incoming data found in the on-boarded consumer's request into the consumer's status, that in turn will populate the ODF info CM's ConnectedClients via the StorageCluster's reconciler (addressed as part of #2440).

/test ocs-operator-bundle-e2e-aws

/cc @umangachapagain

/cc @umangachapagain @nb-ohad

/test ocs-operator-bundle-e2e-aws

/lgtm


Resource deletion: unable to pass --force when using query

Hi,
I'm trying to delete VMs in bulk and need to pass the --force switch.
az vm delete --ids "$(az vm list -g IPv6 --query "[?contains(name,'XREVM')].id" -o tsv)" --force
&
az vm delete --force --ids $(az vm list -g IPv6 --query "[?contains(name,'XREVM')].id" -o tsv)

Both give the error: az: error: unrecognized arguments: --force

The param is --yes (it was renamed).
Adding --help should indicate the available params.

Thanks.  Note that the help file still references –force (under examples):
Delete a VM without a prompt for confirmation.
az vm delete -g MyResourceGroup -n MyVm --force
<EMAIL_ADDRESS>From: Derek Bekoe<EMAIL_ADDRESS>Sent: Friday, April 21, 2017 1:13 PM
To: Azure/azure-cli<EMAIL_ADDRESS>Cc: Teo De Las Heras<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/azure-cli] Resource deletion: unable to pass --force when using query (#2953)
The param is --yes (it was renamed).
Adding --help should indicate the available params.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F2953%23issuecomment-296249710&data=02|01|teodelas%40microsoft.com|743d5da9d5da4c30a85508d488d9992d|72f988bf86f141af91ab2d7cd011db47|1|0|636283915536748040&sdata=UCJGUOy%2FAjDJv85C7sYCOGVUouj1CzO5sJpdqAgluQA%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFIsLQLjoK8l2LnzOHoMiG9CThcEcCw1ks5ryON_gaJpZM4NEc8Y&data=02|01|teodelas%40microsoft.com|743d5da9d5da4c30a85508d488d9992d|72f988bf86f141af91ab2d7cd011db47|1|0|636283915536748040&sdata=wOiqj2sEOu%2B%2BB%2BCRjNe1ACVEZ17lZfQYpWWQf4xSB%2FY%3D&reserved=0.

What version of the CLI do you have?
I see:
Examples
    Delete a VM without a prompt for confirmation.
        az vm delete -g MyResourceGroup -n MyVm --yes

azure-cli (2.0.1)
From: Derek Bekoe<EMAIL_ADDRESS>Sent: Friday, April 21, 2017 1:30 PM
To: Azure/azure-cli<EMAIL_ADDRESS>Cc: Teo De Las Heras<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/azure-cli] Resource deletion: unable to pass --force when using query (#2953)
What version of the CLI do you have?
I see:
Examples
Delete a VM without a prompt for confirmation.

    az vm delete -g MyResourceGroup -n MyVm --yes

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F2953%23issuecomment-296253871&data=02|01|teodelas%40microsoft.com|2958c7a8d1cf4397d20408d488dc14b2|72f988bf86f141af91ab2d7cd011db47|1|0|636283926202893305&sdata=pV7SD1pwHtKZ9Tot%2Bga0bzBSKMwHE8H9%2Fh2YUEKDN3w%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFIsLXmJSAUL-iyC6h8OrA2rpYoY1I6bks5ryOepgaJpZM4NEc8Y&data=02|01|teodelas%40microsoft.com|2958c7a8d1cf4397d20408d488dc14b2|72f988bf86f141af91ab2d7cd011db47|1|0|636283926202893305&sdata=nyChPQXuHhQmHPHiWg44Vk%2BBrjkJhjXEbj2UVqjdzao%3D&reserved=0.

The current latest is 2.0.3 so would suggest upgrading to that.


FSRW test goes sideways after load..

The fsrw test#2 code from here:  http://forums.parallax.com/discussion/171619/
Appears to compile and load, but something goes wrong immediately after loading and appears to reboot...
This file works with both PNut and Fastspin.

Have you tried loading to flash to see if it runs properly after the apparent reboot?  Just curious; it should work just like with PNut.
This is one the P2-ES Rev B with a prewritten microSD card inserted?

I played with the jumpers to load to flash and the boot from flash...
Same thing happens when booting from flash.
I guess that means it compiled wrong.
Not sure how that is possible.  Works with PNut...

I can see it load up about 1/4 of the VGA screen from the bitmap on the uSD card and then stop.

All the files needed to reproduce are in the top link.  Just copy bitmap2.bmp to uSD and set all jumpers off, except P59 pullup.

Ok, yes, looks like file directive is broke.  Changing to this works:
orgh
BitmapFile 'something to show before load is complete and also storage for bitmap
'file "TestPat1.bmp" '640 x 480, 8bpp bitmap
byte 0[350000] 'a little padding just in case metadata and/or block size makes SD file bigger

@GitRayman I believe I have fixed it.  Can you retest your fsrw test#2.spin2 object, and any other examples you have, to verify?  Use that attached version of Propeller Tool v<IP_ADDRESS>-Alpha.
Propeller_v<IP_ADDRESS>-Alpha.zip

Thanks.  Just tried test#2.spin2 and it works!

You're welcome, and thank you too.  The problem was something I had marked as questionable in code but failed to get back to before first alpha, and the result of the error showed up in a way I didn't expect.  I'm glad to get confirmation that it works now.  They'll be another official Alpha on 6/17 that includes this fix.

This is included in v<IP_ADDRESS>-Alpha.


Fix msgpack install

I was having trouble building, getting the error
[10/594] Performing install step for 'msgpackProject'                                                                                FAILED: msgpackProject-prefix/src/msgpackProject-stamp/msgpackProject-install                                                                                cd /home/ljoswiak/build/msgpackProject-prefix/src/msgpackProject-build && /usr/local/bin/cmake --build . --target install && /usr/local/bin/cmake -E touch /home/ljoswiak/build/msgpackProject-prefix/src/
msgpackProject-stamp/msgpackProject-install                                                                                [0/1] Install the project...                                                                                -- Install configuration: ""                                                                                -- Installing: /usr/local/lib/libmsgpackc.so.2.0.0
CMake Error at cmake_install.cmake:52 (file):                                                                                file INSTALL cannot copy file                                                                                "/home/ljoswiak/build/msgpackProject-prefix/src/msgpackProject-build/libmsgpackc.so.2.0.0"                                                                                to "/usr/local/lib/libmsgpackc.so.2.0.0".

FAILED: CMakeFiles/install.util
cd /home/ljoswiak/build/msgpackProject-prefix/src/msgpackProject-build && /usr/local/bin/cmake -P cmake_install.cmake
ninja: build stopped: subcommand failed.

This change seems to fix it.
General guideline:

If this PR is ready to be merged (and all checkboxes below are either ticked or not applicable), make this a regular PR
If this PR still needs work, please make this a draft PR

If you wish to get feedback/code-review, please add the label RFC to this PR

Please verify that all things listed below were considered and check them. If an item doesn't apply to this type of PR (for example a documentation change doesn't need to be performance tested), you should make a strikethrough (markdown syntax: ~~strikethrough~~). More infos on the guidlines can be found here.
Style

[ ] All variable and function names make sense.
[ ] The code is properly formatted (consider running git clang-format).

Performance

[ ] All CPU-hot paths are well optimized.
[ ] The proper containers are used (for example std::vector vs VectorRef).
[ ] There are no new known SlowTask traces.

Testing

[ ] The code was sufficiently tested in simulation.
[ ] If there are new parameters or knobs, different values are tested in simulation.
[ ] ASSERT, ASSERT_WE_THINK, and TEST macros are added in appropriate places.
[ ] Unit tests were added for new algorithms and data structure that make sense to unit-test
[ ] If this is a bugfix: there is a test that can easily reproduce the bug.

It seems that I had msgpack installed locally, so it didn’t test this part of the code


This needs .Net 4.6?

Not necessarily, didn't try

in dotnet 3.5 I get this
`interpolated strings' cannot be used because it is not part of the C# 4.0 language specification

Oh crap, I can fix it easily

What version of Unity are you using?

2017 LTS at the moment, working my way to 2020.1

Oh, very strange, because NG Unity Versioner has been developped against 2017 LTS as well.
If you could provide me any log to help me in the task, I would greatly appreciate

This is how my scan parameters looks like:

Can you try to scan with one single version? To check if the crash is global.

Can you try to scan with one single version? To check if the crash is global.

sorry I don't understand

2017.4.39f1

empty project with PUN2 and post processint stack v2

Take this version. I made it especially for you (Targetting .NET 3.5)
Keep me in touch if you see anything

This version.
NG Unity Versioner.zip

thanks! it works :)

Oh yeah, good to hear!
Don't forget to put a filter namespace, otherwise it will scan your entire codebase!
Also, I never tested it with complex and big projects. I can't tell how reliable is the algorithm.
Feel free to contact me if you see anything!


fixed video reload on changing quality

fixes:#142
After this pr:
The video reloads only once after selecting a quality. The video does not reload if the selected quality is already applied.
https://github.com/code100x/cms/assets/111998184/b71bdba7-fb56-4457-8d36-16ea698a04de

This isn't the ideal solution for this, ideally the quality selector should be in the video player
Please add it there in case youre thinking of doing this
closing for now

Ok sir I will try again.


Visualizing the images from himax camera.

Hello everyone.
I would like to know if there is a good and easy way to retrieve the images being captured from the camera.
I would like to see the captured image, and the model output from that image.
There is a way to send the image thought USB for example?
I think it is a silly question, but I don't have much experience with this kind of hardware. Any help is welcome.
Thank you so much.

@PCPJ hi,
You can reference LAB1_capture_image example under the below repository. https://github.com/HimaxWiseEyePlus/bsp_tflu/tree/master/HIMAX_WE1_EVB_example/scenario_app/workshop_example
see the captured image:

Step1: The easy way is to use our  PC_TOOL_Linux, you can download here.
Step2: please key-in following command on console.

chmod 777 PC_TOOL_Linux

Step3: Install FT4222 linux driver:
You can reference the user guide here to download and install FT4222 linux driver.
Step4:

Then you will see the captured images on the PC and just need to connect by the USB cable.
The more detailed introduction about this tool you can reference the below example.
https://github.com/HimaxWiseEyePlus/WE_I_Plus_User_Examples/tree/main/HIMAX_Yolo_Fastest_Person_Detection_Example

If you want to see the model output, you can reference the SPI TOOL code (https://github.com/HimaxWiseEyePlus/bsp_tflu/tree/master/SPI_Tool#spi-tool-for-receive-meta-data-and-images-from-himax_we1_evb) and change it to receive meta data which you define yourself for your own model output.

Thanks for the help @kris-himax regarding the visualization using SPI Tool. However for some reason I am not able to see anything on the image window when I connected my Himax WE-I Plus via USB. I am sharing the screenshots of my console output and the SPI Tool UI. Do you have any idea what might be the issue?


Golden Record Process Consolidation

Description
see https://jira.catena-x.net/browse/CXAR-939
Impact

Additional information

[ ] I'm willing to contribute to this feature

Closed as aligned with expert group.


fix: allow hyphenated services

Please read CONTRIBUTING.md for additional information on contributing to this repository!

What this PR does / why we need it

This addition ignores a linter error thrown when re-stenciling services with a _ in their service name.
According to the templated comment, we should allow these names:

Why: We allow [-_].

When re-stenciling a data-sharing-service (gocode service uses a _ in the name), the following error is thrown:

api/data-sharing-service/doc.go:8:1: package "data_sharing_service" should not contain - or _ in name (doculint)

Jira ID
[XX-XX]

Notes for your reviewers

@malept Closing this PR, as it sounds like this will take a significant amount of additional work. Other than this repo and terraform monitor repos, I'm unaware of any other stencil packages that would need this change. If you think this is doable without significant refactoring, please let me know which other stencil repos would require work. Thanks!


test_somoclu_summarizers.py failing

Running an a 2021 macbook M1 (so apple silicon, which might be the cause of my woes...)
when trying to build and run the RAIL package locally, I get the following errors on the somoclu test:
FAILED tests/test_somoclu_summarizers.py::test_SomocluSOM - NameError: name 'wrap_train' is not defined
FAILED tests/test_somoclu_summarizers.py::test_SomocluSOM_with_mag_and_colors - NameError: name 'wrap_train' is not defined

there seems to be a related issue tracked in the somoclu repo noting this on windows machines from 4 years ago. It seems like they pushed a <IP_ADDRESS> update to fix it but the pip install doesn't actually work locally, due to a metadata issue:
Discarding https://files.pythonhosted.org/packages/bf/1d/8f2a89cd9a0f37fe3a8ba114f43e310a832f5f30f7f3d886e6379ab2e722/somoclu-<IP_ADDRESS>.tar.gz (from https://pypi.org/simple/somoclu/): Requested somoclu from file:///Users/maxwest/Library/Caches/pip/wheels/47/89/ab/0932352d0a0a8ceebb91b937bd4de854ec6927f4ab6c78b7b4/somoclu-1.7.5-py3-none-any.whl has inconsistent version: expected '<IP_ADDRESS>', but metadata has '1.7.5'

They also have  a 1.7.6 release out from ~2 years ago but it's not on pypi 🙁

Yes, I think that this is yet another instance of package developers updating conda and not PyPI, if you do a conda install -c conda-forge somoclu it should grab an updated version of the package and things should work.  We should add this to install instructions, and maybe ping the somoclu developers to see if they're willing to update the package on PyPI.

for some reason conda can't find the somoclu package, even though it's listed on conda-forge 🤔 weird, could be a problem with my local setup.
I added a comment to the related issue and I'll probably ask them to put 1.7.6 on pypi.

I think the issue here is primarily that some of the underlying packages aren't up to date for M1 mac.

the somoclu people updated the package on pip and conda, so I have 1.7.6 working on my M1! but the somoclu tests are still failing 🤔
        # test loading model by name rather than via handle
        summarizer2 = summarizer_class.make_stage(
            name=key,
            model=f"tmpsomoclu_"+key+".pkl",
            aliases={
                "model": "somoclu_summarize_test2_model",
                "input": "somoclu_summarize_test2_input",
                "spec_input": "somoclu_summarize_test2_spec_input",
            },
        )
        _ = summarizer2.summarize(phot_data, spec_data)
        fid_ens = qp.read(
            summarizer2.get_output(
                summarizer2.get_aliased_tag("single_NZ"), final_name=True
            )
        )
        meanz = fid_ens.mean().flatten()
>       assert np.isclose(meanz[0], 0.14414913252122552)
E       assert False
E        +  where False = <function isclose at 0x104591550>(0.13940019753973246, 0.14414913252122552)
E        +    where <function isclose at 0x104591550> = np.isclose

tests/test_somoclu_summarizers.py:62: AssertionError

Yeah, this is a generic issue with ML algorithms.  When they change the code version, the values you get change slightly.
Maybe check with ziang that the new value is reasonable and then loosen the np.isclose() requirement.
-e

On Mar 6, 2023, at 4:19 PM, Max West @.***> wrote:
the somoclu people updated the package on pip and conda, so I have 1.7.6 working on my M1! but the somoclu tests are still failing 🤔
    # test loading model by name rather than via handle
    summarizer2 = summarizer_class.make_stage(
        name=key,
        model=f"tmpsomoclu_"+key+".pkl",
        aliases={
            "model": "somoclu_summarize_test2_model",
            "input": "somoclu_summarize_test2_input",
            "spec_input": "somoclu_summarize_test2_spec_input",
        },
    )
    _ = summarizer2.summarize(phot_data, spec_data)
    fid_ens = qp.read(
        summarizer2.get_output(
            summarizer2.get_aliased_tag("single_NZ"), final_name=True
        )
    )
    meanz = fid_ens.mean().flatten()

  assert np.isclose(meanz[0], 0.14414913252122552)

E       assert False
E        +  where False = <function isclose at 0x104591550>(0.13940019753973246, 0.14414913252122552)
E        +    where <function isclose at 0x104591550> = np.isclose
tests/test_somoclu_summarizers.py:62: AssertionError
—
Reply to this email directly, view it on GitHub https://github.com/LSSTDESC/RAIL/issues/300#issuecomment-1457258826, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADRIGIVN4JSUFLI5PIY76STW2Z5HVANCNFSM6AAAAAAU2Z7S5I.
You are receiving this because you commented.

I think this same issue popped up with pzflow, and if you look at the pzflow test in tests/test_algos.py  you can see that we have a np.isclose with an atol=0.05 because there is a minor numerical instability that led to some small changes in the results when some of the normalizing flow dependencies were being updated to newer versions.  My guess is that we should just do the same thing here.

overtaken by events a while back


⌘R chords not working, when integrated terminal has focus

It's present in insiders build.

VSCode Version: 1.31
OS Version: OSX 10.14.2

Steps to Reproduce:
Open integrated terminal and focus it, then use chords ⌘R, Shift+R => you'll see it doesn't open available scripts from package.json. It stucks on ⌘R was pressed. Waiting for second key of chord..., then i press Shift+R and it just prints "R" in a terminal.

As you can see from a screenshot, i don't have anything else mapped on ⌘R and When condition is empty, so it should work when integrated terminal has focus, but it doesn't. It's been working in a previous version, but not in 1.31.
Does this issue occur when all extensions are disabled?: Yes/No

You probably need to add command Id to the skipShell commands in Settings:
https://code.visualstudio.com/docs/editor/integrated-terminal#_forcing-key-bindings-to-pass-through-the-terminal

Just found an issue with this approach: https://github.com/Microsoft/vscode/issues/63271

editor.action.toggleTabFocusMode
workbench.action.debug.continue
workbench.action.debug.pause
workbench.action.debug.restart
workbench.action.debug.run
workbench.action.debug.start
workbench.action.debug.stepInto
workbench.action.debug.stepOut
workbench.action.debug.stepOver
workbench.action.debug.stop
workbench.action.firstEditorInGroup
workbench.action.focusActiveEditorGroup
workbench.action.focusEighthEditorGroup
workbench.action.focusFifthEditorGroup
workbench.action.focusFirstEditorGroup
workbench.action.focusFourthEditorGroup
workbench.action.focusLastEditorGroup
[...]

You don't need to add all these lines.

@usernamehw they fixed it already?

Fixed what, exactly?
These commands excluded by default (the big list)
"terminal.integrated.commandsToSkipShell" from your settings are added to that list.

@usernamehw i've read, that you needed to paste all these actions in order to override default ones for commandsToSkipShell.

That was before vscode 1.31.0.

@usernamehw actually improved that 💪 https://github.com/Microsoft/vscode/pull/66179
See the release notes: https://code.visualstudio.com/updates/v1_31#_improvements-to-commandstoskipshell-setting


AzureServiceBusTopicAutoConfiguration should import AzureServiceBusAutoConfiguration

AzureServiceBusTopicAutoConfiguration should import AzureServiceBusAutoConfiguration, same as ServiceBusQueueBinderConfiguration, otherwise we'll get following exception when we have multiple binders (Queue and Topics)
Parameter 3 of method topicClientFactory in com.azure.spring.cloud.autoconfigure.servicebus.AzureServiceBusTopicAutoConfiguration required a bean of type 'com.azure.spring.integration.servicebus.factory.ServiceBusConnectionStringProvider' that could not be found.

This should submit to azure-sdk-java repo


Add liquidity constraint to perfect foresight model

This PR updates model notebooks to account for the existence of BoroCnstArt in the perfect foresight model.  It should only be merged after this feature has been included in a public release of HARK, otherwise the notebook will break; likewise, it should be merged immediately otherwise the notebook will break.

assigned

KinkedRconsumerType.ipynb and PerfForesightConsumerType.ipynb now are part of HARK/examples/ConsIndShockModel so the changes of this PR should be migrated there.

These changes have been migrated to HARK. See https://github.com/econ-ark/HARK/pull/795


A bit more tests for config deletion

More tests for bulk deletion - check working of old-style single config deletion

Codecov Report
Base: 61.74% // Head: 37.14% // Decreases project coverage by -24.60% :warning:

Coverage data is based on head (2c07dd4) compared to base (82c0391).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1795       +/-   ##
===========================================
- Coverage   61.74%   37.14%   -24.61%
===========================================
  Files         178      178
  Lines       13756    13756
===========================================
- Hits         8493     5109     -3384
- Misses       4199     7891     +3692
+ Partials     1064      756      -308

Flag
Coverage Δ

acceptance-api
?

acceptance-apps
35.54% <ø> (-0.59%)
:arrow_down:

acceptance-cli
?

unittests
8.49% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

 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.


Build both Java and Scala sources, add implicit conversion for Scala's Tuple2

There are a couple of changes, but the title lists the most important.

Merged, thanks!


@tanstack/table explorations: add web component setup in monorepo and basic examples

With a greater number of teams utilizing web components, we should offer web component versions for all of the DataTable/@tanstack/table examples that we're building.
### Tasks
- [x] Update monorepo to rename `packages` to `react`
- [x] Add a `web-component` directory at project root
- [x] Update readmes, discussing the purpose of the monorepo and links to stackblitz and csb

### Lit examples
- [x] Basic example
- [x] Batch actions
- [x] Customizable columns (drag and drop interaction on hold until `@dnd-kit/dom` supports sortable lists/becomes more stable)
- [ ] Editable cells
- [ ] Filter flyout
- [ ] Filter panel
- [x] Global filter (included in batch actions example)
- [ ] Infinite scroll
- [x] Nested rows
- [x] Pagination (included in batch actions example)
- [x] Resizable columns
- [x] Row click
- [ ] Row expansion
- [ ] Sortable columns
- [x] Sticky columns
- [x] Virtualized

Initial setup and support for web components is complete and examples for some of the less complex extensions have been added. I'll focus next sprint on some of the more complex extensions (ie editable cells, filtering, etc.).
https://github.com/matthewgallo/tanstack-carbon/tree/main/web-components


Is there a way to disable TLS Secure Renegotiation

Hello,
I need to make a test consuming and api with TLS Secure Renegotiation completely disabled.
For this I've configured the apache httpd 2 server with only TLS 1.2 enabled and I think it rely on openssl to discover if TLS Secure Renegotiation is available or not.
I've tried to edit the openssl.cnf file adding the following properties:
[system_default_sect]
Options = SSL_OP_NO_RENEGOTIATION

After this I've restarted the apache http server and then made the test with the command:
openssl s_client -connect localhost:443 -tls1_2

Even after this setup, the output of the command above still shows that Secure Renegotiation is enabled:
New, TLSv1.2, Cipher is ECDHE-RSA-AES256-GCM-SHA384
Server public key is 2048 bit
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : ECDHE-RSA-AES256-GCM-SHA384
    Session-ID: D9A880F24D13040E379D7BE35AFEFBDF5FE09CE408F0BE71195D1B441DC23098
    Session-ID-ctx:
    Master-Key: 03CB453428DE0DA3D537A56DBCF523AA520E75B20F2797EFDB99DCF2DF3304FD72F51C041C0523300C88887CCABD4B71
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    Start Time:<PHONE_NUMBER>
    Timeout   : 7200 (sec)
    Verify return code: 18 (self-signed certificate)
    Extended master secret: yes
---

Is there a effective way to disable it?

I think you should be specifying NoRenegotiation instead of SSL_OP_NO_RENEGOTIATION in the configuration.
This is documented but not in the most obvious place.

You cannot disable secure renegotiation. OpenSSL will always attempt to negotiate it. You can only influence what happens if the peer does not support it, or disable renegotiation completely. Even disabling renegotiation completely does not prevent the client and server from agreeing secure renegotiation support - it just prevents renegotiation from actually taking place.


Use any version available instead of throwing an error

So my problem currently with mise is that we have in our project .node-version and .ruby-version and instead of using available versions mise throws an error saying that mise ERROR Tool not installed for shim: node even though it is installed is just that .node-version contains 16, but I have 20 installed.
My feature request or question if that is already possible: how to still have warning about it, but fallback to currently installed version even though it might not match the one defined in the file?
mise ERROR Tool not installed for shim: node
Missing tool version<EMAIL_ADDRESS>Install all missing tools with: mise install
mise ERROR Run with --verbose or MISE_VERBOSE=1 for more information

So instead of an error here, I would love if mise still uses Node version which is 20 (I have installed), but gave me warning that it fallback and saying that project expects 16 as defined in .node-version.
mise ls
Tool    Version            Config Source                                Requested
bun     1.1.33             ~/.config/mise/config.toml                   latest
go      1.23.2             ~/.config/mise/config.toml                   latest
node    16.20.2 (missing)  ~/Developer/mobile-application/.node-version 16
node    20.18.0
python  3.13.0             ~/.config/mise/config.toml                   latest
ruby    2.7.5              ~/Developer/mobile-application/.ruby-version 2.7.5

I think this is a bad idea. You have a configuration file specifying a version, that should be the version that gets used. You can ignore idiomatic files if you want.

How I can then ignore it in mise?


Add the ability to specify MPN IDs when creating app registrations

Is your feature request related to a problem? Please describe.
We're currently reviewing the manual steps required for creating app registrations. We've managed to remove the vast majority of manual steps, however two prominent steps remain - setting a logo (covered by #153) and setting our MPN ID on the registration. Currently we get the following error when attempting to do this:
{
  "code": "DeploymentFailed",
  "target": "/subscriptions/<id>/resourceGroups/DS-Infrastructure/providers/Microsoft.Resources/deployments/test",
  "message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.",
  "details": [
    {
      "code": "Forbidden",
      "message": "verifiedPublisher properties cannot be set during Application creation. Graph client request id: 802f3c5e-18df-46aa-85b9-cb522bf1c7a9. Graph request timestamp: 2024-12-18T03:38:24Z."
    }
  ]
}

with the following Bicep file:
extension microsoftGraphV1_0

resource appReg<EMAIL_ADDRESS>= {
  uniqueName: 'test-registration'
  displayName: 'Test Reg'
  verifiedPublisher: {
    verifiedPublisherId: '<id>'
  }
}

Describe the solution you'd like
It would be great to be able to specify the MPN ID on our registrations in the Bicep script.

@SheaSmith thanks for filing this issue.  Setting the verified publisher is accomplished through a separate API from /applications - https://learn.microsoft.com/graph/api/application-setverifiedpublisher?view=graph-rest-1.0&tabs=http.  This is something we were not aware of on the Graph Bicep team.
Not sure why this was separated out into a service action on a different endpoint, but it definitely represents a problem for us to expose the functionality in a declarative and idempotent fashion.
We'll need to sync with the App Platform team (who own these Graph APIs) to understand what the options are here.


Fix index functions

Index functions don't really work well.  I am working on fixing them in a branch called funky-index-functions.

I kind of solved this.  There may be some optimisations left to do, by which we can use memcpy more in the code generator.


Add more icons

Closes #13
This PR

adds all element icons
removes the markers for now
displays the header icon into a 32x32 viewbox

Don't know what's wrong with Codecov here...

I'll double-check. Probably some spelling mistakes with the types.

I updated the identifiers to show the correct icons.

I updated the identifiers to show the correct icons. I also removed the legacy "default" icon, which was the old process icon. For now, nothing will be displayed if no icon was found.
The only missing icon seems to be for Collaboration though.

I added the missing icon, thanks to @andreasgeier (cf. https://github.com/camunda/camunda-modeler-design/issues/21)

I found a functional issue. When dropping the first pool to the canvas, the participant is selected but the properties panel shows the collaboration props. This happens only for the first pool, additional pools will show up in the panel correctly.

Let's open an issue for that 👍

Conditional flow icon is not showing up

default flow is missing as well

With https://github.com/bpmn-io/properties-panel-next/pull/56/commits/b204eb98b6be36f2603fb96030771c35d04e4e0f I added support for conditional + default flows. The icons were already present, just the detection failed.

Should be ready to review + merge now @barmac @MaxTru @andreasgeier. To not blow up this PR even further, let's consider open issues for further improvements (as missing icons).

Is this expected that the icon changes its position for a super-long name of the element?

Probably not. I fixed that to ensure the display box of the icon is stable.


Create an endpoint to find Monthly EMI Calculation

What bug or feature you wants to report.
No response

I would like to work on this issue


Implement entity liquid physics

Resolves #319.

There seem to be some issues that need to be worked out,

Items appear to be pushed by liquids faster than in vanilla Minecraft
Items get stuck when trying to both turn and go down

@DaPigGuy Any update on this?

Behavior may have changed in 1.19.30


Revisit Functional Test Pattern Part 5: New Begginnings

Switch relational specification tests to SharedStoreFixtureBase
Make cross store test to actually share a model
Reenable SqlServerDatabaseCreatorTest on CI
Part of #9098

This has merge conflicts because of my ongoing changes in reveng tests.
Can you leave reveng tests (perhaps design.Specs.tests) as is since most of them are going away.


GH-35170: [CI][Packaging][Conan] Build grpc-proto

Rationale for this change
Because Conan doesn't provide suitable binary.
What changes are included in this PR?
Build grpc-proto instead of using binary.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

@github-actions crossbow submit -g conan

+1


Selected text label for enhanced select components

RMWC Version 8.0.3
The enhanced version of the <Select> component allows for labels to be JSX elements in addition to strings. For example, the code below:
      <Select
        enhanced
        options={[
          { label:

Cookies

Example 1

, value: "1" },
          { label:

Pizza

Example 2

, value: "2" },
          { label:

Icecream

Example 3

, value: "3" },
        ]}
        value="1"
      />

When used this way, however, there doesn't seem to be a way to control what is displayed as the text when the menu is closed.
Binding something to value would have controlled this at one point, but the bug fix in #801 changed the code so that it will always find the selected item and use the textContent of the node for the selected text. Given the example above, Cookies Example 1 would end up being displayed in the UI.
The desired use case would be to have a more complex menu item structure for display but to still use a simple string representation for the collapsed state.
There are two possibilities I've considered for fixing this:

Adding an optional display text field to the options

Updating the FormattedOption definition for selects include a selectedDisplayText field
In the foundation code, if the selected option has selectedDisplayText set use it instead of the textContent

Adding an optional function to set the selected text

Updating the SelectProps to include a function that accepts a FormattedOption and returns a string
In the foundation code, if the component has a text selection function use it to determine the text to display instead of textContent

Before making a change and submitting a PR:

Is this functionality already possible without a change?
If not, which of the two approaches best matches usage patterns of the library?

@ajalexander Sorry for not getting back to you sooner. We've been occupied with the release of version 14. While we hope to address your issue soon, there are still a few bugs on our priority list that need attention. Feel free to submit a pull request – contributions are always welcome!


How to get date onSelectSlot

Hello,
I am trying to store Date onSelectSlot. Actually, When User click on any date in Month view from calendar I want to store that date in localstorage. could someone  please help me how to solve this problem
Thanks

Try looking at onDrillDown - that one is fired when user click the date on month/week view.
If you want to stick to onSelectSlot:
const handleSelectSlot = ({start,end,resourceId}) => {
    console.log("Selected", start, end);
    writeToLocalStorageHere(start);
}
...

return (
  <Calendar
      onSelectSlot={handleSelectSlot}
     ...
   />)


[Feature]: Select Widget allows entry of free text

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
A select widget requires selection of one of the alternatives listed.
However, there is a use case for such a widget that also allows entry of free text,  i.e., text that is not in the list of
options.
For example,  I have a database of articles by member organisations.  When entering an article,  the member organisation is well-defined and should be a "constrained" selection.  But the authors change frequently.  A member might forward the request to provide an article to an individual I have never seen before.  With the current functionality,  I have to make the Author field a free text entry,  but it would save me time if it was a list of everybody who was previously an author, and allow me to add an author not on that list.
Why should this be worked on?
It saves time for users of this feature,  adding to the power of the user-interface.

Thanks for the feature request. We're tracking this here #6928

For what it's worth,  I did find a workable workaround.
I have a select widget to select among existing authors.   I add an "" entry to the select options.
I add an onSelectChanged javascript to check for "" and show a modal for free text entry of a value.  This is written to the table.updatedRows entry.  This gets sent to the backend by the multiple-row save javascript function.
The only awkwardness with this is that "" is shown in the table after a value has been supplied by the modal.  This placeholder is replaced by the user supplied value once the change is written to the backend,  and once the updated dataset has been read.


plugin.query should have deny win conflicting parent access settings

say user A has a two teams t1 and t2 , and user B was given view-user access to t1 but denied view-user access to t2. user B should then be denied view access to user A. This is currently the case in the document methods such as $isAllowed but does not work correctly in plugin.query.


Add autolabel to production

Summary
I would like to modify the netlify CMS UI look and feel on a client by client basis by dropping a CSS file in my admin/index.html file.
React emotion computes hashed class names.
const brownStyles = css({ color: 'brown' }) // css-1q8eu9e

Adding autoLabel: true to the config automatically includes the label property to styles so that class names generated by css or styled include the name of the variable the result is assigned to.
const brownStyles = css({ color: 'brown' }) // css-1q8eu9e-brownStyles

With the label included in the class name I can tweak the Admin theme at my own discretion.
Test plan
Labeling the CSS class names shouldn't have any overhead or cause any issues to my knowledge?

I thought this would have an obvious impact on our bundle size, but it doesn't at all. Will merge, hope it helps 👍


added property page template to WordPress pages

This is a default field in the WP API for pages that allows to check whether a template was assigned to a page.

Code looks fine, and all tests are passing for me. I haven't tested this on a live site but I'm not sure how I hook it up to do that, someone may need to show me at some point

Thanks @mbklnd


Bacalhau fails to boot cluster due to new persistent node store

error creating node: failed to create node info store using NATS transport connection info: failed to create key-value store: context deadline exceeded

The nodes are stuck in a boot loop on staging, all failing with this message after a few seconds.
% gcloud compute ssh bacalhau-vm-stage-1
$ sudo journalctl --follow --unit=bacalhau

This is currently happening because we set BACALHAU_NODE_NETWORK_CLUSTER_PEERS to be a single address, which is the address of the current machine.  Jetstream doesn't like seeing an empty cluster setting, or a cluster containing the local node - it attempts to set up a RAFT network and fails.
To fix this I've removed the variable on the staging machine until I can update the relevant bit of ops code.


HPC-8242 - FTS Admin setup

Moving this to Pending prior merge as it depends on HPC-8326 now that has been moved to a separate PR


Setup with NG-CLI 7

Hello,
I have done an integration of that lib into an Angular 7 project but everytime I launch my e2e tests, it gets errors like : http//localhost:4200/ngapimock/runtime.js cannot be found 404...
Is there something i'm missing here ?

I don't know if it will help you but you might find some things this repo that I put together here with the help of @mdasberg it works as intended and should help guide you in a good direction.

thanks @Local9 ! that was very useful for me

Someone has a example project with Angular 9 and Protractor 5?

@vinibas and @rall I have updated my repo to now use the latest version of Angular.


dev: test util in indexer to generate Starknet calldata

Description
In our Typescript indexer, we should introduce a utility function that generates random StarkNet calldata, providing a comprehensible implementation for each element of the array.
Currently, all the tests passes, but it's difficult to understand how the calldata is constructed and whether it is correct or not. The goal is to make this process more transparent and accessible.
Expected Feature
We need a function that:

Accepts key parameters (e.g., to, etc.).
Returns clean, well-documented calldata.
Makes it easy for developers, including first-time contributors, to generate test calldata confidently.

By having a clear and reusable utility function, we can ensure that the test cases are easier to understand, and contributors can generate calldata reliably without needing deep knowledge of the internals.
Additional Context
This utility function will help improve the clarity of the test suite, making it easier for developers to construct valid calldata and reducing the potential for errors.
We should replace the existing test calldatas by call to this function.
Related https://github.com/kkrt-labs/kakarot-rpc/pull/1414#issuecomment-2382374615

What is the status of this?
Including it in your next Tech strategizing workshop with @ClementWalter

What is the status of this @tcoratger

Not prio right now since this is just "a nice improvement" and not something we absolutely need right now. Will spend more time on it soon.


Update to Erlang 26, Elixir 1.15 and Node 20

📖 Description
Update to the latest LTS version of every element of the stack.

Erlang 26
Elixir 1.15
Node 20

🦀 Dispatch

#dispatch/elixir

🦀 Requesting reviewers for this pull request:

@MathieuLegault1 (reviewer for the elixir stack)


[KOGITO-9322] Setup OCP PR checks

https://issues.redhat.com/browse/KOGITO-9322

PR job #4 was: FAILURE
Possible explanation: Pipeline failure or project build failure
You can find test error logs here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/4//artifact/test/logs/error/
Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/4/display/redirect
See console log:

Console Logs
Copying config sha256:5ab747fa31333fbb468b114a3a35aa2f9ebc3c09781e4f8de107f7c8c68675e1Writing manifest to image destinationStoring signatures--> 5ab747fa313Successfully tagged localhost/kogito-serverless-operator:2.0.0-snapshotSuccessfully tagged localhost/kogito-serverless-operator:latest5ab747fa31333fbb468b114a3a35aa2f9ebc3c09781e4f8de107f7c8c68675e12023-06-12 12:36:50,546 podman.py:62         INFO  Image built and available under following tags: kogito-serverless-operator:2.0.0-snapshot, kogito-serverless-operator:latest2023-06-12 12:36:50,547 cli.py:501        INFO  Finished!podman tag kogito-serverless-operator image-registry.openshift-image-registry.svc:5000/openshift/kogito-serverless-operator:pr-644a8fb[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Push Operator Image(s) to Openshift Registry)[Pipeline] tool[Pipeline] envVarsForTool[Pipeline] withEnv[Pipeline] {[Pipeline] sh+ set +xLogin<EMAIL_ADDRESS>line 3: syntax error near unexpected token `('[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Run e2e tests)Stage "Run e2e tests" skipped due to earlier failure(s)[Pipeline] stage[Pipeline] { (Running e2e tests)Stage "Run e2e tests" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] }[Pipeline] // script[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/4/api/json[Pipeline] readJSON[Pipeline] sh

PR job #6 was: FAILURE
Possible explanation: Pipeline failure or project build failure
You can find test error logs here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/6//artifact/test/logs/error/
Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/6/display/redirect
See console log:

Console Logs
openshift-network-diagnostics                      network-check-target-tzkft                                        1/1     Running     0             16dopenshift-network-diagnostics                      network-check-target-v26kv                                        1/1     Running     0             16dopenshift-network-operator                         network-operator-6b7968774f-mthnz                                 1/1     Running     0             16dopenshift-operator-lifecycle-manager               catalog-operator-7b5d974c6b-lznd7                                 1/1     Running     0             16dopenshift-operator-lifecycle-manager               collect-profiles-28109535-s86cr                                   0/1     Completed   0             38mopenshift-operator-lifecycle-manager               collect-profiles-28109550-qq4s2                                   0/1     Completed   0             23mopenshift-operator-lifecycle-manager               collect-profiles-28109565-ftv48                                   0/1     Completed   0             8m23sopenshift-operator-lifecycle-manager               olm-operator-f5b9db4c8-wm2w5                                      1/1     Running     1 (3d ago)    16dopenshift-operator-lifecycle-manager               package-server-manager-7f54df8479-nr24r                           1/1     Running     0             16dopenshift-operator-lifecycle-manager               packageserver-cccd79994-jzxcr                                     1/1     Running     0             16dopenshift-operator-lifecycle-manager               packageserver-cccd79994-qwcdp                                     1/1     Running     1 (3d ago)    16dopenshift-roks-metrics                             metrics-9fd6fd97f-n2d2v                                           1/1     Running     0             39dopenshift-roks-metrics                             push-gateway-7dd6b9c999-ffczz                                     1/1     Running     0             39dopenshift-service-ca-operator                      service-ca-operator-7c99c77499-8c6xd                              1/1     Running     0             16dopenshift-service-ca                               service-ca-5fd7645d7b-cg4pv                                       1/1     Running     0             16dteste                                              httpd-sample-7bdf74f5cd-b2rbf                                     1/1     Running     0             69dtigera-operator                                    tigera-operator-7cb9f67f44-7nwkl                                  1/1     Running     0             39d[Pipeline] }[Pipeline] // scriptPost stage[Pipeline] archiveArtifactsArchiving artifacts[Pipeline] junitRecording test results[Checks API] No suitable checks publisher found.[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // timeout[Pipeline] }[Pipeline] // stage[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] }[Pipeline] // script[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/6/api/jsonNo test report files were found. Configuration error?[Pipeline] readJSON[Pipeline] sh+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/6/consoleText+ tail -n 50

PR job #7 was: FAILURE
Possible explanation: Pipeline failure or project build failure
You can find test error logs here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/7//artifact/test/logs/error/
Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/7/display/redirect
See console log:

Console Logs
openshift-multus                                   network-metrics-daemon-8rk87                                      2/2     Running     0             16dopenshift-multus                                   network-metrics-daemon-p9vdk                                      2/2     Running     0             16dopenshift-network-diagnostics                      network-check-source-64d5ff5774-sntpw                             1/1     Running     0             16dopenshift-network-diagnostics                      network-check-target-tzkft                                        1/1     Running     0             16dopenshift-network-diagnostics                      network-check-target-v26kv                                        1/1     Running     0             16dopenshift-network-operator                         network-operator-6b7968774f-mthnz                                 1/1     Running     0             16dopenshift-operator-lifecycle-manager               catalog-operator-7b5d974c6b-lznd7                                 1/1     Running     0             16dopenshift-operator-lifecycle-manager               collect-profiles-28109565-ftv48                                   0/1     Completed   0             30mopenshift-operator-lifecycle-manager               collect-profiles-28109580-xhj8t                                   0/1     Completed   0             15mopenshift-operator-lifecycle-manager               collect-profiles-28109595-hvbxt                                   0/1     Completed   0             37sopenshift-operator-lifecycle-manager               olm-operator-f5b9db4c8-wm2w5                                      1/1     Running     1 (3d ago)    16dopenshift-operator-lifecycle-manager               package-server-manager-7f54df8479-nr24r                           1/1     Running     0             16dopenshift-operator-lifecycle-manager               packageserver-cccd79994-jzxcr                                     1/1     Running     0             16dopenshift-operator-lifecycle-manager               packageserver-cccd79994-qwcdp                                     1/1     Running     1 (3d ago)    16dopenshift-roks-metrics                             metrics-9fd6fd97f-n2d2v                                           1/1     Running     0             39dopenshift-roks-metrics                             push-gateway-7dd6b9c999-ffczz                                     1/1     Running     0             39dopenshift-service-ca-operator                      service-ca-operator-7c99c77499-8c6xd                              1/1     Running     0             16dopenshift-service-ca                               service-ca-5fd7645d7b-cg4pv                                       1/1     Running     0             16dteste                                              httpd-sample-7bdf74f5cd-b2rbf                                     1/1     Running     0             69dtigera-operator                                    tigera-operator-7cb9f67f44-7nwkl                                  1/1     Running     0             39d[Pipeline] }[Pipeline] // scriptPost stage[Pipeline] archiveArtifactsArchiving artifacts[Pipeline] junitRecording test results[Checks API] No suitable checks publisher found.[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // timeout[Pipeline] }[Pipeline] // stage[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] }[Pipeline] // script[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/7/api/json[Pipeline] readJSON[Pipeline] sh

Openshift tests job #10 was: UNSTABLE
Possible explanation: This should be test failures
Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/10/display/redirect
See console log:

Console Logs
openshift-multus                                   multus-admission-controller-bw8wj                                 2/2     Running     0              16dopenshift-multus                                   network-metrics-daemon-8rk87                                      2/2     Running     0              16dopenshift-multus                                   network-metrics-daemon-p9vdk                                      2/2     Running     0              16dopenshift-network-diagnostics                      network-check-source-64d5ff5774-sntpw                             1/1     Running     0              16dopenshift-network-diagnostics                      network-check-target-tzkft                                        1/1     Running     0              16dopenshift-network-diagnostics                      network-check-target-v26kv                                        1/1     Running     0              16dopenshift-network-operator                         network-operator-6b7968774f-mthnz                                 1/1     Running     0              16dopenshift-operator-lifecycle-manager               catalog-operator-7b5d974c6b-lznd7                                 1/1     Running     0              16dopenshift-operator-lifecycle-manager               collect-profiles-28109595-hvbxt                                   0/1     Completed   0              42mopenshift-operator-lifecycle-manager               collect-profiles-28109610-nm92b                                   0/1     Completed   0              27mopenshift-operator-lifecycle-manager               collect-profiles-28109625-nxwmw                                   0/1     Completed   0              12mopenshift-operator-lifecycle-manager               olm-operator-f5b9db4c8-wm2w5                                      1/1     Running     1 (3d1h ago)   16dopenshift-operator-lifecycle-manager               package-server-manager-7f54df8479-nr24r                           1/1     Running     0              16dopenshift-operator-lifecycle-manager               packageserver-cccd79994-jzxcr                                     1/1     Running     0              16dopenshift-operator-lifecycle-manager               packageserver-cccd79994-qwcdp                                     1/1     Running     1 (3d1h ago)   16dopenshift-roks-metrics                             metrics-9fd6fd97f-n2d2v                                           1/1     Running     0              39dopenshift-roks-metrics                             push-gateway-7dd6b9c999-ffczz                                     1/1     Running     0              39dopenshift-service-ca-operator                      service-ca-operator-7c99c77499-8c6xd                              1/1     Running     0              16dopenshift-service-ca                               service-ca-5fd7645d7b-cg4pv                                       1/1     Running     0              16dteste                                              httpd-sample-7bdf74f5cd-b2rbf                                     1/1     Running     0              69dtigera-operator                                    tigera-operator-7cb9f67f44-7nwkl                                  1/1     Running     0              39d[Pipeline] }[Pipeline] // scriptPost stage[Pipeline] archiveArtifactsArchiving artifacts‘test/logs/**/*.log’ doesn’t match anything: ‘test’ exists but not ‘test/logs/**/*.log’No artifacts found that match the file pattern "test/logs/**/*.log". Configuration error?[Pipeline] junitRecording test resultsNone of the test reports contained any result[Checks API] No suitable checks publisher found.[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // timeout[Pipeline] }[Pipeline] // stage[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/kogito-serverless-operator-PR-check/10/api/json[Pipeline] readJSON[Pipeline] sh

 error: error validating "/home/jenkins/agent/workspace/kogito/kogito-serverless-operator-PR-check/test/testdata/order-processing/05_sw.kogito_v1alpha08_kogitoserverlessworkflow_devmode_orderprocessing.yaml": error validating data: ValidationError(KogitoServerlessWorkflow.spec.flow): missing required field "specVersion" in org.kie.kogito.sw.v1alpha08.KogitoServerlessWorkflow.spec.flow; if you choose to ignore these errors, turn validation off with --validate=false

known issue and should be solved in https://github.com/kiegroup/kogito-serverless-operator/pull/154


Remove gardener-extension-provider-* dependencies from KMC component

Description
Gardener apis are used inside the KMC component to count the number of virtual networks.
This results in several repeated security findings. To make our life easier, we should get rid of the gardener dependency.
Reasoning

Gardener pulls in many dependencies such as helm, kubernetes etc (see gardener go.mod file).
We do only use the pkg/apis folder of the gardener project because we want to use a data structure.

The code using gardener-extension-provider-* libs
// source: https://github.com/kyma-project/control-plane/blob/d1a89b5c1326ec574146b4a65f17b021455d049e/components/kyma-metrics-collector/pkg/process/event_stream.go#L113
		case Azure:
			// same pattern as for GCP
		case AWS:
			// same pattern as for GCP
		case GCP:
			decoder := serializer.NewCodecFactory(scheme.Scheme).UniversalDecoder()
			infraConfig := &gardenergcpv1alpha1.InfrastructureConfig{}
			if err := runtime.DecodeInto(decoder, rawExtension.Raw, infraConfig); err != nil {
				return nil, err
			}
			if infraConfig.Networks.VPC != nil && infraConfig.Networks.VPC.CloudRouter != nil {
				vnets += 1
			}

Solution
Let's use a dynamic k8s client (k8s.io/client-go/dynamic)  together with a unstructured.Unstructured struct to access the virtual network. There is an example PR for removing gardener from KEB component.

I briefly looked into this and here are my conclusions.
What's wrong?
KMC uses gardener (and gardener-extensions for AWS, GCP and Azure). There are some concerns about security since gardener uses a lot of dependencies that in theory also increases the used dependencies of KMC itself. At least code scanning could get influenced by the amount potential dependencies.
The interesting part is, that we hardly use any functionality of gardener:
It looks like all the interactions in KMC take place at
control-plane/components/kyma-metrics-collector/pkg/process/event_stream.go
For the extensions KMC uses the struct InfrastructureConfig that allows us to communicate with gardener clusters here (azure), here (aws) and here (gcp). In the line underneath this struct gets send to DecodeInto(). For this the struct needs to implement the Object interface which means it needs the DeepCopy and DeepCopyInto methods which usually gets autogenerated. If one just copy-pastes the struct into a new module all the autogeneration code breaks. So, this autogeneration instructions would need some adjustments.
For plain gardener KMC also uses a struct dubbed shoot in the the aforementioned event_stream.go file. At different places in this file different fields of shoot get read. In control-plane/components/kyma-metrics-collector/pkg/process/process.go the value for shoot gets written. For now to me it's unclear, if there are similar limitations as for the extension's structs.
What might be the solution?
Copy the code into a new repo, delete as much unneeded code (and thus dependencies) as possible, make the autogeneration of deepcopy work and uses this in KMC instead.