Get rules by id when editing branch protection rule

When users rename an existing branch protection rule, a new rule with the new name will be created and the old rule will still exist.

The reason is that the SettingsProtectedBranchPost function only get branch protection rule by name before updating or creating a rule. When the rule name changes, the function cannot find the existing rule so it will create a new rule rather than update the existing rule. To fix the bug, the function should get rule by id if it fails to get rule by name.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0ab22a1). Click here to learn what that means.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main   #22932   +/-   ##
=======================================
  Coverage        ?   47.33%
=======================================
  Files           ?     1121
  Lines           ?   150300
  Branches        ?        0
=======================================
  Hits            ?    71145
  Misses          ?    70711
  Partials        ?     8444

Impacted Files
Coverage Δ

routers/web/repo/setting_protected_branch.go
34.55% <0.00%> (ø)

services/forms/repo_form.go
39.50% <ø> (ø)

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

LG-TM bot


Bulk Data Import should cause FHIR-Notifications to be issued if data is created or updated.

Is your feature request related to a problem? Please describe.
When doing a bulk data import and the data is added or updated in the db the fhir-notifications should be issued.
Describe the solution you'd like
A new entry for Create or update caused by bulk data import should be issued to the fhir-notifications.
Acceptance Criteria

GIVEN [a precondition]
AND [another precondition]
WHEN [test step]
AND [test step]
THEN [verification step]
AND [verification step]

Additional context
Add any other context or screenshots about the feature request here.

fhir-notification uses fhir-peristence's interceptors.  This is the design on the REST server.
The bulkdata application does not fire the before or after create/update interceptors (or any interceptors).
To support the bulkdata application sending notifications or not:
1 - a configuration to enable or disable notifications specific to bulkdata should be set (enabling an admin/provider to create a new fhir server without generating notifications).  This could use the fhir-server-config.json's bulkdata storageprovider.  An alternative would be to add a QueryParameter or Parameters.Parameter.value to support turning off or on notifications.
2 - The bulkdata application would then upon $import create/update (skipping skipped resources) would trigger a call to AbstractNotificationUtil in the fhir-server-spi which would generate the connection and load the KafkaService or Nats Service.
3 - nats/kafka and server-spi would need to be added to the bulkdata webapp.


HEVC codec support (iPhoneX)

I am having "fun" with Apples new video codec HEVC/h265 and am VERY surprised, that nobody else has any issues with this codec. In our application, we use a mobile device to record video (or choose from the phones mediathek) and then send it to another computer for playback (in a browser as HTML5 video).
The problem we are facing is a different result, weather I choose a video from the phones mediathek versus recording the video live. If I record a video with the native iOS camera app, then choose this video inside our app from the phones mediathek, everything works quite fine and the video is encoded with h264. But if I record the video directly inside our app, the video is encoded with HEVC/h265 instead of h264!!! Result in Chrome, stillframe of first image and only audiostream is playing...

I even do not know exactly, what the solution would be, other than converting h265 to h264... Why do other users not face this issue? Am I overseeing something trivial? Please let me know, if you need more information, I am happy for any hint or comment!
Thanks so much in advance!!!
EDIT:
At least a similar Stackoverflow post:
https://stackoverflow.com/questions/51103054/html-input-file-video-iphone-x-codec-issue

Maybe I just don't understand what you've wrote here, but I don't think this has any relation to this component as such, this only wraps the dropzonejs for it to be used in React. If I'm mistaken maybe someone will correct me.

Hi there, sorry for the late response and inability to describe the problem properly. I think this issue can be closed. It still persists in our project, but as no other user seems to have similar issues, the false behaviour has to be somewhere else in our stack. Currently there is no time to investigate this problem in more depth, if it would really be inside your wrapper, I would reopen the issue. Thanks either way!


functionapp deployment failes with Getting scm site credentials for zip deployment

az feedback auto-generates most of the information requested below, as of CLI version 2.0.70

Describe the bug
After updating from 2.0.57 to 2.0.70 my deploment of a zip to a functionapp does no more work. Error is: Getting scm site credentials for zip deployment.
Command is:
az functionapp deployment source config-zip  -g "XXX" -n "XXX" --src "deploy.zip"
To Reproduce
Use
az functionapp deployment source config-zip  -g "XXX" -n "XXX" --src "deploy.zip"
on a deployed function app
Expected behavior
A clear and concise description of what you expected to happen.
Environment summary
Windows 10
az -v shows
azure-cli                         2.0.70
command-modules-nspkg               2.0.3
core                              2.0.70
nspkg                              3.0.4
telemetry                          1.0.3
Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\Wehrled.azure\cliextensions'
Python (Windows) 3.6.6 (v3.6.6:4cf1f54eb7, Jun 27 2018, 02:47:15) [MSC v.1900 32 bit (Intel)]
Legal docs and information: aka.ms/AzureCliLegal
Your CLI is up-to-date.
Additional context
With a other pc running still 2.0.57 it still works fine.

@ankitkumarr can you please take a look?
@DanielHWe can you please provide the output of the command with the --debug flag?

I switched from powershell to cmd getting more information:
Getting scm site credentials for zip deployment
Starting zip deployment. This operation can take a while to complete ...
The command failed with an unexpected error. Here is the traceback:
Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\knack\knack\cli.py", line 206, in invoke
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 603, in execute
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 661, in _run_jobs_serially
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 654, in _run_job
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\six\six.py", line 693, in reraise
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 631, in _run_job
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 305, in __call__
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\__init__.py", line 485, in default_command_handler
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli\azure\cli\command_modules\appservice\custom.py", line 296, in enable_zip_deploy_functionapp
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli\azure\cli\command_modules\appservice\custom.py", line 322, in enable_zip_deploy
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli\azure\cli\command_modules\appservice\custom.py", line 2326, in _check_zip_deployment_status
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\requests\requests\models.py", line 897, in json
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\json\__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\json\decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\json\decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

To open an issue, please run: 'az feedback'

with debug I got in the end:
Starting zip deployment. This operation can take a while to complete ...
urllib3.connectionpool : Starting new HTTPS connection (1): update20functionapp2019a3.scm.azurewebsites.net:443
urllib3.connectionpool : https://update20functionapp2019a3.scm.azurewebsites.net:443 "POST /api/zipdeploy?isAsync=true HTTP/1.1" 202 0
urllib3.connectionpool : Starting new HTTPS connection (1): update20functionapp2019a3.scm.azurewebsites.net:443
urllib3.connectionpool : https://update20functionapp2019a3.scm.azurewebsites.net:443 "GET /api/deployments/latest HTTP/1.1" 503 27
cli.azure.cli.core.util : The command failed with an unexpected error. Here is the traceback:

The command failed with an unexpected error. Here is the traceback:

cli.azure.cli.core.util : Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\knack\knack\cli.py", line 206, in invoke
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 603, in execute
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 661, in _run_jobs_serially
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 654, in _run_job
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\six\six.py", line 693, in reraise
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 631, in _run_job
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 305, in __call__
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\__init__.py", line 485, in default_command_handler
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli\azure\cli\command_modules\appservice\custom.py", line 296, in enable_zip_deploy_functionapp
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli\azure\cli\command_modules\appservice\custom.py", line 322, in enable_zip_deploy
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli\azure\cli\command_modules\appservice\custom.py", line 2326, in _check_zip_deployment_status
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\requests\requests\models.py", line 897, in json
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\json\__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\json\decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\json\decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\knack\knack\cli.py", line 206, in invoke
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 603, in execute
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 661, in _run_jobs_serially
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 654, in _run_job
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\six\six.py", line 693, in reraise
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 631, in _run_job
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\commands\__init__.py", line 305, in __call__
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli-core\azure\cli\core\__init__.py", line 485, in default_command_handler
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli\azure\cli\command_modules\appservice\custom.py", line 296, in enable_zip_deploy_functionapp
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli\azure\cli\command_modules\appservice\custom.py", line 322, in enable_zip_deploy
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\azure-cli\azure\cli\command_modules\appservice\custom.py", line 2326, in _check_zip_deployment_status
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-rtk110ve\requests\requests\models.py", line 897, in json
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\json\__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\json\decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\json\decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
cli.azure.cli.core.util :
To open an issue, please run: 'az feedback'

What is the SKU of the app you are trying to deploy to?

Im not sure what SKU means in this context. I try to upload a function app (Runtime version: 1.0.12615.0 (~1)) by use of "az functionapp deployment source config-zip" to a resource in westeurope.

SKU is the type of plan you are on. Are you on a Consumption plan or an App Service Plan? Also what OS is your functionapp on, if you know?
Or you can also share your function app name, and we can get the SKU and OS information. If you would prefer to share it anonymously, you can do so by following these steps https://github.com/Azure/azure-functions-host/wiki/Sharing-Your-Function-App-name-privately

We are on a Consumption Plan. Id of function is 22d01435-a299-4ae7-884b-83b8a62b6f12.
I deploy from powershell on windows 10. The function is written in .Net Framework.

Does still not work, any updates?

I apologize that this got lost somewhere. Would you mind sharing the ID along with an approximate time-period for me to dig this down.
I will prioritize this for this week and get back to you as soon as I have the above information.

I encountered same error when trying zip deploy after non zip deploy like func azure functionapp publish xxx.
In this case azure service returns error text which is not recognized as a json like following.
Deployment 'latest' not found.

Is this information helps?

I encountered same problem with azure cli 2.0.57.
But after updating azure cli tools to 2.0.75. it works fine without any problem.

2.0.57 works fine, but 2.0.70 and 2.0.75 not (Some colleges have no problem with 2.0.70 on other systems). Also the problem is only in powershell. A cmd is working fine.

I had the same issue and error, and on the PS it was outputting this info..
azure-cli                         2.0.77 *
command-modules-nspkg              2.0.3
core                              2.0.77 *
nspkg                              3.0.4
telemetry                          1.0.4
Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users{USERNAME}.azure\cliextensions'
Python (Windows) 3.6.6 (v3.6.6:4cf1f54eb7, Jun 27 2018, 02:47:15) [MSC v.1900 32 bit (Intel)]
Legal docs and information: aka.ms/AzureCliLegal
You have 2 updates available. Consider updating your CLI installation. Instructions can be found at https://docs.microsoft.com/en-us/cli/azure/install-azure-cli
did the upgrade on the cli
Invoke-WebRequest -Uri https://aka.ms/installazurecliwindows -OutFile .\AzureCLI.msi; Start-Process msiexec.exe -Wait -ArgumentList '/I AzureCLI.msi /quiet'
after the installation did exit from powershell and reopened. Ran the script again to see if it generates the error. There were still same errors unfortunately..

