Adding environment variables to docker run command via .kitchen.yml

Adding driver_config option for loading environment variables in docker run command.  I'm not convinced it is the best place to put the configuration, so let me know if you want me to move things around.

What is your use case? These wouldn't really be used during Chef testing and I'm not sure this is a good tool for testing containers in a more general fashion.

Docker frequently uses environment variables for runtime configuration.  I'm building docker images via Chef and packer, but I utilize test-kitchen for testing and prototyping.  Enabling environment variable support enables me to test how the container will run and orchestrated not only in test-kitchen, but in real environments.  Otherwise my prototype environment is distinct from staging, production or any non test-kitchen environment.

The primary use case for this gem is testing Chef cookbooks, but this is probably unobtrusive enough to do. That said, Test Kitchen is not built to be a Docker image testing tool, so this is unlikely to be the only pitfall you'll hit.

 If you're looking for purely external testing, I would use Serverspec or InSpec directly in their remote execution modes.

Actually this opens a far more interesting topic that I have not seen addressed anywhere, and is beyond the scope of Chef (let alone test-kitchen).  I have struggled to answer this.  If you have any thoughts / suggestions on the matter I would love to hear it.
How do we repeatedly test and configure docker containers?  Most people generate Docker images via Dockerfiles, which are effectively BASH scripts.  I used to do this professionally 10+ years ago, and it was non-repeatable, very hard to test, highly dependent upon the starting state, and error prone due to naturally non idempotent steps.  Yes, yes, a Dockerfile is theoretically repeatable, but only if you start from the same starting state and the situation hasn't changed (aka, package updates breaking something?) since you last ran it.  Theoretically one will always have that docker image stored for all time thereafter, but the knowledge of how it was created may be impossible to effectively reproduce (someone manually tweaks the container and commits it as a new image).  The image is passed around between multiple developers, each one taking new commits.
In my experience, Docker images are less infrastructure as code and more "snapshots of what I googled".  I strongly prefer Chef cookbooks.  Though cookbooks undoubtedly add complexity to the unfamiliar, they build upon prior domain knowledge, repeatable steps, and testing frameworks.  For example, installing nginx the naive way (sudo apt-get -y install nginx) works, but with the Chef cookbook, it provides real, extensible code, built upon prior domain knowledge of the product.  This is in stark contrast to downloading the package, using sed to hard code some values someone read off of StackOverflow, and otherwise building upon zero domain knowledge outside of that.  In contrast, using the nginx cookbook, it was intelligently configured, and with just a few attribute tweaks, it can be highly customized to a good, maintainable, and testable installation.
This is a question I am personally struggling to answer in an adequate way.  If you have any thoughts on it, please let me know.

The overall structure of Test Kitchen can definitely work for testing Docker images, but this gem is not for that and I don't want to try and add it given how different it would be from testing cookbooks (or similar, modules, playbooks, etc).
The way I would imagine it working would be a driver that is similar to this one but rather than doing a Dockerfile and a build, it just runs a given image as is. This would probably be paired with a custom verifier plugin that uses InSpec's docker execution mode so we don't have to install/configure SSH access. Totally in favor of someone writing that driver/verifier, it just isn't this one.


MapService removed but map remains

Received from another group:
fast calls to  IMap.addMapService(...) & IMap.removeMapService() can result in map service being removed but map staying on the screen.

waiting for loop interval option to be added to the tester


demo: add Tabs integration example.

What kind of change does this PR introduce? docs update

This change is

Codecov Report

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

@@           Coverage Diff           @@
##           master     #722   +/-   ##
=======================================
  Coverage   94.26%   94.26%
=======================================
  Files          17       17
  Lines         767      767
  Branches      221      221
=======================================
  Hits          723      723
  Misses         44       44

Continue to review full report at Codecov.

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


Add support for schema create or update for Swagger, WSDL and OpenApi

Fix based on REST Spec https://github.com/Azure/azure-rest-api-specs/pull/5824

@dsgouda the build failure is related to an EventHub test.

@weshaggard Please take a look at the failures.

@dsgouda  I wanted to add an extension to one of the models for better usability. Give me an hour.

@dsgouda I re-queued the failing test leg but I don't believe it is related to the changes in this PR so if it fails again merge anyway.
@jsquire looks like another EventHubs test reliability failure
Failed   Microsoft.Azure.EventHubs.Tests.ServiceFabricProcessor.OptionsTests.RuntimeInformationTest
Error Message:
 Assert.True() Failure
Expected: True
Actual:   False
Stack Trace:
   at Microsoft.Azure.EventHubs.Tests.ServiceFabricProcessor.OptionsTests.RuntimeInformationTest() in D:\a\1\s\sdk\eventhub\Microsoft.Azure.EventHubs\tests\ServiceFabricProcessor\OptionsTests.cs:line 191

I added a note about this test in issue https://github.com/Azure/azure-sdk-for-net/issues/5995, and if we see it keep failing then I'll disable it.

@dsgouda I have pushed my changes. Feel free to merge it as soon as CI passes or fails with EventHub test failure.

@dsgouda can you merge this.


resolves #2

This PR addresses issue #2 by checking the violations of points between the extremes of each interval, in function ExactPiece

I'm sorry for only reviewing this PR now.
There are a lot of random lp files and commits that seem unrelated to LinA.
Most of the actual changes seem good!
Could you modify the PR to only include the relevant changes?
Thank you for the PR!


Docker image for minaprotocol/mina-generate-keypair not up to date

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues: https://github.com/MinaProtocol/mina/issues
[X] This issue is not a question, feature request, RFC, or anything other than a bug report. Please post those things in GitHub Discussions: https://github.com/MinaProtocol/mina/discussions

Summary
According to the docs, under section "Usage", 3b

In Docker on Windows/MacOS/Linux: Generate your keys using the minaprotocol/generate-keypair docker image. When prompted, type in the password you intend to use to secure this key. Do NOT forget this password.

docker run  --interactive --tty --rm --volume $(pwd)/keys:/keys minaprotocol/mina-generate-keypair:1.2.2-feee67c --privkey-path /keys/my-wallet

Docker can not find image mina-generate-keypair:1.2.2-feee67c, because the last available image is of version 1.2.0, see docker hub  hub for mina-generate-keypair
Steps to Resolve this Issue
Either update the docker image to version 1.2.2, or change the docs to fit version 1.2.0, if no compatibility issues exist

The current documentation says to use version 1.3.0-9b0369c however I can only see tag 1.2.0-fe51f1e

Thank you @cscetbon and @Trivo25, works for me with image mentioned by @cscetbon:
> docker run --interactive --tty --rm --volume $(pwd)/:/keys minaprotocol/mina-generate-keypair:1.3.0-9b0369c --privkey-path /keys/my-wallet

Password for new private key file:
Again to confirm:
Keypair generated
Public key: B62qiaLLK5vLyog5iEbGseka9n3uTVuCo6hBCviQvawECXQwSH7QkQ1
Raw public key: 039B0CE01C981C1672FC52B3FAC3AA9B70EA88FF8441E403285E8B656D44088E


support opening a folder not in the same directory as the .git folder

we have a monorepo that is not using submodules
In order to get other extensions to work, I open each subfolder in a different instance of vs code, but this then causes this extension to not be able to pick up the url.
If possible it would be nice to have a setting where I could set the path to the .git folder

Sounds like a good improvement.
I won't have time to add this in the near future, but would be happy to review a pull request for this update :)

