#1579 Add Redis backend environment variables

What does this pull request aim to achieve? 💯
Adding backend environment variables for Redis, which will point to our ElastiCache cluster. The cluster has been provisioned in different Terraform code.
Branched off of feature/message-modal/1457 and will be merging into that branch so that it can be used by Socket.IO for direct messaging.
Closes #1579

[x] I have checked that no one else is working on similar changes.
[x] I have read the latest README and understand the development workflow.
[x] The name of this branch is: feature/<branch_name>, or hotfix/<branch_name> if this is a hotfix, where <branch_name> briefly describes the issue(s) resolved and is prefixed with the issue number(s) (e.g. 1127-1130-update-git-branching-model). The branch is created in a cloned version of this repo, not a forked repo.
[x] This branch is rebased/merged with the latest staging (or the latest production if it is a hotfix).
[x] There are no merge conflicts.
[x] There are no console warnings and errors.
[x] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".
[x] The title describes the issue(s) being addressed using format: <issue_numbers> - <brief_issue_description> (e.g. #1127/#1130 - Update Git Branching Model)

@Zaydme @kterry1 Feel free to merge this into your branch so you can test out private messaging in the review environment!


Remove Flex from Platform

remove the flex-box structure from the Platform UI that was overlaid onto the grid system.

resolves #372

completed by the merge of fabric8-ui PR #1419


[Documentation] Pairing without the Intex App

Hi, not sure if this is a good place to post this, but I wanted to share my findings somewhere.
We bought a intex spa with wifi (SB-HSWF20), but sadly their app is terrible and cloud based.
I just wanted the spa in Home Assistant.
After a bit of poking and prodding, I managed to get it connected without the intex app.
Steps: TLDR;

The Intex Spa uses ESPTouch (from Espressif), pair it using some open source app using broadcast option (not multicast)
When the spa is paired, it requires something to listen to it's ports or it seems to disconnect

How I got it to pair without the intex cloud and app:

Using: https://github.com/EspressifApp/EsptouchForAndroid

I ran the EspTouch to pair the spa
After it connected, the wifi light did not stay lit :-(
Either my wifi connection was poor, or some kind of handshake was needed
The spa device disconnected from the wifi :-(

Notes on EspTouch:

Requires local network data in order to broadcast the details to the spa
(I'm not sure how this protocol operates, but this is what I concluded) as having wifi data disabled prevented the app from working.
I blocked all internet for my phone on my router, just to be sure no info could leak

In Home Assistant - HACS : find and install the Intex intergration

In HA prepare all the data to add the new intex spa device (but wait with connecting)
Press and hold the C/F Wifi button on the spa unit, the wifi will start blinking for a minute
Use the EspTouch app to connect the unit again
As soon as it connects, connect the HA intex device setup, the wifi light will remain lit

I figure this out today by taking a peek in inside of the Management APK to check what they were doing and what the protocol was being used.
I found the espressif ESPTouch code, my spa is using V1 not V2.
Perfect, their app is worthless anyways and now my spa is not connected to the internet, only to my local HA network.
Thanks for this

Very interesting, thanks @FreezyExp !
I may add it the docs, and add a page to mathieu-mp/intex-spa python package. It could help people !

perhaps you could even find a python implementation for the ESPTouch code
Seems like others have already done that work, but... as it is a onetime setup that requires a wireless interface, perhaps declaring it as a setup step to hint on how to pair it would work just as well without adding third party dependencies.
https://github.com/KurdyMalloy/EsptouchPython
https://github.com/CaptainDouche/esp_smartcfg_linux

This seems far outside the scope of this integration. More, the python package is an alpha version.
I will suggest this path to the user in the documentation, but I won't add this feature.

hello,
I also have an SB-HSWF20, I managed to configure it directly and integrate it into HA thanks to your documentation
Could you tell me if you can manage it with automation?
When I try to automate, the SPA shuts off after 2 hours
Thanks

Could you tell me if you can manage it with automation?
Yes !
When I try to automate, the SPA shuts off after 2 hours
The duration is set on the panel control side, not on the integration side. Use the spa panel control to change the duration of the feature you want to automate. Look for the HST/HDT settings in the user manual.
After that, the HDT will be reused when the feature is triggered from the integration.


Longitude & latitude attributes of stream entity break map in sidebar - C210

Description
Because the HD and SD stream have Longitute and latitude values in the attributes they are messing with the map in the sidebar. The map is completely zoomed out and you see the whole world.
As a test I removed the values from the attribute via the developer settings and the map started to work again.
It would already help to rename these values.
Reproduction Steps

Enable the HD or SD Stream in Home assistant.
Via the left pane open the Map
The map is completely zoomed out and unusable

Expected behavior
The stream entity should not mess with the map.
If applicable, add error logs.
No response
Device Firmware
1.3.0 Build 220830
Using stream component
Yes
Does camera work via official integrations?
N/A
Camera has all attributes filled out in developer tools
Yes
HASS Environment
Home Assistant OS on ODROID N2+
Search for similar issues
Yes
Additional information

What camera is this? My C200s do not expose anything like this. Is this possible to be set up via the Tapo app?

The camera is set up via the Tapo app.
The properties look like this

longitude: 164903
latitude: 484042

I confirm the same behaviour, I have both a C200 and C210 and if the C210 is enabled the map behaves as described.

I have the same issue with C225

Managed to get rid of the location, I've rejected Tapo App location permissions, deleted the camera from Tapo app, and re-added it. Then I reconfigured Tapo Control in Home Assistant.
Looks like it sets the location when you configure the camera in Tapo app, which makes sense to me since this camera is near a window and the other is deep inside the house where there is no GPS signal and was the first I've configured.

Fixed in https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/releases/tag/5.1.6 available with May HA release.


build(deps): bump react-native from 0.69.10 to 0.69.12

This PR resolves []
Description
Bumps react native 69 to latest version because they fixed the Oppo and 1Plus device crashes from runAnimationStep.
https://github.com/facebook/react-native/issues/35766
Also makes XCode 15 work with eigen.
⚠ Had to run cd ios && bundle exec pod update hermes-engine --no-repo-update due to this react native issue: https://github.com/facebook/react-native/issues/36945 to make CI pass (TLDR; podfile.lock wasnt updating hermes correctly on podfile.lock) and update podfile.lock file (resulted in the last commit diff)

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

bump react-native from 0.69.10 to 0.69.12 - gkartalis

Need help with something? Have a look at our docs, or get in touch with us.

Will prob merge on the next release to have two weeks time to see if any issues arise out of this

I just have a thought of adding the command to package.json, and removing it when we upgrade to 72 or above
Just in case we upgrade to a version that's lower than 72 to update hermes on pod-install 👀

I just have a thought of adding the command to package.json, and removing it when we upgrade to 72 or above
Just in case we upgrade to a version that's lower than 72 to update hermes on pod-install 👀

What do you mean exactly? Doing it on which step?

What do you mean exactly? Doing it on which step?

I was thinking either in yarn pod-install or inside post-pod-install.rb script, but maybe it won't be needed

@MrSltun Yeah, thought of something like that, but that will add exponentially more time every time that we need to run anything that includes this extra step which will only be needed to run on react-native upgrade prs. I would say lets leave this  out for that reason unless you can think of another workaround on this?

@gkartalis I'm with leaving it out! It's quite an edge case :)


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit 22fa6cc:
:clock10: Full build: the average full build time is 6 min(s) 5 sec(s), this is based on the last 346 build(s) for this repository.
:warning: This is a FULL build due to the incremental build being disabled in configuration file (.openpublishing.publish.config.json in 'live' branch). For more detailed info, please click here.

Docs Build status updates of commit 22fa6cc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Abstract and fix draining

Centralize draining logic in one helper function
Fix inbound draining (HBONE).  This meant new connections would go to the old
ztunnel on a ztunnel restart.
Simplify inbound draining; do not re-create the force shutdown logic,
and instead let the common abstraction do it (which does it slightly
better)
socsk5: add propery draining with force shutdown. Remove double-spawn,
which adds some complexity around the proxy_to_cancellable.

This is primarily tested in https://github.com/istio/istio/pull/51710,
which sends a large stream of requests and restarts ztunnel and the
backend app (2 different tests). With this change, these tests pass.
It would be good to get more isolated tests in this repo in the future
as well

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
315efe75b55070485c0ada9fc917ced6eeef2e40
link
true
/test test

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

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
00a2c29d50f9a76540e09696494d981d2b1abaad
link
true
/test test

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

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
c0c1732118e2183ad9090cb4ad50cf74efa923e5
link
true
/test test

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

I think most of this is good to go regardless but in light of https://github.com/istio/istio/pull/51805 I might change do some work to not drain on pod shutdown, only ztunnel. or I can do that in a followup, either one works forme

I think most of this is good to go regardless but in light of istio/istio#51805 I might change do some work to not drain on pod shutdown, only ztunnel. or I can do that in a followup, either one works forme

Fair enough!

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
a2b20c685bf83ffe7a4ead3c55bac9460dea6abe
link
true
/test test

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

/test all

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
77b41e2dc7c6e42f4b5eacfe64a6c3a941697ba2
link
true
/test test

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

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
caffa42131332e48333bb394235d5a48354bfb38
link
true
/test test

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

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
1d809d7c4cb58c1a68e1707be05710f6ff032246
link
true
/test test

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

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
54468b8c4de0ee26dfc05c41d74c4f811e3ff01d
link
true
/test test

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


[WIP]Service to provide Nuget with info to Run Nuget restore

This is the initial stab to create a service, which will provide information to Nuget to perform a Nuget restore.
The APIs will change as I keep introducing things.
Things to do:
[ ] List of Project References
[ ] List of Package References
[ ] Intermediate folder path
This PR wont be checked in until I have all the pieces ready.

Before you go to far with this - please wait for our sync-up tomorrow with NuGet.


fix: remove cron monitoring

This didn't appear to work.
Looks like recently merged by @cleptric in https://github.com/getsentry/sentry-laravel/pull/659 and can't quite spot my mistake. Macros must not be loading somehow.
https://leaf-hi.sentry.io/share/issue/0971d4b094e54beca3fe5a78b3a1581d/

Will revisit in weekend.

@iBotPeaches We just released version 3.3.1 which should fix the issue you're seeing.


Collect compiler info for both languages C/C++

Collect compiler includes and standards for both
language (C/C++) independent from the actual compilation
command and select the right values at analyzer command
build time.
It is possible that in a compile command json the same
compiler binary is used to build C and C++ source.
In this case the include paths and the standard will be
detected for the first command but it will be used for the
command too which is wrong.

New PR #2193 was created, we can close this.


Any paper or blogpost to introduce the working mechanism?

Issue with current documentation:
No response

Hey @hijkzzz We are working on it, will be released soon.


Select End of Range Only

When using the SelectRanges, is it possible to pre-define the startDate so the user only needs to pick the endDate?
I've two inputs (start and end) separately and would like to give the user the change to click on end and edit only the final value... is somehow possible to achieve those?

Yes, there's a startDate prop you can use.


Update block_and_allow.py

ALLOW filter does not take an entity_type.

I've tested with this command:
python block_and_allow.py

and got
['My name is [NAME_1] and my friend is [NAME_2] and we live in Barcelona, not [LOCATION_CITY_1]. [CUSTOM_ENTITY_1]']
```.


Add per-user postgres db for awi-ciroh

Add per-user postgres db
Document how to set this up

Ref https://2i2c.freshdesk.com/a/tickets/175

When this is done, users can access their personal postgres server with psql -h localhost from the commandline, or passing localhost as hostname in any python connecting code.


Set pgp passphrase in buildSettings

I have tried multiple combinations for how to define this setting from
plugins but always failed. It seems this does the trick however, for
some reason I don't know why ¯_(ツ)_/¯
If this works, we can remove one step from the readme. Gonna manually test on CI first.

cc/ @2m this might avoid the pgpPassphrase step for build.sbt

Great!

scratch that, doesn't work https://travis-ci.org/olafurpg/sbt-ci-release/jobs/390678401#L538 I have no clue why because show pgpPassphrase is Some([.asda]). Anyways, back to the original copy-paste solution

Maybe the settings in the buildSettings still need the Global scope?
So it becomes:
override def buildSettings: Seq[Def.Setting[_]] = List(
  Global/PgpKeys.pgpPassphrase := sys.env.get("PGP_PASSPHRASE").map(_.toCharArray())
)

But I am not completely sure about that.


Fix process keeps running in the background

The image-roll process keeps running in the background after I close the window. It only happens if I use the open menu. It doesn't keep running in the background if I use the command-line to open an image or if I double-click on an image.
If I move the destroy call out of the file choosers signal handler then the bug goes away. It makes sense that the file chooser should not be destroyed until it is completely done, but I don't know exactly what is going on.
The code could alternative be re-written to not use a signal handler at all. The run method "blocks" by entering a recursive main loop so the result of the call can just be used directly instead (see issue #32). Let me know if you want that instead.
Fixes issue #32.

Merged! Thank you for finding and fixing this bug :)

Thanks for taking the time to merge my fix! 😄


docs: typo errors

Ah, sorry for the late reply. Can you fix the typo in packages/versions/README.md and run npm run docs to update other README.md? It should auto update this file (root readme) and the npm readme.


Fix warning in iron_juniper

The warning was:
warning: unnecessary parentheses around function argument
   --> juniper_iron/src/lib.rs:268:43
    |
268 |             _ => return Ok(Response::with((status::MethodNotAllowed))),
    |                                           ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove these parentheses
    |
    = note: #[warn(unused_parens)] on by default

Codecov Report

Merging #147 into master will decrease coverage by 0.39%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master     #147     +/-   ##
=========================================
- Coverage   91.28%   90.89%   -0.4%
=========================================
  Files          95       95
  Lines       16528    16202    -326
=========================================
- Hits        15088    14726    -362
- Misses       1440     1476     +36

Impacted Files
Coverage Δ

juniper_iron/src/lib.rs
67.4% <0%> (ø)
:arrow_up:

juniper/src/lib.rs
84.61% <0%> (-11.13%)
:arrow_down:

juniper/src/tests/model.rs
91.79% <0%> (-5.37%)
:arrow_down:

juniper/src/schema/model.rs
89.45% <0%> (-3.49%)
:arrow_down:

juniper/src/executor_tests/variables.rs
95.35% <0%> (-3.49%)
:arrow_down:

juniper/src/executor_tests/executor.rs
98.38% <0%> (-1.62%)
:arrow_down:

juniper/src/schema/meta.rs
84.52% <0%> (-0.9%)
:arrow_down:

juniper/src/validation/input_value.rs
91.58% <0%> (-0.47%)
:arrow_down:

juniper/src/executor/mod.rs
91% <0%> (-0.41%)
:arrow_down:

juniper/src/executor_tests/enums.rs
97.8% <0%> (-0.37%)
:arrow_down:

... and 13 more

Continue to review full report at Codecov.

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


SSL Client certificates

The documentation doesn't seem to highlight the risks of using the same SSL certificate on all clients.
Is this an issue that should be revised?  I'm going to add a notice about it in the demo readme for sensu-puppet, but I don't know if it should be highlighted in the demo docs?
I noticed sensu-chef might also be facilitating this issue, but haven't verified this.

I'm using the rabbitmq-auth-mechanism-ssl plugin on github to provide something similar to the desired functionality. https://github.com/rabbitmq/rabbitmq-auth-mechanism-ssl
I use the common_name functionality, which (unfortunately) means if a certificate is compromised that CN is burned forever.


(C)UDN CRD: update CEL and add IPAM section

based on https://github.com/openshift/enhancements/pull/1725
Whoever reviews that, I beg you to run some manual tests by applying CRs with various wrong and right configs checking the CEL rules work as expected. To do that, run a kind cluster with these changes, create namespace demo, then apply and delete various CR versions, e.g.
apiVersion: k8s.ovn.org/v1
kind: UserDefinedNetwork
metadata:
  name: db-network
  namespace: demo
spec:
  topology: Layer2
  layer2:
    role: Primary
    mtu: 1400
    ipam:
      mode: Disabled
      lifecycle: Persistent

should return
The UserDefinedNetwork "db-network" is invalid:
* spec.layer2: Invalid value: "object": Disabled ipam.mode is only supported for Secondary network
* spec.layer2.ipam: Invalid value: "object": lifecycle Persistent is only supported when ipam.mode is Enabled

cc @maiqueb @ormergi this is the IPAM changes PR, but it has a surprise part with some CEL tweaks :)

based on openshift/enhancements#1725 Whoever reviews that, I beg you to run some manual tests by applying CRs with various wrong and right configs checking the CEL rules work as expected.

It was a trap !!!

To do that, run a kind cluster with these changes, create namespace demo, then apply and delete various CR versions, e.g.
apiVersion: k8s.ovn.org/v1
kind: UserDefinedNetwork
metadata:
  name: db-network
  namespace: demo
spec:
  topology: Layer2
  layer2:
    role: Primary
    mtu: 1400
    ipam:
      mode: Disabled
      lifecycle: Persistent

should return
The UserDefinedNetwork "db-network" is invalid:
* spec.layer2: Invalid value: "object": Disabled ipam.mode is only supported for Secondary network
* spec.layer2.ipam: Invalid value: "object": lifecycle Persistent is only supported when ipam.mode is Enabled

Could you help me out by pasting the most complex UDN you can come up w/ ? JoinSubnets and everything ...

It was a trap !!!

I promise it wasn't supposed to be :D you can easily skip the CEL-changing commit especially if you are not an expert in CEL and just say it in the comment.


Fixed longitude wrapping bug

This fixes https://github.com/louis-e/arnis/issues/101

Needs further testing, not working as expected


Are there any best practices for configuration?

First, I want to say that as someone with no machine learning experience, I really like this project. It's ready to use out of the box, and the concept is very simple.
However, due to my lack of experience in machine learning, I don't have much expertise in how to configure each setting in the configuration options according to the current project's environment. For example, how many should be set for n_jobs, and what factors determine this value?
Is it possible to add documentation about configuration best practices?

Some of the configs are explained in the documentation:
https://gorse.io/docs/master/concepts/how-it-works.html


Handle "don't ask again"

If a user denied a permission, and checked the "don't ask again" checkbox, then shouldShowRequestPermissionRationale will return false. This only applies if the permission was already requested (because of the permission wasn't requested yet it'll also return false).
It would be cool to access that information from this library.

Hi @eygraber,
I'm aware that shouldShowRequestPermissionRationale function will return false for first permission request and for situation where "don't ask again" checkbox is checked. In order to distinguish if false value is returned because of checkbox or because of first permission request, app needs to have some inner state(flag isFirstPermissionRequest) and needs to handle changes of this state.
Since I designed this extension functions to be stateless this will require some bigger changes in design.
This would be really good enhancement for lib, so I'll definitely take a look how to provide this information in one of future lib versions.
Thanks for suggestion.


Update Config “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for festive-saha-f7160e processing.
🔨 Explore the source changes: e4127e28df26c1e5cde1f41694e18a0dd01deec8
🔍 Inspect the deploy log: https://app.netlify.com/sites/festive-saha-f7160e/deploys/618cec5711dc720008355aec


Published app in GooglePlay it still uses testing cards etc

It seems the application payments are still done vs google's test sandbox server because I can see only testing cards in the payment widget. This happens in the application published to google play and installed from the google play!
I went through the docs but I cannot see anything about it, so I expected the library handling it itself.
What am I doing wrong here, please?
Version Number of Plugin: 7.1.0
Device Tested On: Samsung S21FE
Version of VS: the latest now
Version of Xamarin: the latest MAUI .net 8

So I did some more checks and it seems like it works on other devices but mine. I was beta tester, testing the dev version on my device. And for some reason now the purchases on the production app still act like it was the testing version on my device :/ I already left the beta program and tried to reinstall the app. Still the same, I can still see only "Testing cards" during the purchases...

Correct, if you are in a beta group or linked account then it will show those. This is on google's side, not this library  https://developer.android.com/google/play/billing/test

Oh.... I thought it will just ADD these possibilities not that it will completelly remove the real cards :) I will remove my account from there then. Thank you, James! :)


fix LdsBlockSizePerPad=-1 assignment

LdsBlockSizePerPad=-1 assignment before rejection of incorrect config.

rocblas-test passed on gfx908, gfx906, gfx900
rocBLAS version: <IP_ADDRESS>1-4b7530c (new Tensile client)


Look at exposing IORING_CQE_F_SOCK_NONEMPTY

Indication that more data was left to read in a socket.

Simillarly SOCKET_URING_OP_SIOCINQ and SOCKET_URING_OP_SIOCOUTQ for sockets.


Multiple files/images don't work

If a add a new field with "Media" type to a schema and select Allow multiple values, GET /api/content/{schema} doesn't include that field in a response

Thank you, please wait while we are checking the problem.


Undefined reference to custom plugin

Hello,
I'm trying to use my simple MAVSDK plugin in my program, but I have problem defined here: https://discuss.px4.io/t/undefined-reference-to-custom-mavsdk-plugin/18070 which make using this plugin impossible.
Please answer on forum.

It is very difficult to tell you what's wrong in the code just from the information you are giving. Can you show the code that is not compiling? Can you explain what you did that resulted in this error message?

added explanation and source code on forum :)


Implemented draft of Finding data model, and some basic unit tests for it.

Signed-off-by: AWSHurneyt<EMAIL_ADDRESS>Issue #, if available:
Description of changes:
Implemented draft of Finding data model, and some basic unit tests for it.
CheckList:
[x] Commits are signed per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #260 (36c3ce5) into document-level-alerting-dev (ea7d526) will decrease coverage by 0.80%.
The diff coverage is 23.36%.

@@                        Coverage Diff                        @@
##             document-level-alerting-dev     #260      +/-   ##
=================================================================
- Coverage                          78.53%   77.72%   -0.81%
  Complexity                           218      218
=================================================================
  Files                                173      174       +1
  Lines                               6968     7075     +107
  Branches                             915      921       +6
=================================================================
+ Hits                                5472     5499      +27
- Misses                              1007     1088      +81
+ Partials                             489      488       -1

Impacted Files
Coverage Δ

...in/kotlin/org/opensearch/alerting/model/Finding.kt
23.36% <23.36%> (ø)

.../kotlin/org/opensearch/alerting/core/JobSweeper.kt
71.50% <0.00%> (-0.50%)
:arrow_down:

...in/kotlin/org/opensearch/alerting/MonitorRunner.kt
72.05% <0.00%> (+0.67%)
:arrow_up:

...lerting/transport/TransportExecuteMonitorAction.kt
73.33% <0.00%> (+2.22%)
:arrow_up:

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 ea7d526...36c3ce5. Read the comment docs.


APNG: note for edge

Could you add into Edge's note for APNG info about proposal in their feedback center?
Something like this:

Suggestion for impementation is registered in MS Developer Feedback center. You can vote for this suggestion

The link already is in the resources tab:

And since its already "under review", I don't think its necessary to place the link in a more prominent place…

Thanks, did not noticed it in resources, and still think notes is much better place for this.
It's "under review" since 2015. In this case that status means "we'll not implement it in this century unless it will become most popular suggestion in our dev feedback center", sadly.


Disconnect between player character and view

Yarm 0.7.15 with Factorio 0.12.29.
In multiplayer environment, monitor a pile of resources. Use "eye" icon to remotely view that pile, then use movement keys to try to pan screen.
Expected outcome: Panning should fail silently.
Actual outcome: Screen moves, immediate and repeated desync. If lucky enough to be able to disable remote view, desyncs stop. Deleting all YARM monitors not always possible, and now control of character is lost.

I'm sorry, I can't reproduce this -- using Factorio 0.12.29, a Linux headless server and a Windows client, with YARM 0.7.15, EvoGUI 0.4.14, and no other mods, I cannot move the fake player character when remote-viewing a resource site.
It is possible another mod you have is messing around with the player character definition -- as you can see in the YARM prototypes, the fake player should be unable to move. Can you make your set of mods available?

Only semi-relatedly, I'd love to know how this happened:
23:31:00 <Demosthenex> crap. so YARM has disconnected my view from my character
23:31:11 <Demosthenex> i can pan around the map, but not move my char

...Which I mention specifically because YARM has code to prevent you from remote-viewing a site when you're likely to get stuck in the remote view.

For future reference, there is a remote interface function that can reset you back to your character -- you can invoke it by typing /c remote.call("YARM", "reset_player", game.local_player.index) into the console and pointing your cursor at your player character before hitting Enter. Note that this is a debuggish feature and may let you do crazy things, so try not to misuse it.

I removed every one of my YARM monitors, and then had to reset my character using the mouse selection and lua. It's repeatable. I am on Linux both client and server side though. I just broke it again earlier. To be fair, there's two stages. If I can disable remote viewing during the desync loop, the site I was viewing stays stuck in a viewing state in the list. It was while purging my list of sites that I got completely disconnected. Unfortunately the logging messages only goto the in game chat, I don't see them on the console or the logs.
Mod list:
color-coding_1.0.7.zip
KS_Power_0.1.1.zip
Concrete_Lamppost_0.1.0.zip
Landfill_2.1.7.zip
rso-mod_1.5.0.zip
fluid-barrel_1.0.0.zip
Loot-able Rocks_1.0.0.zip
YARM_0.7.15.zip

We can always discuss on IRC. Feel free to PM.

Oh! While trying to fix it I found the cause of the complete player disconnect. YARM asked me to try and run reset_player, and when I did the camera completely disconnected.

Please forgive the slow response, I've found myself otherwise occupied this week-end.
When I remote-view a YARM site, I cannot move the fake player character I'm inhabiting.
None of the mods you're referencing should be messing around with YARM, either, so I can't even begin to imagine what's causing this behaviour in your case.

Slow response isn't an issue. You're providing an excellent mod for free! Thank you for investing your time.
As to the reproduction, if we can't reproduce it, let's just close the ticket. Thanks for trying.

All right; feel free to reopen if you ever find out what went wrong.
Can I ask you, for the sake of my own curiosity, whether you can reproduce it on your machine, in a new world, with only YARM (and, at that, with a fresh download of it, just in case)? You will probably find these useful for faster setup:
/c game.always_day = true
/c game.peaceful_mode = true
/c game.local_player.insert{name="resource-monitor"}


Make Tied Ranking Optional

Add config to make tied ranking optional. Why?

RDM gives the option for tied, aka Competition Ranking (1,1,3,4,5) as well as non-tied, aka Ordinal Ranking (1,2,3,4,5), so why not let the mapper choose between the two?
Tied ranking confuses both users and mappers since many of the main PvP IV sites like GO Stadium do not use tied rankings. This leads to the constant question of "why did the map say this was a rank 1 when it isn't a rank 1?"
As mappers, we should report the best mon to users. The only people who get excited about Functional Rank 1's are users who don't have a map. So I (personally) don't think tied-ranking should be the default.

Hey! Thanks for idea. As mentioned on the other service, I don't plan to work on it for longer time.
If anyone wants to pick it up, feel free, I'm open for customization with current defaults.

maybe @Fabio1988?

That's a great idea! Get him to stop working on a project for everyone to fulfill your desire. I'm sure the rest will be grateful as well! /s

Just pointing out a bug in the pvp logic that feeds into the main project. Don't think anyone wants Golbat to be inferior to RDM in any aspect, which it currently is due to ohbem's pvp restrictions.

Hey, would you like to also add an option of specifying how to rank tied mons? Maybe have an option of array of sorting things, like first by stat product descending, then by cp descending, then by atk IV descending, then by def IV descending, then by sta IV descending, then by level ascending? Maybe we should also allow people to sort stat product ascending because having fewer options would make it inferior? :)

sorting by statproduct then CP to align to the most used PvP calculators seems sufficient :)
again, only an option that is up to the mapper. both are correct.

Just pointing out a bug in the pvp logic that feeds into the main project. Don't think anyone wants Golbat to be inferior to RDM in any aspect, which it currently is due to ohbem's pvp restrictions.

Welcome to Open Source world! You are unhappy with current implementation, that happens :) People either create PR or Fork a project if owner is against merging. But! as I noted before, I don't mind extending API if it won't break current flow and is configurable. That still doesn't mean we are all going to jump into this, we all have better and much, much more important things to do. Feel free to stay on RDM if you are going to complain about PvP results, nobody will force you to change the software you use.

Hold on a second. You said "have to settle", implying that there is an inherent difference between the two tied rankings. How is this true actually?
On the flip side, I do realize that maybe we should sort by stat product descending, and in addition then by attack IV descending due to CMP (charged move priority) ties.

Also from my understanding, the thing with PvP ranks (except for master league) is not that simple. Since your CP is capped, there could be many IV combinations that each have their pros and cons. Indeed, in the real world (both pro players and IV deep dive analyses from various websites), people sometimes prefer rank 2 over rank 1 due to winning CMP ties in the mirror. I can imagine that due to this, the Nash equilibrium of the PvP meta might involve using different IVs of the same Pokemon each with varying probabilities. At the end of the day, sorting by stat product is only a heuristics that often work but won't necessary give you the best of the best. (In fact, you probably need many different IV combinations and use one of them at random to be best of the best.)
With that said, some IV combinations are inherently worse than others. See also: https://github.com/Mygod/ohbem/issues/1, https://codepen.io/Mygod/full/eYgoGXO

Anyway those comments above are only addressing your third point. As for the second point you raised, the confusion is caused by a disagreement between GO stadium and Ohbem(go), and for the reasons that I explained above, for PvP functionally rank 1 (by which I mean all 3 stats being equal to another IV combination) is really rank 1 since you are not going to use that Pokemon for anything else really (like powering it up to a different level), unless I am missing some use cases here? Maybe having a 67% IV would make one feel better than having a 64% IV, but then they are debatably not really just tracking a "perfect IV for great/ultra league" any more.


add: handle edge cases & tests for macos

Tests currently would only run on macos, but since we don't have a CI it shouldn't be an issue, when we add a CI we should support linux and other platforms as well for these tests.

@biswaroop1547 r?

lgtm! also works for previously failing case i.e:
{"aarch64-apple-darwin": None, "universal-apple-darwin": Some("xyz.com"), "x86_64-apple-darwin": None}

@tiero merge?

utACK https://github.com/premAI-io/prem-app/pull/477/commits/65b7ebf91d008253e8cf94fff1f4918ef19ab4e2


CannotVerifyCopySource error - recursive copy from blob to file share on same storage account

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
azcopy version 10.3.1
Which platform are you using? (ex: Windows, Mac, Linux)
Linux-5.0.0-32-generic-x86_64-with-Ubuntu-19.04-disco
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
azcopy copy https://[stg-account].blob.core.windows.net/blob-container?[sas-token] https://[stg-account].file.core.windows.net/file-share?[sas-token] --recursive=true

What problem was encountered?
The file was copied but contains null chars instead of the text 'abcd' as expected

How can we reproduce the problem in the simplest way?

Create a storage account called storage-account, with a blob container blob-container and a file share called file-share
Create a SAS token that isn't limited by IP or otherwise restricted on any scopes
Upload a simple text file containing the text "abcd" to the blob container
Attempt to copy the contents of blob-container into file-share using the aforementioned command

Have you found a mitigation/solution?
None - tried to use blobxfer and that failed too, see https://github.com/Azure/blobxfer/issues/108
Error logs
Despite being a 'fresh' SAS token with no explicit restrictions I get an auth failure code CannotVerifyCopySource
Authentication failed, it is either not correct, or expired, or does not have the correct permission -> github.com/Azure/azure-storage-file-go/azfile.newStorageError, /home/vsts/go/pkg/mod/github.com/!azure/azure-storage-file-go@v0.6.0/azfile/zc_storage_error.go:42
===== RESPONSE ERROR (ServiceCode=CannotVerifyCopySource) =====
Description=This request is not authorized to perform this operation.
RequestId:c6673793-b01a-004a-207b-9069d3000000
Time:2019-11-01T06:13:13.2581669Z, Details:
   Code: CannotVerifyCopySource
   PUT https://[st-account].file.core.windows.net/file-share/testfile.txt?REDACTED
   Content-Length: [0]
   User-Agent: [AzCopy/10.3.1 Azure-Storage/0.6.0 (go1.12; linux)]
   X-Ms-Client-Request-Id: [d850241c-89fd-4599-62fb-388537110d75]
   X-Ms-Copy-Source: [https://[st-account].blob.core.windows.net/blob-container/testfile.txt?REDACTED
   X-Ms-Range: [bytes=0-4]
   X-Ms-Source-Range: [bytes=0-4]
   X-Ms-Version: [2019-02-02]
   X-Ms-Write: [update]
   --------------------------------------------------------------------------------
   RESPONSE Status: 403 This request is not authorized to perform this operation.
   Content-Length: [248]
   Content-Type: [application/xml]
   Date: [Fri, 01 Nov 2019 06:13:12 GMT]
   Server: [Windows-Azure-File/1.0 Microsoft-HTTPAPI/2.0]
   X-Ms-Client-Request-Id: [d850241c-89fd-4599-62fb-388537110d75]
   X-Ms-Error-Code: [CannotVerifyCopySource]
   X-Ms-Request-Id: [c6673793-b01a-004a-207b-9069d3000000]
   X-Ms-Version: [2019-02-02]

First time azcopy user - apologies if I'm holding this wrong.

Hi, there are two issues here.
The first issues is the authentication failure.  This seems to be that one of your SAS tokens (probably the blob one, I think) isn't giving AzCopy the necessary rights.  I can't tell from the info whether that's because the token you constructed didn't actually give the necessary rights, or if your token was fine but it wasn't interpreted properly. Or, there's one other possibility: if I recall correctly, you can't do service-to-service copies between the public (i.e. global) Azure cloud and the national (aka sovereign) clouds. (E.g. the US Govt one).
Assuming your source and destination are both in the some cloud (e.g. both in the main Azure public cloud) or both in US Govt, or both in Azure China etc, then it's probably just some kind of SAS issue.  I'd suggest that you fire up Storage Explorer, and right click on the source container and the destination file share, and choose on each the "Get Shared Access Signature". For the source, use the default permissions (Read and List) and for the destination tick all the boxes (read, list, write, update, create etc).  If your copy works with those SAS's then it must be something about the SAS's that was causing your failure.  Please let us know how you get on.
The other issues is that, after failures, AzCopy is supposed to delete destination files that have been created but which don't yet have any content.  So, instead of seeing a file full of zeroes, you should have seen no destination file at all.  Unfortunately, there is one case where that automatic cleanup doesn't happen: authentication errors.  I've added a task to our backlog to make that cleanup happen after authentication failures.

Thanks for the quick response,
I tried creating separate SAS tokens at the container level and didn't get far. I didn't think to create a brand new storage account before now though - which works just fine!
However, it is starting to look like the storage accounts network configuration is a factor. We aren't on a govt cloud here but we are doing some custom things in our network configuration.
The one that worked has public networking:

However, in my failing target environment it was configured like this:

(I confirmed that IP address of the machine running the azcopy commands is in that firewall list)
There is a single virtual network on a private /16 with an Endpoint Status of enabled under main.
I tried allowing read access to logging and metrics and that didn't work either. Is this an unsupported scenario, or should I try adding some ranges in this firewall list?

I've added a task to our backlog

Re: cleanup - ah ok that makes sense - thanks for letting us know!

Oh, I know why its happening (I think).  In Service to Service copies (e.g. Storage to File) it is the destination service that reaches out to the source to get the data. So the IP address that is getting blocked is the address of the destination Azure Files Service.  I don't know exactly if there's an easy way for you to add that address to the list (because its not just one address, IIRC).
I'll ask colleagues and see what I can find.

I can reproduce the problem when copying from Blob to File.  Am making inquiries and will update here when have some more info.

The team has replied.
It's supposed to be enough to grant the machine running AzCopy access in your firewall rules, as you have done.  And in fact that works fine for blob to blob copies.  In that case, the system is smart enough to securely use the client IP address to authenticate the service-to-service copy.
The same behaviour is not yet enabled for Blob to Azure Files.  It will be enabled soon - e.g. in about 2 weeks.  If that's going to hold up your work, please email me and we'll see what we can do.  We might be able to do something sooner for you. My email address is in my profile page.

I'm going to close this one now, because it's service-side issue, not a problem with AzCopy itself. As noted above, the service-side fix should be rolled out soon.

Oh, and the second issue here, the failure to cleanup destination files after authentication errors, was fixed in yesterday's release of AzCopy 10.3.2.


Planning module fails to generate a proper path to the goal right behind the car

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
Running environment: docker container [image]
While I run Autoware on Carla Map Town01 and set the destination to the right behind of the ego vehicle, Autoware fails to drive as usual. [video] [rosbag file]
In the video, I changed the goal again and again, and I finally found that the planning module works well when I set the goal point behind the car but not on the same lane(the lane defined from openstreetmap file)
Expected behavior
I expected the car to drive, even when I selected the point right behind him as a goal.
Actual behavior
It seems like there are some misbehaviors on the planning module and the car does not move at all.
Steps to reproduce

Run Autoware on docker container [image] [map]
Set the goal right behind the car.

Versions
I build and run on the docker container with the image here (same with the image link right above)
Possible causes
Sorry. I'm novice to Autoware, so I cannot localize the possible causes well.
But it seems like the problem is on planning module, since I've observed that the planned trajectory starts from the front of the ego vehicle, but it does not in this case.
Additional context
No response

thanks for the report.
as you found currently the goal behind ego in the same lene is not supported and there is a similar issue https://github.com/autowarefoundation/autoware.universe/issues/2354

Oh, I should have searched more.. Sorry for bothering you. I didn't noticed there was the same issue.


Using array.sort and array.reduce in a view.reduce

Hi, we are trying to use sort and reduce array methods inside our reduce function in a view. We realise that these methods are not available on the javascript array object that is provided with the documents array in the views reduce function. Any ideas or ways that this could be accomplished?

keys & values aren't "real" JavaScript Arrays.
You can fix that by using:
function(keys, values, rereduce) {
keys = Array.prototype.slice.call(keys);
values = Array.prototype.slice.call(values);
// real JavaScript Arrays now
/* your reduce code */
}

Well ... a little late, but maybe it helps somebody.

Addressed by https://github.com/couchbase/couchbase-lite-java-javascript/pull/10


Manually enabling SSH

I know SSH is by default enabled automatically by sdm. But somehow it turned out SSH was not enabled in one of the SD cards I burned into using sdm.
My question is: is there a way to manually enable SSH in sdm?
Thanks!

I would like to understand how somehow SSH was not enabled.
Please post or email me the file /etc/sdm/history from the system in question.

Ah, just discovered an issue with setting up ssh on bookworm. The location for the 'ssh' file has moved from /boot/ssh to /boot/firmware/ssh
This will be fixed in sdm V10.2. In the meantime, you can easily --explore the IMG or the burned disk and touch the file yourself.
Apologies for the duff.

/etc/sdm/history

Hi, I couldn't find the file /etc/sdm/history. Could you please point me out what I missed?
root@my-rpi:~# cat /etc/sdm/history
cat: /etc/sdm/history: No such file or directory

root@my-rpi:~# cat /etc/sdm/history
cat: /etc/sdm/history: No such file or directory

Hi, I think you need to use sudo
sudo cat /etc/sdm/history

root@my-rpi:~# cat /etc/sdm/history
cat: /etc/sdm/history: No such file or directory

Hi, I think you need to use sudo sudo cat /etc/sdm/history

Hi, still the same unfortunately...
root@my-rpi:~# sudo cat /etc/sdm/history
cat: /etc/sdm/history: No such file or directory

Closing due to no further activity. OP, please feel free to re-open this or a new issue as needed.


New version: NMFk v1.1.20

Registering package: NMFk
Repository: https://github.com/TensorDecompositions/NMFk.jl
Created by: @montyvesselinov
Version: v1.1.20
Commit: c2c92810ec343d4dc0c8913778da72c0eedbf768
Reviewed by: @montyvesselinov
Reference: https://github.com/TensorDecompositions/NMFk.jl/commit/c2c92810ec343d4dc0c8913778da72c0eedbf768#commitcomment-48609796

[no block]
Again the same problem here. BIGUQ pops in for Mads. Again I am not sure why. It has been not a part of Mads for a long time.
@giordano  if you find what is the problem please let me know how to fix it.
Thank you very much!

I don't have the time to debug this at the moment, I'm simply going to merge it, other checks are good


Test order flake in flutter_tool test/general.shard/dart/language_version_test.dart

https://ci.chromium.org/ui/p/flutter/builders/prod/Linux tool_tests_general/15533/overview
It looks like the global variable here is causing some cross-test pollution: https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/dart/language_version.dart#L20

Closing since this was done in:

https://github.com/flutter/flutter/pull/142378

Ah, sorry this was unclear. This issue was meant to track fixing the tests in the file so that the no-shuffle flag could be removed.

Ah my mistake, just going through issue triage and this came up


[Communication] - PhoneNumbers - Simplify status code assertions

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Garo Wallbox - charge timer: modbus: length in response header '12081' must not be zero or greater than '254'

Hello everyone,
I have a Garo wallbox connected to my homeassistant via wifi.
The garo_wallbox addon is running on the homeassistant and also displays all values.
I can also access the wallbox's web interface via the local IP.
Unfortunately, I always get the following error message in the log with the evcc addon:
"[lp-1  ] ERROR  charge timer: modbus: length in response header '12081' must not be zero or greater than '254'"
Current configuration:
HA-Core
2024.3.3
HA-Supervisor
2024.03.1
HA operating system
12.1
HA frontend
20240307.0
Garo Wallbox: GLBDCMW-T211FC connected via wifi
No wallbox is displayed under the device settings - only the Volvo.
On the overview page, however, the Volvo is always shown as "not connected" even if the charging cable is connected and I start the wallbox via Home Assistant or web interface

Seems like this is not related to the add-on but maybe to the main project https://github.com/evcc-io/evcc.


Replace a hard coded path with the default path of  image in map.yaml.

When turtlebot3_navigation.launch is run, an error will occur with default map_file param.
The file path in map.yaml is odd.

Did you set the image value of the map.yaml file to the correct location as follows. I recommend you check again that you have set the correct path.
image: /home/your/map.pgm
resolution: 0.050000
origin: [-10.000000, -10.000000, 0.000000]
negate: 0
occupied_thresh: 0.65
free_thresh: 0.196

This issue will be closed since there were no actions for a while. You can reopen this issue to show this issue to the users whenever. Thanks.


test/helpers: fix GetBPFPacketsCount

Switch to the json output for the 'cilium bpf metrics list' command so
that we can reliably get the packets count regardless of how many spaces
are there in the description of the drop reason.
Signed-off-by: Gilberto Bertin<EMAIL_ADDRESS>

test-me-please

test-me-please


New version: Gruntwork.Terragrunt version 0.67.2

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

 Url: https://github.com/gruntwork-io/terragrunt/releases/download/v0.67.2/terragrunt_windows_amd64.exe Result: 0x80004005

(Automated response - build 895.)

@wingetbot run

@wingetbot run

This package installs and launches normally in a Windows 10 VM.

This package installs and launches normally in a Windows 10 VM.


Specified project path does not exist

Hi guys please help i build my own build.js using the following code
var build = require('./taco-team-build.js');
build.configure({
projectPath: "C:\Users\Mohammad Salim\Desktop"
}).done();
platforms = ["android", "windows"],
args = { android: ["--release", "--device", "--gradleArg=--no-daemon"], windows: ["--release"] };

build.buildProject(platforms, args).done();
but i am getting error , "Specified project path does not exist"
i hope i can compile project which are on another directory

What version of node.js and Cordova are you running?  Try double slashing your path and see if that resolves the issue.  We have seen problems with certain versions of Node.js not properly resolving paths.  Ex: C:\Users\Mohammad Salim\Desktop

hi , but now i am getting error cannot read property done()

The .done() is a doc bug - I will resolve.
Please include a your log along with the following information: The version of node.js and Cordova you are building.
Certain versions of Cordova have known issues with certain versions of node. There's not enough information here to help you out.

Thanks, please include the output log as requested as well.

after executiing build.js
i am getting
C:\Users\Mohammad Salim\Desktop\teco try\taco-team-build-master>node build.js
Module cache at C:\Users\Mohammad Salim\AppData\Roaming\taco_home\node_modules
Installing cordova to C:\Users\Mohammad Salim\AppData\Roaming\taco_home\node_mod
ules\cordova\6.0.0. (This may take a few minutes.)
after few minuts i am back to
C:\Users\Mohammad Salim\Desktop\teco try\taco-team-build-master>
and my project files are on desktop thanks
C:\Users\Mohammad Salim\Desktop\taco

I assume build.js is your own code. Do you have a pointer to it?
Likely given what you are saying you are either doing ".done()" too early which stops the promise chain or are missing a .done() or .then().
You should use .then() if you intend to do another promise.   Now that you've provided one, we can take a look.
That said, build.buildProject(platforms, args) is in fact missing .done() in build.txt.
Try:
build.buildProject(platforms, args).done()
Next, I'm not sure where you are expecting to see the build output. If the build actually executing, and succeeding, then the output will be in platforms/android/build/outputs/apk
I attempted to reproduce your issue but was unable to do so.  I did the following:

Created a folder with new cordova project in it
Created a different folder with your build.js in it
Executed "npm install taco-team-build" in the folder with build.js in it
Updated the script as follows:

var build = require('taco-team-build');
build.configure({
    projectPath: "C:\\Projects\\cordova\\yelling test\\project is here\\stubproject"
})

platforms = ["android"],
args = { android: ["--release", "--device", "--gradleArg=--no-daemon"]};

build.buildProject(platforms, args).done();

node build.js executed successfully.
The only thing I can think of that might be different is I am using the npm version of taco-team-build and not referencing taco-team-build.js

thanks thats code work now , one last i want to know
can i get the path of compiled apk to print , in the done function
like
.done(function(){
console.log(compile file path );
}
thanks for your Help  :)

The location varies by Cordova version unfortunately and is not returned by the library. The sample gulp file contains the locations the files tend to be in: https://github.com/Microsoft/taco-team-build/blob/master/samples/gulp/gulpfile.js
    paths = {
       apk:["./platforms/android/ant-build/*.apk",
            "./platforms/android/bin/*.apk",
            "./platforms/android/build/outputs/apk/*.apk"],
       ipa: ["./platforms/ios/build/device/*.ipa",
             "./platforms/ios/build/device/*.app.dSYM"],
       appx: "./platforms/windows/AppPackages/**/*"
    };

You can use the glob npm package with these paths to get the filenames.


Call method

How to PUT or GET product list using call function?

Consult the documentation here...
Original Wrapper Code and Docs
Best of luck :)

the documentation you have mentioned as for laravel not for codeigniter


Feature/add flush policies

Codecov Report
Base: 78.14% // Head: 83.99% // Increases project coverage by +5.85% :tada:

Coverage data is based on head (7dbb847) compared to base (7fc55c4).
Patch coverage: 81.96% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #122      +/-   ##
============================================
+ Coverage     78.14%   83.99%   +5.85%
- Complexity      443      468      +25
============================================
  Files            64       63       -1
  Lines          5761     5056     -705
  Branches        725      651      -74
============================================
- Hits           4502     4247     -255
+ Misses          693      265     -428
+ Partials        566      544      -22

Impacted Files
Coverage Δ

...n/java/com/segment/analytics/kotlin/core/Events.kt
44.84% <12.50%> (-1.40%)
:arrow_down:

...nalytics/kotlin/core/platform/EventPipelineTest.kt
59.16% <53.12%> (+4.88%)
:arrow_up:

...kotlin/core/platform/plugins/SegmentDestination.kt
71.73% <66.66%> (-2.62%)
:arrow_down:

...lin/core/platform/policies/FrequencyFlushPolicy.kt
72.22% <72.22%> (ø)

...ore/platform/policies/FrequencyFlushPolicyTests.kt
88.88% <88.88%> (ø)

...com/segment/analytics/kotlin/core/Configuration.kt
94.11% <100.00%> (+0.36%)
:arrow_up:

...nt/analytics/kotlin/core/platform/EventPipeline.kt
97.08% <100.00%> (+1.21%)
:arrow_up:

...in/core/platform/policies/CountBasedFlushPolicy.kt
100.00% <100.00%> (ø)

...ytics/kotlin/core/platform/policies/FlushPolicy.kt
100.00% <100.00%> (ø)

...otlin/core/platform/policies/StartupFlushPolicy.kt
100.00% <100.00%> (ø)

... and 14 more

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.


feat: Support IAS in OAuth2Service

Context

SAP/cloud-sdk-java-backlog#391.
Alternative to #269
This PR extends our OAuth2Service implementation to handle IAS token retrievals.
In contrast to the changes suggested in #269, we are not introducing any new public API.
Feature scope:

[x] IAS Binding to Destination transformation

[x] Destination is multi-tenant

Definition of Done

[x] Functionality scope stated & covered
[ ] Tests cover the scope above
[ ] Error handling created / updated & covered by the tests above
[x] ~Documentation updated~ (→ not to be documented yet)
[x] ~Release notes updated~ (→ not to be documented yet)

Left some comments for understanding how things are supposed to work.
Besides those, this looks already good, especially that the API didn't receive any breaking changes!


add Advanced search and Particular search

This commit is a mixture of Nirav's work and mine.
I made some minor tweaks in Nirav's work and added some new things which enables the user to make queries on any model about any particular thing say only inhouse projects so the query will look like this
GET api/projects/?is_inhouse=false&approved=true
also for any random search
GET api/search/?generic=<search anything and it will return response from contributer and  project model based on name>
Other Issues also fixed.
Work left is the addition of permission classes and will be done today.
so basically the Advanced Search is implemented.

@Rashmil-1999 your build is failing, please run black command. Then I can merge your PR.


Promote Jon Pulsifer to full PSC member

Following the process documented in https://github.com/kubernetes/security/pull/37. Once this PR merges, I'll follow the rest of the steps - but let's treat this one as the source of truth.

/lgtm
/approve
/hold
I will wait for lazy consensus until Tuesday of next week on this and then cancel the hold.

/lgtm

/lgtm

/hold cancel

Opened community repo PR
Added to mailing lists
Added to opsgenie
Added to github.com/kubernetes-security

Welcome to the team @jonpulsifer !


Refactor application manager caches

https://github.com/kurator-dev/kurator/blob/17dcbefbeb8ad87d733126c3d8939580bd392453/pkg/fleet-manager/application_controller.go#L63-L65
We should not use package vars instead the cache should be under ApplicationManager

cc @Xieql

fix by #321

/close


check_defaults: Have a default return case for README

Otherwise the return from the function is NoneType
Signed-off-by: Konrad Rzeszutek Wilk<EMAIL_ADDRESS>

this issue has been address in another pull request


Permallinks to comments

A feature request to add generation of permalinks to comments and other entities that has date. Just, as it works now in GitHub / Stack Overflow / many many other places and just as it works currently in Gogs for permalinking commits. Thanks!

Thanks your feedback!
Can you given an example? I'm not sure what do you mean exactly...

Hover any time (date) field in comment's title (header) and you will see, that it is a permallink to this particular comment. For example:

Unknwon commented an hour ago

an hour ago = https://github.com/gogits/gogs/issues/1654#issuecomment-141105798
(example)
This does not work (yet) in Gogs. Dates in comments are not clickable links.
IMHO this is a very useful feature (and shouldn't be hard to code), which allows you to send / share direct links to a particular comment in longer discussion in a certain issue. Currently, you can only link issue itself in Gogs and add something like "Look for the third Unknwon's comment".
Does it sound more detailed right now? :>

Thanks for the details!
But I think you are using very old version of Gogs because it has been implemented for quite a while...

That is possible, because I'm not responsible for updating software in my company! :(
Sorry for bothering and thanks for an information! :)

Hmm.. well :joy:


php artisan queue:retry fails

Bug description
Hey everyone,
Whenever I have a failed job, calling php artisan queue:retry all fails.
The error message I am getting is:
>php artisan queue:retry all

   InvalidArgumentException

  Database connection [tenant] not configured.

  at \vendor\laravel\framework\src\Illuminate\Database\DatabaseManager.php:152
    148▕         // If the configuration doesn't exist, we'll throw an exception and bail.
    149▕         $connections = $this->app['config']['database.connections'];
    150▕
    151▕         if (is_null($config = Arr::get($connections, $name))) {
  ➜ 152▕             throw new InvalidArgumentException("Database connection [{$name}] not configured.");
    153▕         }
    154▕
    155▕         return (new ConfigurationUrlParser)
    156▕                     ->parseConfiguration($config);

  1   \vendor\laravel\framework\src\Illuminate\Database\DatabaseManager.php:115
      Illuminate\Database\DatabaseManager::configuration("tenant")

  2   \vendor\laravel\framework\src\Illuminate\Database\DatabaseManager.php:86
      Illuminate\Database\DatabaseManager::makeConnection("tenant")

Steps to reproduce
Setup an empty Laravel project with this package.
Create a job.
Make sure the job fails.
Try running php artisan queue:retry all
Expected behavior
Calling php artisan queue:retry all should properly retry the job.
Laravel version
8.75.0
stancl/tenancy version
3.4.5

Can you show the code of the job?

This should be fixed in https://github.com/archtechx/tenancy/commit/73a4a3018cadca2ba0fb5f2130fca1718a2b3670. Could you try running composer require stancl/tenancy:dev-master to see if the change fixes this?

Fixed in https://github.com/archtechx/tenancy/commit/73a4a3018cadca2ba0fb5f2130fca1718a2b3670

I'm still having the issue in stancl/tenancy version 3.5.0 and Laravel 8.38.0.
My config is multi database. I have QueueTenancyBootstrapper and DatabaseTenancyBootstrapper in tenancy.php config.
I have forced the database queue driver to use the central connection with
'connection' => 'central',
in config/queue.php
Note that my job got dispatched from the tenant context.
I don't know what else to do. Thanks!

Can you show the code of the job?

Sure
<?php

namespace App\Jobs;

use App\Models\Tenant\Order;
use App\Models\Tenant\Route;
use App\Models\Tenant\RouteRegistry;
use App\Models\Tenant\Visit;
use App\StateMachines\OrderStatusStateMachine;
use Carbon\Carbon;
use Exception;
use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldBeUnique;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Queue\SerializesModels;
use Illuminate\Support\Facades\Log;
use Illuminate\Support\Str;

class RoutePendingVerificationOrder implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    /**
     * The order instance.
     *
     * @var \App\Models\Tenant\Order
     */
    protected $order;

    /**
     * Create a new job instance.
     *
     * @return void
     */
    public function __construct(Order $order)
    {
        $this->order = $order;
    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle()
    {
        $this->order->load('customerAddress.suburb.zipcode');

        $zipcode = null;

        // Obtenemos un arreglo de zipcodes unicos de todos las ordenes
        if ($this->order->customerAddress) {
            $zipcode = $this->order->customerAddress->suburb->zipcode->name;
        }

        if (is_null($zipcode)) {
            $this->fail(new Exception('La dirección del contrato no tiene colonia', 500));
        }

        $centralDatabase = config('tenancy.database.central_db_name');

        $route = Route::where('route_type_id', 2)->whereHas('zones.zipcodes', function ($query) use ($zipcode, $centralDatabase) {
            $query->from("{$centralDatabase}.zipcodes")->where('name', $zipcode);
        })->with('zones.zipcodes')->first();

        if (is_null($route)) {
            $this->fail(new Exception('No hay rutas que tengan ligadas una zona con el CP correspondiente.'));
        }

        $routeRegistry = RouteRegistry::where('route_id', $route->id)->whereDate('date', Carbon::now())->first();

        if (empty($routeRegistry)) {
            $routeRegistry = RouteRegistry::create([
                'routeId' => $route->id,
                'date' => Carbon::now(),
                'userId' => $route->inchargeUserId,
                'transportId' => $route->transportId
            ]);
        }

        $visit = new Visit();
        $visit->routeRegistryId = $routeRegistry->id;
        $visit->creatorId = $route->inchargeUserId;
        $visit->date = Carbon::now();
        $visit->uniqueId = (string) Str::uuid();
        $visit->visitTypeId = 3;
        $visit->orderId = $this->order->id;
        $visit->customerId = $this->order->customerId;
        $visit->saveQuietly();

        $this->order->status()->transitionTo(OrderStatusStateMachine::VERIFIED_PENDING);

        // TODO send push notification
    }

    public function fail($exception = null)
    {
        Log::error('job failed');
    }
}

Hmm, could I see your:

queue connection config
database connections (the ones you're using)
and the exact code that dispatches this?

And does the issue happen on queue:work, or only on queue:retry?

config/queue.php
'database' => [             'driver' => 'database',             'table' => 'jobs',             'queue' => 'default',             'retry_after' => 90,             'after_commit' => false,             'connection' => 'central'        ]
config/database.php
'connections' => [
     'central' => [
            'driver' => 'mysql',
            'url' => env('CENTRAL_DATABASE_URL'),
            'host' => env('CENTRAL_DB_HOST', '<IP_ADDRESS>'),
            'port' => env('CENTRAL_DB_PORT', '3306'),
            'database' => env('CENTRAL_DB_DATABASE', 'forge'),
            'username' => env('CENTRAL_DB_USERNAME', 'forge'),
            'password' => env('CENTRAL_DB_PASSWORD', ''),
           ... (same as defaults)
        ],

        'mysql' => [
            'driver' => 'mysql',
            'url' => env('DATABASE_URL'),
            'host' => env('DB_HOST', '<IP_ADDRESS>'),
            'port' => env('DB_PORT', '3306'),
            'database' => env('DB_DATABASE', 'forge'),
            'username' => env('DB_USERNAME', 'forge'),
            'password' => env('DB_PASSWORD', ''),
          ... (same as defaults)
        ]
],

.env
CENTRAL_DB_HOST=mysql
CENTRAL_DB_PORT=3306
CENTRAL_DB_DATABASE=****
CENTRAL_DB_USERNAME=****
CENTRAL_DB_PASSWORD=****

DB_CONNECTION=central
DB_HOST=mysql
DB_PORT=3306
DB_DATABASE=****
DB_USERNAME=****
DB_PASSWORD=****

The code that dispatches the job is a StateMachine linked to a model property (status). When I transition the status some hooks are triggered.
In some controller:
$order->status()->transitionTo(OrderStatusStateMachine::PENDING_VERIFICATION);

In OrderStatusStateMachine.php
<?php

namespace App\StateMachines;

use App\Jobs\RoutePendingVerificationOrder;
use App\Models\Tenant\Order;
use Asantibanez\LaravelEloquentStateMachines\StateMachines\StateMachine;
use Carbon\Carbon;

class OrderStatusStateMachine extends StateMachine
{

.......

public function afterTransitionHooks(): array
    {
        return [

            self::PENDING_VERIFICATION => [
                function ($from, $model) {
                    if ($from === self::PROSPECT && $model->order_origin_id == 1) {
                        // -------------- Dispatch the job
                        RoutePendingVerificationOrder::dispatch($model);
                    }
                }
            ],
        ];
    }
}

The issue happens only on queue:retry

Hmm, that's strange. Could you add some dd()s to the QueueTenancyBootstrapper to see what code gets executed? Specifically if this listener is executed, and what the initializeTenancyForQueue() method does afterwards.
Also, what makes the job fail?

I can reproduce the issue in 3.4.6, with 3.5.0 fixing it.
➜ samuel@MacBook-Pro  ~/Sites/queue-test  art queue:retry 3f1943c8-2c50-4730-9eb7-6e393a742565

   InvalidArgumentException

  Database connection [tenant] not configured.

  at vendor/laravel/framework/src/Illuminate/Database/DatabaseManager.php:152
    148▕         // If the configuration doesn't exist, we'll throw an exception and bail.
    149▕         $connections = $this->app['config']['database.connections'];
    150▕
    151▕         if (is_null($config = Arr::get($connections, $name))) {
  ➜ 152▕             throw new InvalidArgumentException("Database connection [{$name}] not configured.");
    153▕         }
    154▕
    155▕         return (new ConfigurationUrlParser)
    156▕                     ->parseConfiguration($config);

      +12 vendor frames
  13  [internal]:0
      App\Jobs\TestJob::__unserialize()

      +16 vendor frames
  30  artisan:37
      Illuminate\Foundation\Console\Kernel::handle(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))

➜ samuel@MacBook-Pro  ~/Sites/queue-test  composer update
Loading composer repositories with package information
Updating dependencies
Lock file operations: 0 installs, 18 updates, 0 removals
...
  - Upgrading stancl/tenancy (v3.4.6 => v3.5.0)
...
Use the `composer fund` command to find out more!
> @php artisan vendor:publish --tag=laravel-assets --ansi --force
No publishable resources for tag [laravel-assets].
Publishing complete.

➜ samuel@MacBook-Pro  ~/Sites/queue-test  art queue:retry 3f1943c8-2c50-4730-9eb7-6e393a742565
The failed job [3f1943c8-2c50-4730-9eb7-6e393a742565] has been pushed back onto the queue!

➜ samuel@MacBook-Pro  ~/Sites/queue-test  art queue:work
[2022-01-05 11:46:33][10] Processing: App\Jobs\TestJob
[2022-01-05 11:46:33][10] Failed:     App\Jobs\TestJob

➜ samuel@MacBook-Pro  ~/Sites/queue-test  art queue:failed
+--------------------------------------+------------+---------+------------------+---------------------+
| ID                                   | Connection | Queue   | Class            | Failed At           |
+--------------------------------------+------------+---------+------------------+---------------------+
| 3f1943c8-2c50-4730-9eb7-6e393a742565 | database   | default | App\Jobs\TestJob | 2022-01-05 11:46:33 |
+--------------------------------------+------------+---------+------------------+---------------------+

➜ samuel@MacBook-Pro  ~/Sites/queue-test  art queue:retry 3f1943c8-2c50-4730-9eb7-6e393a742565
The failed job [3f1943c8-2c50-4730-9eb7-6e393a742565] has been pushed back onto the queue!

Hey @stancl: it was Laravel version.
The JobRetryRequested listener was added in version 0.64.0 and my previous version was 8.38.0, so the listener wasn't executed.
Thank you so much for this awesome library you made and for your quick replies!

No worries, appreciate that you tested it! It was a bit half-blind debugging for me as I haven't encountered this myself before partially reproducing it in some tests, so the feedback is helpful.


Cannot see db results when no workspace is open

Do we need to address this? We have a warning message for this reason...

Should be fixed with other save bugs


store: support adding existing structures

Describe your changes
fixes #136
implemented the extend trait for the merkle store, consume an iterator over inner nodes. this can be used to add existing structures into the store.
Checklist before requesting a review

Repo forked and branch created from next according to naming convention.
Commit messages and codestyle follow conventions.
Relevant issues are linked in the PR description.
Tests added for new functionality.
Documentation/comments updated according to changes.

One other thing: I was thinking that we'd remove most with_* and add_* methods after this. Is there a reason to keep them around?

MerklePath and MerklePathSet need it (these don't have the inner node iterator), should I add them?

One other thing: I was thinking that we'd remove most with_* and add_* methods after this. Is there a reason to keep them around?

MerklePath and MerklePathSet need it (these don't have the inner node iterator), should I add them?

Let's remove all methods that we can and we'll remove the remaining ones later.


tinkey command line help is lacking

Help us help you
tinkey does not provide any help about what it does or what each subcommand does. Besides the name of the sub-command, the user does not have anything to go on.
What I would like to see:
tinkey --help

create-keyset : Create a new keyset with a key
add-key: Add a new key to an existing keyset....
...

Have you considered any alternative solutions?
The only other alternative solution is to download args4j-tools and have it generate HTML documentation from source code of the tinkey project. But then we will have to add a feature to print out the HTML file when tinkey --help is run.
Would you like to add additional context?
tinkey requires some getting used to. Having documentation of the tools, and of each subcommand will make it much easier for the user to use.
Also... i think that this is a limitation of using args4j. It does not provide a way to print usage() where the usage could be inlined with the code.

Thanks for the suggestion. I added this in https://github.com/google/tink/commit/456d15c9a8fe826caed7249665cd9827237d6779.


Windows: guest_redirect_dns checking for iptables

The corresponding Vagrantfile is as follows, where the config.landrush.guest_redirect_dns has been set to true, which uses iptables to redirect traffic in the VM. However, iptables are specific to Unix world and are not being used in Windows guest, which is why the functionality should be modified for Windows guest VMs.

create a private network
config.vm.network :private_network, ip: "<IP_ADDRESS>"
config.vm.communicator = "winrm"
config.winrm.username = 'vagrant'
config.winrm.password = 'vagrant'
config.vm.guest = :windows
enable landrush DNS
if Vagrant.has_plugin?("landrush")
config.landrush.enabled = true
config.landrush.guest_redirect_dns = true
config.landrush.host_redirect_dns= false
config.landrush.host 'win8x86-enterprise', '<IP_ADDRESS>'
end

The error I get when running vagrant up is the following, which clearly states the Windows guest doesn't have the iptables_installed** functionality.

==> vagrant-win8x86-enterprise: Running 'pre-boot' VM customizations...
==> vagrant-win8x86-enterprise: Booting VM...
==> vagrant-win8x86-enterprise: Waiting for machine to boot. This may take a few minutes...
vagrant-win8x86-enterprise: WinRM address: <IP_ADDRESS>:5985
vagrant-win8x86-enterprise: WinRM username: vagrant
vagrant-win8x86-enterprise: WinRM execution_time_limit: PT2H
vagrant-win8x86-enterprise: WinRM transport: negotiate
==> vagrant-win8x86-enterprise: Machine booted and ready!
Vagrant attempted to execute the capability 'iptables_installed'
on the detect guest OS 'windows', but the guest doesn't
support that capability. This capability is required for your
configuration of Vagrant. Please either reconfigure Vagrant to
avoid this capability or fix the issue by creating the capability.

This can be avoided by setting the following options, however then we have to manually set the DNS settings inside the VM (for the appropriate network interface).

config.landrush.guest_redirect_dns = false

We should add appropriate CMD (by using ssh communicator) or Powershell (by using winrm communicator) command which set up the appropriate DNS settings in the Windows guest.

Right, Windows guests are not really supported. At the very least the guest DNS configuration won't work, due to the use of iptables. I am not familiar with winrm. Does it have support for changing DNS settings? Or is it just a way to communicate with the VM in order to execute administration task.
My experience is that DNS configuration on Windows is not trivial. Do you have any concrete steps in mind?

WinRM is a way to communicate with the VM in order to execute arbitrary commands - there is basically a WinRM service which is scheduled to come up when the Windows boots, which is why vagrant is waiting for WinRM to come up prior to executing those command (it shouldn't take long into the booting process).
Once the WinRM is up and accessible, vagrant can execute any number of commands. We should execute the following commands (just an example - haven't tried it):
To obtain the interfaces:

# netsh interface show interface

Then we need to parse the output of that command to obtain the names of the interfaces (when not set to default values) and set the DNS address with the following.

# netsh interface ipv4 add dnsserver "Ethernet" address=<IP_ADDRESS> index=1

Can you pinpoint the place in the code where the commands are run in order to try to change the DNS settings (for Windows that is). I'm not that familiar with Ruby, so a little help is needed!

Can you pinpoint the place in the code where the commands are run in order to try to change the DNS settings

There are several. Supporting Windows guests means to add OS type checks at various places. The first thing you ran into was the capability to check whether iptables is installed - install_prerequisites.rb. This should not happen for Windows. Then there are various other guest capabilities which deal with installing and configuring iptables - guest capability folder
Most importantly one would needs a Windows redirect_dns guest capability which then is called in RedirectDns
So definitely possible, just needs some work. Something you are interested in? I am happy to give some guidance ;-)


main entry is missing

Here https://github.com/krasimir/lsbridge/blob/master/package.json#L5
There should be a path to the main library.
Below code doesn't work.
import lsbridge from 'lsbridge';

Below code works
import lsbridge from 'lsbridge/src/lsbridge';

Thanks. It's fixed in the newly released 0.2.0 version.


[BEAM-1799] JdbcIOIT now uses writeThenRead style

Removes JdbcTestDataSet's main, since it is no longer necessary for
loading data.
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[X] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[X] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[X] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[X] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

I've now switched JdbcIOIT to do a write then a read. This code:

creates a table during test class setup (this runs on the client)
during the write test, generates data and populates data into that table
reads the data from the table and verifies it via HashingFn + count

I really like the data generation - feels very clean, and I love the fact that using CountingSource means this part can be parallelized for larger data sets. I suspect that we may choose to refactor the "generate data" portion out into a helper method alongside HashingFn in io-common.
R @dhalperi

Coverage increased (+0.006%) to 70.185% when pulling 26fb99021b1edd48f6660b95a106ecd9116c6f16 on ssisk:jdbc-it-writeThenRead into ccf247c1daa868cdaf441f918775c94ae6e3aa0b on apache:master.

cc @echauchot

@dhalperi do you think you'll have time to look at this in the next day or two? Or should I find another reviewer?

R: @dhalperi so it gets picked up in the dashboard.
I'm a little confused by this PR. For functional / integration tests, it seems important to test that reading works without the write transform, and that writing works without the read transform.
Why doesn't this lose coverage?
[In another discussion I recall discussing we didn't need to do this for performance tests, specifically because we would already have test coverage of correctness in the "standard" IOITs]

To interpret a little bit: you want to make sure we don't make it harder to diagnose problems with ITs. I agree. I also think we want to make sure we are able to diagnose problems with the larger scale (performance) tests.
To that end, I had planned later work to add in other validation besides the hash where it'd be easier to determine the cause (eg. by outputting the expected/actual rows.) It would do sampling of the data set for detailed comparison/output of results. However, I don't have a detailed plan for that yet, so let me do some work to make that more concrete and add it here, then we can see if you think that'll make it appropriately easy to diagnose ITs.

Sounds right -- happy to merge (or ask another committer) as is or when you're happy with it, now that we're on the same page.

R: -@dhalperi for now

k, I'm still thinking about this, hoping to resume in the next week or two now that I'm back.

@dhalperi I updated the code - check it out. This will now allow devs to see more information about failures - a sample of rows that don't match (if they're in the first or last 500 rows)

R: @kennknowles I'll take a peek

Changes Unknown when pulling f02850cc5998685f865edde24cc1cd6ec0d2addc on ssisk:jdbc-it-writeThenRead into ** on apache:master**.

Changes Unknown when pulling f02850cc5998685f865edde24cc1cd6ec0d2addc on ssisk:jdbc-it-writeThenRead into ** on apache:master**.

R: @echauchot I'll take a look as well

okay! I've done some reorganization.
They boil down to:

TestRow is now a First Class Class - abstract out everything that I think is generic and could be used by other tests related to passing around data, and it pairs well with HashingFn.
JdbcTestDataSet -> JdbcTestHelper - there is still some code that I want to share between the unit tests and the ITs.

These aren't quite the abstractions you asked for, but I think they're helpful and get at the root of what you were asking - "what can we refactor/share in this code?"
PTAL

Changes Unknown when pulling c5f9eb396a935cb2fbb18d7cebe508f3cd7aad4c on ssisk:jdbc-it-writeThenRead into ** on apache:master**.

Nothing terribly substantive - g2g modulo little touches.

okay, updated based on feedback, PTAL

Changes Unknown when pulling 2c01ecad8d8e1543944a2255944a14a4baf7523f on ssisk:jdbc-it-writeThenRead into ** on apache:master**.

run java postcommit

false - that doesn't add coverage

LGTM


Elaborate the agent protocol for Cody Ignore

Removes check/isCodyIgnored, which was unused
Adds client -> server method, ignore/forUri which gets a policy result ('use', 'ignore')
Adds server -> client notification, ignore/didChange for invalidating client cached policy results
Adds client -> server method testing/ignore/overridePolicy for setting a cheesy regex-based policy, for testing

Part of #3641
Test plan
pnpm -C agent test

Thanks for your help with this @olafurpg .
@valerybugakov I'm aware this needs to be integrated with the async filter, I will do that in a follow-up. Just want this to for JetBrains development for now.


Error: node-loader, the specified module could not found

Building the latest version on main-v2 with xal-node I get the following on Windows 10 when running the .exe in the dist/win-unpacked folder (same happens when using the installer).

Any idea? 🤔

It looks like the xal-node library didn't build successfully. Can you try to run yarn run build-deps and see if that will solve the issue?

My bad. It seems I've provided the shared OpenSSL libs I've built with vcpkg by setting OPENSSL_DIR for the cargo build.
Just for future reference for myself and others who make this mistake:

Install openssl:x64-windows-static with vcpkg
Set OPENSSL_DIR to '<vcpkg_root>\installed\x64-windows-static'
Set OPENSSL_NO_VENDOR to 1
Set RUSTFLAGS=-Ctarget-feature=+crt-static (as described here)
Do the build :)