Same problem over here when using ps...
azure-cli  2.11.1
core 2.11.1
telemetry 1.0.5 *

@DanielHWe Apologies for the late reply. Could you please let me know if you are still facing the same issue with most recent version of Az CLI? Awaiting you reply.

Why MSFT answers now? It's too late!

Hi, this issue is closed with the current version of azure cli, I can not reproduce it any more.

@DanielHWe One day, It will come back!  It's a resilient bug :-)

how did you just close this issue? it's just unbelievable Microsoft replies after 2 years and still can't replicate when half the world is facing this issue!!!!!! az functionapp deployment source config-zip --src "build.zip" --name f-snowflakedbupdater-dv --resource-group datalink-dv-e2us-rg --verbose --build-remote true This is the command we have been trying to run over the weekend. And let me make this crystal clear: We spent 90% time of the time deploying this thing and only 10% on testing.... it came to a point that the only way to get out of this error is to head out and grab a coffee which took me 15 minutes then that error morphed into HTTPSConnectionPool(host='f-snowflakedbexecutor-dv.scm.azurewebsites.net', port=443): Max retries exceeded with url: /api/deployments/latest (Caused by SSLError(SSLError(1, '[SSL: TLSV1_ALERT_INTERNAL_ERROR] tlsv1 alert internal error (_ssl.c:2548)'))) and then I waited for 15 more minutes to actually see this finished successfully.

@DanielHWe One day, It will come back! It's a resilient bug :-)

LOL this bug has been sitting there for 4 years. It seems like Microsoft never cared to actually replicate this issue.


Avoid IndexOutOfRange exception when running Both mode

Codecov Report

Merging #26 into master will decrease coverage by 2.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
- Coverage   84.04%   81.59%   -2.46%
==========================================
  Files           8        8
  Lines         326      326
  Branches       20       20
==========================================
- Hits          274      266       -8
- Misses         40       48       +8
  Partials       12       12

Impacted Files
Coverage Δ

Telepathy/Common.cs
70.51% <0%> (-5.13%)
:arrow_down:

Telepathy/Server.cs
71.84% <0%> (-3.89%)
:arrow_down:

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 94fad86...75201e7. Read the comment docs.


Windows 10 ZXing is undefined

I'm trying run scanner in my Lumia550 with Windows 10. When the scanner is running i get output error

/www/plugins/phonegap-plugin-barcodescanner/src/windows/BarcodeScannerProxy.js
0x800a1391 - JavaScript runtime error: 'ZXing' is undefined

I'm trying remove plugin and add it,   but it not helps. My Cordova version is 6.3.1. Thanks for help.

@sebastianor are you specifying an architecture?
It is required on Windows 10 and the plugin should currently alert you in Visual Studio TACO or in Cordova CLI.


bug in apt.repo()?

I'm running into an issue using apt.repo().
Am I missing something?
$ cat mike.py
from pyinfra.modules import apt

SUDO = True

apt.repo(
    {'Install VirtualBox repo'},
    'deb https://download.virtualbox.org/virtualbox/debian bionic contrib',
)
$ rm pyinfra-debug.log
rm: pyinfra-debug.log: No such file or directory
$ pyinfra -vv --debug  @docker/ubuntu:bionic mike.py
    [pyinfra_cli.main] Checking potential directory:
    [pyinfra_cli.main] Setting directory to:
--> Loading config...
--> Loading inventory...
    [pyinfra_cli.inventory] Creating fake inventory...
    [pyinfra.api.inventory] Expanding inventory connector: docker
    The @docker connector is in Alpha!
    [pyinfra_cli.inventory] Looking for group data in: group_data/all.py
    [pyinfra_cli.inventory] Looking for group data in: group_data/bsd.py
    [pyinfra.api.inventory] Expanding inventory connector: docker

--> Connecting to hosts...
localhost: >>> docker run -d ubuntu:bionic sleep 10000
localhost: a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05
    [pyinfra.api.state] Activating host: ubuntu:bionic

--> Preparing operations...
    Loading: mike.py
    [pyinfra.api.operation] Adding operation, {'Install VirtualBox repo'}, called @ mike.py:7, opLines=(0, 7), opHash=97d8032458c50f8c46e458a4ca15c67a69be92b3
    [pyinfra.api.facts] Getting fact: apt_sources (ensure_hosts: (ubuntu:bionic,))
    [pyinfra.api.util] Building command (shell_executable: sh): docker exec a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05 sh -c "cat /etc/apt/sources.list /etc/apt/sources.list.d/*.list || true"
[pyinfra.api.connectors.local] --> Running command on localhost: sh -c 'docker exec a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05 sh -c "cat /etc/apt/sources.list /etc/apt/sources.list.d/*.list || true"'
[ubuntu:bionic] >>> sh -c 'docker exec a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05 sh -c "cat /etc/apt/sources.list /etc/apt/sources.list.d/*.list || true"'
[ubuntu:bionic] # See http://help.ubuntu.com/community/UpgradeNotes for how to upgrade to
[ubuntu:bionic] # newer versions of the distribution.
[ubuntu:bionic] deb http://archive.ubuntu.com/ubuntu/ bionic main restricted
[ubuntu:bionic] # deb-src http://archive.ubuntu.com/ubuntu/ bionic main restricted
[ubuntu:bionic]
[ubuntu:bionic] ## Major bug fix updates produced after the final release of the
[ubuntu:bionic] ## distribution.
[ubuntu:bionic] deb http://archive.ubuntu.com/ubuntu/ bionic-updates main restricted
[ubuntu:bionic] # deb-src http://archive.ubuntu.com/ubuntu/ bionic-updates main restricted
[ubuntu:bionic]
[ubuntu:bionic] ## N.B. software from this repository is ENTIRELY UNSUPPORTED by the Ubuntu
[ubuntu:bionic] ## team. Also, please note that software in universe WILL NOT receive any
[ubuntu:bionic] ## review or updates from the Ubuntu security team.
[ubuntu:bionic] deb http://archive.ubuntu.com/ubuntu/ bionic universe
[ubuntu:bionic] # deb-src http://archive.ubuntu.com/ubuntu/ bionic universe
[ubuntu:bionic] deb http://archive.ubuntu.com/ubuntu/ bionic-updates universe
[ubuntu:bionic] # deb-src http://archive.ubuntu.com/ubuntu/ bionic-updates universe
[ubuntu:bionic]
[ubuntu:bionic] ## N.B. software from this repository is ENTIRELY UNSUPPORTED by the Ubuntu
[ubuntu:bionic] ## team, and may not be under a free licence. Please satisfy yourself as to
[ubuntu:bionic] ## your rights to use the software. Also, please note that software in
[ubuntu:bionic] ## multiverse WILL NOT receive any review or updates from the Ubuntu
[ubuntu:bionic] ## security team.
[ubuntu:bionic] deb http://archive.ubuntu.com/ubuntu/ bionic multiverse
[ubuntu:bionic] # deb-src http://archive.ubuntu.com/ubuntu/ bionic multiverse
[ubuntu:bionic] deb http://archive.ubuntu.com/ubuntu/ bionic-updates multiverse
[ubuntu:bionic] # deb-src http://archive.ubuntu.com/ubuntu/ bionic-updates multiverse
[ubuntu:bionic]
[ubuntu:bionic] ## N.B. software from this repository may not have been tested as
[ubuntu:bionic] ## extensively as that contained in the main release, although it includes
[ubuntu:bionic] ## newer versions of some applications which may provide useful features.
[ubuntu:bionic] ## Also, please note that software in backports WILL NOT receive any review
[ubuntu:bionic] ## or updates from the Ubuntu security team.
[ubuntu:bionic] deb http://archive.ubuntu.com/ubuntu/ bionic-backports main restricted universe multiverse
[ubuntu:bionic] # deb-src http://archive.ubuntu.com/ubuntu/ bionic-backports main restricted universe multiverse
[ubuntu:bionic]
[ubuntu:bionic] ## Uncomment the following two lines to add software from Canonical's
[ubuntu:bionic] ## 'partner' repository.
[ubuntu:bionic] ## This software is not part of Ubuntu, but is offered by Canonical and the
[ubuntu:bionic] ## respective vendors as a service to Ubuntu users.
[ubuntu:bionic] # deb http://archive.canonical.com/ubuntu bionic partner
[ubuntu:bionic] # deb-src http://archive.canonical.com/ubuntu bionic partner
[ubuntu:bionic]
[ubuntu:bionic] deb http://security.ubuntu.com/ubuntu/ bionic-security main restricted
[ubuntu:bionic] # deb-src http://security.ubuntu.com/ubuntu/ bionic-security main restricted
[ubuntu:bionic] deb http://security.ubuntu.com/ubuntu/ bionic-security universe
[ubuntu:bionic] # deb-src http://security.ubuntu.com/ubuntu/ bionic-security universe
[ubuntu:bionic] deb http://security.ubuntu.com/ubuntu/ bionic-security multiverse
[ubuntu:bionic] # deb-src http://security.ubuntu.com/ubuntu/ bionic-security multiverse
[ubuntu:bionic] cat: '/etc/apt/sources.list.d/*.list': No such file or directory
[pyinfra.api.connectors.local] --> Waiting for exit status...
[pyinfra.api.connectors.local] --> Command exit status: 0
    Loaded fact apt_sources
    [pyinfra.api.facts] Getting fact: find_in_file (ensure_hosts: (ubuntu:bionic,))
    [pyinfra.api.util] Building command (shell_executable: sh): docker exec a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05 sh -c "grep "^.*deb https://download.virtualbox.org/virtualbox/debian bionic contrib.*$" /etc/apt/sources.list || (find /etc/apt/sources.list -type f > /dev/null && echo "__pyinfra_exists_/etc/apt/sources.list")"