From: Mordechai Zuber<EMAIL_ADDRESS>Sent: Wednesday, June 20, 2018 6:40:26 AM
To: differentmatt/vscode-copy-github-url
Cc: Subscribed
Subject: [differentmatt/vscode-copy-github-url] support opening a folder not in the same directory as the .git folder (#10)
we have a monorepo that is not using submodules
In order to get other extensions to work, I open each subfolder in a different instance of vs code, but this then causes this extension to not be able to pick up the url.
If possible it would be nice to have a setting where I could set the path to the .git folder
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdifferentmatt%2Fvscode-copy-github-url%2Fissues%2F10&data=02|01||c004383f463941ea4f4b08d5d6b361c6|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636650988276785203&sdata=0FhH3C9jIGH7Hi%2B5LLIZ5Bs2N9oEvczOU3%2BWGmenM3E%3D&reserved=0, or mute the threadhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAC0-tKtJpM9JyMbZM3GkcsBgevk5s1Teks5t-lDKgaJpZM4UvS3e&data=02|01||c004383f463941ea4f4b08d5d6b361c6|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636650988276785203&sdata=TPeRA7L4S0K0hyvcCc7IWYPLn%2Btz24ChHNQmLl7Vmb8%3D&reserved=0.


fix(email address back link): add different back routes

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

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


Feedback New welcome page

Testing #2251 on OS X
Feedback:

We need to ensure that the welcome page loads fast. The first time I've tried it took > 10s to load.
I use Safari in Full screen mode, when the welcome page showed up in the browser it switches to Safari in full screen and I got disoriented. The welcome page should have some hint that I've installed VS Code and can now go back to it.

Is there a way hide the welcome page from the welcome page so that I can easily get back to VS Code?

FYI @SofianHn @bgashler1

@egamma is the 10s loading time consistent? this sounds pretty bad, the welcome page now resolves to our docs home page and nothing in the website should take that long. did you try multiple times?
@bgashler1 is working on a design of the welcome page and it will definitely include details that is relevant for someone who just installed the product. Once that is ready, we will update the FWD link.

Is there a way hide the welcome page from the welcome page so that I can easily get back to VS Code?

not AFAIK, javascript can't close a window that it does not open. Even if we do that there is no guarantee that safari will exit the fullscreen mode. I get your point about the distraction tho, lets see if we get any complains.

Re: page load times - do we really want the first-run experience to be so tightly dependent on internet connection quality or network configuration?
Re: context-switching - full-screen or not, popping up a whole new window without explicit action by the user is an altogether jarring experience because it makes users feel less in control. It's fine if there's a link that a user clicks to trigger the action, though. Have we considered providing some basic UI within VSCode, and just populating it with a feed? This would result in a smoother first-run experience, instant load time of some predefined set of content, while keeping the benefit of being able to update it more frequently.

It was a long and difficult conversation that made us go with this design, we are aware of the downsides of it but hoping that the upside is bigger. Being on the website will allow us to provide tailored getting started content, as well as running experiments and improving our measurement. We will be listening for feedback, and again if this turns to be an issue we can always pull it out.

Gotcha, makes sense! :+1: I agree - there's certainly plenty of benefit for very little engineering cost, and it'll also help us quickly experiment with and iterate on the content, and ultimately provide the best on-boarding experience.
In the meantime, we should at least consider checking that the the user is connected to the internet before attempting to load the page. see #2400

cool. Closing this for now.


⚠ Bofrost.fr has degraded performance

In 53634a3, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1545 ms

Resolved: Bofrost.fr performance has improved in 12b6824 after 9 minutes.


Hamiltonial Problem

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

I would like to work on this issue as a part of LGM_SOC

I would like to work on the C++ and Java programming languages as a part of LGM_SOC.

https://www.geeksforgeeks.org/hamiltonian-cycle-backtracking-6/ .The question Link On GFG.


[Bug]: Error: Couldn't reach supabase. Either the address doesn't exist or the network might not be on. Retrying in 10s

General Info

[X] I checked for similar bug report
[X] I am using the latest version
[X] I checked the troubleshooting page for similar problems
[X] I enabled logging and checked the logs

Version(s)
1.1.0
Target(s)
JVM
What happened? (include your code)
While returning to my project to work on, I re-run the app and I always get logged-out with the log shared below.
Here is my setup:
bindSingleton<SupabaseClient> {
        val (url, key) = SupabaseConfigModel()
        createSupabaseClient(
            supabaseUrl = url,
            supabaseKey = key
        ) {
            defaultSerializer = KotlinXSerializer(instance())
            install(GoTrue)
            install(Postgrest)
            install(Realtime)
            install(GraphQL) {
                apolloConfiguration {
                    addHttpInterceptor(LoggingInterceptor(LoggingInterceptor.Level.BODY))
                }
            }
            install(Functions)
            install(Storage)
        }
    }
 bindSingleton<GoTrue> { instance<SupabaseClient>().gotrue }

After few relaunches, the app works just fine and the error is gone, otherwise logging in fixes it for the meantime.
Steps To Reproduce (optional)

signIn:

override suspend fun signIn(email: String) {
        goTrue.sendOtpTo(Email, createUser = true) {
            this.email = email
        }
    }

verifyOtp:

 goTrue.verifyEmailOtp(OtpType.Email.MAGIC_LINK, email, otp)

successful login.
keep the desktop app open for awhile
rerun the app fron Intellij IDE again.
observe the behaviour of logging out the user and getting the error log below.

Relevant log output (optional)
Error: Couldn't reach supabase. Either the address doesn't exist or the network might not be on. Retrying in 10s
java.lang.NullPointerException: Cannot invoke "io.github.jan.supabase.network.KtorSupabaseHttpClient.request(String, kotlin.jvm.functions.Function1, kotlin.coroutines.Continuation)" because the return value of "io.github.jan.supabase.SupabaseClient.getHttpClient()" is null
	at io.github.jan.supabase.network.SupabaseApi.rawRequest$suspendImpl(SupabaseApi.kt:24)
	at io.github.jan.supabase.network.SupabaseApi.rawRequest(SupabaseApi.kt)
	at io.github.jan.supabase.gotrue.AuthenticatedSupabaseApi.rawRequest(AuthenticatedSupabaseApi.kt:23)
	at io.github.jan.supabase.network.SupabaseApi.request(SupabaseApi.kt:20)
	at io.github.jan.supabase.gotrue.GoTrueImpl.refreshSession(GoTrueImpl.kt:641)
	at io.github.jan.supabase.gotrue.GoTrueImpl.importSession(GoTrueImpl.kt:382)
	at io.github.jan.supabase.gotrue.GoTrueImpl.loadFromStorage(GoTrueImpl.kt:429)
	at io.github.jan.supabase.gotrue.GoTrue.loadFromStorage$default(GoTrue.kt:301)
	at io.github.jan.supabase.gotrue.GoTrueImpl$1.invokeSuspend(GoTrueImpl.kt:79)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
	at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
	at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:584)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:793)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:697)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:684)

Publishing version 1.1.1-dev, can you check if that exception still occurs?

Left the app open since yesterday when posting this, I did update to 1.1.1-dev and relaunched the app and i dont see the issue. I will keep monitoring and let you know if I encountered it again.
Can you point me to the commit? im curious to see what was it 🙈

https://github.com/supabase-community/supabase-kt/commit/2f20c6d8ae454634211b649f99d2f5aed2ed88bc. The only reason I could think of was that the plugins get created before the HTTP Client which cause this NPE

Hi Jan,
now, the silent error is as below:
Error: Couldn't reach supabase. Either the address doesn't exist or the network might not be on. Retrying in 10s
java.lang.NullPointerException: Cannot invoke "io.github.jan.supabase.plugins.PluginManager.getInstalledPlugins()" because "this_$iv" is null
	at io.github.jan.supabase.gotrue.AuthenticatedSupabaseApi.rawRequest$lambda$1(AuthenticatedSupabaseApi.kt:70)
	at io.github.jan.supabase.network.KtorSupabaseHttpClient.request(KtorSupabaseHttpClient.kt:45)
	at io.github.jan.supabase.network.SupabaseApi.rawRequest$suspendImpl(SupabaseApi.kt:24)
	at io.github.jan.supabase.network.SupabaseApi.rawRequest(SupabaseApi.kt)
	at io.github.jan.supabase.gotrue.AuthenticatedSupabaseApi.rawRequest(AuthenticatedSupabaseApi.kt:23)
	at io.github.jan.supabase.network.SupabaseApi.request(SupabaseApi.kt:20)
	at io.github.jan.supabase.gotrue.GoTrueImpl.refreshSession(GoTrueImpl.kt:641)
	at io.github.jan.supabase.gotrue.GoTrueImpl.importSession(GoTrueImpl.kt:382)
	at io.github.jan.supabase.gotrue.GoTrueImpl.loadFromStorage(GoTrueImpl.kt:429)
	at io.github.jan.supabase.gotrue.GoTrue.loadFromStorage$default(GoTrue.kt:301)
	at io.github.jan.supabase.gotrue.GoTrueImpl$1.invokeSuspend(GoTrueImpl.kt:79)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
	at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:108)
	at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:584)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:793)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:697)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:684)

Made some more internal changes, can you try out version 1.1.1-dev-2?

I'll do that later today and get back to you.
Thanks for the quick response and support 🙏

so far, I didn't encounter the issue, I will keep this open until Sunday when I will work again on the project im currently doing and keep you posted.
Danke Jan.

so far, I didn't encounter the issue, I will keep this open until Sunday when I will work again on the project im currently doing and keep you posted.
Danke Jan.

Alright, included this fix in 1.1.1 for now!

so far, I haven't encounter the issue and also the auth issue seems to be gone.
Cheers.


Struct-level [requires] aren't enforced on field writes

Suppose I have the following Emboss definition:
struct Foo:
  [requires: bar == 0]
  0 [+1] UInt bar

If I were to write a non-zero value into bar, emboss_prelude.h:185 EMBOSS_CHECK(result); wouldn't execute and the write would succeed. It wouldn't be until I called bool Ok() that I would find out that something was written incorrectly. At that point, I just get a boolean true or false and not a directed, clear result of what went wrong.
This isn't the case if I place the [requires] block on the field itself:
struct Foo:
  0 [+1] UInt bar
  [requires: this == 0]

This second Emboss definition would catch the non-zero write via emboss_prelude.h:185 EMBOSS_CHECK(result);. Regardless of where the [requires] block is placed, we should catch incorrect writes when they happen.

This might be difficult for complex struct-level requires, as the fields would need to be written in an order such that the requires is always true.


Unnamed NFTs on Moonriver

It seems that there all of the NFTs from MoonriverBoredApeYachtClub are nameless which results in weird effect while exploring the gallery, i.e. there's empty popup on hover on card.

Are these NFTs indexed properly? should they be nameless?
If yes, (and there are in fact going to be more NFTs without names) can we handle the gallery overview in a better way? Not sure how, maybe @roiLeo can suggest something

Instead of hiding it we should show "NameOfCollection - #ID"

👋


Update howto-sspr-windows.md

corrected v1909.. as confirmed by customer if this is enabled on lower version also password reset fails not limited to v1909. Hence changed to below and added note for WHFB PIN reset at lock screen
If Ctrl+Alt+Del is required by policy in Windows 10 , Reset password won't work.

