"Handling the RFC command failed :("

See https://github.com/polkadot-fellows/RFCs/pull/8#issuecomment-1763848583
As instructed, I'm opening an issue.

Thanks for the report.
There hasn't been any changes since the last time it worked.
The error log suggests a permissions issue.
It's not clear but I suspect that it's a setting preventing first time contributors from running workflows. You are in fact opening a first PR in that particular repo, whereas the previous time it worked the author was already a contributor at that time.
I'll try to confirm this guess and think about what we can do with it.

Update: this is NOT a first-time-contributor thing as I initially thought.
It's still seems to be a permissions issue but I don't know what's missing (nothing's been changed on my side, and it continues to work in a testing org/repo).

The issue has been fixed and the PR is not merged: https://github.com/polkadot-fellows/RFCs/pull/8#issuecomment-1777113262
I have tried maany things and it's still not known what was the exact cause of this problem, all we know is that merging was denied by github on this particular PR.
The issue has been resolved by changing the merging code from using github's REST API to graphl API.


VMSS probe deploy fix

Which issue this PR addresses:
Recent deployment issues we saw in INT when trying out some changes to deploy.go.
What this PR does / why we need it:

Adds UpdateVMSSProbes() back in with corrections to the code that was checking for the CannotModifyProbeUsedByVMSS error.
Makes the check for the already familiar ResourceNotFound error with the load balancer more robust.
Adds nil checks before checking VMSS health statuses

These changes are needed to make the deployments more robust to failures. They should also make it easier to diagnose the root cause when we run into other types of failures.
Test plan for issue:
I wrote a script to duplicate the known LB ResourceNotFound error so that I could look at the structure of the Azure ServiceError and know how to get to the errors nested inside. I updated the unit tests accordingly.
Is there any documentation that needs to be updated for this PR?
No

@facchettos , would appreciate your review on this since you were originally working on addressing the issue with the health probes. 🙂

before we move forward, I think this needs a deeper discussion because we need a proper strategy if we ever decide to allow the probe to be changed. The problem is that the gateway has to be up all the time to allow for egress lockdown to work, and when we remove the probe from the vmss and loadbalancer, there will be some downtime which potentially will bring some down time because some of the gateways will be unresponsive for a while when updated and since they would have no probe, the load balancer probably won't know which ones are ready.

before we move forward, we should probably have a deeper discussion if we indeed plan on modifying the probes, since an update to the probe can lead to some downtime in the gateway which is required to be up for egress lockdown to work. Having the deployment failing if a probe is modified might be a safeguard in the end.

/azp run e2e

/azp run ci-go


IPP-2.1.test does not exist

Describe the bug
It seems the git repo does not come with ipp-2.1.test, causing failure when running make install.
To Reproduce
Steps to reproduce the behavior:

Clone libcups repo
Run ./configure
Run make
Run make install

Expected behavior
ipp-2.1.test should ideally exist in the repo
System Information:

OS: Debian 12 Bookworm (Raspbian)
Browser Chrome and Firefox
Version v3.0b2

Additional context
I attempted to simply make a copy of ipp-2.0.test, renaming it to ipp-2.1.test to no avail. I am attempting to install this beta version of OpenPrinting's CUPS because I couldn't get ssl working at all (even with default files) on CUPS 2.5b1.

I managed to find examples/Makerfile and edited it to remove the lines:
lib-2.1.test
lib-2.2.test
ipp-backend.test
ipp-everywhere.test
Which allows make install to run successfully, however /etc/cups is missing and the WebUI is unavailable.

Please note: the libcups repository only provides the CUPS library and "ipp" tools for a future set of CUPS servers (cupslocald and cupssharingd) in the other cups-xxx projects.

Also, please report your SSL issues to the CUPS project since that will be the next stable version.

version 6fe4c090763590df9084b726a1815198f2c3233e (3.0b2) fails with
======== install in examples ========
make[1]: Entering directory '/build/source/examples'
Installing sample ipptool files in /nix/store/hkvzza6c6zrfmmcl44p579r39rqw2riv-libcups-3.0b2/share/libcups3/ipptool...
install: ipp-2.1.test does not exist

version 7a8ff5880e878c10e2604c1a59897f8f77ff2d80 fails with
======== all in pdfio ========
make[1]: Entering directory '/build/source/pdfio'
make[1]: *** No rule to make target 'all'.  Stop.

OK, I've restored and updated the IPP/2.x conformance (and related) files for the 3.0rc1 release.


DWPose for preprocessing

This is an exciting project. Could you upload the DWPose code for preprocessing used in EMTD_dataset/preprocess.py

DWPose will provided asap

Would this allow to generate new poses like the contents of "assets/halfbody_demo/pose/01"?

#45

https://github.com/antgroup/echomimic_v2/blob/main/demo.ipynb


Remove the Handler only after it has initialized the channel

Motivation:
Previously, any 'relative' pipeline operations, such as
ctx.pipeline().replace(), .addBefore(), addAfter(), etc
would fail as the handler was not present in the pipeline.
Modification:
Used the pattern from ChannelInitializer when invoking configurePipeline().
Result:
Fixes #9131
If there is no issue then describe the changes introduced by this PR.

@RoganDawes can you please add a unit test ?

I see there is no test at all for handler/src/main/java/io/netty/handler/ssl/ApplicationProtocolNegotiationHandler :-(
I honestly don't know how best to set up the test, given that it requires a pipeline with an existing SslHandler in it to fire a handshakeCompletionEvent.

Can I add it here: https://github.com/netty/netty/blob/4.1/handler/src/test/java/io/netty/handler/ssl/SniHandlerTest.java#L366
Will an assert inside the handler work to break the build if it fails? Or should I use some other mechanism?

@RoganDawes I think it should work... why not try to add the assert stuff in there without your pr merged and see if it fails ;) ?

I added the assert as discussed, and got this in the test output:
09:10:07.751 [nioEventLoopGroup-3-1] WARN  i.n.channel.DefaultChannelPipeline - An exceptionCaught() event was fired, and it reached at the tail of the pipeline. It usually means the last handler in the pipeline did not handle the exception.
java.lang.AssertionError: null
	at org.junit.Assert.fail(Assert.java:86)
	at org.junit.Assert.assertTrue(Assert.java:41)
	at org.junit.Assert.assertNotNull(Assert.java:712)
	at org.junit.Assert.assertNotNull(Assert.java:722)
	at io.netty.handler.ssl.SniHandlerTest$3$1.configurePipeline(SniHandlerTest.java:367)

The tests failed, but not because of this assertion, rather because the countdownlatch did not reach the expected value. Would it not make more sense for the failure to be more specific as to the cause?
To do that, I guess there needs to be some way to propagate the exception from the event thread to the primary JUnit thread, right?

Ok, I addressed my issue above by using a CountDownLatch, and checking its value on the main thread.

Changed to use AtomicBoolean as suggested, which also resolved the braces on the if statement :-)

@netty-bot test this please

@RoganDawes please fix check style error:
13:04:30 [INFO] --- maven-checkstyle-plugin:3.0.0:check (check-style) @ netty-handler ---
13:04:32 [INFO] Starting audit...
13:04:32 [ERROR] /code/handler/src/main/java/io/netty/handler/ssl/ApplicationProtocolNegotiationHandler.java:103: an empty line before } [RegexpMultiline]
13:04:32 Audit done.

Done!

@netty-bot test this please

@RoganDawes thanks a lot!

Thanks Norman!


Update shifts-connector-ukg-known-issues.md

Updated on the: A user can’t perform some actions in Shifts in the Teams web app after signing in with a different account section

I'm no longer working on frontline. Assigning to Heidi


Create the GraphQL query for search flights (allFlights)

allFlights query
The first use will be in the search form. design
The allFlights query should handle the search for a flight.
The query should accept these parameters:

from:  (type Location, required)
to: (type Location, required)
date:(type Date, required)
returnDate: (type Date, optional)
passengers: (type Passengers, optional with default value adults=1)

Tequila API: https://tequila.kiwi.com/portal/resources/search_api
for inspiration: https://github.com/kiwicom/graphql/blob/master/src/flight/queries/AllFlights.js

Let's wait how we decide about having input types for queries and mutations

That link and article is it not more related to input types for mutations?
On a general basis, I would think of it as a function. Should I add a parameter to the function, or should it take an object as parameter?
If there is just one value say an id, it does not make that much sense to wrap it into an object does it? On the other hand, if I pass 5 parameters, it might be better to create an inputtype
@FilipMessa @michalsanger


[Feature]: Rich Text Editor (RTE) should expose toolbar icons for the TinyMCE loaded plugins

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Looking at the Rich Text Editor widget source code here:
https://github.com/appsmithorg/appsmith/blob/6accb9e364dc30a055cb7b9e68af826425b52eb3/app/client/src/widgets/RichTextEditorWidget/component/index.tsx#L76
It appears that a multitude of great TinyMCE editor plugins are loaded, however the configured toolbar buttons here:
https://github.com/appsmithorg/appsmith/blob/6accb9e364dc30a055cb7b9e68af826425b52eb3/app/client/src/widgets/RichTextEditorWidget/component/index.tsx#L39
... do not include (expose) the features from the plugins. I am specifically interested in the image/media plugin and some others. Is it possible to expose the configuration of the toolbar buttons and/or plugins for a given RTE widget via the properties panel. I think it would be fine to pass through a JS snippet if that makes the feature happen sooner.
Why should this be worked on?
TinyMCE is configured to load plugins, however the plugin features are not exposed via the hard-coded toolbar config. Our team needs these buttons in the toolbar.

One additional note: It seems inefficient (network/cpu-wise) to load these plugins that cannot effectively be used by the editor because the buttons aren't included in the toolbar. I recognise that one wouldn't want ALL the buttons in the toolbar b/c it would be overwhelming to most, however there should be a way to specify which buttons are in the toolbar.

@ruswerner thanks for the feature request and the additional context you have provided 🎉 . we will start looking into this now and prfioritize the fix.

@Tooluloope is there a reason these are not exposed today?


package.json lists node version as 0.10.32

"engines": {
  "node": "0.10.32"
}

I'm not sure what tools use this field, but I know I looked at it to determine which version of node I should be using.
Things did not work for me on 0.10. They did on v4.1.1. It looks like you have travis config for v0.12 and v4, implying that you may not intend to build on v0.10 anymore.
It might be worth bumping this value in package.json if that is the case.

That was for Heroku, but you're right, worth bumping.


Add change log to enumerate/save when instruments last passed testing (with drivers test unit).

drivers should also have versions and last tested date / properties

Versioning for drivers:
0.1
first decimal represents: breaking changes
second decimal represents: bug fix, new feature e.g. new property or edit ranges