[pyinfra.api.connectors.local] --> Running command on localhost: sh -c 'docker exec a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05 sh -c "grep "^.*deb https://download.virtualbox.org/virtualbox/debian bionic contrib.*$" /etc/apt/sources.list || (find /etc/apt/sources.list -type f > /dev/null && echo "__pyinfra_exists_/etc/apt/sources.list")"'
[ubuntu:bionic] >>> sh -c 'docker exec a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05 sh -c "grep "^.*deb https://download.virtualbox.org/virtualbox/debian bionic contrib.*$" /etc/apt/sources.list || (find /etc/apt/sources.list -type f > /dev/null && echo "__pyinfra_exists_/etc/apt/sources.list")"'
    | fact: find_in_file('/etc/apt/sources.list', '^.*deb https://download.virtualbox.org/virtualbox/debian bi    / fact: find_in_file('/etc/apt/sources.list', '^.*deb https://download.virtualbox.org/virtualbox/debian bi    - fact: find_in_file('/etc/apt/sources.list', '^.*deb https://download.virtualbox.org/virtualbox/debian bi    \ fact: find_in_file('/etc/apt/sources.list', '^.*deb https://download.virtualbox.org/virtualbox/debian bi[pyinfra.api.connectors.local] --> Waiting for exit status...
[pyinfra.api.connectors.local] --> Command exit status: 1
    Loaded fact find_in_file: ('/etc/apt/sources.list', '^.*deb https://download.virtualbox.org/virtualbox/debian bionic contrib.*$')
    [ubuntu:bionic]  Ready: mike.py

--> Proposed changes:
    Groups: @docker
    [ubuntu:bionic]   Operations: 1   Commands: 1

--> Beginning operation run...
--> Starting operation: Install VirtualBox repo
    [pyinfra.api.operations] Starting operation Install VirtualBox repo on ubuntu:bionic
    [pyinfra.api.util] Building command (shell_executable: sh): docker exec a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05 sh -c "# If the file now exists
                if [ -f "/etc/apt/sources.list" ]; then
                    # Grep for the line, sed if matches
                    (grep "^.*deb https://download.virtualbox.org/virtualbox/debian bionic contrib.*$" "/etc/apt/sources.list" && sed -i="" "s/^.*deb https:\/\/download.virtualbox.org\/virtualbox\/debian bionic contrib.*$//" /etc/apt/sources.list) ||                     # Else echo
                    echo "deb https://download.virtualbox.org/virtualbox/debian bionic contrib" >> /etc/apt/sources.list

                # No file, just echo
                else
                    echo "deb https://download.virtualbox.org/virtualbox/debian bionic contrib" >> /etc/apt/sources.list
                fi"
[pyinfra.api.connectors.local] --> Running command on localhost: sh -c 'docker exec a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05 sh -c "# If the file now exists
                if [ -f "/etc/apt/sources.list" ]; then
                    # Grep for the line, sed if matches
                    (grep "^.*deb https://download.virtualbox.org/virtualbox/debian bionic contrib.*$" "/etc/apt/sources.list" && sed -i="" "s/^.*deb https:\/\/download.virtualbox.org\/virtualbox\/debian bionic contrib.*$//" /etc/apt/sources.list) ||                     # Else echo
                    echo "deb https://download.virtualbox.org/virtualbox/debian bionic contrib" >> /etc/apt/sources.list

                # No file, just echo
                else
                    echo "deb https://download.virtualbox.org/virtualbox/debian bionic contrib" >> /etc/apt/sources.list
                fi"'
[ubuntu:bionic] >>> sh -c 'docker exec a426aa3919c74b6d09a5d839b4b89f5f1863114c67824842655044000cff3e05 sh -c "# If the file now exists
                if [ -f "/etc/apt/sources.list" ]; then
                    # Grep for the line, sed if matches
                    (grep "^.*deb https://download.virtualbox.org/virtualbox/debian bionic contrib.*$" "/etc/apt/sources.list" && sed -i="" "s/^.*deb https:\/\/download.virtualbox.org\/virtualbox\/debian bionic contrib.*$//" /etc/apt/sources.list) ||                     # Else echo
                    echo "deb https://download.virtualbox.org/virtualbox/debian bionic contrib" >> /etc/apt/sources.list

                # No file, just echo
                else
                    echo "deb https://download.virtualbox.org/virtualbox/debian bionic contrib" >> /etc/apt/sources.list
                fi"'
[ubuntu:bionic] https://download.virtualbox.org/virtualbox/debian: 4: https://download.virtualbox.org/virtualbox/debian: Syntax error: end of file unexpected (expecting ")")
[pyinfra.api.connectors.local] --> Waiting for exit status...
[pyinfra.api.connectors.local] --> Command exit status: 2
    [ubuntu:bionic] Error
    [pyinfra.api.state] Failing hosts: ubuntu:bionic
localhost: >>> docker commit a426aa3919c7
localhost: sha256:db4497e361e507180b287c769f7777969102d1ffd5d009a6772ccfb9a4c218a6
localhost: >>> docker rm -f a426aa3919c7
localhost: a426aa3919c7
    [ubuntu:bionic] docker build complete, image ID: db4497e361e5
--> pyinfra error: No hosts remaining!
$ ls pyinfra-debug.log
ls: pyinfra-debug.log: No such file or directory
$

Turns out this is an issue with Docker specifically - not quoting commands correctly; this should be resolved by https://github.com/Fizzadar/pyinfra/commit/0cc6433f6569d92596d5570423cba571208b7e0c.


Search not working if [showSingleCategory] is true

The searched emojis are not shown when searching, but it selects an emoji (tested when pressing enter after a search, it selects the emoji). I think it has something to do with not displaying all category. Maybe disable showSingleCategory when searching?
Also, when pressing the enter key on a blank search bar, an error on the component occurs 'emoji is undefined'
ngx-emoji-mart: v0.15.0
angular: v7.2.0
npm: v5.6.0
node: v8.11.1

:tada: This issue has been resolved in version 1.0.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Ship tf-controller/aws-primitive-modules with the default installation

We want to ship AWS Primitive Modules for TF-controller by default.
It's going to be an OCIRepository with name aws-package-v0.42.0.

via Helm chart
via the default yaml

Fixed in #395


-Added '$' to the identifier for email property to be sent to Localytics. This is required to be able to identify users on Localytics.

By analyzing the blame information on this pull request, we identified @none to be a potential reviewer.

@lbatr3s  Thanks for the PR!
This has been fixed in the latest version 1.2.0. You don't need to explicitly send in $email, rather, send in the email with the method setCustomerEmail, which appears in Localytics as $email.
We decided to keep the current behavior mapping to email as well so this will not be a breaking change (i.e. some users may have Audiences/reports built off of those traits). These reserved traits are used to create personalized emails, where the general custom traits cannot.


Run-configurations reworked based on Rider 2020.2 + run/debug individual scenes

fixes https://github.com/JetBrains/godot-support/issues/1 and https://github.com/JetBrains/godot-support/issues/8 and https://github.com/JetBrains/godot-support/issues/16 and https://github.com/JetBrains/godot-support/issues/2

closing this because I will publish directly from net202


Generalizing the Webhook class

Is there someway we can generalize the webhook class so that we could use it in lambda or API gateway sense.
Was looking at perhaps using Chalice: https://github.com/aws/chalice/blob/master/chalice/app.py. Or should I just subclass and override the init method?

I think it would just be a case of changing this line to do whatever is needed for Chalice or whatever, rather than Flask. Is that what you mean?

Yeah pretty much, but how do that without subclassing?

Right now, yes, but I think it would be good - as you said - to generalise it in some way. Maybe by leaving it up to the app to add the rule, we could provide some helper functions for common frameworks.
It will be a bit challenging to do this nicely and keep backwards compatibility but I'll have a think and CC you on the PR

I'm happy to help out if you would like. Do you think this type of change has benefit for other users, or is just me being selfish about it. If its just me/my use case, then it might be worth the risk.

Any movement on this @fophillips, I'm not part of that team who managed this in my previous role, but it still might be helpful or else I'll just close this issue

I would suggest using flask with either awsgi or zappa.


Test bug report issue

What steps did you take and what happened?
Test
What did you expect to happen?
Test
How can we reproduce it (as minimally and precisely as possible)?
No response
Cluster-api version
v1.3.x
Kubernetes version
No response
Anything else you would like to add?
No response
Relevant label
kind/bug
Area labels
One or more /area label. See https://github.com/kubernetes-sigs/cluster-api/labels?q=area for the list of labels.

/close


[QUESTION] Is this only meant to be used with IONIC ?

I'm finally trying out this package and I'm trying to reproduce the Z-Stack example in my hybrid Vue project and I'm noticing a lot of dependencies on IONIC which I wasn't prepared for.
I don't use IONIC and wanted to know if this is usable without IONIC? Do you have examples that are not dependent on IONIC CSS/JS being included?

Hello @vesper8
Library not depended on ionic framework and can be used with pure js + html.
Just create html file with context:
<!DOCTYPE html>

  <meta charset="UTF-8">
  <title>Cupertino Pane</title>
  <meta name="viewport" content="viewport-fit=cover, width=device-width, initial-scale=1.0, minimum-scale=1.0, maximum-scale=1.0, user-scalable=no">

      <ion-drawer>

Header

Content

      </ion-drawer>

Thank you for confirming @roman-rr
I'm getting close to having the demo of Z-Stack working in Vue now.. I got the buttons working and they display the correct cards.. but the animations/transitions are not working. They work only when I manually slide the cards, but when I activate them via the buttons then they don't work.. any idea why this is happening? Do I have to include the transition css manually since I'm not using Ionic?

nevermind, figured it out!

Any idea how to integrate this in Vue project? I tried but no animation occurred when import with es module. Can someone provide any basic example?

@sasafister
This is working for me:
<template>

    <ion-slides class="swiper-container">
      <ion-slide>
        <button-rounded-blue
          id="button-1"
          :active="activeNum === 1"
          @clicked="activatePane(1, this, true)"
        >
          #1
        </button-rounded-blue>
      </ion-slide>
      <ion-slide>
        <button-rounded-blue
          id="button-2"
          :active="activeNum === 2"
          @clicked="activatePane(2, true)"
        >
          #2
        </button-rounded-blue>
      </ion-slide>
      <ion-slide>
        <button-rounded-blue
          id="button-3"
          :active="activeNum === 3"
          @clicked="activatePane(3, true)"
        >
          #3
        </button-rounded-blue>
      </ion-slide>
      <ion-slide>
        <button-rounded-blue
          id="button-4"
          :active="activeNum === 4"
          @clicked="activatePane(4, true)"
        >
          #4
        </button-rounded-blue>
      </ion-slide>
      <ion-slide>
        <button-rounded-blue
          id="button-5"
          :active="activeNum === 5"
          @clicked="activatePane(5, true)"
        >
          #5
        </button-rounded-blue>
      </ion-slide>
    </ion-slides>

      <ion-drawer class="card-1">

Card #1

          Content

      </ion-drawer>
      <ion-drawer class="card-2">

Card #2

          Content

      </ion-drawer>
      <ion-drawer class="card-3">

Card #3

          Content

      </ion-drawer>
      <ion-drawer class="card-4">

Card #4

          Content

      </ion-drawer>
      <ion-drawer class="card-5">

Card #5

          Content

      </ion-drawer>

</template>

You're welcome : )

Thnx...I don't want to use 'ion' elements...can I get rid of them?

Also, wrapper for VUE 3 available here. Don't sure it with good support, but works.
https://github.com/Devrax/v-cupertino

Can someone please provide a React Example?

Hey @rgoldiez
Please show to @I-am-abdulazeez some working example with React code.

@I-am-abdulazeez here is a basic example in react using this package as a re-useable component.  There’s no dependency on Ionic but it works equally well with Ionic/React.
https://codesandbox.io/s/cupertino-example-ny4wp?file=/src/App.tsx

@rgoldiez Thank you very much !

@I-am-abdulazeez here is a basic example in react using this package as a re-useable component. There’s no dependency on Ionic but it works equally well with Ionic/React.
https://codesandbox.io/s/cupertino-example-ny4wp?file=/src/App.tsx

Worked Well!!!

@I-am-abdulazeez here is a basic example in react using this package as a re-useable component. There’s no dependency on Ionic but it works equally well with Ionic/React.
https://codesandbox.io/s/cupertino-example-ny4wp?file=/src/App.tsx

how would you go about changing the width of the bottom panel?
I've been struggling for hours...

@10000multiplier
.cupertino-pane-wrapper .pane {
  width: 150px;
}

@10000multiplier
.cupertino-pane-wrapper .pane {
  width: 150px;
}

Thank you !!!


CentroidTracker::update crash

I guess you probably know about this crash.. we are deleting map elements inside the range based loop resulting in invalid iterator. Do we plan to fix this anytime soon ?

Has it been fixed yet?

Hi @n-mam @YannickLecroart, apologies for the delay!
This issue has been fixed in the fix_deletion branch. Please checkout to this branch and test it out, I will merge to master after.
Thanks! :)