@NagaCSC : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@NagaCSC : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


New design for event view

First, I would like to appreciate your work on this amazing calendar widget. It's really hard to find a suitable widget that can fit my requirement.
Event view
In this PR, the event view is reworked to make it feels like the Apple built-in calendar widget.

The events are grouped by date
Number of displayed event is not hardcoded. It will depends on remaining space and display as much as possible.
Display event location if available
Tap the event item will open calendar app to the event date (only tested with Apple Calendar)
For all day event, display an icon instead of text to save one row of space

Calendar view
I have added some options to:

Enable/disable smaller date size for previous/next month
Text color for previous/next month

It is just because I personally like consistent size and use grey color to distinguish previous/next month

For text today, tomorrow and days after, I don't think there is built-in i18n for them, so I have made a small translation file for putting those translation. Currently English, Chinese and Japanese are available (I only know these languages). We could invite people to help translating them, or just use Google Translate. It is up to your decision.

oh wow, that looks great. thank you so much for this. that is a lot of features
anyways, i don't use this widget script enough anymore to fully appreciate your effort. i feel i should have archived this repository some time ago. so i'm not exactly sure what's the best path forward. something i've been thinking about is that i'll archive this repository and if you're up to it, i'll point to your fork as the continuation of this project so anyone who happens to end up here knows where to find the latest and greatest features.

for some reason or other i cannot actually get this pull request to consistently work in a widget. c6f1e91 seems to be the last commit that eventually renders something. it does work within scriptable with the debug flag. on a mac even my own script doesn't work due to some calendar permission issue. these problems may resolve over time, but i wouldn't want to leave this pull request hanging here until then. i could branch the current main into a "previous version", merge this pull request as the new "main version" and anyone who runs into any issues with this could always try the previous one.

for some reason or other i cannot actually get this pull request to consistently work in a widget

I think the problem is I am using Device.locale API to get user device locale but the API is returning some weird locale code. I've changed the default locale setting back to en_GB. The widget should work with default settings now.

something i've been thinking about is that i'll archive this repository and if you're up to it, i'll point to your fork as the continuation of this project

I'm happy to take the work as there's no calendar widget that suit my need. A customizable calendar widget is just perfect for me. I'll continue to work on this until I don't use calendar/iPhone or I found a better calendar app.

I think the problem is I am using Device.locale API to get user device locale but the API is returning some weird locale code. I've changed the default locale setting back to en_GB. The widget should work with default settings now.

I already played around with this, because in my case, Device.locale returns the tag separated by _, but Intl.RelativeTimeFormat expects the tag to be separated by -.

I'm happy to take the work as there's no calendar widget that suit my need. A customizable calendar widget is just perfect for me. I'll continue to work on this until I don't use calendar/iPhone or I found a better calendar app.

Thank you, I've updated the readme to point to your repository now.


JB-599 - Add 'extension' data to cycles & payouts

What does this PR do and why?
Closes JB-599 : Add 'extension' data to cycles & payouts
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3862

PR #3863   👈

This comment was auto-generated by Graphite.

@wraeth-eth started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@wraeth-eth merged this pull request with Graphite.


Hamdi/fee challenge

1.) Refactored libraries. Contract is too expensive to deploy - exceeds gas limit
2.) Fixed fee vulnerabilities. User must commit to fees payed when exiting.
3.) Renamed contract from RootChain -> PlasmaMVP
Fixes #84

Add challenging for fee amount of deposits being exited.


Compilation Error: MOSEK library linking issue

Dear project maintainer,
I'm encountering an error while trying to compile the time_optimizer package. The error occurs during the linking process with the MOSEK library. Here are the details:
Environment

Operating System: [Ubuntu 20.04]
ROS Version: [e.g., ROS Noetic]
Architecture: [ x86_64]

Error Message
I'm receiving the following error when running catkin_make:
/usr/bin/ld:/home/cwkj/mavros_ws/src/time_optimizer/include/time_optimizer/third_party/lib/x86/libmosek64.so:0: syntax error
collect2: error: ld returned 1 exit status
make[2]: *** [time_optimizer/CMakeFiles/time_optimizer.dir/build.make:631：/home/cwkj/mavros_ws/devel/lib/libtime_optimizer.so] Error 1
make[1]: *** [CMakeFiles/Makefile2:4340：time_optimizer/CMakeFiles/time_optimizer.dir/all] Error 2
make: *** [Makefile:144：all] Error 2
Invoking "make -j2 -l2" failed
Copy
Steps to Reproduce

Cloned the time_optimizer repository
Installed dependencies as per the README
Ran catkin_make in the workspace

What I've Tried

Verified the existence of the MOSEK library file
Checked file permissions
Ensured the MOSEK version matches my system architecture

Questions

Is there a specific version of MOSEK required for this project?
Are there any additional setup steps for MOSEK that aren't mentioned in the README?
Could you provide guidance on how to correctly link the MOSEK library in this project?

Any assistance or guidance would be greatly appreciated. Thank you for your time and for maintaining this project.

Are you able to build the entire framework? https://github.com/Zhefan-Xu/CERLAB-UAV-Autonomy

Sorry that I did not build a complete project, I am a beginner, is it correct that I put this in the src directory of mavros? Could you please give me more detailed instructions on which working plate I should put it in🥺🥺
---- Replied Message ----
| From | Zhefan @.> |
| Date | 10/13/2024 20:53 |
| To | Zhefan-Xu/time_optimizer @.> |
| Cc | sunyuan1111 @.>,
Author @.> |
| Subject | Re: [Zhefan-Xu/time_optimizer] Compilation Error: MOSEK library linking issue (Issue #1) |
Are you able to build the entire framework? https://github.com/Zhefan-Xu/CERLAB-UAV-Autonomy
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Thank you for your help! I just went to learn ros .This problem was solved by creating a workspace.
mkdir -p ~/catkin_ws/src
cd ~/catkin_ws/
catkin_make
Thank you again for your patience🥰
---- Replied Message ----
| From | Zhefan @.> |
| Date | 10/13/2024 20:53 |
| To | Zhefan-Xu/time_optimizer @.> |
| Cc | sunyuan1111 @.>,
Author @.> |
| Subject | Re: [Zhefan-Xu/time_optimizer] Compilation Error: MOSEK library linking issue (Issue #1) |
Are you able to build the entire framework? https://github.com/Zhefan-Xu/CERLAB-UAV-Autonomy
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Thanks. Feel free to reopen the issue if you have further questions.


Rename field name -> jobName to follow /deployment api

Rename job name field used for dev deployments to be the same as used in /deployment API:
https://github.com/vespa-engine/vespa/blob/20f84687ed1362a7f76404e2f20a6619fca05119/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/JobControllerApiHandlerHelper.java#L630

Thanks.


Add blog post about public roadmap

This PR adds the upcoming blogpost about our open roadmap.
:memo: Reviewer Checklist
Review this pull request by ensuring the following items:

[ ] http://localhost:3000/roadmap (→ https://vast.io/roadmap) works as expected
[ ] The blog post reads well
[ ] The roadmap repo is open-sourced

[sc-35833]

Thanks for taking the first stab at it, Anthony, even prior to :set spell. 😉

Adding the blocked label until scheduled release next Wed.


Info prompt to restart Code after uninstalling extension, but not installing

When you uninstall an extension, you see this:

But when you install an extension, you have to see Enable in the extensions pane. It seems like you should see an info bar for both cases (or neither).

Fixed.


[CORE] Optimize some code in HashAggregateExecBaseTransformer

What changes were proposed in this pull request?

We should move aggregateFunction validation to doValidateInternal.
For getAttrForAggregateExprs, we can use simpler implementation to get the correct attributes from aggregateExpressions.
For ConverterUtils.getAttrFromExpr, we can use the Expression.references method to obtain the desired Attribute. In the original implementation, when iterating over BinaryLike types, the left child is used for iteration. We can use reference.head to get the same Attribute.

Due to the inclusion of custom agg in ClickHouse, the original logic has been retained, and the getAttrForAggregateExprs has been abstracted as an interface to accommodate different logics implemented by Velox and ClickHouse.
How was this patch tested?
Exists CI.

@rui-mo @zzcclp @PHILO-HE @ulysses-you Could you help review?

cc @rui-mo if you have other comments

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_4459_time.csv
log/native_master_01_22_2024_19a863c0d_time.csv
difference
percentage

q1
35.48
32.31
-3.173
91.06%

q2
23.91
24.07
0.167
100.70%

q3
37.71
37.08
-0.636
98.31%

q4
37.22
38.49
1.272
103.42%

q5
68.62
68.33
-0.287
99.58%

q6
5.44
7.26
1.820
133.48%

q7
81.59
83.81
2.220
102.72%

q8
85.01
83.38
-1.628
98.08%

q9
122.28
117.04
-5.236
95.72%

q10
44.50
40.47
-4.033
90.94%

q11
20.09
20.27
0.178
100.88%

q12
25.92
27.53
1.611
106.22%

q13
45.69
46.48
0.794
101.74%

q14
19.05
15.48
-3.576
81.23%

q15
26.94
28.47
1.534
105.69%

q16
13.91
15.34
1.422
110.22%

q17
100.24
100.56
0.320
100.32%

q18
144.93
145.52
0.589
100.41%

q19
14.15
12.41
-1.741
87.69%

q20
26.42
26.33
-0.087
99.67%

q21
222.39
225.25
2.857
101.28%

q22
13.60
13.60
-0.003
99.98%

total
1215.09
1209.47
-5.617
99.54%


Switch is not deleted

The switch is not deleted in the Home app after deleting it from the configuration file

@Caban-ru and @oigroigm. Found a bug that made the receiver disappear. Sorry for inconvenience. It should be fixed in 2.1.1.

just tested 2.1.1
receiver now fine) works after restart as well
about switches:
I can see them in HB web ui (and they works), but they are not appearing in HomeKit

Ok good that it is working again. Thank you for testing. I'm not sure why it is not appearing in Homekit but is in the web UI. What I can think of is removing the Homebridge from the Home app and after removing the persist and accesories folder, readding. This is the only thing I can think of. You will have to setup all other plugins too in the home app though :/

@oigroigm Do you still have issues in the latest version?

I installed the latest version but hombrige crashes

@oigroigm Hmm. can you post your Homebridge log when you run it with debug enabled? If you don't know how that works, you can add the "debugToInfo": true, line to your config.json:
"platforms": [{
    "platform": "DenonAVR",
    "debugToInfo": true,
    "devices": [{

[2/22/2020, 10:34:45 AM] [DenonAVR] DEBUG: pollForUpdates: <IP_ADDRESS>
[2/22/2020, 10:34:46 AM] TypeError: Cannot read property '0' of undefined
at /usr/lib/node_modules/homebridge-denon-heos/index.js:376:49
at Parser. (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:304:18)
at Parser.emit (events.js:198:13)
at SAXParser.onclosetag (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:262:26)
at emit (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:624:35)
at emitNode (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:629:5)
at closeTag (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:889:7)
at SAXParser.write (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:1436:13)
at Parser.exports.Parser.Parser.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:323:31)
at Parser.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:5:59)
at exports.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:369:19)
at Request._callback (/usr/lib/node_modules/homebridge-denon-heos/index.js:370:6)
at Request.self.callback (/usr/lib/node_modules/homebridge-denon-heos/node_modules/request/request.js:185:22)
at Request.emit (events.js:198:13)
at Request. (/usr/lib/node_modules/homebridge-denon-heos/node_modules/request/request.js:1154:10)
at Request.emit (events.js:198:13)
[2/22/2020, 10:34:46 AM] Got SIGTERM, shutting down Homebridge...
[2/22/2020, 10:34:48 AM] [DenonAVR] DEBUG: pollForUpdates: <IP_ADDRESS>
[2/22/2020, 10:34:48 AM] TypeError: Cannot read property '0' of undefined
at /usr/lib/node_modules/homebridge-denon-heos/index.js:376:49
at Parser. (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:304:18)
at Parser.emit (events.js:198:13)
at SAXParser.onclosetag (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:262:26)
at emit (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:624:35)
at emitNode (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:629:5)
at closeTag (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:889:7)
at SAXParser.write (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:1436:13)
at Parser.exports.Parser.Parser.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:323:31)
at Parser.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:5:59)
at exports.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:369:19)
at Request._callback (/usr/lib/node_modules/homebridge-denon-heos/index.js:370:6)
at Request.self.callback (/usr/lib/node_modules/homebridge-denon-heos/node_modules/request/request.js:185:22)
at Request.emit (events.js:198:13)
at Request. (/usr/lib/node_modules/homebridge-denon-heos/node_modules/request/request.js:1154:10)
at Request.emit (events.js:198:13)