[openimageio] build with libraw support

Patch OpenImageIO CMake files to find 'libraw' as distributed by vcpkg ('raw_r' library renamed to 'raw') and enable libraw support in OIIO build.

I think better support as Feature https://github.com/Microsoft/vcpkg/blob/3c16addc8c8e6516a4ee4edcb11c1fc428d38b96/ports/ceres/CONTROL#L7-L9

https://github.com/Microsoft/vcpkg/blob/3c16addc8c8e6516a4ee4edcb11c1fc428d38b96/ports/ceres/portfile.cmake#L29-L32

Thanks for the PR!


Two small improvements for trace_model.py

When initial working with demo/trace_model.py I ran into a couple of changes needed, so I could run the code.
I wanted to submit this PR because maybe it takes a few iterations to fully flush out running trace_model.py, so I wanted to submit some small changes to start off.
First
The image paths were for local images. I didn't have the local images, so I changed the images to flickr URLs. One is from demo/Mask_R-CNN_demo.ipynb and the other is from COCO dataset 2014 training data.
Second
I initially tried to run:
python demo/trace_model.py

but I got an error for "../configs/caffe2/e2e_mask_rcnn_R_50_FPN_1x_caffe2.yaml" because it was using a relative path, so you have to be in the demo dir to run the code. This is an optional fixup, but now you can run the script from the demo dir or from the project home dir.
References
https://github.com/facebookresearch/maskrcnn-benchmark/pull/138

Thanks!


Set-OMEPowerState missing Devices module

Set-OMEPowerState.ps1 is missing the line below at the beginning.
using module ....\Classes\Device.psm1
Without this line the error below is given.
Unable to find type [Device].

Working on a new release, I'll get this fixed. Thanks!

Hi Trevor,
When do you plan the next release?
DO you have a roadmap, what is going to be in the new release?
It is a pity that the module is not in powershell gallery and that versioning in the manifest is not maintained.

The plan is to have it in the PowerShell Gallery eventually. Working on that. This is a side project for me and things will happen when they happen.
I'm currently working on incorporating all the scripts from https://github.com/dell/OpenManage-Enterprise in to the module.

Fixed in latest release https://github.com/dell/OpenManage-PowerShell-Modules/releases/tag/2.3.1
Added a better CHANELOG and started testing git actions for publishing to PSG.


docker network create --option="foo.bar=baz"

Provide a way to pass options to the network driver when creating a new user defined network. This would allow (for example) the setting of the com.docker.network.bridge.name allow friendly bridge names on the host and allow the re-use of an existing bridge on the system (providing the bridge driver handled this correctly).
Example use:
docker network create --option="com.docker.network.bridge.name=br0" br0

Environment
Environment information, although not sure its needed for a feature request:
docker version
Client:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   4419fdb-dirty
 Built:        Thu Nov 26 02:50:10 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   4419fdb-dirty
 Built:        Thu Nov 26 02:50:10 UTC 2015
 OS/Arch:      linux/amd64

docker info
Containers: 2
Images: 38
Server Version: 1.9.1
Storage Driver: btrfs
 Build Version: Btrfs v4.0.1
 Library Version: 101
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.3.0-coreos
Operating System: CoreOS 877.1.0
CPUs: 4
Total Memory: 7.655 GiB
Name: corefs
ID: ANSS:L2RV:YMWL:ZS3X:VYHV:GGQC:HGXQ:YUZS:LGFT:4OXN:QC4P:63G4

uname -a
Linux corefs 4.3.0-coreos #2 SMP Thu Nov 26 02:49:19 UTC 2015 x86_64 Intel(R) Core(TM) i3-2100 CPU @ 3.10GHz GenuineIntel GNU/Linux

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Taking a look at source code now and it seems something may already be implemented, or in the works.
api/client/network.go seems show that the network create subcommand takes a -o or --opt argument.
It seems to be present in the client version I am using and so I am testing functionality now.
This also appears to be documented so I am not sure how I overlooked this.

Seems to work as expected. Sorry for the mix up. Thanks.


Error Crawlee + Cheerio Apify Typescript template

Which package is this bug report for? If unsure which one to select, leave blank
None
Issue description
Using the apify create command and selecting Typescript > Crawlee + Cheerio gets me the following error.
PS C:\Users\name\Documents\apify> apify create
? Name of your new Actor: test4
? Choose the programming language of your new Actor: TypeScript
? Choose a template for your new Actor. Detailed information about the template will be shown in the next step. Crawlee + Cheerio
? Do you want to install the following template?
 Crawlee + Cheerio:
 A scraper example that uses Cheerio to parse HTML. It's fast, but it can't run the website's JavaScript or pass JS anti-scraping
challenges. Install template
Run: npm.cmd install --omit=optional
npm warn deprecated<EMAIL_ADDRESS>This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm warn deprecated<EMAIL_ADDRESS>Glob versions prior to v9 are no longer supported
npm warn deprecated<EMAIL_ADDRESS>Rimraf versions prior to v4 are no longer supported
npm warn deprecated<EMAIL_ADDRESS>Use @eslint/config-array instead
npm warn deprecated<EMAIL_ADDRESS>Use @eslint/object-schema instead
npm warn cleanup Failed to remove some directories [
npm warn cleanup   [
npm warn cleanup     'C:\\Users\\name\\Documents\\apify\\test4\\node_modules',
npm warn cleanup     [Error: EPERM: operation not permitted, rmdir 'C:\Users\name\Documents\apify\test4\node_modules\eslint-plugin-react'] {
npm warn cleanup       errno: -4048,
npm warn cleanup       code: 'EPERM',
npm warn cleanup       syscall: 'rmdir',
npm warn cleanup       path: 'C:\\Users\\name\\Documents\\apify\\test4\\node_modules\\eslint-plugin-react'
npm warn cleanup     }
npm warn cleanup   ]
npm warn cleanup ]
npm error code 1
npm error path C:\Users\name\Documents\apify\test4\node_modules\esbuild
npm error command failed
npm error command C:\WINDOWS\system32\cmd.exe /d /s /c node install.js
npm error C:\Users\name\Documents\apify\test4\node_modules\esbuild\install.js:134
npm error     throw new Error(`Expected ${JSON.stringify(versionFromPackageJSON)} but got ${JSON.stringify(stdout)}`);
npm error           ^
npm error
npm error Error: Expected "0.23.1" but got "0.21.5"
npm error     at validateBinaryVersion (C:\Users\name\Documents\apify\test4\node_modules\esbuild\install.js:134:11)
npm error     at C:\Users\name\Documents\apify\test4\node_modules\esbuild\install.js:282:5
npm error
npm error Node.js v20.17.0
npm error A complete log of this run can be found in: C:\Users\name\AppData\Local\npm-cache\_logs\2024-10-01T19_21_37_128Z-debug-0.log
    Error: npm.cmd exited with code 1

If I afterwards try to use apify run I get:
Error: Actor is of an unknown format. Make sure either the 'package.json' file or 'src/__main__.py' file exists or you are in a migrated Scrapy project.

Code sample
No response
Package version
3.5.4
Node.js version
20.17
Operating system
Windows
Apify platform

[ ] Tick me if you encountered this issue on the Apify platform

I have tested this on the next release
No response
Other context
No response

Tried again today without making any changes and worked without any issues. So, perhaps this is solved? No idea why I got the error in the first place.

Error: Expected "0.23.1" but got "0.21.5"

This is the error that breaks, and it's from esbuild (so most likely from tsx), not from our code. Sounds like you managed to install different versions of esbuild somehow.
Closing, since this is not a problem in the tooling either. And next time please be more patient and don't open issues about the very same thing in multiple places.


Bump ember-cli-moment-shim

@jasonmit I could not find a changelog for your addon, could you give me a short summary what changed between the versions?

Any update @jasonmit?

The more significant of the changes is it pulls in moment/moment-timezone from npm instead of bower.  In >= 1.1.0, fastboot is supported.

cc @stravid

Thanks for the explanation @jasonmit!


Add setting.AppSubUrl for LANDING_PAGE = explore

I have an URL containing a /git/ subdirectory for the service.
If I change LANDING_PAGE to explore, I'm redirected to /explore instead /git/explore.
This patch add AppSubUrl to LANDING_PAGE.

Thanks for your PR!
But we don't accept PR to master, please re-PR to develop!

No problem, I created another one #1055


[MHR] Change Interpid-heart icon

Having Interpid Heart and Frostcraft use the same icon makes it impossible to tell which one is charging up

HunterPie uses the same icons the game does, any suggestions?


OpenSSL.SSL.Error on booting

I re-installed my Workflow to v2.9.1 after deleting v2.4.0 by kubectl delete ns deis.
some values extracted by helm inspect values deis/workflow | sed -n '1!p' > values.yaml are modified.
And I deployed Deis by calling helm install deis/workflow --namespace deis -f values.yaml
Database migration looks done with success. But I got an exception from Django.
The log is https://gist.github.com/monaka/317b5e812bb10337301896f37d2ef2f7
Even though this error, Controller is keep running. Builder is also running but '0/1' ready. Router is under CrashLoopBackOff.
Where is the checkpoint against this issue?

Additional information. My K8s API endpoint is encrypted with self signed certs.

Conrtoller expects /var/run/secrets/kubernetes.io/serviceaccount/ca.crt available. But there have no ca.crt on my K8s. It seems to be caused my misconfiguration. Close this by myself.

I got still an error after adding ca.crt. I guess this is caused SSL verification.
Can I make skip verification steps? like
context.verify_mode = ssl.CERT_NONE
context.check_hostname = False

Or can I set URL to API server using an environment variable or something?
  File "/usr/local/lib/python3.5/dist-packages/requests/packages/urllib3/util/ssl_.py", line 308, in ssl_wrap_socket
    context.load_verify_locations(ca_certs, ca_cert_dir)
  File "/usr/local/lib/python3.5/dist-packages/requests/packages/urllib3/contrib/pyopenssl.py", line 392, in load_verify_locations
    self._ctx.load_verify_locations(cafile, capath)
  File "/usr/local/lib/python3.5/dist-packages/OpenSSL/SSL.py", line 525, in load_verify_locations
    _raise_current_error()
  File "/usr/local/lib/python3.5/dist-packages/OpenSSL/_util.py", line 48, in exception_from_error_queue
    raise exception_type(errors)
OpenSSL.SSL.Error: []

it's not in a release yet, but https://github.com/deis/controller/pull/1158 will enable that.

@bacongobbler I see. #1158 is it to me. I'll build Controller image with the master branch and continue  upgrading.
Thank you for your support.