Merged to master.


PV (Persistent Volumes) support

Can this tool be used to migrate PV (Persistent Volumes)? It created PVC but not PV

@abaidgulshan - Sorry, the tool doesn't support migrating underlaying persistent volume


fix isatty parameter behaviour #84

fix for issue #84

Coverage remained the same at 91.545% when pulling 2079c3c41de02e7343948f0cd23315754660ece3 on heni:fix/isatty-force-attribute into<PHONE_NUMBER>bc39057f824c545f2decab49c019ad on xolox:master.

Hi and thanks for the feedback! Today I published coloredlogs release 14.1 which fixes this issue.


Problem accessing affiliate agreement linked to from /Affiliates page

here is error report:

The agreement was deleted by mistake. I'm very sorry about that, I have now un-deleted it and it's available for download again.
Thank you for reporting this.


🐛 Update owners to allow me review etc 🤗

What is this PR About?
Want to be able to review and merge PRs.
How do we test this?
cc: @redhat-cop/rego-policies

/lgtm

/approve


Documentation

Do you have some sort of wiki on how to use?

I've put this project aside for work. It is fully functional for LTI 1.3. Not so LTI Advantage (Although there is a branch that is capable of obtaining the token for said service. More not code encapsulated as facades).
There is a brief description to understand how the roles work. I understand that without proper documentation it is difficult to implement for new people and reviewing the code can take a lot of time.
I hope this weekend I can start with better documentation, but to be more helpful, could you tell me what is most critical for you to know right now.
Implementation / Configuration? ; Launching? How to start developing?
Sorry for my English, I used the translator :D

Hello. Thanks for the reply. I appreciate it.
I'm playing around with the code and checking if it fits with what i'm looking for (We are adding LTI 1.3 functionality, we are in LTI 1.0).
Maybe i can email you instead if i have any questions?
Thanks!

Sure, feel free to contact me. As long as I know the answer I will help you. If not, I might be able to help you find a solution.
Anyway, if you think it's a query that can help someone else in the future, you could leave it as an issue, it's not easy for any developer to understand LTI and the information is somewhat scarce.

Is there a way to pass custom attributes?

Yes, in the LTI configuration within the LMS. These variables will always be passed for all instances of your LTI, you cannot apply filters and conditions to pass different variables depending on the context (As far as I understand).
This varies depending on the LMS, but many provide variable substitution (not standardized) where the data is converted to contextual.
You can also define your own key-value variables, which you must then retrieve from the initial LTI launch object (where the resource link, user data, etc.)
I found this cheat sheet to quick variable substitution for the LMS Canvas:
https://gist.github.com/jbasdf/15fb8abed66fbaf623cdfbdd45bfb1c4
If you need to serve contextual and custom variables that are not within your LMS's variable substitution, you can handle these values as a setting within your LTI. In the first launch check if there is a configuration for your resourcelink and/or course, if not, force the user with the corresponding role to enter the configuration with the necessary parameters to continue.
You can keep a default configuration or block with some screen like "We are configuring the resource" for roles that are waiting.

I'm trying to figure out how to do grade push (line items) - i saw the lti advantage branch. Is that a good place to start?

That branch only remained in the intention. But it has a code snippet that will allow you to understand how to obtain a valid token and then consume the lti avantage endpoints. In fact, the example uses the names and roles service to obtain such information.
In the Lti1p3Controller -  Nrps::init($instance); this is a facade and its class is in lti1p3/src/Classes/Nrps.php ; We can notice that it extends to service that is in the same location.
First of all, the method has all the code to consume the service, it was in the testing context. My intention was to encapsulate the core in the service class to later have a facade for the name and role services and notes, reusing what is common within the service class (Obtaining the token and consuming the webservices).
Then simply structure the provided results into classes with friendly methods.
As a bonus, you may notice that the URL to get the token is different, this should be provided by the LMS documentation and when we add an instance, we should add this url there.
Another project in PHP that helped guide me and in fact the author solved some doubts (which I really appreciate). It is the following:
https://github.com/celtic-project/LTI-PHP
The problem with it is that it mixed LTI 1.1 ; 1.0 ; 1.3; Making it hard to understand some things.

Hello. Is the LTI Advantage branch working?

No, it is outdated. The main branch brings improvements and is better adapted to what the standard says. Also now supports LTI Deep Linking.
I still keep the advantage branch because they have a working example to consume the services in moodle and maybe in other LMS. After that, the LMS response would have to be encapsulated in more manageable classes.
The problem is in Canvas LMS, some things don't work as expected and it's hard to get answers from the company. Just a few months ago they fixed some errors but the generation of tokens to consume adventage services does not work.
If you are using CanvasLMS I recommend you to complement with the package to integrate Oauth2 (canvasoauth) and consume the canvas API which will allow you to do the same thing as adventage and much more (combining the best of both worlds).
lti1p3 and canvasoauth work well together, plus you can extend classes or override methods at any time after publishing the vendor.
In practice, forget about the adventage branch as functional for now, just use them as a reference to consume the service (in moodle), to understand better, check the following file that has the only thing salvageable from that whole branch:
https://github.com/xcesaralejandro/lti1p3/blob/lti_advantage_test/src/Classes/Nrps.php
You can tell me the LMS is working to better help you, even though LTI is a standard LMSs often add their own flavor to the standard by doing a few different things.
I have particularly worked with moodle and canvas, if you are using another we could talk to be able to integrate the package and be able to solve possible errors, I work on this for my free time, it is for the love of art.

I'm still trying to understand the flow. I am more interested with grade passback. Where do you think is a good place to start?

First, you must understand that you cannot use LTI Advantage (Ratings Service) without first using "regular" LTI.
That is, it requires creating an LTI to launch your application under the standard and then consume the extra services, which should not be more complex than exchanging credentials and consuming a webservice. (The respective endpoints should come in the original LTI message).
Depending on the LMS you will be able to configure your LTI in different sections, for example: the course menu, a module resource, etc. There may be locations where the LMS limits the ratings service, try to start by setting your LTI to a location that should obviously be able to rate, such as a resource.
What I will say now may confuse you a bit, once you have an LTI message and store the respective data, you could consume the ratings service from your application without the need of an LTI launch (being outside of the LMS). This would be possible because you will have stored the respective data to consume the service, this will depend a little on the business model that your application implements and the technical solution that you use, I will tell you so that you know that it is possible.
Please note that for security reasons the ratings service is limited, this means that it works in its respective context. In other words, you will only be able to manage the rating of your tool instance (Add, View, Modify, etc.) you will not have access to other LMS ratings.
Where to start?
The first thing is to decide if you want to use this or another package, or build your own code from scratch.
If you want to build your code from scratch, go see the documentation for the LTI standard and LTI Advantage. Although it is somewhat confusing to understand, no one better than the creators of the standard to tell you how to do things.
If you want to use a package, possibly integrating is easier and you only have to configure things. This package as of today will fix all LTI integration, but does not yet have support for LTI Advantage. Being developed for Laravel, it has all the advantages of this framework.
I have not yet implemented LTI Advantage (which allows to obtain the list of members of a course and manage grades) to this package because it does not work properly for all LMS (differences from LMS to follow the standard).
If you use moodle, the consumption example I mentioned above should be enough to understand (with reverse engineering) how to consume an LTI Advantage service, if you use canvas, better combine the current LTI package with the Canvas Oauth package to work with the Canvas API.
If you are using another LMS, I have not tested its integration, if you manage to give me access to an account as a teacher, I could test the integration and correct possible exceptions.
Lately my work life consumes me, daily work, extra projects, all this to survive in Latin America. Although my time is limited lately I think I could help you in a better way, maybe with a step by step video of how to start with the development/configuration or something like that, but I need to know what LMS you are working on so that everything works properly.