Hmm. It looks like you get a different result from the status page.
Can you try to put the following line in your browser and post the result?
http://<IP_ADDRESS>:8080/goform/formMainZone_MainZoneXmlStatusLite.xml or http://<IP_ADDRESS>:80/goform/formMainZone_MainZoneXmlStatusLite.xml

STANDBY

AUXD

-60.0

off

Ok, clear. Apparently your receiver doesn't have the option to switch between relative and absolute volume display, so it misses that info too. This means that it was reading data that wasn't available. In 2.3.6 this should be fixed.

i tried the new version and it seems to work.
Thanks

Great thanks for reporting. will close this issue. If there are other issues please let me know!


Bump andrewrothstein.kpt version to 0.0.3

The kpt Galaxy role has been recently updated. This change has somehow affected the version used by this project. This change updates the version used by the Sandbox installation process.
Fixes: https://github.com/nephio-project/nephio/issues/317
\cc @denysaleksandrov

/approve
/lgtm

/override e2e


Header Section in Mobile Screens

In the Mobile Screens I want to add the Three Bars icon to make the nav bar Responsive .Whenever the User clicks on the Horizontal Bars Icon then the div will dropdown with all the nav items.

The Above Image is just to show my idea .I can Beautify it more according to our color palette.
I Would be grateful if you assign me this issue under gssoc-ext,hactoberfest . My Github Username is @Bhanuprakash842

@Bhanuprakash842 sure go ahead

@Bhanuprakash842 its been two days and no pull request was made so we are closing this issue


Small Abseil fixes for dependents.

Enable PIC so that shared libraries can use the Cartographer
library.
Fixes the config so that dependent builds can find Abseil.
Signed-off-by: Wolfgang Hess<EMAIL_ADDRESS>

This is a follow up of #1570.
FYI @bochen87 @ojura


Added backend support to generate N images

Generate N images

issue #100

Added new function  generateImages in the file openaiController.js .
The function generates N images depending on what the user wants.
Added the function to the routes file.

@SurajPratap10  please merge #101 before merging this because i created almost 3-4 new branches and worked multiple times.
This can be easily be redone and merged the next time.
My pr is big hence requires lot of time to make a new pr.

any demo video? @markyasser  plus do make this available keeping in mind the recent website as it keeps on updating via the PRs of other folks

any demo video? @markyasser plus do make this available keeping in mind the recent website as it keeps on updating via the PRs of other folks

I couldn't make a demo video because I don't have OpenAI key and GPT is not available in my country.
But the function should work properly.
I have synced my branch with the main branch.

any demo video? @markyasser plus do make this available keeping in mind the recent website as it keeps on updating via the PRs of other folks

I couldn't make a demo video because I don't have OpenAI key and GPT is not available in my country. But the function should work properly.
I have synced my branch with the main branch.

No issues, good job


Can "pretty sure" become "certain", please? This is a DoS vulnerability waiting to happen.

wl_closure_lookup_objects checks if objects are of the correct interface. It's called in wl_client_connection_data in the server code, which is called when a connection receives data.

Sweet


Check github CI

Shall the CI support all Integritee's feature?

Teeracle/EVM are not supported by the Encointer node. It hasn't the pallets.
Integration Tests fail with Integritee node. The existential deposit is different.

We need a docker image for the node. Wait for https://github.com/encointer/encointer-node/issues/278

We need a dev image of branch community-sidechain built with  "--features skip-ias-check"


Make the database connection "on demand"

To try and enable us to utilise the queue for the source tree descent this proposes a way to make the database connection object only open once an Analyser Task needs to make use of it.  This plays into the way multiprocessing works as you cannot pass "live" connections to parallel workers.
However, doing things this way comes at a slight cost - there is some overhead in initialising a connection object (rather than re-using one which was previously opened).  The hope is that being able to utilise multiple workers in the source tree descent offers enough of a benefit on its own that any slight extra overhead in the workers having to create connections becomes worth it

Can we do some before and after tests on a sizeable source tree to quantify any perforamnce cost?

Can we do some before and after tests on a sizeable source tree to quantify any perforamnce cost?

I'll see what I can do - perhaps I can script the generation of a big set of source files that depend on one another in a semi-random (but non-circular) way

Added the following page also describing some benchmarking work:
https://github.com/metomi/fab/wiki/Database-Connection-Test

Ready for another look I think - hopefully I've understood what you were getting at with the testing?


Controls do not disappear on fullscreen

I'm not sure if this is a bug or a feature, but the controls / index do not disappear on fullscreen. Most probably this is because the pointer is always on top of the control. Is there any way to fix this? Would be nice if the controls disappear after some time and only come back when there is some movement.

Is there a quick fix that can I easily apply?
Thanks in advance :-)

Sorry for the late answer.
There is no quick fix but I will create a new release within the next few days

Fixed in latest version :)


Trusted types tests

Builds on top of https://github.com/Polymer/lit-html/pull/1153
Needs https://github.com/Polymer/tools/pull/3503 to be released before it will work

Ah, I guess we're not testing in Chrome canary currently


fix issue 432. tested on iphone

the on-touchstart, on-touchend and on-touchmove attributes just above them are Polymer-specific.  removing the browser ontouch* attributes solves the issue.

Yes! Good fix, it's a Polymer thing, I recognize this kind of issue, well done! Good work on the linting as well. I'm merging it now.


Setup Travis CI

http://jekyllrb.com/docs/continuous-integration/
https://github.com/gjtorikian/html-proofer

htmlproof _site --disable-external