I pulled quay.io/deis/controller:git-184b54c image and run with K8S_API_VERIFY_TLS `"false".
It worked well.
I see it should be regenerated my CA later. Thanks.


Allow not only IP address as host parameter

Description:
Hue configuration has previously allowed specifying FQDN as host parameter, but recent changes has changed that.
This is understandably inconvenient for users which depend on being able to use FQDN and not IP addresses.
This small change allows FQDN for host parameter again.
It would be nice to update the documentation to reflect this, and I'm more than happy to do so if this small change is accepted.
Related issue (if applicable): fixes #13575
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
hue:
  bridges:
    host: hue-fqdn.example.com

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @feitingen,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Discovery uses IP addresses and we need to match it up. This simple PR is thus not so simple and actually breaks logic.

Bummer.
Thanks for looking at it anyways.

I'm open for a PR to re-introduce host names that covers all the edge-cases and includes tests for them.


[v2.9] Bump hosted operators to v1.9.2

Bumping hosted operators versions ahead of the 2.9.2 release.

LGTM


do not require github insights widgets to prompt login

do not require github insights widgets to prompt login
:heavy_check_mark: Checklist

[ ] Added tests for new functionality and regression tests for bug fixes
[x] Added changeset (run yarn changeset in the root)
[ ] Screenshots of before and after attached (for UI changes)
[ ] Added or updated documentation (if applicable)

Is this PR still relevant?

It is still relevant, but Ill close it for the moment. Maybe Ill get time again to fix the tests.


Fixes 4670: show last error and status for templates

Summary

Adds a status column to the template list page
Displays any errors if there's an Invalid status

Additional changes

Also adds 2 commands to run the app against stage or prod (npm run start:stage and npm run start:prod) without needing to select from a menu - thank you @marusak for the suggestion!

Testing steps

Add a custom and RH repository, let them snapshot, and create a template using latest content. The template status in the UI should change to In progress and then Valid once the update-template-content task completes.
Edit the custom repo URL used in the template to create a new snapshot. Once the snapshot finishes, stop the server and remove the pulp URL. Restart the server. The template status should now be Invalid and display the error from the update-latest-snapshot task when clicking on it.
Edit the template. The status should still be Invalid, with the error from the update-template-content task displayed on click.

For QE - this PR can only be tested in stage since it depends on candlepin. It would also be difficult to simulate an error here, so validating the happy path should be sufficient.

https://issues.redhat.com/browse/HMS-4670

Hi
If you create the template before the included RHEL repos, e.g. Ansible, are introspected the template shows Invalid and never becomes Valid after the RHEL repo reaches Valid status. Is that out if scope?

hi @swadeley, i think this is the same issue as the associated backend PR, because candlepin isn't in ephemeral and so the task that the status depends on doesn't run. i can adjust the logic here to check if we're in ephemeral so that the status isn't always Invalid, but we may only be able to test this fully in stage

Hi
nothing broken in ephemeral, merging to test further in stage.


Shard Travis job into three pieces

@Hixie @yjbanov

Nice!

Before: Ran for 9 min 38 sec
After: Ran for 5 min 58 sec

The long pole is generating the dartdocs.


Gradle doesn't want to pull a specific dependency from a repo it already pulls other dependencies

Expected Behavior
Gradle pull the dependency properly
Current Behavior
Gradle doesn't pull the dependency from the maven repository; instead, pulling the dependency from a local cache which is outdated.
Context
Discord bot. The specific dependency it doesn't want to pull is a command manager for a discord bot.
Steps to Reproduce
Try pulling dependency com.mrivanplays:JDCF:1.0.4-SNAPSHOT from maven repository https://repo.mrivanplays.com/repository/ivan/
See it doesn't work.
Your Environment
IntelliJ IDEA 2019.3 Ultimate.
This is the build.gradle I use to deploy the dependency I talk about if it has something to do with the issue.

The most likely explanation is that another repository, declared first, provides that version as well with an outdated version.
Can you please add information on which repositories are declared in the build that pulls the dependency? If it contains mavenLocal(), please have a look at this mention in the documentation.
Are you also aware of the caching done by Gradle and the default 24 hours cache time for SNAPSHOT? See the documentation on how to tweak that behavior.

The most likely explanation is that another repository, declared first, provides that version as well with an outdated version.
Can you please add information on which repositories are declared in the build that pulls the dependency? If it contains mavenLocal(), please have a look at this mention in the documentation.
Are you also aware of the caching done by Gradle and the default 24 hours cache time for SNAPSHOT? See the documentation on how to tweak that behavior.

No I'm not aware of the caching, I'm new to gradle and I'm curious for every setting I do. I recently switched from maven.
I've removed the mavenLocal() repository but it still doesn't pull it. The repository from where it should pull this dependency is this as mentioned in my initial post on this issue. I'll try applying that setting you told me about the caching. Ill update as soon as I receive any news

The settings suggested fixed everything! Thanks


Quick info panel shows wrong namespace

repro steps:

create solution
add Sample.Core project
add referece to Sample.Core
build
add open Sample.Core
Quick info panel shows wrong namespace (expected Sample.Core but Microsoft.FSharp.Core)

note: tooltip shows correct namespace.

What does tooltip show?

It's very likely that we don't pass enough contextual info to FCS. They may expect long identifiers but we pass short identifier at https://github.com/fsprojects/VisualFSharpPowerTools/blob/e736aa38668d182696b04dd2f6e6dd637b2515f1/src/FSharpVSPowerTools.Logic/QuickInfoMargin.fs#L101.
We may have Sample.Core in the tooltip results, but it could be truncated before displaying in QuickInfo panel.

It seems it works ok for me:

Try to rebuild the solution and reopen VS.

I can repro following the steps above.
(Sent Vasily a repro solution by email.)


Any plan for Regression?

Any plan for adding Regression class?

This should be done - with no guarantees of performance!
I'll close this for now.


Encode Non-ASCII HTTP Headers

To address the issue in #38, add %-encoding of HTTP headers that are non-ascii.
(While latin-1/iso-8859-1 headers are technically allowed, to be extra safe all non-ascii headers are encoded)
If a header such as:
Content-Disposition: attachment; filename="Lancement du Système d’Échange Local (SEL).pdf"

it will be converted to:
Content-Disposition: attachment; filename*=UTF-8''Lancement%20du%20Syst%C3%A8me%20d%E2%80%99%C3%89change%20Local%20%28SEL%29.pdf

before being written to WARC (per https://tools.ietf.org/html/rfc8187#section-3.2.3)
Adds StatusAndHeaders.to_ascii_bytes() to perform this conversion where necessary.

Coverage increased (+0.003%) to 99.844% when pulling eca81826c40ee012b75dd4bd54ad01b04b98740c on percent_encode_non_ascii_headers into 3f6e2d792cc20af001a22fd0515c4a6ebc45dc27 on develop.


FileNotFoundError terminating yt-dlp early

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.04.08 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
When downloading a video using the below command line, I am getting an error at the end of the script that appears to be related to yt-dlp not being able to find a file it downloaded.  I have tried passing in the -k flag to be sure that all files were kept and not deleted, but the error persists regardless.
I do still appear to get a complete .mkv file with no issues despite the error.  But I'm not sure if there's any post-processing yt-dlp should be doing since the program terminates early.
I am on Windows 10 and installed via the binary in the repo description.  I have tried this on both an actual TV episode (from OutTV) and also from a Twitter video and they both result in the same error.
Verbose log
[debug] Command-line config: ['-vU', 'https://manifest.prod.boltdns.net/manifest/v1/hls/v4/aes128/5330942435001/558a1ff1-dce4-4239-ba78-44b5b3edf51c/10s/master.m3u8?fastly_token=NjI3ODQyZGNfMWE0NjRlNmEzYzUwNGNkZWY2ODNlMTFjYmRkYTc4M2Y2ODkyNDE1NGVkY2ViZDgwZGU0MDJiNjQ2MDkxMDM1Yg%3D%3D', '--cookies', '', '-f', 'bestvideo[height=1080]+bestaudio/best', '--convert-subs', 'srt', '--embed-subs', '--remux-video', 'mkv', '--compat-options', 'no-attach-info-json', '-o', '%(title).200s.%(ext)s']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2022.04.08 [7884ade] (win_exe)
[debug] Compatibility options: no-attach-info-json
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19044-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: avprobe -bsfs
[debug] exe versions: ffmpeg 4.2.3
[debug] Optional libraries: brotli, certifi, Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2022.04.08, Current version: 2022.04.08
yt-dlp is up to date (2022.04.08)
[debug] [generic] Extracting URL: https://manifest.prod.boltdns.net/manifest/v1/hls/v4/aes128/5330942435001/558a1ff1-dce4-4239-ba78-44b5b3edf51c/10s/master.m3u8?fastly_token=NjI3ODQyZGNfMWE0NjRlNmEzYzUwNGNkZWY2ODNlMTFjYmRkYTc4M2Y2ODkyNDE1NGVkY2ViZDgwZGU0MDJiNjQ2MDkxMDM1Yg%3D%3D
[generic] master: Requesting header
[debug] Identified a direct video link
[generic] master: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Downloading subtitles: en-CA
[info] master: Downloading 1 format(s): 4279+audio-3-en__Main_
[info] Writing video subtitles to: master.en-CA.vtt
[debug] Invoking downloader on "https://manifest.prod.boltdns.net/manifest/v1/hls/v4/clear/5330942435001/558a1ff1-dce4-4239-ba78-44b5b3edf51c/b684919c-71f1-4385-aaaf-9fe9ecaa4a10/rendition.m3u8?fastly_token=NjI3NzQ3MzNfYzg2ZjNjNGMxZjVjYmRkOWRiM2Y3YTY1MDU5NjY4ZTU4NjU5NmRlMGFlOGUzZTJiZjBhZTIyOTEwODNkNjljYQ%3D%3D"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 45
[download] Destination: master.en-CA.vtt
[debug] File locking is not supported on this platform. Proceeding without locking
[download] 100% of 53.38KiB in 00:01
[SubtitlesConvertor] Converting subtitles
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:master.en-CA.vtt" -f srt -movflags "+faststart" "file:master.en-CA.srt"
Deleting original file master.en-CA.vtt (pass -k to keep)
[debug] Invoking downloader on "https://manifest.prod.boltdns.net/manifest/v1/hls/v4/aes128/5330942435001/558a1ff1-dce4-4239-ba78-44b5b3edf51c/4dd88f44-6222-4a03-a47e-01a24e888bd6/10s/rendition.m3u8?fastly_token=NjI3NzQ3MzNfYzU0NTkxMWI2OTc4YTBlZjAxOTg3MzkwYjg1MGFkODY1MDM0ZjczYzA5NDgwZTQ1Y2I1YzBmZGVmNThhZGYyMg%3D%3D"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 133
[download] Destination: master.f4279.mp4
[download] 100% of 599.40MiB in 00:51
[debug] Invoking downloader on "https://manifest.prod.boltdns.net/manifest/v1/hls/v4/aes128/5330942435001/558a1ff1-dce4-4239-ba78-44b5b3edf51c/307c5afa-26c2-4443-a535-214ecb3b7aee/10s/rendition.m3u8?fastly_token=NjI3NzQ3MzNfZWI4ZmM2MTg4MDUxMTg4NDlmZTJmODk3ODVjNGM2Mzc4MDFlMjk0ZDAzNDA5Y2U3ZTNkODA1NTZiY2ZhNmFiMg%3D%3D"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 134
[download] Destination: master.faudio-3-en__Main_.mp4
[download] 100% of 36.49MiB in 00:15
[debug] ffmpeg command line: ffmpeg -i "file:master.faudio-3-en__Main_.mp4"
[Merger] Merging formats into "master.mp4"
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:master.f4279.mp4" -i "file:master.faudio-3-en__Main_.mp4" -c copy -map "0:v:0" -map "1:a:0" "-bsf:a:0" aac_adtstoasc -movflags "+faststart" "file:master.temp.mp4"
Deleting original file master.f4279.mp4 (pass -k to keep)
Deleting original file master.faudio-3-en__Main_.mp4 (pass -k to keep)
[VideoRemuxer] Remuxing video from mp4 to mkv; Destination: master.mkv
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:master.mp4" -map 0 -dn -ignore_unknown -c copy -movflags "+faststart" "file:master.mkv"
Deleting original file master.mp4 (pass -k to keep)
[EmbedSubtitle] Embedding subtitles in "master.mkv"
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:master.mkv" -i "file:master.en-CA.srt" -map 0 -dn -ignore_unknown -c copy -map "-0:s" -map "1:0" "-metadata:s:s:0" "language=eng" -movflags "+faststart" "file:master.temp.mkv"
Deleting original file master.en-CA.srt (pass -k to keep)
Traceback (most recent call last):
  File "yt_dlp\__main__.py", line 19, in <module>
  File "yt_dlp\__init__.py", line 869, in main
  File "yt_dlp\__init__.py", line 862, in _real_main
  File "yt_dlp\YoutubeDL.py", line 862, in __exit__
  File "yt_dlp\utils.py", line 1583, in save
FileNotFoundError: [Errno 2] No such file or directory: ''
[16404] Failed to execute script '__main__' due to unhandled exception!

Filname '' was given in command as the location of cookie jar, see: --cookies ''.

Well that'll do it :) Thanks!

Did you ever get OutTV to work?  I get unsupported URL


Incompatible with old glibc

The javet native part is incompatible with older glibc versions.
When running on ubuntu 16.04 / 18.04 or centos 7 / 8 (which is equivalent to redhat enterprise linux), you got an UnsatisfiedLinkError: /tmp/javet/86/libjavet-v8-linux-x86_64.v.1.0.7.so: /lib64/libstdc++.so.6: version GLIBCXX_3.4.26' not found (required by /tmp/javet/86/libjavet-v8-linux-x86_64.v.1.0.7.so)`.
I created a minimal hello world test and run it agains multiple linux versions:
https://gitlab.com/tristanlins/javet-compatibility-test/-/pipelines/445343180
As you can see, all dists older than ubuntu 20.04 are not compatible.
I run into very similar problem, with my jsass library.
I solved this by using centos 7 in docker to build the native library.
As far as I understand the build process, it should work to replace ubuntu 20.04 in the dockerfile with an older / other distri.