Inactivity

Inactivity


FINOS transfer tasks

There are still some outstanding tasks that need to be worked on before transferring ownership of this repository to FINOS.

[ ] Rebrand the gh-pages website with FINOS branding
[ ] Follow the instructions on https://github.com/finos/project-blueprint to copy anything missing into this repository
[ ] Prepare an index.html redirect page that will become the file https://scottlogic.github.io/datahelix/index.html and will redirect browsers to https://datahelix.finos.org/

After the transfer has completed, we should also do the following:

[ ] Update the Markdown documentation so that any references to https://github.com/ScottLogic/datahelix are changed to https://github.com/finos/datahelix
[ ] Ask the Scott Logic IT team to make a note that they should never re-use the name "datahelix" for a GitHub repository.

Regarding the following step:

Follow the instructions on https://github.com/finos/project-blueprint to copy anything missing into this repository

The one step I'd probably omit in those instructions is the use of the README template - the current project README has a lot of useful and relevant content. I'd just update it with the FINOS 'shield' and ensure the CCLA related info is captured somewhere.

The one step I'd probably omit in those instructions is the use of the README template - the current project README has a lot of useful and relevant content. I'd just update it with the FINOS 'shield' and ensure the CCLA related info is captured somewhere.

I personally found the current README comprehensive and well written, so I agree that there is no need to strictly follow the template.
I'd suggest to add - right before the License section - a Contributing section that links to .github/CONTRIBUTING.md, so that potential contributors know where to start, requirements, best practices and conventions.

Code transfer done, new repo coordinates are https://github.com/finos/datahelix


Strange search pattern "<

So... I was experimentig with some softwares and mirror fs.
One of those got stuck in an infinite loop of trying to save a file.
It turns out that the software tries to backup the old file using this pattern: filename.[\d{4,4}].ext, before doing so, it asks for all the files matching that pattern, max the matches and move the old file.
it was failing because the search pattern it was using was filename."<.ext.
Now... that software works fine outside dokan mirror fs, substituting the "< with an * also makes it work again.
My doubt is that somewere in dokan there might be a double encoding, or an encoding error in general, any idea how to find it out ?

Before I try several different possibilities.... Can give a more concrete example on how to reproduce this?

Never tried c-mirror.
The software that does that is not open source, long to download and to install and very costly ... so i won't suggest trying with that, but i could try it for you, also i could try to build a demo software that behaves the same...

I've sent you an email with some details.

So i figured out how to test with mirror.c and it works, but the strange search pattern is still there
`###FindFiles 0384
FindFiles :C:\test\source
FindFiles return 7 entries in C:\test\source*
index from 0
FileMatch? : New Text Document.txt (Project1"<.rvt,0,0)
FileMatch? : Project1.0001.rvt (Project1"<.rvt,0,0)
=>return single entry
index to 1
index from 1
FileMatch? : New Text Document.txt (Project1"<.rvt,1,0)
FileMatch? : Project1.0001.rvt (Project1"<.rvt,1,0)
FileMatch? : Project1.0002.rvt (Project1"<.rvt,1,1)
=>return
FileMatch? : Project1.0003.rvt (Project1"<.rvt,1,2)
=>return
FileMatch? : Project1.rvt (Project1"<.rvt,1,3)
index to 3
index from 3
FileMatch? : New Text Document.txt (Project1"<.rvt,3,0)
FileMatch? : Project1.0001.rvt (Project1"<.rvt,3,0)
FileMatch? : Project1.0002.rvt (Project1"<.rvt,3,1)
FileMatch? : Project1.0003.rvt (Project1"<.rvt,3,2)
FileMatch? : Project1.rvt (Project1"<.rvt,3,3)
STATUS_NO_MORE_FILES`

Hi @TrabacchinLuigi ,
I don't know if you contacted @Rondom already but is it possible to reproduce this ourself ?
Otherwise there is processmonitor https://technet.microsoft.com/fr-fr/sysinternals/processmonitor.aspx that can display the request and probably can give some information about it.

Hi @Liryna

Actually yes, just download a copy of Revit from Autodesk, start a trial and try.
As you can see in this screenshot accordingly to procmon it search for filename.*.ext not filename."<.ext

Ok thanks you !
I don't have my computer right now with me.
Do you think you can try to reproduce this with a sample C win32 ?

The output i've attached before it was from mirror.c, i just started the one that get deployed with dokan and attached a debugger to it

I understood :heart: I was meaning a sample to execute a read dir with the same regex that I can use on the mirror later to reproduce it easily.

I can't 😭 i don't know how to code in c...
I could do it in C# but i fear that since every string is unicode in C# this won't happen...

Ok I see @TrabacchinLuigi , It will take some time until I find some :cry:

Sorry, forgot to report back here.
I tried reproducing by running
echo foo > Project1.0001.rvt
dir "Project1.*.svt

and I got the strange output in the mirror log. But the output of dir is as expected. So I thought that this is a display error.
Also from the procmon-screenshot you posted I can see that the response looks quite normal, from what I can see. It returns Project1.0001.rvt, Project1.0002.rvt, Project1.0003.rvt. Project1.0004.rvt. Or am I misunderstanding something? I think I do, because you write that replacing "< with * makes it work, i.e it is NOT "only a display error"
@Liryna Btw, do you know of any tool that takes a recording from ProcMon and converts it to code?. This would be extremely useful for reproducing things and creating regression tests.

Great!! i didn't expected that using cmd it would have failed too. thanks @Rondom 👍
I do actually get wrong string in Mirror.cs->FindFilesWithPattern, so, yes it's a problem...

I've managed to fork, clone, install WindowsSDK, WDK and build dokany 😌, but no way i can debug it 😞, i just don't have enough experience with building/debugging drivers...
But i do really think the problem could be here.