Attribute data-proofer-ignore should be added to image tags using srcset to avoid the following error:
  *  internal image /assets/images/lighten-darken-mix_small.png 540w, /assets/images/lighten-darken-mix_medium.png 900w, /assets/images/lighten-darken-mix_large.png 1200w, /assets/images/lighten-darken-mix_huge.png 1590w does not exist (line 209)
  *  internal image /assets/images/sass-wallpaper_small.jpg 540w, /assets/images/sass-wallpaper_medium.jpg 900w, /assets/images/sass-wallpaper_large.jpg 1200w, /assets/images/sass-wallpaper_huge.jpg 1590w does not exist (line 619)
  *  internal image /assets/images/sassdoc-preview_small.png 540w, /assets/images/sassdoc-preview_medium.png 900w, /assets/images/sassdoc-preview_large.png 1200w, /assets/images/sassdoc-preview_huge.png 1590w does not exist (line 560)

Done in #322.


Add chartconfig resource chart

Towards https://github.com/giantswarm/giantswarm/issues/1902

Yep, TIL you can have multiple charts but their names must match <project-name>-*-chart, fixed.


Fix call with wrong arguments in Callable.callv() when passing a readonly (const) Array

Use a duplicate of the passed array when it is read-only.
Prior to this fix, when the array was marked as read-only, the temporary address of p_arguments->read_only was used for all parameters. Consequently, this caused all parameters to reference the last element of the array.
Fixes #93600.

This would really benefit from having a test added under modules/gdscript/tests/scripts/runtime/features to make sure it actually fixes the bug and makes sure it remains fixed

As this is not GDScript specific, shouldn't it be a unit test in tests/core/variant/test_callable.h ?

We do many of these tests there
Also how did you confirm this was not script specific? I at least suspect it's specific to arguments in script methods being called, due to how the bindings and data passing works

Given some of the details here I think I have a potential alternative solution that might be better, I'll look at it along with my other changes in:

#88905

I'm not sure that the call to duplicate has that much overhead since the underlying Vector uses CoW if I recall correctly. But I might be wrong.

I'm not sure, that's a good point and it might not matter, but i don't think the code is significantly less readable and it avoids potential weirdness

I did a quick check, and indeed the duplication is done using CoW. If it were up to me I'd keep the duplicate as it's simpler IMO, but I don't have strong opinions about it. You tell me. I'll add the test in modules/gdscript/tests/scripts/runtime/features in the meantime.

You decide, both are fine by me, and let's see what the core team says

A few thoughts:

CoW involves dealing with the atomic refcount, with performance implications.
This looks like an ad hoc fix for an issue due to how read-only arrays are implemented. It may be better to assess solutions to the root issue.

I think it might be difficult to handle the issue with read access on the array itself, given the nature of the reference, but I'd say those points reinforce the use of my suggestion above, it avoids the COW issues, and avoids any issues with the data access
I can't think of a reliable way to handle the read only status of array without breaking compatibility, or allocating more data somehow, but regardless I think this situation warrants at least a temporary local solution

This also needs to be done for Object::callv

Indeed this solution is not perfect, and we should consider modifying how read-only arrays are implemented. I will try to think of a nice solution when I have more time.

Made an issue tracking the broader issues of the read-only state:

https://github.com/godotengine/godot/issues/93702


chore: split Measure.Trim definition from MeasureSpace.lean

The file MeasureSpace has more than 4000 lines: let's move some results out of it.

bors r+


feat(ltr): add fallback styles for ltr + additional support.

Description
Part of https://github.com/Teradata/covalent/issues/323
Instead of assuming there will be a dir="ltr" or dir="rtl", there is a need to set defaults and then overrides when its dir="rtl", since there are use cases where devs wont use dir at all.
What's included?

feat(rtl): added default rules for ltr support with overriding rlt rules.
chore(rtl): polish usage of rtl
bugfix(rtl): fixed cases where rtl wasnt being applied.
feat(file-upload): added support for rtl.
feat(json-formatter): added somewhat support for rtl (there is still issues around the strings.
feat(search): added support for rtl.
feat(paging): added suport for rtl.
bugfix(notifications): fixed rtl support.
bugfix(data-table): fixed rtl support.

Test Steps

[ ] ng serve
[ ] Go to http://localhost:4200/#/components
[ ] Test components on both ltr and rtl

General Tests for Every PR

[ ] ng serve --aot still works.
[ ] npm run lint passes.
[ ] npm test passes and code coverage is not lower.
[ ] npm run build still works.

NOTE: we still need to figure out how to handle strings like Array [n] , etc etc in rtl
Also need to figure out how are we gonna handle rtl for utility classes like pad, push and pull

for utility classes, i think naming like pad-start & pad-end is more semantic.

@akhedrane yeah, i was actually thinking something like that or pad-before and pad-after to be inline with material

We can leave the current left/right/top/bottom for explicit versions (they do this also in some places) and add additional before/after/above/below that are dynamic


Support Project Loom in Jetty 10 & JDK 17+

As of version 10, Jetty supports Project Loom and virtual threads
https://github.com/eclipse/jetty.project/issues/8007
We want to start porting our code to use Loom as soon as it goes GA.
Please add out of the box support for configuring Jetty with virtual threads in the standard HTTP Server config

Before we can integrate this into the framework, we'd have to modify the InstrumentedQueuedThreadPool of dropwizard-metrics to support setting a thread factory.

+1

hi, we already have plans to start looking into Loom as soon as JDK 21 is released next month.
Any news on this? It would be a major issue if we were blocked from using Loom solely because of Dropwizard not supporting it :-/
Thank you, much appreciated

Hi @jfurmankiewiczpros. I've just opened #7457 for basic loom support.

Resolved by #7457
Resolved by #7746

When is the target for 3.0.3 release?

Was this also merged into 4.0.x ?
Any plans to upgrade Jetty to 12.x - where Virtual Threads are enabled by default ?

Was this also merged into 4.0.x ?

enableVirtualThreads option is available since 4.0.3

@grnadav The upgrade to Jetty 12 won't be done for Dropwizard 3.x and 4.x but will be introduced in Dropwizard 5.

@zUniQueX gotcha.
Anywhere I can track the progress towards 5.x ?

@grnadav Jetty 12 extracted the servlet support from the server core and moved the old artifact to separte modules for different EE versions. That causes many breaking changes for the request handling. So we can't upgrade without breaking changes for our users and therefore defer the upgrade to our next major version.
We don't have a timeline for the release of Dropwizard 5 yet. Currently, the Jakarta EE 10 upgrade is blocked by the next release for the Jersey 3.1 branch. After this release is available, we can upgrade to EE 10 and move to a Java 17 baseline which will be the main change for Dropwizard 5.

@zUniQueX dang thats a lot of deps lineup to track. good job with that!
Why is DW5 aiming for Java 17 and not 21? its LTS as well. More deps. issues?

@grnadav We try to support the widest range of Java versions we can to make it easier for users to upgrade their applications in the same system environment. The previous Java baseline changes in Dropwizard were introduced because some libraries dropped support for older Java versions.
With Jetty 12 there is a minimum Java version of 17 required to run Jetty. Therefore we also use Java 17 as our baseline. But newer Java versions are supported as well and we'll run all pipelines against the newest LTS versions once they are released (as we did with Java 21).


Support 'plain' or 'json' output from list

To make programmatic consumption of transient list output easier, we should have some way of not doing the fancy table output.

I'll take it.


runtimeVersion using nvm-windows when it should use WSL

VSCode Version: 1.32.3
OS Version: Windows_NT x64 10.0.17134

Steps to Reproduce:

Have nvm installed on WSL, nvm-windows installed on Windows
Configure VS Code to run/debug applications on the WSL
Add a launch configuration for a NodeJS project
Add a runtimeVersion to the launch configuration with a version that you have not installed on nvm-windows/Windows, but have installed on nvm/WSL.
Run the application --> you get the error "Node.JS version 'x.x.x' not installed for 'nvm-windows'.
Install that version on nvm-windows/Windows.
Run the application --> the application is started in VS Code's debug console, on the WSL

So it seems the runtimeVersion is checked in Windows before running the app in the WSL. It also works if you just remove the runtimeVersion setting.
Does this issue occur when all extensions are disabled?: Yes
Does this issue occur in the Insiders build?: Yes, as of 20/03/2019

I took a deeper look and believe it's related to the vscode-node-debug repo. If you want, I can resubmit it there. If I read the code correctly, it checks the current platform. And of course, I'm running VS Code on Windows, but my application on WSL/Linux.

Apparently, the Debug Console is still running on Windows, instead of the WSL. In light of that, the error message makes sense of course. Yet I thought it was possible to have your application run/debug on WSL, while still using VS Code in Windows?
I have terminal.integrated.shell.windows and terminal.external.windowsExec set to bash.exe. If anyone can point me to some documentation on how to run VS Code in Windows, but have the Debug Console use bash, that would be great.

If you set "useWSL" to true in your launch config, the debug target will be launched in WSL and the Debug Console will be connected to that debug target properly.
The Debug Console is not "running anywhere" because it is not a terminal. The debug console just shows the output of the process being debugged. If the process being debugged runs remotely, the Debug Console will show the output from the remote process but it does not "run remotely".

Ok, makes sense. So I've now managed to get it working again, and my initial report still stands. Here you can see it's connected to bash, but it still complains about nvm-windows:

The output in the debug console is from a previous run, using runtimeVersion 8.10.0, which is installed with nvm-windows. Changing that to a version that isn't installed, will produce the error in the message box.

Currently node.js version managers like "nvm" are not supported for "useWSL".
But we are working on this...

Ok, thanks for answering.


Fix regex to prevent cutoffs caused by '#'

I ran into an issue where if you had an env with spaces around the = AND a # IN the value, then it would cut off the value at the hash.
Example: foo = '123#456' # comment
This fixes the regex to account for that. An example can be found in the spec.

@fergmastaflex can you re-open/re-create this PR and I'll review it and move it forward?


chore: veANC gov fixes + unittests

Summary of changes
Add missing unittests coverage for veANC gov changes + minor fixes.
Report of required housekeeping

[ ] Github issue OR spec proposal link
[ ] Wrote tests
[ ] Added a relevant changelog entry to CHANGELOG.md

(FOR ADMIN) Before merging

[ ] Added appropriate labels to PR
[ ] Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
[ ] Confirm added tests are consistent with the intended behavior of changes
[ ] Ensure all tests pass

 veANC - more unittests to Gov changes


WIP: remove secrets from config yaml and terraform configuration files

Resolves #13.
Proposal for managing secrets... this approach:

Makes use of the fact that terraform variables can be set by setting an environment variable with the same name but prefixed with TF_VAR_
Could be used for all secrets/tokens.
Makes it harder for secrets to leak as they are never stored as plain text.
Does not expose secrets during a render (I need to confirm this, I may have to define the variable as sensitive in variables.tf).
Can serve as an interim prior to more thorough managment of secrets using something like vault/keycloak.
In some cases will require the removal of some values from the schema. e.g. client_id and client_key for google oauth.

Further thoughts on this with respect to breakage of schema compatibility. Currently the schema sets client_id/client_key for a number of different auth use-cases.
With the current approach we would have to remove these from the schema (breaking backwards compatibility) and  in each case document the env variable that needs to be set in order to define the appropriate terraform variable.
We could alternatively use a convention whereby the current fields are filled with a placeholder value instead of the current key. For example:
security:
  authentication:
    type: custom
    authentication_class: "oauthenticator.google.GoogleOAuthenticator"
    config:
      oauth_callback_url: 'https://qhub.ouraes.com/hub/oauth_callback'
      client_id: QHUB_SECRET_google_oauth_client_id
      client_secret: QHUB_SECRET_google_oauth_client_secret
...

The above convention would trigger a search for the env variables google_oauth_client_id and google_oath_client_secret and would use them to set TF_VAR_google_oauth_client_id etc which then populates the corresponding terraform variables. This approach prevents schema breakage and could likely be backwards compatible with the current implementation of storing secrets (if we wish to continue to support that). It seems a hacky though...
For now I will continue to implement this with schema modification...

So @costrouc  pointed out that it is not just .tf files that the secrets are exposed in during the render. Sometimes it is put in other files that won't work as cleanly as we had envisaged. So for example, the auth client_id and client_key are leaked via the jupyterhub config:
data$ grep -r oops_secret .
./qhub-config.yaml:      client_id: oops_secret
./qhub-config.yaml:      client_secret: oops_secret
./infrastructure/jupyterhub.yaml:      c.GitHubOAuthenticator.client_id = "oops_secret"
./infrastructure/jupyterhub.yaml:      c.GitHubOAuthenticator.client_secret = "oops_secret"

Until  we implement something more systematic for secrets management it seems like avoiding rendering these values is too challenging and if security is the imperative on a CI provider based deployment the best option is to simply not render.

So @costrouc pointed out that it is not just .tf files that the secrets are exposed in during the render. Sometimes it is put in other files that won't work as cleanly as we had envisaged. So for example, the auth client_id and client_key are leaked via the jupyterhub config:

That's true, that those SECRETS are engrained in the files. My thought was to have placeholders in those files and and replace them in terraform during deployment via terraform variables, in here:
https://github.com/Quansight/qhub/blob/08c94b91428f68d47270ba003a7461c45ef64f48/qhub/template/{{ cookiecutter.repo_directory }}/infrastructure/modules/kubernetes/services/meta/qhub/main.tf#L12-L13

This implements both approaches based on the two use-cases:

The user wishes to remove secrets from the config yaml. This can be done for any field using the QHUB_SECRET_ pattern. These values will often continue to be rendered but if the user turns off rendering it goes a long way to protecting the values.
Due to the way the Prefect integration is implemented the required secret can be provided to terraform through the TF_VAR_prefect_token environment variable. This variable is now explicitly checked for. Overall this is a nice pattern and should be mimicked for future integrations.

@aktech's suggested is a nice addition to this PR.. .namely, we implement a extra bit of logic to replace the QHUB_SECRET placeholder after rendering (so we don't leak secrets there) but before the relevant file is provided to terraform. This may have to be implemented on a case by case basis because the relevant values might be in any file... or perhaps we could search every file for the placeholder patterns...