GLIBCXX_3.4.26
Hope #26 be helpful to you.
Ubuntu 16.04 / 18.04 / CentOS 7
Please refer to this doc. Basically, private builds are required. I haven't created private builds for almost 7 months. You are welcome trying the legacy builds and let me know if they work. Actually, some Javet users managed to have the legacy builds work successfully on those Linux distributions.
Building Javet on Ubuntu 16.04 is a different story. The dockerfile won't work. Even to me, I will have to spend tens of hours figuring out how to build the latest version on Ubuntu 16.04 nowadays.

Sorry, I missed the discussion (only searched for issues :see_no_evil: ).
I gave it a try and close this issue for now.

No problem. I just submitted this commit to the dev branch. Do you mind trying it in your env?

I will try out

Looks good: https://gitlab.com/tristanlins/javet-compatibility-test/-/pipelines/445405631

Sorry, I'm not able to visit your project. Is it a private project? So, my last commit makes it work?

Oops, I missed the visibility when creating the project :see_no_evil:
It is public now

That looks nice.

hello @tristanlins,
you demo test v8, did you try nodejs ?


Allow Unassigned task creation in team projects with one active user …

I create some team projects and start to creating tasks to see the whole picture.
Users are not assigned to projects yet, so all of the tasks are assigned to me because of missing Unassigned option in assignee selection list.
This makes difficult to distinguish between these newly created tasks and tasks that are actually assigned.
I see a parameter for getAssignableUsersList() function named single_user.
I think it would be better to allow Unassigned option for Team projects.

Sorry, it seems that rebasing pull-request code was a wrong decision %)


Move SQL-related methods out from models

Introducing new extension traits, protected by sql-ext feature flag.

After talking to @dpetrick decided to just keep all of this in prisma-models but make the sql-features opt-in to not bring quaint to crates that don't need it.

Also moved the prisma-models to libs to keep only crates that output binaries in the project root.


update endowed positions dataset

READY FOR REVIEW
Summary

Updated endowed positions data set.

Review By (July 10, 2023)

Our hope is to push live to prod on July 11 with a day for review by the stakeholders via dev.

Criticality - 4

This update only affects the Endowed Positions page(s).

Review Tasks
Setup tasks and/or behavior to test

Confirm the date in the footer of the page shows last updated July 11.
Spot check a few of the pages to confirm they are displaying results.

Front End Validation

[ ] Is the markup using the appropriate semantic tags and passes HTML validation?
[ ] Cross-browser testing has been performed?
[ ] Automated accessibility scans performed?
[ ] Manual accessibility tests performed?
[ ] Design is approved by @ user?

Backend / Functional Validation
Code

[ ] Are the naming conventions following our standards?
[ ] Does the code have sufficient inline comments?
[ ] Is there anything in this code that would be hidden or hard to discover through the UI?
[ ] Are there any code smells?
[ ] Are tests provided? eg (unit, behat, or codeception)

Code security

[ ] Are all forms properly sanitized?
[ ] Any obvious security flaws or new areas for attack?

General

[ ] Is there anything included in this PR that is not related to the problem it is trying to solve?
[ ] Is the approach to the problem appropriate?

Affected Projects or Products

Does this PR impact any particular projects, products, or modules?

Associated Issues and/or People

JIRA ticket(s)
Other PRs
Any other contextual information that might be helpful (e.g., description of a bug that this PR fixes, new functionality that it adds, etc.)
Anyone who should be notified? (@mention them here)

Resources

AMP Tool
Accessibility Manual Test Script
HTML Validator
Browserstack and link to Browserstack Credentials

@forrestglick
This should be ready on dev in a few moments.

Thanks @sherakama for pushing to Dev. I've just sent to the client for review. If all looks good, we will plan for a push to Prod tomorrow, July 11.


wrong return type when \\u{${hex}}

Bug Report

🔎 Search Terms

🕗 Version & Regression Information

This is a crash
This changed between versions ______ and _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
I was unable to test this on prior versions because _______

⏯ Playground Link

https://www.typescriptlang.org/play?ssl=5&ssc=2&pln=1&pc=1#code/KYDwDg9gTgLgBAMwK4DsDGMCWEVwPowQCqKmaEAJsABRoAWAhlAMKXABccKSAtgEbAoAGjgB3KAzAB+TnwgQANsAYoAlAFgAUAG8tASHIoAzvDqg4AXjj0mrKgDpCAZRhRMKAObUAjADZVANz6UMAwSFC41OKScAA+sXBmIPZKnjB0cAB8cAAsqnBScAAGADolSNoAJNpJAL61RXAMRtY4JnCcpeXVdY3NrcYwQZq1QA
💻 Code

export function _toUnicode(charCode: number, wrap?: boolean)
{
	const hex = charCode.toString(16);
	return (wrap || hex.length > 4) ? `\\u{${hex}}` as const : `\\u${hex}` as const;
}

🙁 Actual behavior

export declare function _toUnicode(charCode: number, wrap?: boolean): `\\u${string}`

🙂 Expected behavior

export declare function _toUnicode(charCode: number, wrap?: boolean): `\\u${string}` | `\\u{${string}}`

`\\u{${string}}` is valid `\\u${string}` - type just simplified(reduced?) to most permissible type as they are compatible.
Same as '{}' | string where result type would be string


Crash with corrupted audio

When handling certain cases of corrupted audio, WaveFormatConversionProvider's conversionStream is null, possibly due to strange things happening with Windows Multimedia. It seems conversionStream  isn't checked for null and throws an exception in Dispose(). This crash occurs in my own testing and in NAudioWpfDemo, 5-10 seconds after attempting to load the file.
I am not well-versed in .NET, but this condition is a problem because I'm unable to trap the exception in try-catch from external code.
File link (too big for GitHub) here.

thanks for reporting, have checked in a fix


Still cannot get full text

I have update my rev to b0b242f.
And I try the example
for post in get_posts('NintendoAmerica', base_url="https://mbasic.facebook.com", start_url="https://mbasic.facebook.com/NintendoAmerica?v=timeline", cookies=self.fb_cookie_path, pages=1):
print(post)
The post on December 6th was truncated on the web page, and the post results I got were also truncated 'text' instead of 'full_text'.
The 'text' show below :
There's no need to say goodbye just yet! The Seattle Aquarium’s partnership with Nintendo of America is continuing through the end of March 2024. 💙
That means you'll have more time to see some of your favorite characters from the Animal Crossing: New Horizons game all while exploring the Aquarium...

i tried it now, and it got me the full text from the latest nintendo post, although the text was duplicated. but full_text has all the text in it anyway

@moda20 Thanks so much. I can get full text, although the text was still duplicated.
I think I can close this issue first.


fix segmentation fault

add a check to ensure inverse projection is defined, before calling it (needed to prevent segfaults for proj versions 4.9.2 and before).
This solves issue #43 on my system. The inverse call now throws a proper python exception if the inverse projection is not defined, in the case that the module is build without bundled proj sources.

Would you mind adding a unit test for this?

Sure, I just added some test code that shows the segmentation fault for the unpatched pyproj version when build with unbundled proj version 4.9.1 or 4.9.2. When the patched _proj.pyx from this pull request is applied all tests pass. Currently it does 135 tests, 1 for each projection, but if this is too much, its easy to do all tests in a single test as well.