I read the C-Code that you linked and the replacement of characters seems to be by design, because that is how MS handles the filters internally (@Liryna correct me, if I am wrong). This is not documented anywhere. :-(
See https://github.com/dokan-dev/dokany/blob/062d80474d90627364d52fee437fe18d1c1c0442/dokan/directory.c#L616-L635
The problem does not occur with the C-Mirror, because it does not implement FindFilesWithPattern and the Dokan library does the filtering itself. So, I think that the C#-Mirror has to be fixed to account for the filtering characters and the filtering characters itself should be documented in both the C and the C# documentation of FindFilesWithPattern().

Some more details:
The string does change between the calls, if i set a break point here i get the right string

Instead setting a break point here  i get the wrong string. Going back in the call stack i can see this:

That's why i think it is a problem of c implementation not dotnet

As I said above I think the "wrong string" is in fact the right string, because this is how the filter-string looks in the kernel. See the link to the Microsoft-documentation described in the Dokan-Library-source-code I linked in my previous comment.  (This my knowledge gained from reading the comments in the C-Code linked and highlighted in the pervious comment)
Thus, the C#-Mirror needs to fixed. It might be difficult/impossible to convert from a Kernel-Pattern to a pattern supported by the .NET-API (because the <-character (DOS_STAR) has no equivalent in the .NET API DirectoryInfo.GetFiles Method (String)

Oh... I think I understood... so those chars have been mapped as DOS_DOT = " and DOS_STAR = < and the other one as >
So... Those chars are invalid as part of a path and could be safely substituted? I can provide a quick fix very soon and maybe a test too

Hi guys 😃 ,
Looks like you guys worked a lot on this 👍
I agree that it would be great to convert the DOS to regex that can be used in C# but we have to be sure to not make things "worst".

DokanIsNameInExpression is in the C-API, but is not exposed in the C#-API. Instead of calling the C-version from C# and induce overhead for converting types, it might be faster to use the C#-version, but it might also be the other way round. No matter what (C# or C), it would be useful to have that method in the dokan-dotnet as well.

I totally agree with this. If we cannot make a safe conversion we should use DokanIsNameInExpression.
I had no time to look on the PR @TrabacchinLuigi .

I could write some unit test if we find out what should match and what not 🔨

Could you guy please tell me why searchPattern is always "" when I try to use search box for searching (top right explorer)? In this case, I input "test". My expectation is searchPattern equal to "test".

Thanks!

What windows does isn't open sourced, so there is no sure answer.
We can guess that windows search enumerate folders, and open files to search the content of them.
By the way your statement is false, the pattern isn't always an empty string, i just tested, try to use a conditional breakpoint instead of a normal one

Sorry Luigi! I mean the pattern is * (the text editor has an issue for displaying *).

So it's just windows search enumerating files in folders and sub folders.
I guess this is because, windows search won't just look for the name of file but also content and metadata, so it will just ask the operating system for everything and internally do it's search with it's own criteria.

@TrabacchinLuigi, @Liryna
Is it any tests for this functionality (in unit tests and/or winfstest)?

I posted some links to tests in the PR or the issue, but they are under am incompatible licence

@magol There where, we removed them because licence as @Rondom said.

Ahh, now I remember. I have short memory 😣.
But it should not be that difficult to implement these tests ourselves. Incomplete tests are better than no tests at all.

Indeed. I have plans to run Microsoft's IFSTest which might cover come cases in a black-box manner. Challenge is to find a selection of tests that are sensible to support. There are quite a few more NTFS-specific tests in there.


Lineage Navigation Results into Error

Affected module
It is impact at UI definitely
Describe the bug
While user navigate through the lineage (Specially through column lineage) there is an error "Something Wrong (can't read the properties)"

To Reproduce
Please go thought the screen capture video of OM  sandbox instance
Expected behavior
Lineage-Render-Error-2.webm
Lineage-Render-Error-1.webm
Expected behavior is NOT to have such error
Version:
Version 1.1.0 as well as 1.1.1. I used sandbox instance of the OM https://sandbox.open-metadata.org/table/redshift_dbt.dev.dbt_jaffle.orders
Additional context
The issue been raised earlier but was not able to reproduce it. Now we can reproduce with the steps in the video
https://openmetadata.slack.com/archives/C02B6955S4S/p1688624707621659

Fixed with above issue.


Review HttpRequest Body Reading in HttpPipelinePolicies

Review HttpPipelinePolicy operations on how they consume HttpRequest bodies. Particularly, ensure that side-affect operations, such as authorization calculations, don't leave the ByteBuffer in a consumed state (where a call to remaining() is mutated). In cases where the body buffers need to be read a duplicate of the body buffer should be created and consumed to ensure the body is left in an unmodified state when the HttpClient goes to send the request.

Checked all exisiting classes that implements HttpPipelinePolicy. All ByteBuffer that used in the request is in the unmodifiable state.
Report:
Not ByteBuffer read operation Policies:

AddDatePolicy, AddHeadersFromContextPolicy, AddHeadersPolicy, ArmChallengeAuthenticationPolicy,  AzureKeyCredentialPolicy, AzureSasCredentialPolicy, BearerTokenAuthenticationChallengePolicy,
BearerTokenAuthenticationPolicy,  CookiePolicy, HostPolicy, OpenTelemetryHttpPolicy, PortPolicy, ProtocolPolicy, RecordNetworkCallPolicy, RequestIdPolicy, TimeoutPolicy, UserAgentPolicy

Policies that use the ByteBuffer:

HttpLoggingPolicy::writeBufferToBodyStream // already duplicate the ByteBuffer.
RetryPolicy:

  return httpResponse.getBody()
                            .ignoreElements() // since we ignore the elements, duplicate the byteBuffer doesn't needed here.
                            .then(attemptAsync(context, next, originalHttpRequest, tryCount + 1)
                                .delaySubscription(delayDuration));


Netlify builds do not work correctly

Because I modified the Patternfly theme to use Patternfly 4 in a non-standard way, when Netlify builds the site it ends up with a combination of Patternfly 3 and Patternfly 4 React components, so the site doesn't build correctly. I think I'm going to have to create a forked custom theme to fix that so that NPM downloads the stuff the way I need it downloaded.

Fixed. I pushed a pseudo fork of the patternfly theme to my personal github repo and then changed the package to require PatternFly4.  Technically that means the theme is broken, but I have already been overriding the broken parts of the theme in the site src directory, so the theme works fine. At some point we might want to both merge the overrides into the theme and move the theme package to the XBMC github area. But for now I'm gonna call this done.


pipeline.hgetall issue

const pl = redis.pipeline();
pl.hgetall('foo:0');
pl.hgetall('foo:1');
const result = await pl.flush();

console.log(result);
// => [ ArrayReply {}, ArrayReply {} ]

But hashes are exists.
Other command I didn't test.

Hi @akhimchak, thank you for opening this issue! I'd like to know about the behaviour you expect.
In that case, the flush() method currently returns results in the form of Array<ArrayReply>. In contrast, your expected result is Array<Array<string>>. Am I understanding your intension correctly? :thinking:

Hi @uki00a, I mean data exists in the redis, but when executed listed commands hashes are empty.
v0.19.0 works' fine in that case.

I've opened the PR #227 to fix this issue. Could you please review it when you have time? :slightly_smiling_face:


BHBC-1665: Comments removal

Overview
Links to jira tickets
https://quartech.atlassian.net/browse/BHBC-1665
This PR contains the following changes
Removal of “Comments” from Treatment Unit info page

Popup / Details View components

This PR contains the following types of changes

[ ] New feature (change which adds functionality)
[ ] Enhancement (improvements to existing functionality)
[ ] Bug fix (change which fixes an issue)
[ ] Misc cleanup / Refactoring / Documentation
[ ] Version change

Checklist
A list of items that are good to consider when making any changes.
Note: this list is not exhaustive, and not all items are always applicable.
General

[ ] I have performed a self-review of my own code

Code

[ ] New files/classes/functions have appropriately descriptive names and comment blocks to describe their use/behaviour
[ ] I have avoided duplicating code when possible, moving re-usable pieces into functions
[ ] I have avoided hard-coding values where possible and moved any re-usable constants to a constants file
[ ] My code is as flat as possible (avoids deeply nested if/else blocks, promise chains, etc)
[ ] My code changes account for null/undefined values and handle errors appropriately
[ ] My code uses types/interfaces to help describe values/parameters/etc, help ensure type safety, and improve readability

Style

[ ] My code follows the established style conventions
[ ] My code uses native material-ui components/icons/conventions when possible

Documentation

[ ] I have commented my code sufficiently, such that an unfamiliar developer could understand my code
[ ] I have added/updated README's and related documentation, as needed

Tests

[ ] I have added/updated unit tests for any code I've added/updated
[ ] I have added/updated the Postman requests/tests to account for any API endpoints I've added/updated

Linting/Formatting

[ ] I have run the linter and fixed any issues, as needed
See the lint commands in package.json
[ ] I have run the formatter and fixed any issues, as needed
See the format commands in package.json

SonarCloud

[ ] I have addressed all SonarCloud Bugs, Vulnerabilities, Security Hotspots, and Code Smells

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots, if applicable.

Just need to run the formatter, and then its good to go

Opting to remove the description attribute instead. See https://quartech.atlassian.net/browse/BHBC-1678


1.2 Setup database

Setup a MySQL database and setup some basic tables.

We have setup a mysql database with four tables: user, employee, product and shopping_cart. These tables have appropriate attributes with primary keys and foreign keys.

The product table should possibly include a link to an image of the product.


CI improvements

Summary
This updates the CI steps and removes Lando from the equation. This uses native composer to build Drupal and then pass that artifact into a web container to run the tests and ensure the profile installs correctly. This has greatly reduced the time to pass tests.
The ExistingSite tests have currently been disabled as those will need further investigation into the slowness, but that will be saved for another day.
Composer 2 support has been added to the local develop.sh script.
Metadata

Question
Answer

Did you use a meaningful pull request title?
yes

Did you apply meaningful labels to the pull request?
yes

Did you perform a self review first?
yes

Documentation reflects changes?
no

CHANGELOG reflects changes?
yes

Unit/Functional tests reflect changes?
sort of

Did you perform browser testing?
yes

Risk level
Low

Relevant links
RIG-139

Ok looks good pending rebase

/approved


Proposal: Improve metrics collector situation

Overview
This issue discusses two closely related issues:

Prometheus' request context is not inherited/re-used by UNMS API client.
The UNMS metrics contain metrics about the collector itself.

These issues arose when working on #23, where I needed to bump the UNMS API request timeout from 5 to 30s. This however is guess work, as the "correct" timeout value heavily depends on the number of devices in your UNMS instance.
Re-use HTTP request context in UNMS API client
It would be nice, if the (*Exporter).collectImpl() method (and (*Exporter).fetchDeviceData(), introduced in #22) could inherit and re-use the request context available in the handler (i.e. (*http.Request).Context()).
This allows the HTTP client (i.e. Prometheus) to abort the request, which in turn would stop all pending UNMS API requests (they will fail because the inherited context is already cancelled). It also removes the guesswork involved in choosing a "correct" timeout value.
However, since the Prometheus prometheus.Collector interface predates Go's context package, it is not easy to pass the request context down to the collector. One way to work around this issue, is to create a new prometheus.Registry in the HTTP handler and register a new Collector containing the context:
// omitting error handling
handler := func(w http.RequestWriter, r *http.Request) {
  target := r.URL.Query().Get("target")

  if instance, ok := targets[target]; ok {
    reg := prometheus.NewPedanticRegistry()
    reg.MustRegister(&contextWrapper{instance, r.Context()})
    promhttp.HandlerFor(reg, promhttp.HandlerOpts{
      ErrorLog: log,
    }).Serve(w, r)
    return
  }
}

This requires a few additions/changes
A new contextWrapper type (bringing the context.Context into the Exporter struct itself is not a good idea, because that requires getting rid of their internal metrics).
type contextWrapper struct {
  instance *Exporter
  ctx      context.Context
}

func (cw *contextWrapper) Describe(out chan<- *prom.Desc) {
  cw.instance.Describe(out)
}

func (w *contextWrapper) Collect(out chan<- prom.Metric) {
  cw.instance.CollectWithContext(cw.ctx, out)
}

Some changes to the Exporter struct:
// Keep prometheus.Collector interface intact (optional). Just pick
// a default timeout and pass it down.
func (e *Exporter) Collect(out chan<- prom.Metric) {
  ctx, cancel := context.WithTimeout(context.Backround(), 30*time.Second)
  defer cancel()

  e.CollectWithContext(ctx, out)
}

func (e *Exporter) Collect(ctx context.Context, out chan<- prom.Metric) {
  // fetch data from UNMS API
}

Untangle Exporter and UNMS metrics
The current registries contain both the prometheus.NewBuildInfoCollector() and prometheus.NewGoCollector() which yield (almost) duplicate data if you have multiple UNMS instances configured.
I'd like to take the opportunity to move them out of the UNMS metrics and into a separate exporter metrics set, and apply some common patterns found on other exporters:

/metrics would yield build info and Go metrics (go_info, go_gc_*, et.al.)
/metrics?target=name would yield only the UNMS data, as described in the README.
/ would return a simple HTML page listing the available instances, and linking to their metrics endpoint

For this the Prometheus scrape configuration will need an update.
scrape_configs:
- job_name: exporters
  static_configs:
    - exporter.example.org:9100 # node exporter
    - exporter.example.org:9806 # UNMS exporter
    - ...

- job_name: unms_exporter
  # for a static target "unms.example.org", rewrite it to
  # "exporter.example.org:9806/metrics?target=unms.example.org",
  # but keep "unms.example.org" as instance label
  relabel_configs:
    - source_labels: [__address__]
      target_label: instance
    - source_labels: [__address__]
      target_label: __param_target
    - replacement: 'exporter.example.org:9806'
      target_label: __address__
  static_configs:
    - targets:
      - my-unms-instance.example.org

I've implemented this pattern successfully in the past (our cambium-exporter and unifi-sdn-exporter work in a similar way). So far, I've not encountered any issues.
Feedback wanted
Before I start on this, I'd like to gather some feedback, including, but not limited to:

Would you be willing to accept a PR implementing this? These changes will undoubtedly cause work on your infrastructure (reconfigure Prometheus, maybe update Grafana dashboards).
Should I provide a single or multiple PRs? (I don't think separating this into multiple PRs would make much sense, however I'd be OK with putting in the extra work).

this sounds like a good idea, thanks for the thorough explanation!

fixed in https://github.com/ffddorf/unms-exporter/pull/33


Revert PR 19790 (breaks windowsTP4 CI on cache bust)

Signed-off-by: John Howard<EMAIL_ADDRESS>Revert "Combine SetupWorkingDirectory for Linux and Windows"
This reverts commit ec31741ca186278ea60faf49f85087c493e78806.
@darrenstahlmsft @swernli @tiborvass @vdemeester @thaJeztah . Ugh, #19790 has broken the ability to docker build if a RUN statement follows a WORKDIR statement. This has the consequence of breaking windowsTP4 CI if the cache is bust (such as it is on a couple of machines following the test of moving to GOLang 1.6RC1). I'll work with @darrenstahlmsft to get the right fix in on Monday. In the meantime, #19814 should not be merged.

Note that the move to golang 1.6rc1 has been postponed, but we may get CI setup for golang 1.6
/cc @tiborvass

@thaJeztah I know - it just so happened that it caused a builder cache bust which showed up this bug. Regardless, we really need to get this PR merged to ensure windowsTP4 reliability after some machines are now in a bad state.

ah damn :sweat:
LGTM 🙇

LGTM

ping @tiborvass @jfrazelle ptal!

Oh, we have two LGTM's

LGTM


Resolve local paths to defaults relative to the file, not CWD.

Closes #272.

Also, FTR, this matches the behaviour of !include.

Actually I think this patch must be wrong and the tests are passing by coincidence.

This is certainly a bug and we want to get this on Hackage soon.

Actually I think this patch must be wrong and the tests are passing by coincidence.

Huh?  I can't see why.

The second test is actually not testing what it should be testing; all paths are being resolved relative to the original package.yaml, no matter where the defaults.yaml that we're looking at is. Having a look now and seeing if I can fix it...

all paths are being resolved relative to the original package.yaml, no matter where the defaults.yaml that we're looking at is

Ah yes, you are right.
BTW, thew Windows build is broken.  I pushed a commit to master that should hopefully fix it.  Please rebase.

:thumbsup: Synchronicity!

This looks good, except for the tests...

Regarding tests, I think half of the story can be covered by https://github.com/sol/hpack/commit/ab2fcf92cb6d7a27e5ec88f6d9f036c576a84ecf.  This is generally a good idea, as we never want to depend on the current working directory (also relevant for e.g. glob expansion).
What is left is that we adjust the path for recursive defaults.  With the current code structure it's hard to unit test this; we could have a second acceptance tests for it, but adapting the existing acceptance test is what I did https://github.com/sol/hpack/commit/0698800099c498d0c46c545f2dacaee906690432.

@quasicomputational please don't be offended by my harsh comments.  I think it's completely fine to write these tests in the first place to probe for possible bugs.  But then we either throw them away or turn them into unit tests.  Acceptance test driven development has scalability problems, if we now do exhaustive acceptance testing then that would even go one step further.

An last but not least, thanks for fixing this!  I'll try to get it on Hackage soon, probably as a minor release, I consider this a bug fix, the old behavior was hardly useful.


Broken link on getting started guide - Alpine Linux Distro

Broken link in 'Alpine Linux Distro' in following sentence:
Now we will create a minimal node.js container based on the slim Alpine Linux distro. We do this by adding the following lines to our Dockerfile.
Link to section of GSG - https://www.balena.io/os/docs/raspberrypi3/getting-started/#Creating-a-Project-from-Scratch
Broken url - https://www.balena.io/os/docs/raspberrypi3/getting-started/link-to-alpine
Update - This is broken on all versions of the gsg so needs updating for each device type

The right link is https://hub.docker.com/r/balenalib/raspberrypi3-alpine-node
This is the page where the link is wrong https://github.com/balena-os/balena-os/blob/master/pages/docs/gettingstarted.md

Closing issue in favor of https://github.com/balena-os/balena-os/pull/560


[5.7] Prevent swift code samples from highlighting wrong keywords

Rationale: Fixes a syntax highlighter bug for swift code
Risk: Medium
Risk Detail: Changes the regex that matches some swift keywords.
Reward: High
Reward Details: Users will get properly highlighted code listings.
Original PR: https://github.com/apple/swift-docc-render/pull/325
Issue: rdar://93358075
Code Reviewed By: @mportiz08
Testing Details: Updated tests and tested manually in the browser

@swift-ci test


Update CS-RIN-RU-ENHANCED-external.user.js

I could only change the [Not on RIN] color and bring back the cracked text (not everyone will open it and change it after all) and added a Turkish name. 👍

I found how to change the color of all text.👍👍👍


Request Az.Resources version 3.0.0 and above to deploy azure template spec

I need to deploy Azure Resource Manager template spec.  The cmdlet I need are in Az.Resources 3.0.0 and above which requires Az.Accounts 2.1.0 and above.  I see the Az.Accounts imported by Azure Powershell version 5 is already 2.2.4, but deploy still failed with error
"The term 'Get-AzTemplateSpec' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again."
Please upgrade Az.Resources module to latest released version. Thanks.
Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Feature
Enter Task Name: Azure Powershell
list here (V# not needed):  Version 5
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises?

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
[Include task name(s), screenshots and any other relevant details]
I need to deploy Azure Resource Manager template spec.  The cmdlet I need are in Az.Resources 3.0.0 and above.  I see the Az.Accounts imported by Azure Powershell version 5 is already 2.2.4, but deploy still failed with error
The term 'Get-AzTemplateSpec' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
Please upgrade Az.Resources module to latest released version. Thanks.
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

@hannwang1 On hosted agent, the latest version of az module corresponds to 4.7.0 where Az.Resources module version is 2.5.1
Addition of latest az modules on hosted agent is in our backlog already.
Meanwhile  to unblock yourself, you can use private agent instead of hosted one, install the latest az module on your machine locally where Az.Resources version is 3.0.0 and then specify same version in your task in specify version block

Hi @20shivangi  we don't have private host.  When are you going to update the Az version.  Az 4.7.0 is more than 5 months old.  many releases will be blocked if you can only support 6 months old version.  Can you please provide a timeline when you are going to upgrade version? Or any other workaround I can use newer version in my task? Thanks.

Hi all,
I faced the same situation. As per this comment on #14160,  we don't need private agents and a simple step installing the desired version should be enough.

For Linux:
Save-Module -Path /usr/share/az_5.3.0 -Name Az -RequiredVersion 5.3.0 -Force -ErrorAction Stop
For Windows:
Save-Module -Path $env:SystemDrive\Modules\az_5.3.0 -Name Az -RequiredVersion 5.3.0 -Force -ErrorAction Stop


Hardcoded README.md (#31)

This update makes the README.md path dynamic using the SOFT_SERVE_README_PATH environment variable.

I fixed the build error

Thank you for this! It's really cool (and a good idea). Just a couple of notes... Can you change the ReadmePath arg in the function definitions to rmp to match the style of the rest of the args? Not the cfg.ReadmePath, that's fine. Also, can you squash this PR into a single commit?
Other than that, it looks and works great!

@toby alright, I only named it ReadMePath because it’s referred as that in other parts of the repo.

Not to throw a wrench into this, but does it make sense as a global?
After all where to put the readme (and its name) is repo specific and depends on the project/user.
Wouldn't it be better to make this configurable per repo?
Just to give you an idea, here's a pretty standard cgit config:
readme=:README.md
readme=:readme.md
readme=:README.mkd
readme=:readme.mkd
readme=:README.rst
readme=:readme.rst
readme=:README.html
readme=:readme.html
readme=:README.htm
readme=:readme.htm
readme=:README.txt
readme=:readme.txt
readme=:README
readme=:readme
readme=:INSTALL.md
readme=:install.md
readme=:INSTALL.mkd
readme=:install.mkd
readme=:INSTALL.rst
readme=:install.rst
readme=:INSTALL.html
readme=:install.html
readme=:INSTALL.htm
readme=:install.htm
readme=:INSTALL.txt
readme=:install.txt
readme=:INSTALL
readme=:install

@brunnre8
I could add it to this config:
https://github.com/charmbracelet/soft-serve/blob/e8b51469d2d72d7e86a648333553c4230f76b200/README.md#L62-L111
repos:
   - name: Lorem
     repo: ipsum
     private: false
     note: "dolor sit amet"
     readme: consectetur/README.md

You can now specify a readme file path per repository in your config.yaml, otherwise, it finds the first match of README* and uses for the readme. Example readme: docs/README.md
https://github.com/charmbracelet/soft-serve/pull/98
Closing...


fix(appsync-modelgen-plugin): associatedWith fields in bidirectional …

…hasone

Description of changes

When the CPK feature flag is ON, this change fixes the associatedWith fields that are generated in a Bi-Directional Has-One relationship.
We add an option shouldUseFieldsInAssociatedWithInHasOne to control this change and limit it only to the Model Introspection Visitor.
Issue #, if available

Description of how you validated changes
Unit tests
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] Breaking changes to existing customers are released behind a feature flag or major version update
[ ] Changes are tested using sample applications for all relevant platforms (iOS/android/flutter/Javascript) that use the feature added/modified

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #530 (6055503) into main (453e821) will decrease coverage by 0.00%.
The diff coverage is 90.90%.

@@            Coverage Diff             @@
##             main     #530      +/-   ##
==========================================
- Coverage   85.51%   85.50%   -0.01%
==========================================
  Files         148      148
  Lines        7263     7266       +3
  Branches     1909     1913       +4
==========================================
+ Hits         6211     6213       +2
- Misses        956      957       +1
  Partials       96       96

Impacted Files
Coverage Δ

...psync-modelgen-plugin/src/utils/process-has-one.ts
92.30% <80.00%> (-3.70%)
:arrow_down:

...odelgen-plugin/src/utils/process-connections-v2.ts
92.40% <100.00%> (+0.09%)
:arrow_up:

...rc/visitors/appsync-model-introspection-visitor.ts
98.71% <100.00%> (+0.01%)
:arrow_up:

...nc-modelgen-plugin/src/visitors/appsync-visitor.ts
96.24% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Bug]: Replace twitter with X

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

project 9: food ordering website
Expected behavior
.
Add ScreenShots
.
On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a SSOC'24 contributor
[X] I have starred the repository

@AkhilaSunesh u are assigned


[CARBONDATA-2753][Compatibility] Row count of page is calculated wrong for old store(V2 store)

Row count of page is calculated wrong for V2 store.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[x] Testing done
Manual Testing

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6249/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7497/

retest this please

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6256/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7507/

retest sdv please

retest sdv please

retest sdv please

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6024/

retest sdv please

LGTM

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6046/

Retest sdv please

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6054/

LGTM


Remove automatic assignation of bug issues, as it may brings to skip to proper triaging and not allowing others to contribute on the project.

Remove automatic assignation of bug issues, as it may brings to skip to proper triaging and not allowing others to contribute on the project.
Is still open-source, correct? If a bug is already assigned, nobody jumps-in @samwelkanda

PR: https://github.com/microsoftgraph/msgraph-sdk-python-core/pull/224


More specific recipe removal

I'd like to remove the ability to "recycle" rails back into ingots with the Stoked Crucible. Curretly, it seems like my only option is to remove all recipes that make iron, and then add them all back in just for that 1 item.
Please either add more specific recipe removal, or a way to overwrite the recipe with StokedCrucible.add(foo:bar, null, [minecraft:rail]);
The reason why this is a problem, is that other mods (like railcraft) are adding way to make track more efficiently, or with different material or items. I cant have a way to make infinite iron.

Also, Thanks so much for this mod. You all do an awesome job of fixing things quickly and answering requests. :)

https://github.com/elytra/Unabridged/issues/591

Yeah, I figured that would have ended up being an issue down the line. Just need to refine the solution, and the new version will have it possible for the cauldron, crucible, and mill under remove(output, [input]);.

Any way youd be willing to add a config to just remove that one recipe ... just for the time being?

or an if(railcraft) {tracks smelt to <railcraft:rail>} else {tracks smelt to <minecraft:iron_ingotl>}

It's pretty much all implemented on the MineTweaker part, just need to commit it and upload the update.

oh it's done already? great. I'll just wait patiently then, Thanks :)