Sorry about the style issues. Fixed now. Tests for the get_secret_config_entries function are in tests/test_render.py or did you mean that I should add more?

Sorry about the style issues. Fixed now.

Awesome, thanks!

Tests for the get_secret_config_entries function are in tests/test_render.py or did you mean that I should add more?

Yep, I missed it apologies. Taking a look now.

This looks great now. One last thing, can you add some documentation, maybe here.

Looks great, thanks for working on it!


World generation bugs when touching I:groundLevel

Description
The generation process takes forever if the config option I:groundLevel is set to 10. The game is stuck with the "Building world" screen and you never reach your world
If you set it to 20, you do reach your world, but the chunks won't load. You get this: http://i.imgur.com/OT4FCM6.png
Basic Information

MC Version; 1.11.2
Mod Version: lostcities-1.1x-0.0.12beta
Forge version: net.minecraftforge:forge:1.11.2-<IP_ADDRESS>6:universal

Details
Here's my profile.cfg:

http://rumia.aius.u-strasbg.fr/captures/profile_modernwarfare.cfg

Here's my modlist:

http://rumia.aius.u-strasbg.fr/captures/modlist.txt

It's a wasteland, with no mobs, chisel blocks, taller buildings, and only cities.
Also, I decreased the explosion rate
It works fine if the ground level is set to 71 (default)

A ground level that low is bound to cause all kinds of complications as it has no room to generate the subways, the cellars and various other features that all happen at levels below ground level. Not sure if this is solvable

Perhaps you should change that comment then
# Ground level [range: 2 ~ 256, default: 71]
I'd like to have no underground space because I'm planning to create some kind of PVP-with-guns map and making a base underground is kind of cheating, you know ?

Well at the moment I cannot give you a solution to that. The lowest that I generate stuff is groundLevel - 18 as that is the height where the subway system is generated so groundLevel must at least be greater then 19-20.
Why can't you prevent this in other ways? For example, use adventure mode and don't give access to pickaxes so they can't easily dig down or something

Good idea ! I could add a bedrock layer. Thanks !


Allow parameter escaping in citekey pattern so a colon can be used

Quoting my own comment in #268:
Seeing as quite a few people prefer to use a colon, how about adding a new filter called "prefixcolon". That way, no escaping facility would be required at all and the colon would still not appear in the case of an empty field... just an idea.

I've rather just allowed escaping of parameters with a backslash. Could you give http://tempsend.com/E773422870 a spin?

What can I say, works like a charm! Thank you! Why is the fixed build versioned as 1.1.x by the way? It seems you just released 1.2.1 a couple of hours ago. Is it tuned to my Zotero version or something?

Nope, that's just how fast things move with BBT :) the 1.2.1 version includes this.

iow 1.1.2 was still the current version 19 hours ago