Thanks!


Added SDK9 project

Hi,
I made an SDK9 project for your library.

Thanks @estpla, do you know if SDK9 introduces any new features? I think you should be able to use XRM v9 and this project v8 together without need for an update.

I'm not totally sure about the new features. But, if you use v8 to connect to a CRM in version 9.0, it gives a connection error. Seems to be a problem with TLS 1.2. This error gets corrected with .NET Framework 4.6 and above.
Some links with info:
https://community.dynamics.com/crm/b/abhinavranjandynamics365/archive/2018/01/31/follow-up-on-tls-1-2-issue-in-dynamics-365
https://community.dynamics.com/crm/b/mrdavesdavidyackdynamicscrmblog/archive/2018/01/08/tls-1-0-and-1-0-now-blocked-to-dynamics-365-9-0-orgs


nats: no responders available for request

Description of the bug
I'm building an example of the microservices project built using go-micro with two simple services: https://github.com/begmaroman/go-micro-boilerplate/blob/feature/k8s/docker-compose.yaml
The first service is rest-api-svc and built using go micro web framework. Another one is account-svc built using go-micro.
They use nats server for discovery, transport and as a broker. I tried all other options for the transport but the following bug still appears:
{"id":"go.micro.client.transport","code":500,"detail":"nats: no responders available for request","status":"Internal Server Error"}

The interesting point is that the bug is flaky and appears ~ once per 10 request.
How to reproduce the bug

Clone the repo
Run go mod download
Run make build-base-image
Run docker compose up --build
Send request to GET http://localhost:3004/user few times

Environment
Go Version: 1.23.0

I can't remember the code for nats specifically but it could.be that across restarts the registry is being polluted by dead nodes. Ensure you're using heartbeat and ttls for expiry with the registry and set your client retries to 3. That should mitigate some of the problem.

If you look at the nats registry code in the plugins repo you'll notice that deregistration has no form of broadcasting. So effectively it can result in dead nodes. It's been a long time since I've done any development here so you'll need to investigate yourself I'm afraid.

@asim this problem is related to all registries available in plugins repo so I think this is something related to dead nodes or so. Would you mind giving more advise about the fix because I really stuck here. Thank you very much.

If it's related to all registries then it's an issue with shutdown and services not getting the time to deregister.  If you have ttl and expiry set in the service options then these entries should expire from the registry but you can increase the client retries so it immediately tries a different service entry from the registry.
service.Client().Init(client.Retries(3))

@asim weird thing is that when I'm using a custom selector strategy, I receive an incomplete list of nodes sometimes exactly when a request fails.

Then the issue is likely with your custom selector I guess. What are you using?

@asim I use a self-pinger client https://github.com/begmaroman/go-micro-boilerplate/blob/master/proto/health/pinger.go and the custom selector https://github.com/begmaroman/go-micro-boilerplate/blob/master/proto/health/health.go
So the list of nodes which is coming as an argument has 2 or 1 nodes. When it has 1 node, the request usually fails.

Some more context:

This is how the service gets initialized: https://github.com/begmaroman/go-micro-boilerplate/blob/feature/k8s/services/account-svc/microservice/microservice.go#L28-L42
This is how the client gets initialized: https://github.com/begmaroman/go-micro-boilerplate/blob/feature/k8s/services/rest-api-svc/microservice/microservice.go#L55

Ok so assuming this is a self healthcheck I guess the assumption is going to be the error could potentially occur before the service actually registers if the request to ping is initiated before the entry is in the registry, meaning no nodes are returned matching the instance. That's the only time I could see that error based on my limited understanding of the code

@asim given that I use a regular way to define microservice and communicate with it, should we consider the issue is in the framework?

Ok so assuming this is a self healthcheck I guess the assumption is going to be the error could potentially occur before the service actually registers if the request to ping is initiated before the entry is in the registry

It fails every second request. I send ping and healthchecks after the service is fully started.

@asim given that I use a regular way to define microservice and communicate with it, should we consider the issue is in the framework?

Given no one else has the issue I wouldn't call it an issue with the framework. You're doing something very bespoke with your own selector and nats. If it's happening with any registry then it points to some issue with your k8s setup as opposed to the service itself. I don't know the specifics of it so it's hard to comment. Theoretically it should be fine but if it's failing then either the service is dead or something else is wrong.

How often is the healthcheck fired?

Every second. I tried every 5 seconds as well. The point is that the healthcheck of the service itself and the selfpinger client with the custom selector works well.
Results of executing 2k requests with 50 concurrency:
Without a custom selector strategy:
Status code distribution:
  [200] 1006 responses
  [500] 994 responses

without a custom selector strategy:
Status code distribution:
  [200] 2000 responses

As fas as I can guess, it fails when trying to retrieve a live node, but if we specify which exactly node I want to use, it works well.

I found the problem. Due to some reason, the list of nodes in the service object has a weird item which does not work:
{
        "metadata": null,
        "id": "1cab0b44-71dc-43d6-a145-c58faa936b63",
        "address": "<IP_ADDRESS>:5678"
}

Since the default random selector is used, in 50% cases it tries to use unreachable node. The valid node ID starts with the service name plus node id while this bad node has only ID.
I created a custom selector which filters out bad nodes.


dataclass fields with default values misscategorized as class variables

In this example:
@dataclass
class SemanticVersion:
    __slots__ = ("version", "prerelease", "build_metadata")

    version: tuple[int, int, int]
    prerelease: Optional[tuple[str, ...]] = None
    build_metadata: Optional[tuple[str, ...]] = None

slotscheck reports a failed import, due to prerelease (and build_metadata) being in __slots__ conflicting with class variables.
However clearly, these aren't actually class variables, they're in fact just fields of a dataclass with some default values. Slotscheck should probably be able to distinguish between actual class variables and usage in dataclasses, to prevent this false positive.
Note that sometimes, variables defined like this can actually be considered class variables by the dataclass decorator, but only if they're type-hinted as one, or if they aren't type hinted at all, see this example:
from typing import ClassVar

@dataclass
class Foobar:
    FOO: ClassVar[int] = 5  # actual class variable, not present in generated `__init__`
    BAR = 10  # another class variable, not present in generated `__init__`

    x: int = 2  # instance variable, with default value in `__init__`

Actually, turns out this isn't an issue with slotscheck, it's an issue with python itself.
This ValueError is raised on runtime not just when using slotscheck, as dataclasses just don't seem to properly support slots below python 3.10, which added the slots parameter to the dataclass decorator.

@ItsDrike thanks for posting though, I did a quick check myself and come to the same conclusion. Perhaps I can make the error messages clearer to make it clear that import errors are mostly due to the code itself, not slotscheck.


As a player, I want a wider variety of building materials so I can be more creative while building. - Shawn Ruby

Task 1. Add stair building material

Task 2. Add lighting building materials (torches/lanterns).

Task 3. Add fence building materials.

Closed with pull requests https://github.com/WSU-CEG-6110-4410/ADAD-Craft/pull/46 https://github.com/WSU-CEG-6110-4410/ADAD-Craft/pull/144


[Suggestion] A DEV license valid for all countries

Hi Frreddy,
I'm working in an international project with multiple BC databases around the world with different localizations.
Would be possible for Microsoft provide a DEV license to be used for CI/CD Pipelines that includes all objects ranges of the different localizations?
This would semplify the management of the licenses used in DevOps.

I will discuss this with some people internally

May I ask, which kind of license you are using to run your CI/CD pipelines? If you're using a partner or even the demo license, it should already be open in the ID range from 1 - 50.000, which includes local ranges (10.000 - 30.000 - give or take). With a local partner or demo license, you should be able to compile and publish all other localizations as well!

@JesperSchulz I use a Partner license so I do not have any issue with custom objects range 1 - 50.000.
My question is related to Standard Objects range that may be different per localization.
When I download a Partner license from the Partner Source I need to specify for which version and which localization. Do you mean that it's irrelevant which localization I select I will get same Partner licese with same objects?
Please confirm

Except for the license header, the partner license should be identical regardless of the localization you choose, yes. I haven't actually tried this myself, but I would be surprised if you could not use an IT localization partner license to run an ES localization CI/CD pipeline. I would give it a shot! If anything fails, please let me know and I'll see what we can do.

@JesperSchulz I did not create a failure scenario rather I've downloaded detailed report in txt for a Partner BC 19 license with IT localization and ES localization as you suggested in your example and comparing them there are differences. Maybe you can have a look as well and share your opinion.

I would need your license dumps, as the ones I create may be very different from yours. Could you somehow make them available to me? Maybe reach out to me on Yammer or LinkedIn for a more private forum?

Opinion shared in private conversation. Please close the issue, if you feel like your questions have been answered :-)


Fixed  Battleship game ends abruptly if given bad input #622

Relevant Issues

Closes python-discord#622
Description

self.match = re.fullmatch("([A-J]|[a-j]) ?((10)|[1-9])", message.content.strip())
I changed the regular expression match to full match .
Reasoning

Basically pattern.match will return if any prefix of that pattern matches.
pattern.fullmatch will return true when the whole string matches
Screenshots

Additional Details

Did you:

[x] Join the Python Discord Community?
[ ] If dependencies have been added or updated, run pipenv lock?
[ ] Lint your code (pipenv run lint)?
[x] Set the PR to allow edits from contributors?

@rijusougata13 Thanks for your contribution! Please fill pull request template fully and give why and what you changed. Thanks.

@ks129  is it okay now?

so then we can merge and close the issue


Change clone command to fork + Format

Thanks for your fork! :slightly_smiling_face:

@thariq-shanavas Did you noticed this PR?

Thank you!


Added win32 check for IsUserAnAdmin()

Based on accepted answer to "Cross-platform way to check admin rights in a Python script under Windows?".

Seems alright.

Thanks!

Whoops, forgot to push this to PYPI.  I'll do that now.

Ok uploaded! (0.3)

:cake:


fixes NameError: global name 'http' is not defined

even after your fix the problem described in https://github.com/benoitc/restkit/issues/83 is still valid: there are still some http. left. this fixes the problem.
BUT please review the code well, since i'am not a http-protocol guru neither do I know the http_parser library used well - maybe theres a much better or cleaner way to achieve the same goal.