Well that allowed me to shave about 50 lines from my CraftTweaker file.  Fantastic!

@WesCook knows what's up.


Renamed "specifity" to "specificity"

fixes #463

Thanks a lot - unfortunately, GitHub does not allow me to change the target branch. Any PR needs to target devel. Can you please re-submit? Please make sure you start from devel (there are already a lot of breaking changes to master, so merging them again would be a PITA).


Request: Include typescript definitions

Error: Could not find a declaration file for module 'svelte-confetti'.
<EMAIL_ADDRESS>implicitly has an 'any' type.
  Try `npm i --save-dev @types/svelte-confetti` if it exists or add a new declaration (.d.ts) file containing `declare module 'svelte-confetti';` (ts)
  import { Confetti } from 'svelte-confetti'

@types/svelte-confetti is not in the npm registry

I'll be interested in getting this done. @Mitcheljager will you be fine with it?

@LeoDog896 Absolutely! I don't tend to use Typescript myself, but I know a lot of others do.

@Mitcheljager would it be a wiser idea to make a standard SvelteKit skeleton library instead, that way the main demo page could benefit from SSR (as mentioned in the README)? I can make two separate PRs - one that moves the project to TS/SvelteKit (but leaves the core in JavaScript), then a second one with smaller diffs but more relevant code changes to move to TypeScript.