Yes, you seem to have had a quick turnaround these last 24 hours. My flatmate has a serious problem atm btw which might be related to him having installed 1.2.1 (similar to #175, i.e. German error message about not being able to export, empty .bib file, "BetterBibTeX is undefined" in the log, but doesn't seem to be the same) ... waiting for him to reinstall and send another log over to make a new ticket.

Why reinstall?

That's what non-programmers do as a first reflex when something doesn't work...

Shouldn't be necessary.

Just open a new issue; if "report zotplus error" still works, that will probably give me what I need, if not, I'll get you a walkthrough to get the logs.

(also, if somethings breaks, you have my permission to assume it's something I did rather than you)

Lol, understood & thanks :)


Filler server: No blocks processed [DS:0|SH:0|JQ:1]

I started this project with docker.
These are the logs from the filler server.
2022-02-19T15:45:52.592Z [PID:39] [info] : Ship connect options {"start_block_num":166284797,"end_block_num":4294967295,"max_messages_in_flight":50,"have_positions":"removed","irreversible_only":false,"fetch_block":true,"fetch_traces":true,"fetch_deltas":true}  2022-02-19T15:45:52.682Z [PID:39] [info] : Receiving ABI from ship...  2022-02-19T15:45:54.490Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:45:54.511Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:45:54.502Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:45:54.518Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:45:54.531Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:45:54.536Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:45:54.534Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:45:54.547Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:45:54.515Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:45:54.585Z [PID:39] [info] : Launching deserialization worker...  2022-02-19T15:46:02.600Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3590 seconds  2022-02-19T15:46:07.601Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3585 seconds  2022-02-19T15:46:12.602Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3580 seconds  2022-02-19T15:46:17.603Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3575 seconds  2022-02-19T15:46:22.604Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3570 seconds  2022-02-19T15:46:27.606Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3565 seconds  2022-02-19T15:46:32.607Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3560 seconds  2022-02-19T15:46:37.608Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3555 seconds  2022-02-19T15:46:42.609Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3550 seconds  2022-02-19T15:46:47.610Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3545 seconds  2022-02-19T15:46:52.610Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3540 seconds  2022-02-19T15:46:57.611Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3535 seconds  2022-02-19T15:47:02.613Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3530 seconds  2022-02-19T15:47:07.614Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3525 seconds  2022-02-19T15:47:12.615Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3520 seconds  2022-02-19T15:47:17.616Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3515 seconds  2022-02-19T15:47:22.616Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3510 seconds  2022-02-19T15:47:27.617Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3505 seconds  2022-02-19T15:47:32.617Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3500 seconds  2022-02-19T15:47:37.618Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3495 seconds  2022-02-19T15:47:42.618Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3490 seconds  2022-02-19T15:47:47.619Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3485 seconds  2022-02-19T15:47:52.619Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3480 seconds  2022-02-19T15:47:57.619Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3475 seconds  2022-02-19T15:48:02.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3470 seconds  2022-02-19T15:48:07.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3465 seconds  2022-02-19T15:48:12.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3460 seconds  2022-02-19T15:48:17.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3455 seconds  2022-02-19T15:48:22.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3450 seconds  2022-02-19T15:48:27.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3445 seconds  2022-02-19T15:48:32.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3440 seconds  2022-02-19T15:48:37.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3435 seconds  2022-02-19T15:48:42.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3430 seconds  2022-02-19T15:48:47.619Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3425 seconds  2022-02-19T15:48:52.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3420 seconds  2022-02-19T15:48:57.619Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3415 seconds  2022-02-19T15:49:02.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3410 seconds  2022-02-19T15:49:07.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3405 seconds  2022-02-19T15:49:12.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3400 seconds  2022-02-19T15:49:17.621Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3395 seconds  2022-02-19T15:49:22.620Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3390 seconds  2022-02-19T15:49:27.621Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3385 seconds  2022-02-19T15:49:32.628Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3380 seconds  2022-02-19T15:49:37.627Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3375 seconds  2022-02-19T15:49:42.628Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3370 seconds  2022-02-19T15:49:47.628Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3365 seconds  2022-02-19T15:49:52.628Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3360 seconds  2022-02-19T15:49:57.640Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3355 seconds  2022-02-19T15:50:02.640Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3350 seconds  2022-02-19T15:50:07.640Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3345 seconds  2022-02-19T15:50:12.640Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3340 seconds  2022-02-19T15:50:17.640Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3335 seconds  2022-02-19T15:50:22.641Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3330 seconds  2022-02-19T15:50:27.641Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3325 seconds  2022-02-19T15:50:32.642Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3320 seconds  2022-02-19T15:50:37.640Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3315 seconds  2022-02-19T15:50:42.641Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3310 seconds  2022-02-19T15:50:47.641Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3305 seconds  2022-02-19T15:50:52.641Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3300 seconds  2022-02-19T15:50:57.642Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3295 seconds  2022-02-19T15:51:02.646Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:0] - Stopping in 3290 seconds  2022-02-19T15:51:15.664Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:0|JQ:1] - Stopping in 3277 seconds
On the last line, I can see it change JQ to 1, but the server just stopped here for hours and nothing happened.
Please help me with this one.

Have you started the blockchain node from a snapshot? Seems like it is not returning any history data


Click to symbol does not jump to file but to the import (JSX/JS).

VSCode Version: Version 0.10.14-insider (0.10.14-insider)
OS Version: OSX 10.11.4

Steps to Reproduce:

Cmd+Click on a Component in JSX (also JS).
VSCode jumps to the import and not the file itself.

Expected:
VSCode should just jump to the file referenced in the import as its smart enough to know the specific import.

I get that behavior also on non-JSX functions.

Updated, you are correct.

This is how the TS server currently resolved. IMO the behavior is 'technically' correct but not very user friendly. IMO the tsserver should either:

offer a flag to control that
resolve the definition to the 'implementation' if the import doesn't rename the symbol
jump to the import if it does rename the symbol.

This issue was moved to Microsoft/TypeScript#7930


property for representing product model components

We need a way of representing product model components, i.e., we need an abstract version of ProductObject's hasProductComponent for Product. This should be added to the product module.
Did we have naming convention for the abstract/concrete components?

Looking through the product module again I believe we may actually cover this as part using hasComposition and Composition. Example:
<SomeShoeModel> a product:Product ;
    product:hasComposition [
        a product:ProductComposition ;
        product:associatedWithProductModel <SomeLaceType> ;
        product:compositionQuantity 1
    ] .
<SomeLaceType> a product:Product .

The issue is then instead about including a shortcut for linking to product models to product components.
Suggestion:

Change domain and range of product:hasProductComponent to include also product:Product, or
Split the product:hasProductComponent into two properties (product:hasProductComponent with range Product and product:hasProductObjectComponent with range ProductObject)

I prefer that latter due to the clear distinction between abstract and concrete products. Thoughts?
@evabl444

Let's see if I get this... so you say that we have the Composition class, which is a reification of the composition relation, both for the item level and the model level? And then you suggest to have two separate "shortcuts", one for models and one for items?
But for the reification, is there a corresponding property to "product:associatedWithProductModel" for the case when it is a ProductObject that has a composition, and which should be related to components that are ProductObjects and not models?
Otherwise I also agree with the second bullet point. (But I did not have time to check the actual file today.)

I think currently, at the concrete project object level, although we have physical object is also domain of hasComposition, we don't have some property like "product:associatedWithProductModel" to connect objects' composition to product model level.
I also agree with the second bullet point. As I understand, in such way, we kinda have two paralleled ways for both the product model level and object level. But then my concern is whether we need to have "Composition" at the model and object levels?

But then my concern is whether we need to have "Composition" at the model and object levels? That is a good question... Not sure.
Maybe have:

ProductComposition with associatedWithProduct (abstract)
Direct link product:hasProductComponent (abstract)
ProductObjectComposition with associatedWithProductObject (specific object)
Direct link product:hasProductObjectComponent (specific object)

Or do we want to keep associatedWithProductModel? Any further thoughts on this? @evabl444 @huanyu-li

Do you mean have another property like associatedWithProductObject for ProductObjectComposition?


Release 0.4.0

@JuliaRegistrator register(branch=release)

Registration pull request created: JuliaRegistries/General/1597
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.4.0 -m "<description of version>" eb80db7945019e6bab6b47602461308708b07ab6
git push origin v0.4.0

@JuliaRegistrator register(branch=release)

Registration pull request updated: JuliaRegistries/General/1597
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.4.0 -m "<description of version>" 648792f218672bdade138ffd055b5a3b43937f29
git push origin v0.4.0


Revert "libmsgpack_1_4: remove"

This reverts commit ff89e8189800db204eb251c410f7cd7699b9c36b.
Motivation for this change
There are references to this variable.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

I suggest we revert now and then we can look into proper removal in a later PR.

@vcunat You seem to have been active recently, can I can approval to push this now?

neovim-qt builds is the remaining consumer of 1.4; it builds and appears to run with msgpack 2 here. @7c6f434c might know more about why neovim-qt was configured to build against 1.4.

Sorry missed that one, I'll check that again, and thanks for reverting in the meantime

No problem. Hopefully we can kill it soon :)

Most likely I just went with whatever was recommended for the specific version of NeoVim. I guess there has been an update of NeoVim since then.


Convert repo to typescript

can use rollup plugins after https://github.com/palmetto/palmetto-components/pull/24 is merged

https://github.com/rollup/plugins (babel and typescript)

connect typescript types and proptypes: https://github.com/milesj/babel-plugin-typescript-to-proptypes

comments can be transpiled too https://github.com/milesj/babel-plugin-typescript-to-proptypes#comments-boolean

 Convert Palmetto Components to Typescript


Automate SSL certificate renewal on ci.adoptopenjdk.net

Presumably, this would be a case of using Nagios to monitor if the cert has expired (#1568 ), and if so, run a script containing the instructions that are given on login to the ci.adoptopenjdk.net machine :
cd /opt/letsencrypt/
./letsencrypt-auto --config /etc/letsencrypt/configs/ci.adoptopenjdk.net.conf certonly
service nginx restart

I know nothing about the Jenkins config. If indeed nginx is in front of Jenkins, I'd use certbot and let it handle everything. You can call certbot regularly with cron and it will automatically update the certificates if required. I can send you my Ansible role I use at work.

I've managed to get a working example of a service handler working on the 2 VMs that I have to simulate a Nagios instance. To do this I setup a dummy task in the server config (/usr/local/nagios/etc/servers/XXX.cfg)
define service {
      host_name			      ubuntu_host
      service_description             failing_check
      check_command                   check_by_ssh! exit 1
      ...
      event_handler                   test_handler
}

Fairly self explanatory, but it runs exit 1 on the host via ssh. The event_handler field handles what happens as a state-change. To simulate these, I would just use the Nagios interface to set the "state" manually.
The event_handler field calls the test_handler command, which is defined in /usr/local/nagios/etc/objects/commands.cfg.
define command {
  command_name     test_handler
  command_line     $USER1$/check_by_ssh -t 360 -H $HOSTADDRESS$ -C '/usr/lib/nagios/plugins/test_script.sh $SERVICESTATE$'
}

This is defined like any other command, but it uses the check_by_ssh command to start the test_script.sh on the host machine. $USER1$ is the path to the check_by_ssh executable on the Nagios server.
In this example, all test_script.sh does is print the $SERVICESTATE$ (i.e. 'OK', 'WARNING', 'UNKNOWN', 'CRITICAL') to a file located at: /home/nagios/debug.log, which is located on the host.
When implemented, the actual script will run the commands listed above:
cd /opt/letsencrypt/
./letsencrypt-auto --config /etc/letsencrypt/configs/ci.adoptopenjdk.net.conf certonly
service nginx restart

Note: To get the check_by_ssh command working at all, the Nagios user on the Host machine needs to have the ssh-key for the Nagios user on the server-machine, in the authorized_keys file. @karianna I'm slightly concern this would be a security issue (as it's effectively a single-point of failure). All of the Nagios users on the host machines will have limited permissions, but even so.