:-( for some reason the other commit 603fc1d was included in this pull request too, sorry, looks like github fooled me. looks like i cant remove it.

Fixed in 4355be916de9b5ce4250ab27bdaac834ef76765b . For header, I assume this is another ticket. But you can solve it by passing a string body and using the form object directly in your code.


Add support for yum facts (similar to the apt ones)

Pull Request (PR) description
This PR add a bunch of facts similar to the ones provided by the apt module:

yum_has_updates — Boolean indicating if updates are available;
yum_package_updates — Array[String] list of upgradable packages;
yum_updates — Integer number of upgradable packages;
yum_reboot_required — Boolean indicating if the systems needs to be rebooted.

e.g.
% facter yum_has_updates yum_package_updates yum_reboot_required yum_updates
yum_has_updates => true
yum_package_updates => [
  "puppet-agent.x86_64"
]
yum_reboot_required => true
yum_updates => 1

% facter yum_has_updates yum_package_updates yum_reboot_required yum_updates
yum_has_updates => false
yum_package_updates => []
yum_reboot_required => false
yum_updates => 0

This Pull Request (PR) fixes the following issues
n/a

You might check out the https://forge.puppet.com/albatrossflavour/os_patching module, which already does this sort of thing, but has taken into account the time impact during puppet runs. I would recommend against running yum commands for each facter run.

You might check out the https://forge.puppet.com/albatrossflavour/os_patching module, which already does this sort of thing, but has taken into account the time impact during puppet runs. I would recommend against running yum commands for each facter run.

This is a good thing to keep in mind. I don't know if this reaches out to a subscription service, but we've seen Satellite servers get heavily overloaded when using ensure => latest in large scale deployments.

This is a good thing to keep in mind. I don't know if this reaches out to a subscription service, but we've seen Satellite servers get heavily overloaded when using ensure => latest in large scale deployments.

We deployed it 6 month ago.  We contributed patches to fix it / make it work on the various systems we use…  Unfortunately it's still "meh" and we are considering to remove it :disappointed:.

I had a local commit for supporting a "list updates" task that I never pushed.
Because of the test failure around the cache and as I suggested in https://github.com/voxpupuli/puppet-yum/pull/141#discussion_r308020361 I added two commits to ensure the repositories are not automatically updated when Puppet runs and to allow refreshing the packages with MCollective / Choria / Bolt.

@ekohl sure!  There is still the test suite idempotency discussion that is annoying me because I do not understand the intention but I believe it's not testing what it tries… Should we track this in another issue?
Once everybody is happy with the changes, I'd like to squash the commits: I refrained to do it recently because it's a bit painful for reviewers to focus on the changes, but the history does not really makes sense right now :laughing:

All listed reviewers have approved the change.  Let's squash related commits!


Allow to disable hierarchy

Is your feature request related to a problem? Please describe.
When a couple of tests or whole test suite fails due to the error console or log file from CI is overflooded with the printed hierarchy and in more complex apps or parts of apps this hierarchy can be quite big which results in an unreadable mess.
Describe the solution you'd like
Allow us to pass the param to disable the print of hierarchy for better readability in whole test suites runs.
Describe alternatives you've considered
We can use the disable hierarchy for the whole test suite run and enable it when we debug the failed tests when it's useful to have.

This has been discussed in the past and has been decided against. Please use the search function.

Yes, I found these issues already, before writing the suggestion but I totally disagree with you. Wanted to add some new air into the discussion. Why don't you want to add more freedom to the framework? This hierarchy display is really unneeded to be displayed in the main console if we need to stick with it, it would be more appreciated as another artifact than a wall of text in a test run. Recently you improved the test output by adding the inline reporter, but we can't appreciate that because hierarchy destroys whole test output.


*: use coreos/bbolt as import path

alternatively, could change from bolt_test to bolt for the tests and avoid having to import at all, but maybe that could cause some conflicts. lgtm


Base image support for CSM artifacts

Description
Adds a base image, based upon RedHat UBI Micro, for Dell CSM artifacts to use as a base image. This centralizes and simplifies the management and updating of the image.
Checklist:

[X] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[n/a ] I have verified that new and existing unit tests pass locally with my changes
[n/a] I have not allowed coverage numbers to degenerate
[n/a] I have maintained at least 90% code coverage
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[X] I have maintained backward compatibility

@eric-young, is there an opportunity to provide some information in the README file with some build usage instructions?

@eric-young, is there an opportunity to provide some information in the README file with some build usage instructions?

Done. Also added the ability to build as non-root users

@eric-young, is there an opportunity to provide some information in the README file with some build usage instructions?

Done. Also added the ability to build as non-root users

Thank you @eric-young!

I see one error:
$ make docker
Building base image from registry.access.redhat.com/ubi9/ubi-micro@sha256:9dbba858e5c8821fbe1a36c376ba23b83ba00f100126f2073baa32df2c8e183a and loading dependencies...
cd base-image && ./build-base-image.sh -u "registry.access.redhat.com/ubi9/ubi-micro@sha256:9dbba858e5c8821fbe1a36c376ba23b83ba00f100126f2073baa32df2c8e183a" -t ""sample_registry""/""csm-base-image"":""test"" acl gnutls device-mapper-multipath e2fsprogs gnutls gzip hostname kmod libaio libblockdev libuuid libxcrypt-compat nettle nfs-utils nfs4-acl-tools numactl openssl rpm systemd tar util-linux which xfsprogs
Building base image from registry.access.redhat.com/ubi9/ubi-micro@sha256:9dbba858e5c8821fbe1a36c376ba23b83ba00f100126f2073baa32df2c8e183a
And creating sample_registry/csm-base-image:test
With packages of: acl gnutls device-mapper-multipath e2fsprogs gnutls gzip hostname kmod libaio libblockdev libuuid libxcrypt-compat nettle nfs-utils nfs4-acl-tools numactl openssl rpm systemd tar util-linux which xfsprogs
Error: There are no enabled repositories in "/etc/yum.repos.d/".
Unable to detect release version (use '--releasever' to specify release version)
0 files removed
47e4e5190bd5f42cd7d501edd90fd3ddb4eb3f23c6d6a0e314a1734d6fe11e89
Getting image source signatures
Copying blob 4092f1e3e312 skipped: already exists
Copying blob a7a506debb5f done   |
Copying config 71dc853854 done   |
Writing manifest to image destination
71dc853854514b6dfe4d1f2a79a900fd1ec48d6940891ca1b407dbe202171340
Built base image: sample_registry/csm-base-image:test

I see one error:
$ make docker Building base image from registry.access.redhat.com/ubi9/ubi-micro@sha256:9dbba858e5c8821fbe1a36c376ba23b83ba00f100126f2073baa32df2c8e183a and loading dependencies... cd base-image && ./build-base-image.sh -u "registry.access.redhat.com/ubi9/ubi-micro@sha256:9dbba858e5c8821fbe1a36c376ba23b83ba00f100126f2073baa32df2c8e183a" -t ""sample_registry""/""csm-base-image"":""test"" acl gnutls device-mapper-multipath e2fsprogs gnutls gzip hostname kmod libaio libblockdev libuuid libxcrypt-compat nettle nfs-utils nfs4-acl-tools numactl openssl rpm systemd tar util-linux which xfsprogs Building base image from registry.access.redhat.com/ubi9/ubi-micro@sha256:9dbba858e5c8821fbe1a36c376ba23b83ba00f100126f2073baa32df2c8e183a And creating sample_registry/csm-base-image:test With packages of: acl gnutls device-mapper-multipath e2fsprogs gnutls gzip hostname kmod libaio libblockdev libuuid libxcrypt-compat nettle nfs-utils nfs4-acl-tools numactl openssl rpm systemd tar util-linux which xfsprogs Error: There are no enabled repositories in "/etc/yum.repos.d/". Unable to detect release version (use '--releasever' to specify release version) 0 files removed 47e4e5190bd5f42cd7d501edd90fd3ddb4eb3f23c6d6a0e314a1734d6fe11e89 Getting image source signatures Copying blob 4092f1e3e312 skipped: already exists Copying blob a7a506debb5f done | Copying config 71dc853854 done | Writing manifest to image destination 71dc853854514b6dfe4d1f2a79a900fd1ec48d6940891ca1b407dbe202171340 Built base image: sample_registry/csm-base-image:test

This needs to run on RHEL, not openSuSE. I added a check to make sure that the user is on RHEL


Problems with installing pycryptodome on Windows

This can be fixed by installing visual studio build tools from https://visualstudio.microsoft.com/downloads/#build-tools-for-visual-studio-2019 . This should install MSVC and allow to install pycryptodome.


[No QA] Set up beta checks for policyExpenseChat

@marcochavezf, please review when you get the chance
Details
This is set up for future PRs in this repo that will implement policyExpenseChats.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/195467
Tests
N/A
QA Steps
N/A
Screenshots
N/A

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @marcochavezf in version: 1.1.39-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @timszot in version: 1.1.39-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Error in render_revert_entity when used with asyncpg engine

First of all, thank you for the great library, it is so much more fun than writing (and tracking) function definitions in alembic migration files!
I am using alembic_utils with asyncio sqlalchemy:
alembic==1.7.4
alembic-utils==0.7.3
SQLAlchemy==1.4.25

New functions are generated successfully, but when a function definition has been changed, alembic-utils fails with the following stack trace:
alembic revision -m 'test' --autogenerate
INFO  [alembic.runtime.migration] Context impl PostgresqlImpl.
INFO  [alembic.runtime.migration] Will assume transactional DDL.
INFO  [alembic.ddl.postgresql] Detected sequence named 'projects_id_seq' as owned by integer column 'projects(id)', assuming SERIAL and omitting
INFO  [alembic_utils.depends] Resolving entities with no dependencies
INFO  [alembic_utils.depends] Resolving entities with dependencies. This may take a minute
INFO  [alembic_utils.replaceable_entity] Detecting required migration op PGFunction PGFunction: public.workspace_is_visible( p_workspace_id integer, p_user_id integer )
INFO  [alembic_utils.replaceable_entity] Detecting required migration op PGFunction PGFunction: public.workspace_permission_required( p_workspace_id integer, p_user_id integer, p_permission workspace_permission )
INFO  [alembic_utils.replaceable_entity] Detecting required migration op PGFunction PGFunction: public.workspace_user_permission_list( p_workspace_id integer, p_user_id integer )
INFO  [alembic_utils.replaceable_entity] Detected ReplaceOp op for PGFunction PGFunction: public.workspace_user_permission_list( p_workspace_id integer, p_user_id integer )
INFO  [alembic_utils.replaceable_entity] Detecting required migration op PGFunction PGFunction: public.project_is_visible( p_project_id integer, p_user_id integer )
INFO  [alembic_utils.replaceable_entity] Detecting required migration op PGFunction PGFunction: public.project_permission_required( p_project_id integer, p_user_id integer, p_permission project_permission )
Traceback (most recent call last):
  File "/home/demidov/work/rndflow/api/env/bin/alembic", line 8, in <module>
    sys.exit(main())
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/config.py", line 588, in main
    CommandLine(prog=prog).main(argv=argv)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/config.py", line 582, in main
    self.run_cmd(cfg, options)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/config.py", line 559, in run_cmd
    fn(
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/command.py", line 234, in revision
    scripts = [script for script in revision_context.generate_scripts()]
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/command.py", line 234, in <listcomp>
    scripts = [script for script in revision_context.generate_scripts()]
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/autogenerate/api.py", line 600, in generate_scripts
    yield self._to_script(generated_revision)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/autogenerate/api.py", line 500, in _to_script
    render._render_python_into_templatevars(
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/autogenerate/render.py", line 87, in _render_python_into_templatevars
    _render_cmd_body(downgrade_ops, autogen_context)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/autogenerate/render.py", line 109, in _render_cmd_body
    lines = render_op(autogen_context, op)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic/autogenerate/render.py", line 127, in render_op
    lines = util.to_list(renderer(autogen_context, op))
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/alembic_utils/reversible_op.py", line 155, in render_revert_entity
    with engine.connect() as connection:
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/future/engine.py", line 419, in connect
    return super(Engine, self).connect()
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 3194, in connect
    return self._connection_cls(self, close_with_result=close_with_result)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 96, in __init__
    else engine.raw_connection()
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 3273, in raw_connection
    return self._wrap_pool_connect(self.pool.connect, _connection)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 3240, in _wrap_pool_connect
    return fn()
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/pool/base.py", line 310, in connect
    return _ConnectionFairy._checkout(self)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/pool/base.py", line 868, in _checkout
    fairy = _ConnectionRecord.checkout(pool)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/pool/base.py", line 476, in checkout
    rec = pool._do_get()
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/pool/impl.py", line 256, in _do_get
    return self._create_connection()
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/pool/base.py", line 256, in _create_connection
    return _ConnectionRecord(self)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/pool/base.py", line 371, in __init__
    self.__connect()
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/pool/base.py", line 666, in __connect
    pool.logger.debug("Error on connect(): %s", e)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/util/langhelpers.py", line 70, in __exit__
    compat.raise_(
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/util/compat.py", line 207, in raise_
    raise exception
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/pool/base.py", line 661, in __connect
    self.dbapi_connection = connection = pool._invoke_creator(self)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/engine/create.py", line 590, in connect
    return dialect.connect(*cargs, **cparams)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/engine/default.py", line 584, in connect
    return self.dbapi.connect(*cargs, **cparams)
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/dialects/postgresql/asyncpg.py", line 748, in connect
    await_only(self.asyncpg.connect(*arg, **kw)),
  File "/home/demidov/work/rndflow/api/env/lib/python3.9/site-packages/sqlalchemy/util/_concurrency_py3k.py", line 61, in await_only
    raise exc.MissingGreenlet(
sqlalchemy.exc.MissingGreenlet: greenlet_spawn has not been called; can't call await_() here. Was IO attempted in an unexpected place? (Background on this error at: https://sqlalche.me/e/14/xd2s)
sys:1: RuntimeWarning: coroutine 'connect' was never awaited

A temporary solution for me is to use alembic with the synchronous psycopg2 engine.

After more than 40 migrations (including extensions, functions, and triggers) I don't see any problems with the asyncpg engine. This can be closed as far as I am concerned, thanks again!

thanks for the colab!


Safety Timer

You can take help from Ui provided.

We just have to create this screen with that timer right? And when the 'Activate' button is pressed then the timer should start counting down? And after the timer hits 0 then what do we have to do?

I would like to work on this please assign this to me

We just have to create this screen with that timer right? And when the 'Activate' button is pressed then the timer should start counting down? And after the timer hits 0 then what do we have to do?

the timer should start counting down when the 'Activate' button is pressed and send the alert messages to the emergency numbers we provide till it hits 0.

It's alright give it to @Upasana1216


Resource group NetworkwatcherRG is not getting deleted by Terraform Destroy

Description
Resource group NetworkwatcherRG is not getting deleted by Terraform Destroy which resulted in Policy conflict when reapplying the Terraform
Steps to Reproduce
Terraform Apply
Terraform destroy
Steps to reproduce the behavior:
1.
2.
Expected behavior
Actual behavior
Screenshots
Additional context
Operating System:
Terraform Version:
Cloud (public, Azure Government, etc.):

Can you share a log of what which resulted in Policy conflict when reapplying the Terraform is being returned?

This is as expected since Terraform or Bicep doesn't deploy this resource group, it is created when enabling Network Watcher which is global (regional). We can modify the this to use one we create but that is not normal.

Can you share a log of what which resulted in Policy conflict when reapplying the Terraform is being returned?

Part of the log is from bicep deployment after TF destroy in same subscription.... i am not sure this is related to NetworkWatcher. Sunscription and client IDs are redacted.
Deployment failed with multiple errors: 'Authorization failed for template resource 'Deploy VMSS Agents cicdbicep-hub' of type 'Microsoft.Authorization/policyAssignments'. The client 'xxxxxx' with object id 'xxxxxx' does not have permission to perform action 'Microsoft.Authorization/policyAssignments/write' at scope '/subscriptions/xxxxxxxxx/resourceGroups/cicdbicep-hub/providers/Microsoft.Authorization/policyAssignments/Deploy VMSS Agents cicdbicep-hub'.:Authorization failed for template resource 'Deploy VM Agents cicdbicep-hub' of type 'Microsoft.Authorization/policyAssignments'.

Have you recreated service principles? This appears to be related to the user(SP) doing the deployment not having correct permissions to write to policyAssignments.

Example of was done in the earlier script version of MLZ when we created SP's on the fly. 'Resource Policy Contributor' is required and not default as part of contributor.
az role assignment create \
                --role 'Resource Policy Contributor' \
                --assignee-object-id "${sp_object_id}" \
                --scope "/subscriptions/${sub}" \
                --assignee-principal-type ServicePrincipal \
                --output none

Does running the role assignment create command @shawngib described on the object ID and subscription scope described in your error resolve the error?

Does running the role assignment create command @shawngib described on the object ID and subscription scope described in your error (The client 'xxxxxx' with object id 'xxxxxx' does not have permission to perform action 'Microsoft.Authorization/policyAssignments/write' at scope '/subscriptions/...) resolve the error?

nope...:( checking on couple of things

Initial assumption seems to be a resource not exists error that stems from naming collisions with not-existing but not-yet-deprovisioned resources: https://github.com/Azure/missionlz/issues/498
Gonna close for now @vidyambala -- but happy to re-open or reference in another Issue if @shawngib is able to track down a root cause.


burrito tries to load metadata for other versions or other burrito apps than the one being executed.

Host
os: Windows 11 x64
zig: 0.11.0
elxir:
Erlang/OTP 25 [erts-<IP_ADDRESS>] [source] [64-bit] [smp:48:32] [ds:48:32:10] [async-threads:1] [jit:ns]

Elixir 1.16.2 (compiled with Erlang/OTP 25)

Target
Host OS: Windows 11
Host CPU: x86_64,
Host LibC and Version vs build tools 2022

Burrito tries to load metadata for a burrito app other than the one being run: Failed to load the metadata file \path\to\other\version\or\other\burrito\app\_metadata.json
I will invesitgate further and add more details...

It occurs on an empty directory (or similar) of a previous version.  Still unsure why it was throwing this error, referring to the example cli app, when running my app.

It looks like the metadata file path didn't get computed properly. Can you make a minimal reproducible repo for this issue so I can debug.


feat: Update feature config before launch to ensure features are loaded before serving content

Also change the polling interval configuration from milliseconds to seconds

:tada: This PR is included in version 1.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to add an image to Extra field of Cloze?

Tried, did not work here. Do you have any working example?  Thanks in advance.  Adding an image on a basic note is working here, but not in Extra field of cloze.

Hey,
Just tested in but something like this does work:
* {{c1::Joe}} is a great man.
# type=Cloze
** Extra field
** [/path/to/image.jpg]

Can you tool allow also inline images, so nothing directly as header line definition, but as content?
Because the advantage of this style would be you could the those images directly in org-mode for example, you see again org-mode.  :smile:
The usability would be much better, while designing your content you would always know and see the content, and not having to guess what was this image again.  The data would show itself already.

That is a really good point actually. I assume you're using org in Emacs?

I'm going to change add that as a feature request but should be easy enough to imply

Yes, org-mode in emacs.  Thanks for all the positive additions in future.

Tried now, the Extra line is even not necessary, it shows it in Extra field automatically great.  Not having the black dot (list element) before the image would be also nice, simpler view, but ok we should ignore this minor aspect and be happy it works at all.
* {{c1::Joe}} is a great man.
# type=Cloze
** [/path/to/image.jpg]

As you wrote main point is modifying a card is not possible, only adding a new one.  Having an option to modify later would be not bad, but one can not have everything it seems.
Defining a quick macro converting an inline image to org_to_anki version works nicely, as a transition solution.

I only added the extra line as an example of text. Probably should have been more clear.
You can turn the bullet points off they are just on by default. There is more docs on it here:
https://github.com/c-okelly/org_to_anki/blob/master/documentation/orgReadme.md#html-list-types--list


How To Access Two Separate DBs From Two Different Container

I am running postgres db container on my machine.
I have two other containers, lets say A and B.
A connects to postgres db (default)
B want to connect to another db (TestDB).
In my case
If one of the db is connected, the connection for other db is refused.

I'm able to connect to multiple databases from different containers using the same user without issue
Setting up the database
$ docker run -d --rm --name postgres --network=net -e POSTGRES_USER=user -e POSTGRES_PASSWORD=pass -e POSTGRES_DB=testdb postgres
014f75e59e542cb5aa89674778b9edc8561221d5746e58cfd7508e4480b3eee9

Connecting with the first client to database testdb
$ docker run -d --rm --name client-1 --network=net postgres
4288db2eba1b7829df433fa7b81f88ed20a658441454a1bad08fb33e4754bb28

$ docker exec -it client-1 psql -U user -h postgres -d testdb
Password for user user:
psql (11.4 (Debian 11.4-1.pgdg90+1))
Type "help" for help.

testdb=#

Connecting with second client to database postgres and getting current connections
$ docker run -d --rm --network=net --name=client-2 postgres
92fe83b4d9748657ef9c4f8f0fe01b4308a61338e7b814dac36b8967360ec7ce

$ docker exec -it client-2 psql -U user -h postgres -d postgres
Password for user user:
psql (11.4 (Debian 11.4-1.pgdg90+1))
Type "help" for help.

postgres=#  SELECT usesysid, usename FROM pg_stat_activity;
 usesysid | usename
----------+---------
       10 | user
          |
       10 | user
       10 | user
          |
          |
          |
(7 rows)

postgres=#

@wglambert
can you suggest a way to do it using dockerfile ?
Can I set Username password and database inside dockerfile - this will create one user
and then run CMD["postgres"]- this will create default user.
then access both db from two different containers ?
I have issue raised regarding this  #598

@wglambert
Thanks.
I want to connect programmatically.
as mentioned before I have two other containers, lets say A and B.
A runs app1 which connected to postgres db.
B runs app2 which want to connect TestDB.
But one of them connects and other get Connection Refused error.

It could be something with the Postgres configuration, like https://www.postgresql.org/docs/10/auth-pg-hba-conf.html
I'd recommend trying the Docker Community Forums, Docker Community Slack, or Stack Overflow. Since these repositories are for issues with the image and not necessarily for questions of usability


Make advanced and advancedSync backwards compatible

Following our discussion in #66 I've renamed the levels and added a deprecation warning for users so we can change the advanced level in 2.0.
@tjlytle you agree this works?

@cbetta Looks great, thanks!

@leggetter mind pushing this change?

@cbetta I'd like to get the JWT stuff #69 (scope has crept, but it needs to) in before pushing a new release.

@leggetter sure thing, though I personally think it's artificial to wait.
As for #69 I'll wait for you and Tim to come down to a bit more of a spec on that one.


Weight Refactor: Do not modify weight in ModifyPokemon

Added a new event, ModifyWeight, to deal with weight changes from items,
abilities, and moves. Add a function getWeight to call the event and
return a Pokemon's current weight.

Once again, calling @Marty-D to verify that all the weight-based mechanics are here.
I forgot to put regression tests for Sky Drop's weight calc, but oh well

Looks like everything is here! The only thing I would recommend is to also check that Transform copies the right weight. Come to think of it, I wonder if it even does that correctly before this change.

It should, I think... because BattlePokemon#transformInto calls BattlePokemon#formeChange, which in turn overwrites pokemon.template with the new template, and that should have the proper weight. The old code did the rest by updating pokemon.weightkg with the template's weight at every update before calling ModifyPokemon, while this getWeight simply goes off the pokemon.template.weightkg.
But thanks for mentioning this, since Autotomize is still using pokemon.weightkg, which by all means should be deprecated now, and should be checking against pokemon.template.weightkg. So I'll have to fix that anyways.

Huh. Why is Heavy Metal giving these sorts of off-by-one errors? And I don't even know what the actual damage is because node's assert module doesn't give me access to a lessThan...

Don't push right now. I suspect that the new RNG is doing something that I don't understand, and since it'll be a pain to merge everything into here I'm going to force Travis to run the build again and grab data from console.log.

Is gamut-was-taken's damage calc slightly off? The damages being rolled are currently 1 less than the reported ranges.
And to think that it was working just fine yesterday...

Can always calculate them by hand

@ascriptmaster Why are you setting the seed for your weight tests?

I'm setting the seed to prevent critical hits from messing up the damage calc... but apparently it's wrong anyways!
What I'll probably do later to prevent messiness with damage calcs is hook the runEvent function and snag the base power provided by Grass Knot's basePowerCallback.

I fixed the tests to now catch the BasePower event by overriding Battle#runEvent.
I also fixed up the abilities/items to no longer use chainModify, since that actually forces the weight to an integer, which is bad.

node's assert module doesn't give me access to a lessThan...

can't you just assert varName < 20, true or something?

Also, is this ready for merge?

can't you just assert varName < 20, true or something?

Yes, and that's what I was doing, but "Assertion Error: false !== true" is always so incredibly informative
Zarel, if you're willing to take the extremely small risk of the weight calc not working with Transform, even though the Transform code indicates that it should work properly, then yes, it's ready for merge.

Why not test with Transform?

Because I trust the code. Transform calls transformInto which calls formeChange which overwrite's a Pokemon's template. And all the weight calcs now use pokemon.template.weightkg instead of pokemon.weightkg, which was reset to pokemon.template.weightkg on every ModifyMove cycle anyways.
Basically, if it worked before this refactor, basically, it'll work after the refactor. And yet, no matter now sure I am of the code, I still can't trust myself enough to simply say "it's ready to merge no questions asked".

...you could just test.

I can't right now, and I guess I should've said that instead of dancing around the issue.
Tomorrow, then.

Sounds good.

My tests say that it works properly with Transform. I'd commit them here but there are some errors in Gen 4's Transform code that need fixing first.

...should I merge now or wait for your Gen 4 fixes?

Merge now. Gen 4 Transform is a separate issue anyways and deserves its own pullreq.

Transform fixes are sent in #1656.


Not working on Safari iOS - mobile iPhone

Hi guys, web works pretty well
but it seems not work for me on my iPhone,
And if I read it right in the documentation says no issues on mobile (only when the battery is in low mode)
I tried to find something on the issues here but I couldn't
Is it a problem only with me?
here is my test page below
renan-stnl-uxdesign.com

Hi guys, I fix that,


Salsa - IntelliSense based on JSDoc, @typedef -> @property -> {Function(Type): Type} not support

TypeScript Version:
Code
/**
 * @typedef TestType
 * @property {Function(String): Boolean} createTestFunction
 */

/** @type {TestType} */
var t;

/** @type {Function(String): Boolean} */
var fn1;

@property does not support {Function(Type): Type}
@type support {Function(Type): Type}
Expected behavior:
type TestType = {
    createTestFunction: Function;
}

Actual behavior:
type TestType = {
    (Missing): Function;
}

I think you meant function and not Function.
/**
 * @typedef TestType
 * @property {function(String): Boolean} createTestFunction
 */

/** @type {TestType} */
var t;

/** @type {function(String): Boolean} */
var fn1;

Function is a type name, and it already has a well defined meaning.
The closure-compiler docs also refer to the lower-case function: https://github.com/google/closure-compiler/wiki/Annotating-JavaScript-for-the-Closure-Compiler#function-type

@mhegazy oh, i don't know 'Function' and 'function' are different 😂
should the wiki explain this?
https://github.com/Microsoft/TypeScript/wiki/JsDoc-support-in-JavaScript

it is:

// Define function type
/** @type {function(string, boolean): number} */
var fn5;


Support sinking to event-specific topics/tables/streams/subjects/etc

Right now there are two "sink outputs" -> good and bad.
It would be cool if there was a third option, for writing events to their own topics/streams/etc.

Dup, closing.


Don't specify -arch and -m{}os-version-min= on Apple platforms

As recommended by Clang developers in https://github.com/llvm/llvm-project/issues/88271#issuecomment-2050425068, we already specify --target, and that should be enough.
Related: https://github.com/rust-lang/cc-rs/pull/1029, the -mxros-version-min option does not exist.
CC @BlackHoleFox.

sounds good to me, IIRC specifying version-min might be related to errors in other projects using cc, maybe even rustc bootstrap, which failed to compile llvm.

Yeah ditching it seems like a fine option. Only thought is maybe compatibility with old (whatever matches 10.12) XCode versions but that might not be a real issue since there is no reason to build iOS (and derivative) apps with anything except the last N-2 XCode releases for app store reasons.
Would the idea be to remove use of -m{}os-version-min and -arch everywhere except macOS?

Maybe  platform_version, it seems to support visionos.
     -platform_version platform min_version sdk_version
             This is set to indicate the platform, oldest supported version of that platform that output is to be used on, and the SDK that the output was
             built against.  platform is a numeric value as defined in <mach-o/loader.h>, or it may be one of the following strings:
             • macos
             • ios
             • tvos
             • watchos
             • bridgeos
             • visionos
             • xros
             • mac-catalyst
             • ios-simulator
             • tvos-simulator
             • watchos-simulator
             • visionos-simulator
             • xros-simulator
             • driverkit
             Specifying a newer min or SDK version enables the linker to assume features of that OS or SDK in the output file. The format of min_version
             and sdk_version is a version number such as 10.13 or 10.14


Static Code Analyzer : Streams

fix with using() statement

using( var fs = new Stream())
{
    // do something
}

This appears to have already been fixed

@csmoore is there a way to show the verified fix? It looks like when @jmccausland logged this, the fix was proposed but I can't find the PR/commit where it was actually fixed.

There were 2 methods with this condition mentioned in the report - OnImportCSVFileCommand(), LoadConfiguration() :
For this one I can't seem to link to the commit either - it seems to be a permission problem with where the commit was made (perhaps a private repo?)
https://github.com/Esri/coordinate-conversion-addin-dotnet/blob/dev/source/CoordinateConversion/CoordinateConversionLibrary/ViewModels/TabBaseViewModel.cs#L167
The other one - I did change in commit https://github.com/Esri/coordinate-conversion-addin-dotnet/commit/cc43203b064d535b649c6742ca56196bcd00a3bd -
https://github.com/Esri/coordinate-conversion-addin-dotnet/blob/dev/source/CoordinateConversion/CoordinateConversionLibrary/Models/CoordinateConversionLibraryConfig.cs#L95


Support for additionalProperties

Context

hapi-swgger version: 9.0.2

What are you trying to achieve or the steps to reproduce ?
Our API returns normalized data, like:
{
   article: {
      id: 'article:1',
      title: 'Hello World!',
      tags: [
         'tag:1',
         'tag:2'
      ]
   },
   tags: {
      'tag:1': {
         id: 'tag:1',
         name: 'Animals'
      },
      'tag:2': {
         id: 'tag:2',
         name: 'Cat'
      }
   }
}

Currently, it's impossible to describe responses with KeyValue pairs (tags object).
Swagger 2.0 supports additionalProperties property for solving this issue.
Suggestion
Transform Joi.object().pattern(regex, schema) to additionalProperties in JSON (currently hapi-swagger ignores object.pattern)
Then following code (hopefully) should generate proper Swagger documentation:
Joi.object().keys({
   tags: Joi.object().pattern(/.*/, Joi.object().keys({
      id: Joi.string(),
      name: Joi.string()
   }))
});

I find myself in need of such a feature myself. Any chance this could be implemented soon?

There is no timeline for this feature request. PRs are welcome.

Heya @robmcguinness! In the meantime I've noticed support has been added for parsing a Joi.object().patter(pat, schema) structure, but output's not quite right... I'll try to have a look into it and get back to you.

Sounds good!


Bokeh Error Cannot read property 'value' of undefined

python 3.5.1
bokeh.version       0.12.10
I just did as README.md says, but it turned out that "Cannot read property 'value' of undefined" in the browser.

Thanks for reporting this issue. It seems to be related to a change in the newer version of bokeh. I'll track that down (the error is incredibly vague...).

Should be all fixed!

There we go! Now it should be good. Let me know if you run into any other issues.

Thank you!