I don't think the demo page should be using Sveltekit, I want it to stay easy to manage via github pages. The project is meant for Svelte, with it working in SSR simply being a nice bonus.

Alright, thanks for the clarification!

For anyone reading this, I simply created a new file called ambient.d.ts in the src folder of my SvelteKit app with the following contents:
declare module 'svelte-confetti' {
  import { SvelteComponentTyped } from 'svelte';

  declare const __propDef: {
    props: {
      /**
       * The max size in pixels of the individual confetti pieces.
       * @default 10
       */
      size?: number;
      /**
       * The max horizontal range of the confetti pieces. Negative is left, positive is right. [-1, 1] would mean maximum of 200px left and 200px right.
       * @default [-0.5, 0.5]
       */
      x?: [number, number];
      /**
       * The max vertical range of the confetti pieces. Negative is down, positive is up. [-1, 1] would mean maximum of 200px down and 200px up.
       * @default [0.25, 1]
       */
      y?: [number, number];
      /**
       * Duration of the animation for each individual piece.
       * @default 2000
       */
      duration?: number;
      /**
       * If set to true the animation will play indefinitely.
       * @default false
       */
      infinite?: boolean;
      /**
       * Used to set a random delay for each piece. A large difference between each number will mean a longer spray time.
       * @default [0, 50]
       */
      delay?: [number, number];
      /**
       * Color range on the HSL color wheel. 0 to 360 is full RGB. 75 To 150 would be only green colors.
       * @default [0, 360]
       */
      colorRange?: [number, number];
      /**
       * Can be used to pick a random color from this array. Set just one array elements to have a single color. Accepts any viable css background property, including gradients and images.
       * @default []
       */
      colorArray?: number[];
      /**
       * Amount of particles spawned. The larger your spray the more pieces you might want. Be careful with too many as it might impact performance.
       * @default 50
       */
      amount?: number;
      /**
       * How many times the animation will play before stopping. Is overwritten by the "infinite" property.
       * @default 1
       */
      iterationCount?: number;
      /**
       * How far each piece falls. Accepts any css property, px, rem, vh, etc, but not 0.
       * @default "100px"
       */
      fallDistance?: string;
      /**
       * Set to true to make each confetti piece rounded.
       * @default false
       */
      rounded?: boolean;
      /**
       * Set to true to make the explosion appear in a cone like shape which might feel more realistic when dealing with a larger amount.
       * @default false
       */
      cone?: boolean;
      /**
       * Set to true to make the particles accelerate at a constant speed without "falling" down. Give it a more explosion like effect.
       * @default false
       */
      noGravity?: boolean;
      /**
       * A number from 0 to 1 that determines how far the particles spread horizontally. A low number will mean the x near the peak and the x near the end are similar.
       * @default 0.15
       */
      xSpread?: number;
      /**
       * By default the elements are removed when the animation is complete. Set to false to prevent this behaviour.
       * @default true
       */
      destroyOnComplete?: boolean;
    };
    events: {};
    slots: {};
  };

  type ConfettiProps = typeof __propDef.props;
  type ConfettiEvents = typeof __propDef.events;
  type ConfettiSlots = typeof __propDef.slots;
  export class Confetti extends SvelteComponentTyped<ConfettiProps, ConfettiEvents, ConfettiSlots> {}
}

@LeoDog896 feel free to use this inside the typings whenever you get the chance.

Thank you so much!


Handle Rely plots in Couchbase

Change CBAppDatabaseManager code to make Rely plots work in Couchbase.

Have completed code to make Rely plots work in Couchbase.


Remove some redundancy in date filter

I was looking #67, I had the filtering out of the Completes, that is fairly trivial. However doing the date based filtering would mean having some duplicated code.
I started on removing some of this redundancy.
However the little hiccup and thats because  that the focus of the filtering is on the 'due' whereas I would have to have this change to be the completed.
If this is accepted I will throw up another PR for the filter by day, as it has a little complication.
As far as I can see the test suite still passes.

@stuartskelton Sorry for the delay, I'll be reviewing this today!

No worries, I have been messing about with #75 while I was waiting.


Can't find or install brew formula: ripgrip

brew config output
HOMEBREW_VERSION: 3.1.11
ORIGIN: https://github.com/Homebrew/brew
HEAD: 9657303ed2c484a4bd8f7527706940f9e4d43354
Last commit: 3 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 776fcbf72fd0a20f673351cba5b9b825863fc144
Core tap last commit: 40 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3_2/bin/ruby
CPU: octa-core 64-bit kabylake
Clang: 11.0.0 build 1100
Git: 2.21.1 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 7.54.0 => /usr/bin/curl
macOS: 10.14.6-x86_64
CLT: <IP_ADDRESS>.1.1576735732
Xcode: N/A

brew doctor output
Your system is ready to brew.

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
Trying to install ripgrip (formulae.brew.sh link), but brew can't find it
What happened (include all command output)?
JZ-MBP:~ johnathan$ brew list ripgrip
Error: No such keg: /usr/local/Cellar/ripgrip
JZ-MBP:~ johnathan$ brew install ripgrip
==> Searching for similarly named formulae...
Error: No similarly named formulae found.
Error: No available formula or cask with the name "ripgrip".
==> Searching for a previously deleted formula (in the last month)...
Error: No previously deleted formula found.
==> Searching taps on GitHub...
Error: No formulae found in taps.
JZ-MBP:~ johnathan$
What did you expect to happen?
brew should've found ripgrip when using brew list, allowing me to run brew install for ripgrip on my Mac
Step-by-step reproduction instructions (by running brew commands)
JZ-MBP:~ johnathan$ brew doctor
Your system is ready to brew.
JZ-MBP:~ johnathan$ brew list ripgrip
Error: No such keg: /usr/local/Cellar/ripgrip
JZ-MBP:~ johnathan$ brew install ripgrip
==> Searching for similarly named formulae...
Error: No similarly named formulae found.
Error: No available formula or cask with the name "ripgrip".
==> Searching for a previously deleted formula (in the last month)...
Error: No previously deleted formula found.
==> Searching taps on GitHub...
Error: No formulae found in taps.
JZ-MBP:~ johnathan$

There really isn't any formula or cask called ripgrip. You might thinking of ripgrep instead.

omg 🤦♀  🤦♀  🤦♀  🤦♀
Well, that's 1.5 hours well spent 😛
Thanks a lot @gromgit !