Why don't you install a cron job on the respective machines that invoke certbot or whatever letsencrypt-auto does? Seems significantly simpler and does not require ssh'ing into the machine.

I'm certainly not opposed to that - I just haven't used Cron before :grin: It also made sense to me to link how #1568 was implemented, to this. However, If we do it with a Cron Job (and based on the list of things you suggested we monitor in #1229), we wouldn't have to use the check_by_ssh plugin at all, which stops my single-point-of-failure concern too :-)
I'll do some testing and put my findings here

Okay, yep, looking at it, cronjobs seem easy to implement. The letsencrypt-auto command renews it for 90 days, I was thinking we run those commands every 80 days(ish).
Cronjobs don't really allow for every X amount of days, so we could just setup the cron job to run everyday, and check it's been 80 days since the last cert renewal in the script ( probably using the check_ssl_cert command to find the expiration date)

The letsencrypt-auto command renews it for 90 days, I was thinking we run those commands every 80 days(ish).

Run it every week by putting it in /etc/cron.weekly. If there's no cert to renew, it will skip it.

If that's the case, isn't there a chance of there being (up to) 6 days with no certificate renewal, if we checked weekly?

Certbot usually renews the certificates well in advance. So the certificate should be renewed before Nagios complains. I haven't seen a single warning since we deployed certbot at work.

Hmm, okay- I'll look at certbot in that case, I was just going to put
cd /opt/letsencrypt/
./letsencrypt-auto --config /etc/letsencrypt/configs/ci.adoptopenjdk.net.conf certonly
service nginx restart

in a script and run it w/ a cronjob

It looks like letsencrypt-auto should do more or less the same that certbot does. Do you know where letsencrypt-auto comes from?

Not specifically, it's part of the login message when you ssh to the ci.adoptopenjdk.net server:
TO RENEW THE HTTPS CERT:

cd /opt/letsencrypt/
./letsencrypt-auto --config /etc/letsencrypt/configs/ci.adoptopenjdk.net.conf certonly
service nginx restart

It could be https://github.com/certbot/certbot/blob/master/letsencrypt-auto? So you could leave it in place and maybe we switch to an OS package after upgrading Ubuntu.

For now, @aahlenst suggested we just put letsencrypt-auto renew --renew-hook "systemctl reload nginx" into a file in /etc/cron.weekly/. From what I can gather, that will renew all of the certificates on the system automatically, when they expire in <30 days. The --renew-hook argument is where the user can specify what's run after a renewal happens. If I'm not missing anything, is everyone happy with that solution?

No objections from my side.

Cron job has been put in, I'll keep an eye to make sure the SSL cert is actually renewed when it gets close, via Nagios. Closing issue


my docker images are able to ping eureka-server using curl in docker terminal but not getting registered

I am using spring boot, spring cloud netfix eureka-server, config-server and docker to run microservices.
Everything is fine in non-dockerized environments, but once I dockerized the eureka server and the microservice, for example user-service, I found that the user-service could not registered to eureka server but able to ping using "curl http://netflix-eureka-service-registry:8761".
My config-server is able to get details from my git repo where I am bale to see using /actuator/default
2024-05-21 18:39:25 2024-05-21T21:39:25.139Z  INFO 1 --- [cloud-config-server] [freshExecutor-0] c.n.d.s.t.d.RedirectingEurekaHttpClient  : Request execution error. endpoint=DefaultEndpoint{ serviceUrl='http://localhost:8761/eureka/}, exception=I/O error on GET request for "http://localhost:8761/eureka/apps/": Connect to http://localhost:8761 [localhost/<IP_ADDRESS>] failed: Connection refused stacktrace=org.springframework.web.client.ResourceAccessException: I/O error on GET request for "http://localhost:8761/eureka/apps/": Connect to http://localhost:8761 [localhost/<IP_ADDRESS>] failed: Connection refused
2024-05-21 18:39:25     at org.springframework.web.client.RestTemplate.createResourceAccessException(RestTemplate.java:915)
2024-05-21 18:39:25     at org.springframework.web.client.RestTemplate.doExecute(RestTemplate.java:895)
2024-05-21 18:39:25     at org.springframework.web.client.RestTemplate.execute(RestTemplate.java:790)
2024-05-21 18:39:25     at org.springframework.web.client.RestTemplate.exchange(RestTemplate.java:672)
2024-05-21 18:39:25     at org.springframework.cloud.netflix.eureka.http.RestTemplateEurekaHttpClient.getApplicationsInternal(RestTemplateEurekaHttpClient.java:145)
2024-05-21 18:39:25     at org.springframework.cloud.netflix.eureka.http.RestTemplateEurekaHttpClient.getApplications(RestTemplateEurekaHttpClient.java:135)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$6.execute(EurekaHttpClientDecorator.java:137)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.RedirectingEurekaHttpClient.executeOnNewServer(RedirectingEurekaHttpClient.java:121)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.RedirectingEurekaHttpClient.execute(RedirectingEurekaHttpClient.java:80)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.getApplications(EurekaHttpClientDecorator.java:134)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$6.execute(EurekaHttpClientDecorator.java:137)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.RetryableEurekaHttpClient.execute(RetryableEurekaHttpClient.java:120)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.getApplications(EurekaHttpClientDecorator.java:134)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$6.execute(EurekaHttpClientDecorator.java:137)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.SessionedEurekaHttpClient.execute(SessionedEurekaHttpClient.java:77)
2024-05-21 18:39:25     at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.getApplications(EurekaHttpClientDecorator.java:134)
2024-05-21 18:39:25     at com.netflix.discovery.DiscoveryClient.getAndStoreFullRegistry(DiscoveryClient.java:1045)
2024-05-21 18:39:25     at com.netflix.discovery.DiscoveryClient.fetchRegistry(DiscoveryClient.java:958)
2024-05-21 18:39:25     at com.netflix.discovery.DiscoveryClient.refreshRegistry(DiscoveryClient.java:1475)
2024-05-21 18:39:25     at com.netflix.discovery.DiscoveryClient$CacheRefreshThread.run(DiscoveryClient.java:1442)
2024-05-21 18:39:25     at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
2024-05-21 18:39:25     at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
2024-05-21 18:39:25     at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
2024-05-21 18:39:25     at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
2024-05-21 18:39:25     at java.base/java.lang.Thread.run(Thread.java:833)
2024-05-21 18:39:25 Caused by: org.apache.hc.client5.http.HttpHostConnectException: Connect to http://localhost:8761 [localhost/<IP_ADDRESS>] failed: Connection refused
Getting this error for all my microservices (api-gateway, user-service, department-service, config-server)
In my git I have this-
eureka.client.register-with-eureka=true
eureka.client.fetch-registry=true
eureka.client.service-url.defaultZone=http://netflix-eureka-service-registry:8761/eureka/
#eureka.instance.hostname=netflix-eureka-service-registry
I tried all posible waus but did not find the issue. I kindly request anyone who can help me. I am learning spring microservices.

Hello @GIRISHARAN have you mapped your docker port to local port? Have you properly set eureka.client.serviceUrl.defaultZone?

Hello @OlgaMaciaszek Yes, I mapped the docker port to my local port. In my git I have properly set eureka.client.serviceUrl.defaultZone. My CloudConfig server is able to get details from my git repo and register itself with Eureka Server but other services which has to get the configuration details from CloudConfig server are not able to fetch details from it.

@GIRISHARAN you are getting a ConnectionRefused; it seems something is misconfigured in your docker environment, especially as the apps work normally while on a different enviroment. It's difficult to know the issue without access to the environment.

@GIRISHARAN  you can use env variables in your configuration for the services something like this -eureka.client.serviceUrl.defaultZone=http://${EUREKA_HOST:localhost}:8761/eureka
If present in the context the value of  EUREKA_HOST will be taken else default will be localhost.
You can pass the hostname for your eureka server you can set custom hostname for your eureka service using
hostname: eureka-server in your docker configuration.

@ansh-devs the issue I am facing now is my services (dept, user) are not able to connect with Cloud Config Server even though I have mentioned configuration details correctly in my application.properties file of that services
spring.cloud.config.import-check.enabled=false
spring.cloud.config.enabled=true
spring.cloud.config.uri=http://CLOUD-CONFIG-SERVER:8083
CLOUD-CONFIG-SERVER is the name of my Docker Container

@GIRISHARAN  try using env variables as i mentioned,
then explicitly assign hostname to your config server container like -  hostname: {your_custom_hostname} in your docker compose file...

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.


Support Postgres Dollar Quotes #1372

Support Postgres Dollar Quotes notation.
Fixes #1372.

 Thank you very much for your contribution, we do look forward receiving more soonest.
In my opinion, the PR looks very good and has incorporated sufficient Unit Test Cases. Well done.