getParameter() super slow on Macs with nvidia GPUs

For pickable layers, getParameter() calls in getBlendMode() are super slow. Seems to be a GPU/platform specific issue that we can solve directly so we need to have a workaround.
Does the layer in our chicago building demo actually need to be pickable?

This will be solved for all similar cases by the new state manager in luma.gl v4. I would be OK with a simpler fix here for now.

This will be solved for all similar cases by the new state manager in luma.gl v4. I would be OK with a simpler fix here for now.

Yes. That will solve the problem once and for all. Currently, we are clearing picking framebuffers, initialize blending mode and set up blend equations even when we don't have any pickable layers. For this specific issue, we might want to patch the code to bypass those.

There seems to be some driver issue around the NVIDIA GPUs used by Mac.
One symptom is that the 64-bit building layer rendered in our Chicago building demo is super slow. This can be mitigate by turning off 64-bit. However, I was regularly hit by GPU kernel panic when looking at this specific demo.
This is not the only GeoJsonLayer we have in our demos. I believe the ArcLayer demo and TripsLayer demo also have GeoJsonLayer inside. Not sure what makes this happen.

There seems to be some driver issue around the NVIDIA GPUs used by Mac.

We did testing on this for v3 on nVidia when we released right? Can we redo that? E.g. if you run the ChoroplethExample of the current github pages, how is performance on the nVidia Mac? Does it also exhibit low FPS / Kernel Panic.
Always good to know if are we dealing with a regression. If it the issue is not present on the older layer, the v4 fp64 library and the v4 SolidPolygonLayer (I assume this is what is rendering the polys in the GeoJson example) are obvious suspects. Also could make sense to double check uniforms and attribute values in the luma log, as I recall the nVidia GPUs can't handle NaNs.

It turns out that the kernel panic might be due to  https://www.apple.com/support/macbookpro-videoissues/ I'll try to secure another laptop with Nvidia GPUs to test.
Closing.


how color transparent?

if i use color method like this:
chart.area().position('date*quantity').color('l(90) 0:#51E3FC 1:rgba(xxx,xxx,xxx, 0.x)')

it not working, but i can't find any simple way to get i wanted transparent gradient?
any suggestion?

It works

@troy351 oh it's my fault，it works now; thanks


Improve PR template

It needs some separation of PR categories so we don't have to manually delete things.

#727 deals with it


Components example does not run As-Is

Noticed that the components example does not run as is.
All references of
Flags:   discordgo.MessageFlagsEphemeral

needs to be cast to uint64
Flags:   uint64(discordgo.MessageFlagsEphemeral)

Error message: Cannot use 'discordgo.MessageFlagsEphemeral' (type MessageFlags) as the type uint64
Go Version: go1.19 windows/amd64

Examples use master branch as the version (@master). Flags were made discordgo.MessageFlags only there, and were not tagged yet. This works as intended.


Version for Go 1.19

Hi!
Version v1.0.6 requires Go 1.20 (uses new unsafe functions that were added in this version). I have a situation where I'm stuck with Go 1.19 and cannot migrate to Go 1.20 just yet. So, I cannot use v1.0.6.
I also cannot use v1.0.6-go1.18 because I need a fix for https://github.com/open-telemetry/opentelemetry-go/issues/4076 and in general cannot use older OTEL libraries due to other dependencies. Latest OTEL libraries require Go 1.19, so I can use them.
Would it be possible to release a version of the library for Go 1.19? I.e. a version with latest OTEL libraries but without the new unsafe functions. If it's too much hassle then I'll wait until I can migrate to Go 1.20.

Actually, it should be possible to rewrite the code to avoid those functions and make the library compatible with Go 1.19 without removing the methods that use new unsafe code. Then, at some point, the change can be reverted to use stdlib versions of them. That way more people can use the library as it becomes compatible with Go 1.19 too :)

@rueian FYI I no longer need the v1.19 version as I've finally upgraded to Go v.120. Thank you!


Personalization for Windows

Hello, I hope you are well
I had a question.
I have launched your theme in Windows and it is very attractive
But I wanted to use the Monterey style model, you said in the instructions that I should apply changes in the userChrome.css file, but I was a little confused about how to activate the Monterey style model.
Can you introduce me to a procedure?

bro did you find the solution if yes please help me also

https://github.com/AdamXweb/WhiteSurFirefoxThemeMacOS


Provide an RPC to quit the performance benchmarking workers

I propose adding a "Quit" RPC to the performance benchmarking workers so that it's easier to control them with scripts. Arguably this is not needed.
This will conflict with #4995 which is more important. This is also not complete until this RPC is implemented in the wrapped languages that already support these workers and other language stacks.
@yang-g , is this advisable?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

LGTM

@murgatroid99 Please see the JS code here...


feat(FMP-360): track wagmi connection state in button state

Current dependencies on/for this PR:

main

PR #52

PR #53   👈

PR #54

PR #55

This comment was auto-generated by Graphite.

⏳ This pull request is set to merge as part of a Graphite merge job
Stack job ID: VExqmqIMSxeFRAdKggnq.
See details on graphite.dev


Add union_extract kernel

Which issue does this PR close?
Closes #6386
Rationale for this change
What changes are included in this PR?
union_extract kernel implementation, docs and tests
Are there any user-facing changes?
A new public kernel union_extract is added

I am depressed about the large review backlog in this crate. We are looking for more help from the community reviewing PRs -- see https://github.com/apache/arrow-rs/issues/6418 for more

I am depressed about the large review backlog in this crate. We are looking for more help from the community reviewing PRs -- see https://github.com/apache/arrow-rs/issues/6418 for more

I've started reviewing this one, but it's going to take me a bit (learning) to make it thru. Thank you for the documentation and testing.

Thank you for the well documented code.

I pushed like dozens of faster ways of doing the same thing, felt obligated to document it :sweat_smile:
Thanks @wiedld for your review

I'll plan to merge this tomorrow unless there are any other comments or anyone else wants to comment

Looks like the CI failure is not related -- https://github.com/apache/arrow-rs/actions/runs/10984024899/job/30494124939?pr=6387

I restarted the failed CI job https://github.com/apache/arrow-rs/actions/runs/10984024899/job/30494124939?pr=6387
I think it will pass now that https://github.com/apache/arrow-rs/pull/6437 is merged

I took the liberty of merging up from master to try and get a clean CI run for this PR

This CI run looks entirely unrelated - I will close/reopen this PR to retrigger and see if it is still happening

integration CI is failing due to https://github.com/apache/arrow-rs/issues/6448

https://github.com/apache/arrow-rs/issues/6448 is fixed, so merging it in

Thanks again @gstvg  and @wiedld


ARROW-18253: Add additional bounds safety checks

Is this something that can happen due to bad user input? Or are you simply checking for internal invariants?

Is this something that can happen due to bad user input? Or are you simply checking for internal invariants?

A little bit of both.  The only one as far as I can determine that would be a function of bad parquet files is the overflow when down-casting.  But I think there are other safe guard in place here.    In terms of bad user input, if we are talking about code that consumes these libraries I think most of the places do the negative checks are guards against here (the one exception being PathWriter), which I believe is completely internal.  These were raised by another team doing a code audit and "yellow flags".  If any of them strike you as superfluous I can revert them.


[Integration][Flink] RunEvent created with null eventTime, run and job

Hi, thank you for your efforts developing Flink integration with OpenLineage.
I'm currently attempting to use OpenLineage to collect Flink run info, and I referred to the examples provided to add jobListener and register with StreamExecutionEnvironment. However, when I run the Flink job, I received failed to emit openlineage event error. Marquez received the event message and returned error code 422. This error occured when OpenLineageFlinkJobListener executes onJobSubmitted.
The error stacktrace is attached:

04/05 14:37:16.969 [pool-4-thread-1] ERROR io.openlineage.flink.client.EventEmitter - Failed to emit OpenLineage event:
io.openlineage.client.OpenLineageClientException: code: 422, response: {"errors":["job must not be null","eventTime must not be null","run must not be null"]}
at io.openlineage.client.transports.HttpTransport.throwOnHttpError(HttpTransport.java:145)
at io.openlineage.client.transports.HttpTransport.emit(HttpTransport.java:119)
at io.openlineage.client.OpenLineageClient.emit(OpenLineageClient.java:46)
at io.openlineage.flink.client.EventEmitter.emit(EventEmitter.java:37)
at io.openlineage.flink.visitor.lifecycle.FlinkExecutionContext.onJobSubmitted(FlinkExecutionContext.java:59)
at io.openlineage.flink.OpenLineageFlinkJobListener.start(OpenLineageFlinkJobListener.java:82)
at io.openlineage.flink.OpenLineageFlinkJobListener.onJobSubmitted(OpenLineageFlinkJobListener.java:64)
at org.apache.flink.streaming.api.environment.StreamExecutionEnvironment.lambda$executeAsync$12(StreamExecutionEnvironment.java:2099)
at java.base/java.util.ArrayList.forEach(ArrayList.java:1541)
at org.apache.flink.streaming.api.environment.StreamExecutionEnvironment.executeAsync(StreamExecutionEnvironment.java:2099)
at org.apache.flink.streaming.api.environment.StreamExecutionEnvironment.execute(StreamExecutionEnvironment.java:1983)
at org.apache.flink.streaming.api.environment.StreamExecutionEnvironment.execute(StreamExecutionEnvironment.java:1951)
at aero.airlab.flink.systrk.irstoopenatms.IntegrationTest$EnvExecutor.execute$lambda-0(IntegrationTest.kt:130)
at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
at java.base/java.lang.Thread.run(Thread.java:829)

After running in debug mode, I found out that in class FlinkExecutionContext, function buildEventForEventType is called for job start and checkpoint. However, the return statement for this function is:

return openLineageContext
    .getOpenLineage()
    .newRunEventBuilder()
    .inputs(inputDatasets)
    .outputs(outputDatasets)
    .eventType(eventType);

Since only inputs, outputs and eventType are provided, the other 3 parameters are null, and the above error occurred.
It would be great if you could give me some advice on this issue. I look forward to hearing from you! Thank you

Hi @majunting, you're totally right!
Our integration tests verify if Openlineage events contain input / output dataset information, but we do not verify if they're valid OL events 🤦
And they're not as they're missing fields you got within an error:
{"errors":["job must not be null","eventTime must not be null","run must not be null"]}

Thanks for your debugging. I will fix this next days.

Hi @majunting , thanks for opening an issue! Please join us on Slack if you haven't already.


WebSocket Error with microsoft edge

My shinyproxied app does not work with Microsoft Edge (Microsoft Edge 42.17134.1.0; Microsoft EdgeHTML 17.17134), sigh...
In the console I get:
SCRIPT12008: SCRIPT12008: WebSocket Error: Incorrect HTTP response. Status code 200, OK
SCRIPT5007: SCRIPT5007: Unable to get property 'readyState' of undefined or null reference

It works fine with IE11 11.472.17134.0 but apparently does not for some earlier versions.
I manage the websockets as reported in #120
Have you already had this type of problems ?
Is it possible to disable websockets completely, as shiny server supports (cf https://docs.rstudio.com/shiny-server/#disabling-websockets-on-the-server) ?

This might be related to #65 see my comment?

Hi @kforner,
I suspect the Apache configuration must be adjusted. In your post #120 I see this line:
RewriteRule /(.*) ws://shinyproxy:8080/$1 [P,L]
But ws only works for http. For https, you may have to add a wss rewrite as well?

Hi @fmichielssen
If the configuration is incorrect, why is it working with all the other browsers and other edge versions ?
I'll try your suggestion.
thanks.

Hi @kforner ,
I'm also trying to make my app use https, but I'm getting only the text "loading app..." and that's it, nothing else happens. Can you share the virtual host code that worked for you?
Thanks in advance,
Silvano

Hi
We now have documentation on how to use Apache as a reverse proxy for ShinyProxy: https://shinyproxy.io/documentation/security/#https-ssl--tls
This should solve your issue. If you need more help, feel free to re-open or open a new issue.


Changing themes dynamically gets stuck after two changes

In my app, I am trying to provide the ability to change themes dynamically. I have two themes - light and dark.

The app is initialized with the light theme. See below (css.global('body', {...}):

@observer
class App extends Component {
    render() {
        const { paletteType } = rootStore.appStore;
        const palette = {
            primary: {
                main: blue[500]
            },
            secondary: {
                main: pink.A400
            },
            error: {
                main: red.A400
            },
            type: paletteType,
            // Initialize background to white (default is #fafafa)
            // This allows pictures with white background to blend in.
            background: {
                default: paletteType === 'light' ? '#ffffff' : '#303030'
            }
        };
        const theme = createMuiTheme({ palette });

        css.global('body', {
            height: '100%',
            margin: 0,
            background: theme.palette.background.default,
            fontFamily: theme.typography.fontFamily,
            fontSize: theme.typography.fontSize,
            color: theme.palette.text.primary,

            // Helps fonts on OSX look more consistent with other systems
            WebkitFontSmoothing: 'antialiased',
            MozOsxFontSmoothing: 'grayscale',

            // Use momentum-based scrolling on iOS devices
            WebkitOverflowScrolling: 'touch'
        });

        return (
            <ThemeProvider theme={theme}>
                <Provider rootStore={rootStore}>
                    <Shell />
                </Provider>
            </ThemeProvider>
        );
    }
}

export default App;

The user can toggle the theme by clicking a button. When this button is clicked, the paletteType changes to dark and the app is re-rendered with the new theme. This works just fine.

The trouble starts when I click the button again to toggle the theme back to light. I can see that the palette is recomputed with the light theme, however when this theme is applied to the body (css.global('body', {...}), the body never changes! In Chrome inspector, I can see to newly computed themes applied to the body - first the light theme then the dark. However any further changes to the styles are not applied to the body.

What am I missing?
You can see the full working code here: https://github.com/nareshbhatia/mobx-shop-glamorous

Found a solution - thanks to @tikotzky. See https://github.com/paypal/glamorous/issues/401


How to add Tool tip for each tree attribute in angular tree component

how to add ToolTip to angular tree component?

answers pls

Did you get the answer? How did you achieved this?

@Js4Life @Prabukmca
You can use the custom template for this. Take a look at the documentation. With this you can style a node of the tree. There you can do what you like - including adding a ToolTip.
Example:
<tree-root [nodes]="nodes">
  <ng-template #treeNodeTemplate let-node let-index="index">
    <div [title]="node.data.toolTip">{{ node.data.name }}</div>
  </ng-template>
</tree-root>

But the template gives you also the freedom to include other implemenations like the material or ngx-bootstrap tooltips.
Closing this for now. If there are other questions please open a new issue. Thanks


Project 64 Zooming?

I loaded Project64 into Steam using Ice.  The emulator works great in fullscreen at 1920x1080 when I run the exe file from it's normal folder; however, when I choose N64 games from Steam, the game loads but the screen is really zoomed in.  I have messed with the emulator options, but I'm very confused.

Hmmm... thats really odd. Have you tried launching the game the same way Ice/Steam does? That is, find the game in Steam and hit 'Properties' and copy the contents of the "Target" field. Then open a Command Prompt (search for cmd from the start menu) and paste the results in (warning: Command Prompt has been known to be super freaking annoying. You may have trouble pasting the text in :P). See if Project64 launches like you want it to.


Amazon Linux 2 Preconfigured With .NET Core and PowerShell Core

/cc @joeyaiello @SteveL-MSFT

@DarwinJS that's really cool!


Small fix in getObjectLiteralIndexInfo

Fixes #38175

Note that besides fixing the index of node.properties, I also assumed that it should be the same in properties, otherwise it should change to i - offset there.


Added token_to_token auth

This pull request makes it possible to use the grant type token_to_token which allows for creating a new token without expiring the old one, would be really useful for having 2 bots logged in to the same account and it could possibly even help with further development on sharding systems and such.

Before merging I would need you to add the auth to the documentation in api.rst.

Alright, i will make another commit with the added documentation.

Done, can you review again, @Terbau ?

Thanks. I'll review it once im home and then merge it.

After testing I have decided not to merge because the ios client basic token is not allowed to use token_to_token.
Not really important since it won't work either way but you forgot to change refresh_token to access_token.

I will make improvements to this pull request and open another iteration of it later.


Add type definitions for TS support

Describe the feature or improvement you're requesting
Right now, migrating to Typescript or creating a d.ts file manually are few of the options.
If the API is going to be minimal in the long term, then simple index.d.ts created manually should be enough.
Additional context
No response

Typescript is supported, you can see the .d.ts files in the /dist folder here: https://github.com/openai/openai-node/tree/master/dist
If you find that the types aren't working for some reason, please open a separate issue with steps to reproduce the problem.


Add VERSION_INFO

Fixes https://github.com/tensorflow/tensorflow/issues/39795
/cc @mihaimaruseac

I've added the golden update for v2.

Do we need to change that PR label here?

Do we need to change that PR label here?

It's pending API onwers review, hopefully we'll get it today.

It's pending API onwers review, hopefully we'll get it today.

So API owners suggested to use semver or distutils.version.LooseVersion instead of adding a new API.
Means the workaround in https://github.com/tensorflow/tensorflow/issues/39795#issuecomment-751472512 is the recommended way.
Sorry that it gets to be closed after such a long wait.

So API owners suggested to use semver or distutils.version.LooseVersion instead of adding a new API.
Means the workaround in https://github.com/tensorflow/tensorflow/issues/39795#issuecomment-751472512 is the recommended way.
Sorry that it gets to be closed after such a long wait.

@mihaimaruseac As the gist was mine as this PR do you will accept a semver dep PR?

@mihaimaruseac As the gist was mine as this PR do you will accept a semver dep PR?

I think we should ask the API owners on the new PR. I'll add tags to have them notified

I think we should ask the API owners on the new PR. I'll add tags to have them notified

I think we should ask the API owners on the new PR. I'll add tags to have them notified

Thanks I suppose that they meant to push this on the user side (outside Tensorflow) but just to be sure that we don't want a python only PR

I think we should ask the API owners on the new PR. I'll add tags to have them notified

Thanks I suppose that they meant to push this on the user side (outside Tensorflow) but just to be sure that we don't want a python only PR

Yes, I think the same, that they wanted to have users that needed this implement on their code instead of relying on TF.

Yes, I think the same, that they wanted to have users that needed this implement on their code instead of relying on TF.


strange 'getdatatable' nil value error spamming console

Describe the bug
A clear and concise description of what the bug is.
This error keeps popping up in console
Situation for the error to occur
Describe in detail the situation when the error occurs. Check this situation with the addon enabled and disabled before creating a ticket. Complaints in the format "Your addon is breaking the game, please fix it" or "It doesn't work for me" will be rejected.
no clear cause, spamming the console and generally being annoying
Screenshots or video
If possible, add a video or screenshots that show the error.

Desktop:

OS: [e.g. Windows]
Version [e.g. 10]
windows 10 home version 21H2
Additional context
Add any other context about the problem here.
It might be caused by another addon called "Background NPCS"

Should be fixed now.


Publish in PSGallery

It would be nice, if you could publish the module to the PSGallery.

It's long overdue. 🙂 Right now, I'm planning on submitting a PR at the end of October to publish one large new release at the end of the month.
Thank you for #216!

I have several other tests to commit :)


minimum python support version

A minimum Python version of 3.11 is a very high bar given that 3.9 still hasn't reached EOL. In my opinion, a minimum version of 3.10 would be more apt given that it introduced a cleaner syntax for type hinting (which we can't use in 3.9)

3.10 sounds like a good target to me, too.

I have gone through the minimum python support version for all libraries and 3.9 seems to be the highest base version required. This means, we can go with python 3.10

okay sounds good! want to send a pr? :)


Calling syncSettings.viewDifferences causes an error

[2022-07-13 02:04:19.145] [exthost] [error] Error: ENOENT: no such file or directory, lstat '~/.config/VSCodium/User/globalStorage/zokugun.sync-settings/repository/profiles/main/profile.yml' syncSettings.viewDifferences {"value":"zokugun.sync-settings","_lower":"zokugun.sync-settings"}

Instead, if I do Open the profile settings, an empty profile.yml is created, and then it works.
... any ideas what went wrong? Why don't I have that file?

Verified as fixed in https://github.com/zokugun/vscode-sync-settings/tree/v0.13.0 😃


[FEATURE] Add closing comment

Is your feature request related to a problem? Please describe.
Issues are closed due to a branch being deleted or becoming active again.
Describe the solution you'd like
Add comment when closing issues explaining why it was closed
Describe alternatives you've considered
None.
Additional context
None.

Not worth the overhead and dealing with secondary rate limits for large repos.


add documentation to ransac_affine

I've added to documentation from the comments of this notebook: https://github.com/GFleishman/bigstream/blob/master/notebooks/bigstream_intro_tutorial.ipynb

Hi Oren,
I'm back to work and finally going through your PRs. Thanks again for these, I'm really happy you're finding value in bigstream and I'm grateful for the contributions. I wish I had been more diligent and pulled this one when you submitted it since it's spot on - but I've since written a more verbose docstring for this function. So closing this one without merge.

no problem!
Just (literality 4 mins ago) I managed to get a working registration :)

That looks great! A little bit of residual misalignment in the lower left corner at the nominally higher z planes. This alignment looks like a rigid (or maybe affine) only. If you obtained it with the ransac method, you can try running a gradient descent method with the ransac result as the initialization:
from bigstream.align import affine_align

improved_affine = affine_align(
    fix, mov, fix_spacing, mov_spacing,
    alignment_spacing=2.0,
    shrink_factors=(2,),
    smooth_sigmas=(2.,),
    optimizer_args={
         'learningRate':0.25,
         'minStep':0.01,
         'numberOfIterations':2000,
     },
    initial_condition=ransac_affine,
)

The returned transform is the composition of both the ransac result and the gradient descent optimized result, so using only improved_affine for resampling will show the result of both steps together. Alternatively, if you want to keep the two types of affine alignment separate you can do:
from bigstream.align import affine_align

improved_affine = affine_align(
    ...
    static_transform_list=[ransac_affine,]
)

So initial_condition was replaced with static_transform_list. Let me know if you have questions about this option.
Finally, you can tighten things up completely with a deformation if you need to. Deformations take much longer, here's some tips to compromise speed and accuracy:

Use a large alignment_spacing like maybe 2.0. That speeds the alignment up the most.
Use a large distance between control points: control_point_spacing = 128.0 or similar
Always run with really low resolution parameters first just to confirm the alignment completes and is working as expected. Once you're sure it's doing what you want, then you can increase resolutions and compute the "real thing."

Data looks great by the way - hope the rest of the project is going great too!

Thanks a lot @GFleishman that was helpful!.
BTW, I have a different dataset that I try to help with, another lab member managed to get good results with MATLAB 🥸imregdemons, I am trying to get the same results or better with bigstream, but currently I did not manage.
prob_t82.zip
prob_t0.zip

Any tips??
Thanks a lot!


Add plot features

The current plot function is a single theme and the output value is a single value.
It would be useful to add themes and output values, e.g. grads, to this.

The solution to this issue requires these

Devising a color scheme

Intuitive interface, It is important for users to be able to easily switch between modes.


add pybullet blocks to nightly

sorry for the crazy commit history. I need to stop branching off of branches in this new squash paradigm


let g:iron_repl_open_cmd = 'topleft vertical 80 split" error caught in async

Hi:
I added let g:iron_repl_open_cmd = 'topleft vertical 80 split' & used key
autocmd Filetype python nmap <localleader>\\ :call IronSend(getline('.'))<CR><ESC><C-w>jG<C-w>pj0
and I get:

error caught in async handler '/home/hoon/.local/share/nvim/site/plugged/iron.nvim/rplugin/python3/iron:function:IronSend [['from datetime import datetime']]'
Traceback (most recent call last):
File "/home/hoon/.local/share/nvim/site/plugged/iron.nvim/rplugin/python3/iron/init.py", line 166, in send_to_repl
self.send_data(data, repl)
File "/home/hoon/.local/share/nvim/site/plugged/iron.nvim/rplugin/python3/iron/base.py", line 115, in send_data
ft = repl['ft']
KeyError: 'ft'

but  autocmd Filetype python vmap <buffer> <localleader>\\ $<Plug>(iron-send-motion)
key works fine.
Is there another command I should use?
Thank u much,
Hoon,

Thank u - that split timing issue w
ould not have crossed my mind ;)

You might as well try something like this:
nmap <localleader>\\ :call IronSend(getline('.'), &ft) \| IronFocus<CR>
It will automatically focus on the repl buffer.


Update README.md

Note that jenkins.model.Jenkins.RUN_SCRIPTS permission is deprecated

/approve
/assign @akram

@coreydaley we still should send the PRs towards the https://github.com/openshift/jenkins-openshift-login-plugin repostiory, as the change is valid for both repositories.
Could you please create a PR there?

/close


Kong Ingress controller error - "context deadline exceeded"

Discussed in https://github.com/Kong/kong/discussions/8964

Originally posted by anup-krai June 16, 2022
Hi,
We are using Kong with KIC deployed in K8s cluster with Postgres DB.
Kong - 2.7.1
KIC - 2.2.1
We are continuously seeing below error in KIC logs -
time="2022-06-16T10:55:05Z" level=error msg="could not update kong admin" error="2 errors occurred:\n\tfailed to sync all entities: context deadline exceeded\n\twhile processing event: {Update} upstream inventory.443.svc failed: making HTTP request: Put \"https://localhost:8444/upstreams/01158245-1730-4c79-a294-c67001d062fb\": context deadline exceeded\n" subsystem=proxy-cache-resolver

Because of this error new objects (ingress, services etc) are not getting deployed in Kong.
How can we resolve this error?

Issue is resolved after adding proxy_timeout_seconds environment variable in KIC. Thanks.


haskell: Pass dontStrip to cabal. Fixes #43506

Motivation for this change
See #43506
Things done

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

[ ] NixOS
[ ] macOS
[X] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Fits CONTRIBUTING.md.

CC @peti

I've tested this using
NIX_PATH=nixpkgs=. nix-build --expr 'with import <nixpkgs> {}; haskell.lib.overrideCabal haskellPackages.hello (old: { dontStrip = true; postInstall = "${pkgs.file}/bin/file dist/build/hello/hello $out/bin/hello"; })'


Support for PHP input

Form
So, in my website a want a form to take user input. Now I have made it, and I need to use PHP to take in user input.
But in normal EJS you can't actually put the PHP tag (). Now I want the user input to go through some PHP
functions.
How can I do this? Is there a way to actually connect PHP with EJS and then take in the user input?
Note: I came to expressjs because I am too lazy to make the backend of my website with PHP, then obviously its possible!

https://medium.com/@MartinMouritzen/how-to-run-php-in-node-js-and-why-you-probably-shouldnt-do-that-fb12abe955b0

https://medium.com/@MartinMouritzen/how-to-run-php-in-node-js-and-why-you-probably-shouldnt-do-that-fb12abe955b0


Create more frontend tests

In general let's use the yarn test --coverage option to see where we're lacking on tests.  Ideally we would get the most files in the yellow. The following are places to focus
New pages: Signup, Team, Timeline, Dashboard (Skip AdminTimeline and Sweep for now)
Lets start with the following components, open to ideas on component testing frameworks / methodologies
components: Panel.js, PanelLinePlot.js, PanelRunsLinePlot.js, RunViewer.js
I know this is a big task, but as the codebase gets bigger, this is more and more important.

Also, @thekatic let's get your react-scripts plugin fix into master!

@vanpelt here's a new PR that will fix existing tests, and this is the one that implements my update on the script.

@vanpelt I've resolved that issue with PR on react-scripts.


feat: gated fsync kernel (matches coreos-stable)

Thank you for contributing to the Universal Blue project!
Please read the Contributor's Guide before submitting a pull request.

@m2Giles there was a comment made in Discord about the coreos-testing, zfs failure on fedora 40 builds failing... Just calling it out here... the fsync patches themselves don't seem to be conflicting with the zfs kmod build in 6.9. But 6.10 is failing the zfs kmod build even without fsync patches.

For name I think fsync-coreos should get the idea across. fsync-coreos-stable would be clearer but unsure if we need that distinction


Linux install - Service already exists prompt does not wait for user input

===================================================
| Creating Service
===================================================
  Service 'stanza' already exists. Replace it? [y]/n:     Replacing service!
  Creating service file...

Environment:

Observed on Amazon Linux

Could not reproduce on Debian 10. Setting up an Amazon Linux box now

@djaglowski I'm unable to reproduce this on Amazon Linux. Is there anything else about the environment that might have caused this?

@camdencheek Thanks for looking into it. I don't know what else might have been relevant. Will close for now.


Scaffold difference in SP Online only and SP 2016

Hello Malik,
first I'd like to thank you for providing a lot of information for SPFx in general and SPFx with Angular.
Since I'm pretty new in SharePoint development, I just have a raw idea about the difference when you scaffold a new webpart for SP Online only or for SP2016 onwards.
So my question is, is this solution for SP2016 onwards or just SP Online?
Dear regards

The project is for SP Online, although the same concept can be extrapolated to on prem. I am in the process of simplifying this project and accompanying it with a video course so it's easier to follow.


Z-Fighting on used Thought-Knot's held and dropped item models

Modloader
Forge
Minecraft version
1.20.1
Hex Casting version
0.11.1-7-pre-634
Modloader version
Fabric: Loader 1.10.19 + API 0.92.1
Modpack info
https://github.com/walksanatora/HexxyTestPack/blob/main/modpack.zip
The latest.log file
No response
Issue description
There is mild z-fighting between the core and string parts of the Thought-Knot's used(as in it is holding an iota) model when holding the item and when it is dropped.
Steps to reproduce

Craft a Thought-Knot
Write an iota to it with Scribe's Gambit
Drop/hold the item and look at the top of the purple blob

Other information
No response

Related: #618


Tweaking Docs for Readability

I've trimmed some things down, added references to other files we support and increased the references to binder-examples
closes #297 #326

Looks nice!


ci: updated node versions in strategy matrix

WHATS NEW

Replaced node-versions to 10.x, 14.x, and 16.x

CHECK WORKS!


Rewrite

reverted back to single repo to maintain
several other improvements / changes

Thanks...  Building the zip in a clean raspbian-lite 64 bit install gave me:
Done installing OVOS core
Installing systemd files
cp: cannot stat '/home/ovos/raspbian-ovos-rewrite/stage-core/02-messagebus/files/ovos-systemd-messagebus.service': No such file or directory
Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
Collecting sdnotify
Downloading https://www.piwheels.org/simple/sdnotify/sdnotify-0.3.2-py3-none-any.whl (3.5 kB)
Installing collected packages: sdnotify
Successfully installed sdnotify-0.3.2
Enabling service files

fixed

sorry, I didn't see this till now.  we can remove the /ramdisk logs from mycroft.conf or create a ramdisk in the manual_user_install.sh script.  I think that just commenting them out of the config, with a comment saying why will work.
give me a few

@Bulwagga give it a shot now

No worries. Please dont feel any pressure from me.  We are troubleshooting a freezer issue here. You're doing a great job!  I'm really excited to see those servers come online.+}
-------- Original message --------From: builderjer @.> Date: 7/2/23  9:56 PM  (GMT-05:00) To: OpenVoiceOS/raspbian-ovos @.> Cc: Bulwagga @.>, Comment @.> Subject: Re: [OpenVoiceOS/raspbian-ovos] Rewrite (PR #61)
sorry, I didn't see this till now.  we can remove the /ramdisk logs from mycroft.conf or create a ramdisk in the manual_user_install.sh script.  I think that just commenting them out of the config, with a comment saying why will work.
give me a few
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Any reason I can't go ahead and delete my Bulwagga/raspbian-ovos repository?  I'm thinking of keeping things as simple as possible..

I would like to see an option to install the Gui In the manual install  script  eventually

Do the contents of the stage-* file trees contain enough information to create the manual_user_install.sh script automatically? I'm assuming you run a script on the contents of raspbian-ovos and it creates the image file as specified in raspbian-ovos-config... where could I find it? Thanks!

short answer, no, I do not think so
explanation:
So the RPi foundation has a script for building images called pi-gen.  With that, run by itself, you can build a vanilla raspberryPiOS-lite or full image, and even a noobs image.  What this repo does is takes the first few stages from that, which builds a minimal "lite" image, and extends that to have ovos installed as the OS instead of raspbian.  When this first started, all of this was in a single stage, and just installed the pip files.  It has grown into what it is now in just a few months.  the raspbian-ovos-config file will be, or already is removed from the latest.  It is/was there so that there was a config file to follow to build your own complete image.  you can look at this file for how it is processing the stages for the official build.  I have tried to separate the stages into what is happening, mainly to make debugging easier.  On a local build, you can skip stages that have already been built correctly and continue with debugging from there.
manual_user_install.sh script:
This was put there originally so something was there while I created the stages to build an image.  It was never actually meant to stay and be used, but I have found that several people use it.  SOOOOOO, this is what I would like it to become.
Not sure how long you have used mycroft/ovos/neon, but the original mycroft-cor had a dev_setup.sh script that would setup mycroft on your device for you and try to detect the OS that is being used.  I would like something like this as a final step for this script.
Wanna help?
p.s.  you should really join the matrix channels. communicating is easier, and you can see a whole bunch of development going on.  https://matrix.to/#/#openvoiceos-support:matrix.org https://matrix.to/#/#openvoiceos-general:matrix.org https://matrix.to/#/#openvoiceos-development:matrix.org

Yeah, I'm interested in helping. I've been watching matrix for a few weeks to get the flow.  I'm interested in helping, but my relevant skill level tops out with script building. I fully support  your vision for creating something similar to dev_setup.sh for OVOS.  I have several RPi devices doing multiple tasks, and the value I see is using a PC as a tts/stt server and having the pi's continuing to do their tasks while listening for wake words. I ran mycroft on two Ubuntu PCs starting with dev_setup.sh. Having one script supporting multiple OSs seems like the way to go.  My only doubt about the concept is that if OVOS as an OS works decently enough on the PIi, there won't be much demand for client software on desktops. But that is a ways in the future.  For now I'm trying to simply get the script current with the newly merged environment. Thanks for the encouragement.  Please suggest things as you think of them and I'll do my best to stay oriented in a useful direction.


Use requests instead of urllib in mailman

Fixes #581.
This changes the urllib code to use requests instead. Requests handles certificates better, so it successfully verifies SSL certificates.

Codecov Report
Patch coverage: 33.33% and project coverage change: +0.36 :tada:

Comparison is base (41327f1) 73.37% compared to head (8984634) 73.73%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #584      +/-   ##
==========================================
+ Coverage   73.37%   73.73%   +0.36%
==========================================
  Files          30       30
  Lines        3635     3632       -3
==========================================
+ Hits         2667     2678      +11
+ Misses        968      954      -14

Flag
Coverage Δ

unittests
73.73% <33.33%> (+0.36%)
:arrow_up:

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

Impacted Files
Coverage Δ

bigbang/ingress/mailman.py
67.82% <33.33%> (+6.35%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


3D Mesh Reconstruction

3D mesh reconstruction with naive exterior entity classification. This PR aims to address #554 and allows for the reconstruction of a proteus mesh into a SCOREC-readable mesh data structure. Exterior entities are classified as model entities for simplicity and material types are preserved during this reconstruction.
This is functional in both serial and parallel. The marin case was run as a test example in parallel up until T=1s for a moderate discretization (~30k elements). The following is an isovolume comparison at that time for a baseline case and one using the same mesh but one that is reconstructed and passed back into Proteus.
The isovolume is colored by node type to indicate the preservation of the node material types through reconstruction.

Duplicated in #589 with comparison to the update_adapt branch as opposed to master branch.


Improve installation process

At the moment, the installation process is just putting the files into the correct spot and making them executable. There is no solid process for applying updates. I was wondering if there was some simple way to update the process so that pulling in updates was easy? I would do a pull request, but I am no unix pro so I figured I would ask for advice first.
Asside from this being part of the mainstream package managers, my first thought would be to check out the git repo, so that you can git pull on it, then symlink notes into /usr/local/bin/notes and notes.bash_completion to /usr/share/bash-completion/completions/notes. Would this cause issues or be difficult to maintain? I guess this would assume the user has git installed...
Down the road you could even apply some sort of auto-update process. If it becomes part of a package manager then it would be built in, but if we did something like the above idea with git it could do a git pull in the  background when you run the notes command every so often. Maybe the notes script could check for updates and curl them in automatically so it does not rely on git.
Thoughts?

Yes, that sounds excellent. I'm very open to ways to better distribute this.
First, thoughts:
Having git as an option is fine, but I don't think it should be the main one. Firstly because the git repo contains a bunch of things that are irrelevant to users (the tests, the full repo history, etc), and also because it requires you to manually think about releases yourself as an end user. You probably don't want to be on master, but that is what you want to get by default if you clone the project. Instead you want to be on the latest tagged release. And on top of that, yes, being tied to git will be a bit inconvenient for some users.
We could build an install/update script without git pretty easily, because Github gives us most of the information anyway. You can easily query github for the latest release (curl -s https://api.github.com/repos/pimterry/notes/releases/latest), get the tag (grep tag_name | ...), and pull down the latest version of notes (curl https://cdn.rawgit.com/pimterry/notes/$TAGNAME/notes > /usr/local/bin/notes). That might be a reasonable starting point. Need to think about what happens when the list of files to download changes. Maybe a script that downloads a manifest of files and locations first, which potentially changes per-version, and then installs what it's given.
Looks like just moving to the Bash Package Manager might get close to this.
In a perfect world, I'd like a separate distribution channel that we explicitly push releases to (potentially automatically by pushing tags to github), that contain only the files that end users actually care about (i.e. notes and notes.bash_complation), and which automatically handles updates by integrating with normal system updates, rather than notes having to trigger updates separately. Proper package managers will also do things like uninstalling and prompting users for updates nicely all for us.
To do that, we'd want native per-platform packages (RPMs, DEBs, whatever you need to publish to homebrew, etc). Notably we're already published to the Arch User Repository (see https://github.com/pimterry/notes/issues/7) - that's a package that internally manages it's own updating with Git, which is a bit crazy long-term, but works for now.
Building our own packages is harder, but probably not that hard. fpm should let us build them, and then we'd need scripts to automatically push the results to each of the relevant distributions channels.

So, given that wall of text, where should you start. If you're interested, trying doing one of:

Seeing what the Bash Package Manager actually gives us, what it doesn't, and how much work that might be to publish to.
Writing an install script so that users can do something like curl https://cdn.rawgit.com/pimterry/notes/master/install.sh | bash and get the latest version up and running (and then do the same again later to update). Needs to think about making sure updates work, and handling later changes to the set of files we publish in releases (e.g. when zsh completion gets added), etc.
Trying to build a package for one platform (e.g. deb), working out how automated publishing would work for that kind of package, and then trying to work out a way we could easily do that automatically when we want to do a release (which we can expand to cover  more platforms later).

They're approx that order of difficulty I think. Let me know if you're keen, and where you want to start.

Great! Let me read up on each of these and see what I can do.

I think haveing platform specific packages (deb, rpm, etc) that are integrated into the package managers (apt, packman, etc) is the right way to go, but I started looking at the documentation and I think that is a bit out of my league at the moment, even with fpm.
bpkg would provide an easy option, but that is yet another tool to install. It would be pretty simple to add later on, but I dont think it is the way to go at the moment.
It sounds to me like the best option for right now would be a setup script to run remotly via curl. I can get to work on this and submit a PR, but I have a few questions first:
How do you want to handle updates?

~Encourage~ force users to stay with the newest version via background auto update?
Prompt if they want to update to the newest tagged release, or only for major version releases?
Add an update command that checks and does the update?

What do you think about this update process? The notes script has code that knows how to uninstall itself and how to remotly run the setup script so that if the list of files or dependencies change it can clean itself up and invoke the new process.

Phew, exciting. Ok. I'd rather not integrate the updating code into notes directly, largely because it's a big can of worms, especially if you get real package managers involved too later. Even in the existing arch case for example, it seems like it's going to be really easy to accidentally end up with two independently managed install on your machine, and to not be able to track down or uninstall them again cleanly in future.
I think that rules out auto-update and active prompting for now. We should start with a simple script that just does installs of the latest version, and build everything up from there. Once we've got that in place, we can build up notifications and autoupdate on top very easily.
I'd aim to first write a script that:

You can curl and run one command to get an immediately working notes setup on a clean typical machine on the latest version
You can curl and run again to update a notes setup on a typical machine that ran the script before to the latest version
Prompts if it's not sure what to do about anything (e.g. confirm before overwriting /usr/local/bin/notes)
Outputs the list of files created afterwards, for manual uninstallation (we can do nice uninstalling later)

'Typical' includes at least normal Ubuntu installs and OSX with GNU tools installed. Latest version is defined by releases on github. If there's more you want to strip out of that to keep it even simpler, feel free.
Later steps might include:

Prompting for updates
Doing bash (and zsh: #33) auto-completion automatic installation as part of this
Uninstalling easily
Installing not the latest version
Listing changelogs and new features when you install
Integration into proper package managers

But we don't need to worry about any of that to get a useful working first step in place, and we can then start building from there.


GitHub usernames don't always begin with a letter

I'd love to switch my site over to Utterances, but my username seems to be tripping up the repo regex 😅 Would there be any issues with removing the [a-z] from the beginning of it?

Doh! Sorry about that! Will get this shipped ASAP. Thanks for the PR

🚢shipped- apologies for the delay, thanks for the PR 👍

Cheers - looking forward to trying out utterances :D


Draft: Just attributes

This is a draft PR for a generalized filter attribute library for ext_proc and wasm. @gbrail is mentoring
Questions:

[ ] should the authority pseudo-header be included as a top level property? Or maybe it just overrides 'host'? @gbrail
[ ] Can someone familiar with grpc status look at how I've implemented that logic? The rub here is knowing when the exchange was via grpc vs plain http.
[ ] regarding filter_state, the FilterState seems to be an opaque data store, should I impl an iterator for it? If so should the values just be Protobuf::Any?
[ ] It looks like in order to get the necessary code coverage I will need to handle the case where the my token enums are out of range, is it suggested that I write tests for this?

Todo:

[x] headers impl
[x] request headers test
[x] response headers & trailers test
[x] getMetadata tests
[x] tests around absent headers & trailers

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

This seems fine to me -- but again, I'm not someone who can actually approve this and I imagine that they will have more suggestions.
At this point, I'd suggest the following:

Get the presubmit tests to run -- that will likely take some work and require another merge from main
Check the code coverage on your tests, because the standard is pretty much 100%
Turn this into a non-draft PR, which will result in a reviewer being assigned and it'll help ensure that it gets actually approved.

Thanks!

@PiotrSikora @kyessenov @lizan can you folks help review this PR?

@htuch got the CI passing, what are next steps in the review? Also I have a few questions for the team left as indicated in the summary.

@estk I think @kyessenov is the best person to take on primary review here.
@wjtracey is looking at extensible CEL right now, so I think he might want to take a look at as well.

@kyessenov  would you be willing to take a pass?  If not @htuch you could suggest someone else for first pass?

I'll take a pass. I was out for a few weeks and missed the notification.

@gbrail I don't think there is incompatibility. I was just recommending to keep the internals protobuf-less and only output Structs at the end. Protobufs perform poorly as runtime data structures from our experience. CelValue seems suitable as a runtime value but it requires a bit more care with memory management as it is essentially a "view" of some memory.

I have cycles this week to work on this.

I this this PR is waiting on changes from the author so I'll mark it as waiting.
/wait-any

Bah Humbug GitHub. Still planning on progress from the author in the new year!
/wait-any


feat: add RSK Mainnet and Testnet 1.3.0 contracts

Please add RSK Mainnet and RSK Testnet:
https://github.com/safe-global/safe-deployments/pull/288
Thank you!

Hey @Uxio0!
Could you take a look at this PR too?
Thanks in advance!


[stable/anchore-engine] Add ability to use existing service account for CloudSQL.

What this PR does / why we need it:
Sometimes it is needed to use an existing specific service account with CloudSQL permissions. This PR adds an ability to do that.

Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @btodhunter

/ok-to-test

Thanks for the contribution @sT331h0rs3!

/lgtm

Thanks a lot for review and testing, @Btodhunter! Is anything else needed from my side to have this PR merged?

@sT331h0rs3 nope - the merge will happen automatically after all the automated tests pass. Thanks again!


Linux support

Please support linux too

Hi!
You can download .tgz package from latest release page, extract it and run jar file using java -jar [jar_file] command

Need help to run the GUI on linux. I'm getting:
java -jar trex-stateless-gui-jfx.jar
Error: Could not find or load main class com.exalttech.trex.application.TrexApp
My folder content:
drwxr-xr-x. 2 yossi yossi    4096 Nov  2 02:36 lib
-rwxr-xr-x. 1 yossi yossi 1068687 Nov  2 02:36 trex-stateless-gui-jfx.jar
This is the latest GUI version (trex-stateless-gui-4.2rc.tgz)

Hi,
I think you're using old java or opendjk-jre.
could you show the output of java -version?
Please try using Oracle JRE instead from this page: https://java.com/ru/download/manual.jsp
I tried jre-8u151-linux-x64.tar.gz but other versions should work as well(unless they are too old)
Here is how you can get it
tar xvf trex-stateless-gui-*.tgz
tar xvf jre-*.tar.gz
./jre1.8*/bin/java -jar trex-stateless-gui-jfx.jar

I think it will be good to add an executable script to tgz, we will add for the next release build.
Meanwhile it can be created like this:
cat > trex-stateless-gui <<"END"
#!/bin/sh

BASE=$(dirname $0)
cd $BASE

if ! which java > /dev/null ; then
    echo 'Please download and install latest Oracle Java Runtime environment(JRE)'
    echo ' and add it to PATH environment variable'
    echo 'Java download location: https://java.com/ru/download/'
    echo 'Manually choose package(or different OS): https://java.com/ru/download/manual.jsp'
    echo 'manual run command: $JRE_PATH/bin/java -jar trex-stateless-gui.jar'
    exit 1
fi

java -jar trex-stateless-gui.jar
END
chmod a+x trex-stateless-gui

added https://github.com/cisco-system-traffic-generator/trex-stateless-gui/blob/master/trex-stateless-gui and updated build scripts. this file will appear in the tgz in the next release and can be used on Linux/Mac ( https://cj.xored.com/job/trex-gui-mac/lastSuccessfulBuild/artifact/target/jfx/trex-stateless-gui.tgz )

Thank you so much.. Works fine with oracle java
But, it would be great if it can be made to work with openjdk

Thank you. This is still helpful. To find a java version that can run this.


fix few misspell and one small cleaning

One little tweak, and if you can rebase on top of tip should be good to go.

@dvrkps I updated the branch for you with latest master. @jhendrixMSFT I guess we will do a patch release for this. Do you want a changelog although I would say we can go without that since there is no functional change.

@vladbarosan agreed no changelog update required.


[feat]: Table output "Page Size" option, and list of pages are too dark to see

@mhmdkrmabd could you adjust the colours a bit for this please?

That's weird, can Mr @hshimizu share the set color of this workspace?

@mhmdkrmabd it may be difficult to see based on selecting the workspace colour for this. I think change it just use standard colours that are more visible and not use the workspace select colour.


Nested JSON

I have an issue where the output of my cli tool is JSON but then the nested JSON inside the report field is all messed up and it seems I can't properly json.loads the main result. I wonder if there might be a way to detect json and include it properly rather than escaped inside as a substring?

It's possible to "intercept" the output providing a callback function. See this example.
You should probably be able to write a callback function that does something like:
res: dict = future.result()  # 1. get current report
res["report"] = json.loads(res["report"]) # 2. manipulate report however u want
future._result = res # 3. set new result

Because of #31, I would not recommend using callback functions to modify the future's result. So you are left with 2 other options:

Don't change the behavior on flask-shell2http side, handle the JSON parsing on the client side. Shouldn't be too painful IMO.
Use a decorator to modify the JSON response and return the modified response. See below example:

from flask import make_response, jsonify
def json_parser_decorator(f):
    @functools.wraps(f)
    def decorator(*args, **kwargs):
        response = f(*args, **kwargs)
        resp_json = response.json.copy()
        resp_json["report"] = json.loads(resp_json["report"])
        return make_response(jsonify(resp_json), ret.status_code)
    return decorator

shell2http.register_command(
    endpoint="mytool", command_name="mytool", decorators=[json_parser_decorator]
)


Get all tags from author

Hi, I'm interested in joining this project for this seems to be useful when using discord and I'm a fan of it
I want to start from this issue but have hard time where to start reading the source code from
Would you mind giving me some hints on it?
Also, where can i find the test codes and the code conventions?

Hello @kaestro, thanks for reaching me on this issue.
You can find mostly all the information you need within the contributing guide and Readme.
Here, the main task is to implement a author @commands.slash_command into commands/tag_commands.py who will look into the database for a given author.
You'll need to implement a new database query into /db.py to allow this command to works, and maybe into /context_menu.py if you intend to add a new app-subcommand.
Please don't hesitate to reach me here if you need further explains about the issue, bot setup or this project

@tarto-dev
Do get all tags from author mean that this needs to specify which author has written the tag?
if it's so, I wonder how I can test to create tags from other user.
Or does it mean to get all tags from the user who used the command itself?

Another question
https://github.com/kaestro/tagsy-discord/commit/5d16e7379a76139fc78dde5abb8a8cbab9bffd95
I tried creating the command, and the problem is the if I type in author as my discord account 'kaestro' no message comes out.
My guess is that when saving tags, created by isn't saved as raw string, or does it? Is there any way that I should be formatting before using sqlite?

Hey @kaestro, authors are stored by user_id to handle names changes.
As you can notice here https://github.com/tarto-dev/tagsy-discord/blob/master/modals%2Fadd_tag.py#L61
The trick is to search for users in the current server with that ID and recover the username with it.

This should do the work

async def get_username(guild_id, user_id):
    guild = client.get_guild(guild_id)
    if guild:
        member = await guild.fetch_member(user_id)
        if member:
            return member.name
    return "User not found"

(they're might be some mistake as I'm typing with my phone ^^')

This should do the work
async def get_username(guild_id, user_id):
    guild = client.get_guild(guild_id)
    if guild:
        member = await guild.fetch_member(user_id)
        if member:
            return member.name
    return "User not found"

(they're might be some mistake as I'm typing with my phone ^^')

I couldn't find where get_guild or client are at. Is it supposed to be inside context_menu?
Only place I could spot guild_id was inside context_menu.py and couldn't find where client is initialized.
Also, can you guide me on opening up db of tagsy-discord project? I think this kind of problem is happening for I don't know the structure of db itself, use of disnake, etc.
Thank you for your support even when you couldn't access the pc :)

Hey! Sorry for the late reply, I was in vacation.
Those two methods are herited by the discord SDK (Disnake) and are accessible from Client or Context variables. You should be able to get the information from the documentation.
I can give you a database dump, but you can access it just by running the app and adding a tag. You then should be able to mount/read it with vscode and it's sqlite plug-in.
If you want, I can share my discord username to avoid github delay and give you further explanation 😉
Just let me know if you need anything else

How was your vacation? Did you go to abroad?
It's really glad to hear that you've gone for a vacation had maybe had a good time there :)
I agree with the idea that sharing discord username will help a lot. Send me yours through my email address<EMAIL_ADDRESS>or maybe opening up a channel can help.


OSx Make Error

I am trying to build on a mac and receiving the below errors, I'm not sure what to do to fix it. Can anyone help?
Thank you
In file included from /Users/lrayner/PycharmProjects/playground2/pangolin/python/pangolin.cpp:2: /Users/lrayner/PycharmProjects/playground2/pangolin/external/pybind11/include/pybind11/stl.h:283:46: error: no member named 'experimental' in namespace 'std' template<typename T> struct type_caster<std::experimental::optional<T>>                                         ~~~~~^ /Users/lrayner/PycharmProjects/playground2/pangolin/external/pybind11/include/pybind11/stl.h:283:69: error: 'T' does not refer to a value template<typename T> struct type_caster<std::experimental::optional<T>>                                                                     ^ /Users/lrayner/PycharmProjects/playground2/pangolin/external/pybind11/include/pybind11/stl.h:283:19: note: declared here template<typename T> struct type_caster<std::experimental::optional<T>>                   ^ /Users/lrayner/PycharmProjects/playground2/pangolin/external/pybind11/include/pybind11/stl.h:286:36: error: no member named 'experimental' in namespace 'std' template<> struct type_caster<std::experimental::nullopt_t>

Figured it out, the std.h file was different to that in the pybind11 repository.
I swapped out
`#  if defined(PYBIND11_CPP14) && __has_include(<experimental/optional>)
include <experimental/optional>
define PYBIND11_HAS_EXP_OPTIONAL 1
endif`
for
`#  if defined(PYBIND11_CPP14) && (__has_include(<experimental/optional>) &&
!__has_include())
include <experimental/optional>
define PYBIND11_HAS_EXP_OPTIONAL 1
endif`


Add cocoapods spec

Hi,
Just created a podspec to ease integration into Xcode. It's missing the license part in order to be submitted to cocoapods community.

Hi DHOscar,
Thanks a lot for your contribution.
With the new version of AdyenCSE we added CocoaPods integration.
Regards,
Oleg


[Fix-12981]add worker address validator and clear function.

Purpose of the pull request

This PR will close #12981

Brief change log

Add a worker address validator.
Add a clear function

Verify this pull request

This change added tests

Codecov Report

Merging #13013 (63bc581) into dev (1a8811c) will decrease coverage by 0.01%.
The diff coverage is 43.47%.

@@             Coverage Diff              @@
##                dev   #13013      +/-   ##
============================================
- Coverage     39.37%   39.36%   -0.02%
- Complexity     4270     4274       +4
============================================
  Files          1067     1067
  Lines         40119    40110       -9
  Branches       4605     4591      -14
============================================
- Hits          15796    15788       -8
  Misses        22545    22545
+ Partials       1778     1777       -1

Impacted Files
Coverage Δ

...uler/server/master/registry/ServerNodeManager.java
0.00% <0.00%> (ø)

...phinscheduler/service/registry/RegistryClient.java
1.05% <0.00%> (-0.02%)
:arrow_down:

...apache/dolphinscheduler/common/utils/NetUtils.java
51.94% <76.92%> (+2.30%)
:arrow_up:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...erver/master/processor/queue/TaskEventService.java
75.00% <0.00%> (-5.36%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
42.55% <0.00%> (-2.13%)
:arrow_down:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
51.38% <0.00%> (-1.39%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

... and 21 more

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

Should we add legal address check in RegistryClient.getServerNodeSet, either?

agree with u


Add a discovery extension to support multiple versions of the DeltaSpike - repairs for issue #100

See https://github.com/cdi-unit/cdi-unit/pull/179 in the upstream repository.


drivers: spi: esp32: add basic SPI master support

Do we need the new sample or can you use tests/drivers/spi/spi_loopback ?

No, we don't. We missed the /tests/spi_loopback sample.


add support for extract(field from source)

change documentation and wording for EXTRACT.
otherwise ack.

Coverage increased (+0.03%) when pulling adfa6689d965539e6d26a62c5e214e36de8be109 on j/extract into e081141ba2eba13477075cde7abf66f84fad432c on master.

Changes Unknown when pulling 97f8366cb1058e574ccb6abc94c27c595d18387d on j/extract into ** on master**.


Repair CI after updating ROS2 GPG key

@nbbrooks you have experience with updating the CI because of GPG key.


Add precommit hook for formatting

Codecov Report

Merging #67 (0b4b7d5) into master (06c1ae5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #67   +/-   ##
=======================================
  Coverage   93.91%   93.91%
=======================================
  Files          41       41
  Lines        4615     4615
=======================================
  Hits         4334     4334
  Misses        281      281

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 06c1ae5...0b4b7d5. Read the comment docs.


Mildly suspicious owner

Bejolithic is an owner of this crate, as of https://github.com/zopfli-rs/zopfli/issues/1.
I just happened to notice that all 3 other crates they own are copies of other crates, renamed, stripped of attribution and relicensed:
https://crates.io/crates/forage -> https://crates.io/crates/maimo: https://github.com/FuzzrNet/Forage/issues/6
https://crates.io/crates/wasmpng -> https://crates.io/crates/wasimage: https://github.com/datatrash/wasm-png/issues/1
https://crates.io/crates/bbcli -> https://crates.io/crates/wingcli: https://github.com/losfair/blueboat/issues/90
Probably innocuous, but thought it might be worth raising to nip in the bud potential for a supply-chain attack on users of this crate.

Thanks for raising up this issue!
There were many, many more instances of not only crates, but entire repos, blog posts and more that I found blatantly copied without any form of attribution. This behavior is definitely suspicious and there's nowhere I can find that Bejolithic contributed to zopfli-rs, so I have removed this user from the organization.

This looks wrong indeed. These aren't just forks, but crates with completely replaced authorship metadata without any credit, and inappropriately changed license.
I've removed them from zopfli owners.

Thanks for the swift action both! Agreed, the more I looked the more dodgy that account seemed. And their sudden near-universal disappearance is interesting: https://bio.link/bejolithic https://volt.fm/bejolithic https://keybase.io/bejolithic https://contra.com/bejolithic https://www.reddit.com/r/KeybaseProofs/comments/v0f9m8/my_keybase_proof_redditbejolithic/ Ahmad Ainul Rizki https://twitter.com/bejolithic https://vero.co/bejolithic
Closing as resolved.


Sample of using resumable.js for uploading file before encrypt the file data

i need encrypt the file (data) before upload the file using resumbale.js uploader?
Data means Example: test.txt This file having "Hello World" text i want encrypt the text then upload the file

This is out of scope for the library, so won't be able to help much along ;)


Controller delete the release during the upgrade process without warning

Hello,
We experienced a release deleting during upgrade, is it expected ?
https://github.com/elastic/cloud-on-k8s/issues/4734#issuecomment-892748226
kubectl logs -f helm-install-elastic-operator-jzwnf -n elastic-system
CHART="${CHART//%\{KUBERNETES_API\}%/${KUBERNETES_SERVICE_HOST}:${KUBERNETES_SERVICE_PORT}}"
set +v -x
+ cp /var/run/secrets/kubernetes.io/serviceaccount/ca.crt /usr/local/share/ca-certificates/
+ update-ca-certificates
WARNING: ca-certificates.crt does not contain exactly one certificate or CRL: skipping
+ [[ '' != \t\r\u\e ]]
+ export HELM_HOST=<IP_ADDRESS>:44134
+ HELM_HOST=<IP_ADDRESS>:44134
+ helm_v2 init --skip-refresh --client-only --stable-repo-url https://charts.helm.sh/stable/
+ tiller --listen=<IP_ADDRESS>:44134 --storage=secret
Creating /root/.helm
Creating /root/.helm/repository
Creating /root/.helm/repository/cache
Creating /root/.helm/repository/local
Creating /root/.helm/plugins
Creating /root/.helm/starters
Creating /root/.helm/cache/archive
Creating /root/.helm/repository/repositories.yaml
Adding stable repo with URL: https://charts.helm.sh/stable/
Adding local repo with URL: http://<IP_ADDRESS>:8879/charts
$HELM_HOME has been configured at /root/.helm.
Not installing Tiller due to 'client-only' flag having been set
++ jq -r '.Releases | length'
++ helm_v2 ls --all '^elastic-operator$' --output json
[main] 2021/08/04 12:19:18 Starting Tiller v2.17.0 (tls=false)
[main] 2021/08/04 12:19:18 GRPC listening on <IP_ADDRESS>:44134
[main] 2021/08/04 12:19:18 Probes listening on :44135
[main] 2021/08/04 12:19:18 Storage driver is Secret
[main] 2021/08/04 12:19:18 Max history per release is 0
[storage] 2021/08/04 12:19:18 listing all releases with filter
+ V2_CHART_EXISTS=
+ [[ '' == \1 ]]
+ [[ '' == \v\2 ]]
+ [[ -n '' ]]
+ shopt -s nullglob
+ helm_content_decode
+ set -e
+ ENC_CHART_PATH=/chart/elastic-operator.tgz.base64
+ CHART_PATH=/elastic-operator.tgz
+ [[ ! -f /chart/elastic-operator.tgz.base64 ]]
+ return
+ [[ install != \d\e\l\e\t\e ]]
+ helm_repo_init
+ grep -q -e 'https\?://'
+ [[ helm_v3 == \h\e\l\m\_\v\3 ]]
+ [[ eck-operator == stable/* ]]
+ [[ -n https://helm.elastic.co ]]
+ helm_v3 repo add elastic-operator https://helm.elastic.co
"elastic-operator" has been added to your repositories
+ helm_v3 repo update
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "elastic-operator" chart repository
Update Complete. ⎈Happy Helming!⎈
+ helm_update install --namespace elastic-system --repo https://helm.elastic.co --version 1.7.0
+ [[ helm_v3 == \h\e\l\m\_\v\3 ]]
++ tr '[:upper:]' '[:lower:]'
++ jq -r '"\(.[0].app_version),\(.[0].status)"'
++ helm_v3 ls --all -f '^elastic-operator$' --namespace elastic-system --output json
+ LINE=1.7.0,failed
+ IFS=,
+ read -r INSTALLED_VERSION STATUS _
+ VALUES=
+ [[ install = \d\e\l\e\t\e ]]
+ [[ 1.7.0 =~ ^(|null)$ ]]
+ [[ failed =~ ^(pending-install|pending-upgrade|pending-rollback)$ ]]
+ [[ failed == \d\e\p\l\o\y\e\d ]]
+ [[ failed =~ ^(deleted|failed|null|unknown)$ ]]
+ [[ helm_v3 == \h\e\l\m\_\v\3 ]]
+ helm_v3 uninstall elastic-operator --namespace elastic-system
release "elastic-operator" uninstalled
+ echo Deleted
+ helm_v3 install --namespace elastic-system --repo https://helm.elastic.co --version 1.7.0 elastic-operator eck-operator
Deleted
NAME: elastic-operator
LAST DEPLOYED: Wed Aug  4 12:19:23 2021
NAMESPACE: elastic-system
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
1. Inspect the operator logs by running the following command:
   kubectl logs -n elastic-system sts/elastic-operator
+ exit

++ helm_v3 ls --all -f '^elastic-operator$' --namespace elastic-system --output json
+ LINE=1.7.0,failed

The release was in a failed state. The helm controller handles this by uninstalling the broken release, and installing it again to a known-good state. For information on why the release was in a failed state, you would need to look at logs from the failed job pod.

How to check the logs of the failed pod if it got deleted by the controller quickly?
This behavior is good for stateless things, but I don't recommend to use the controller for stuff like rook-ceph where data loss happens when reinstalling it.

We could perhaps consider an enhancement to the HelmChart CR spec to support enabling different behavior if the release is in a failed state - for example, simply leaving it failed for an administrator to address manually?

Sounds like a good idea to me. I suggest using the safe option to not delete it as a default.
I had some bad experiences with this controller and data loss after a few weeks uptime, luckily on a test environment.

The way we use the Helm controller to deploy core cluster infrastructure, it's usually best for us as distribution maintainers to default to behavior that gets the chart installed at any cost. I can see the value in allowing users to disable that for their own charts though.

We have seen more or less the same issue during our upgrade of RKE2 from 1.20.8 to 1.20.12:
 CHART="${CHART//%\{KUBERNETES_API\}%/${KUBERNETES_SERVICE_HOST}:${KUBERNETES_SERVICE_PORT}}"

set +v -x

+ [[ '' != \t\r\u\e ]]

+ export HELM_HOST=<IP_ADDRESS>:44134

+ HELM_HOST=<IP_ADDRESS>:44134

+ helm_v2 init --skip-refresh --client-only --stable-repo-url https://charts.helm.sh/stable/

+ tiller --listen=<IP_ADDRESS>:44134 --storage=secret

[main] 2021/12/16 08:54:01 Starting Tiller v2.17.0 (tls=false)

[main] 2021/12/16 08:54:01 GRPC listening on <IP_ADDRESS>:44134

[main] 2021/12/16 08:54:01 Probes listening on :44135

[main] 2021/12/16 08:54:01 Storage driver is Secret

[main] 2021/12/16 08:54:01 Max history per release is 0

Creating /home/klipper-helm/.helm

Creating /home/klipper-helm/.helm/repository

Creating /home/klipper-helm/.helm/repository/cache

Creating /home/klipper-helm/.helm/repository/local

Creating /home/klipper-helm/.helm/plugins

Creating /home/klipper-helm/.helm/starters

Creating /home/klipper-helm/.helm/cache/archive

Creating /home/klipper-helm/.helm/repository/repositories.yaml

Adding stable repo with URL: https://charts.helm.sh/stable/

Adding local repo with URL: http://<IP_ADDRESS>:8879/charts

$HELM_HOME has been configured at /home/klipper-helm/.helm.

Not installing Tiller due to 'client-only' flag having been set

++ timeout -s KILL 30 helm_v2 ls --all '^rke2-ingress-nginx$' --output json

++ jq -r '.Releases | length'

[storage] 2021/12/16 08:54:01 listing all releases with filter

+ V2_CHART_EXISTS=

+ [[ '' == \1 ]]

+ [[ '' == \v\2 ]]

+ [[ -n '' ]]

+ shopt -s nullglob

+ helm_content_decode

+ set -e

+ ENC_CHART_PATH=/chart/rke2-ingress-nginx.tgz.base64

+ CHART_PATH=/tmp/rke2-ingress-nginx.tgz

+ [[ ! -f /chart/rke2-ingress-nginx.tgz.base64 ]]

+ base64 -d /chart/rke2-ingress-nginx.tgz.base64

+ CHART=/tmp/rke2-ingress-nginx.tgz

+ set +e

+ [[ install != \d\e\l\e\t\e ]]

+ helm_repo_init

+ grep -q -e 'https\?://'

+ [[ helm_v3 == \h\e\l\m\_\v\3 ]]

+ [[ /tmp/rke2-ingress-nginx.tgz == stable/* ]]

+ [[ -n '' ]]

+ helm_update install --set-string global.clusterCIDR=<IP_ADDRESS>/17 --set-string global.clusterDNS=<IP_ADDRESS> --set-string global.clusterDomain=cluster.local --set-string global.rke2DataDir=/var/lib/rancher/rke2 --set-string global.serviceCIDR=<IP_ADDRESS>/17 --set-string global.systemDefaultRegistry=

+ [[ helm_v3 == \h\e\l\m\_\v\3 ]]

++ helm_v3 ls --all -f '^rke2-ingress-nginx$' --namespace kube-system --output json

++ jq -r '"\(.[0].app_version),\(.[0].status)"'

++ tr '[:upper:]' '[:lower:]'

+ LINE=1.0.2,failed

+ IFS=,

+ read -r INSTALLED_VERSION STATUS _

+ VALUES=

+ for VALUES_FILE in /config/*.yaml

+ VALUES=' --values /config/values-10_HelmChartConfig.yaml'

+ [[ install = \d\e\l\e\t\e ]]

+ [[ 1.0.2 =~ ^(|null)$ ]]

+ [[ failed =~ ^(pending-install|pending-upgrade|pending-rollback)$ ]]

+ [[ failed == \d\e\p\l\o\y\e\d ]]

+ [[ failed =~ ^(deleted|failed|null|unknown)$ ]]

+ [[ helm_v3 == \h\e\l\m\_\v\3 ]]

+ helm_v3 uninstall rke2-ingress-nginx --namespace kube-system

release "rke2-ingress-nginx" uninstalled

+ echo Deleted

Deleted

+ helm_v3 install --set-string global.clusterCIDR=<IP_ADDRESS>/17 --set-string global.clusterDNS=<IP_ADDRESS> --set-string global.clusterDomain=cluster.local --set-string global.rke2DataDir=/var/lib/rancher/rke2 --set-string global.serviceCIDR=<IP_ADDRESS>/17 --set-string global.systemDefaultRegistry= rke2-ingress-nginx /tmp/rke2-ingress-nginx.tgz --values /config/values-10_HelmChartConfig.yaml

In our case this hit ingress-nginx (causing downtime) and longhorn (causing data-loss).
It this the same problem?

That's the same issue I had. I think it's best to have the safe option as a default, and the destructive one as opt in.
As an alternative I suggest writing some damn big warning signs somewhere to not use this controller with any deployments which require any kind of persistence.

I am sure our deployments have been installed and running perfectly well - so not sure why / where we can see a failure and is handled "wrong" for sure..

We tried to reproduce this issue and it seems that we might have found at least one situation where an "uninstall" happens where it should not:
In case the helm upgrade for a given chart / deployment fails i.e. due to hitting a timeout - the deployment is in status "pending-upgrade".
Now the process gets restarted, detects "pending-upgrade", sets the deployment on "failed" and re-runs the helm upgrade.
Here again a failure happens - i.e. this:
Retrying upgrade of longhorn
Error: UPGRADE FAILED: post-upgrade hooks failed: timed out waiting for the condition

In the next re-run the deployment is on failure and this causes the uninstall / reinstall.
Assumption is that this is related to this change of the status from "pending-upgrade" to "failed":
https://github.com/k3s-io/klipper-helm/blob/9a6c7e2ce9f7bf0bafb5d803bb91797e2470b918/entry#L39
Could you verify if this observation is correct?
In general - could we disable the "uninstall / install" in case of any errors completely?
(https://github.com/k3s-io/klipper-helm/blob/9a6c7e2ce9f7bf0bafb5d803bb91797e2470b918/entry#L68)

It is a really bad situation if for what ever reason something that is installed with persistent data gets uninstalled as this will cause data-loss (i.e. with longhorn in our case but also with rook-ceph or elastic,...)
An automated uninstall / install instead of upgrade will cause downtime (nginx-ingress in our case)

Fow now I solve this by switching to flux helm controller which do not suffer from this issue https://github.com/fluxcd/helm-controller.

Fow now I solve this by switching to flux helm controller which do not suffer from this issue https://github.com/fluxcd/helm-controller.

So you deploy this "in addition" to the one we have in rke2/k3s?

OK, so the helm upgrade operation is timing out. It's taking longer than helm gives it by default. Assuming there's not something else broken, the correct fix for this is probably just raise the timeout. I am also assuming that Longhorn itself supports upgrading by just upgrading the chart without any additional preparation.
I see that we didn't get this field added to the HelmChart spec in the RKE2 docs, but there is a timeout field available that takes a duration string (`30m' or the like):
https://github.com/k3s-io/helm-controller/blob/master/pkg/apis/helm.cattle.io/v1/types.go#L30
This is passed through directly to the helm command's timeout arg:
https://github.com/k3s-io/klipper-helm/blob/master/entry#L152

The „timeout“ situation is just one case that we could reproduce - not sure if there are other situations..
We believe that adjusting the timeout is not a valid workaround, because we never know how much time an upgrade in a given environment might need. We must ensure by 100% that there is no automatic uninstall as an uninstall is downtime and data-loss.
So the way how we reproduce the root cause might be only one situation where this „uninstall“ happens - and we have also seen this with the rke2 included ingress-nginx.. Assumption is that any update/upgrade might cause problems in a given cluster or servers / agents in the cluster etc. So to be on the save side - we might have to set the timeout to „never timeout“. How can we configure this as default for everything that happens with the rke2 integrated helm-controller?
To have a real solution - could we get the „uninstall / install“ removed from the helm-controller completely to protect against downtime and data-loss? Or what would be the reason to allow uninstall / install in a production environment?

Tracking this issue since I'm experiencing the same thing with rke2 + rancher-istio. Will test out that timeout field and report back. We're also managing rook-ceph so further prevention of downtime/data-loss is still necessary.

Hi,
I don't see it's mentioned here , and could not find it documented anywhere - but there is a failurePolicy option in the HelmChart CRD. By default it's set to reinstall which will uninstall the helm chart in case of any failure/timeout during install/upgrade. But you can change it to abort - and then helm controller will just stop and won't uninstall (will wait for manual intervention) .
Should be something like that:
 apiVersion: helm.cattle.io/v1
kind: HelmChart
metadata:
  name: some-chart
  namespace: kube-system
spec:
  repo:
  chart: https://%{KUBERNETES_API}%/static/charts/some-chart-1.2.3.tgz
  targetNamespace: some-ns
  failurePolicy: abort
  valuesContent: |-
    someKey: someVal

Yes. That was added several years ago:

https://github.com/k3s-io/helm-controller/pull/137


Auto delete the first 2 characters when inputting a character

Type: Bug
It always auto delete the first 2 characters when inputting a character
VS Code version: Code 1.78.2 (Universal) (b3e4e68a0bc097f0ae7907b217c1119af9e03435, 2023-05-10T14:44:45.204Z)
OS version: Darwin arm64 22.1.0
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
5, 5, 5

Memory (System)
16.00GB (0.04GB free)

Process Argv
--crash-reporter-id 60d76d46-4b2c-44ce-8e62-59bae27cc8f1

Screen Reader
no

VM
0%

Extensions (26)

Extension
Author (truncated)
Version

rails-routes
aki
0.6.3

aws-toolkit-vscode
ama
1.74.0

buildt-vscode
Bui
0.2.7

solargraph
cas
0.24.0

path-intellisense
chr
2.8.4

vscode-eslint
dba
2.4.0

gitlens
eam
13.6.0

chatgpt-gpt4-gpt3-vscode
Eas
0.8.3

auto-close-tag
for
0.5.14

copilot
Git
1.86.82

copilot-nightly
Git
1.86.113

vscode-pull-request-github
Git
0.64.0

rails-snippets
Hri
1.0.8

endwise
kai
1.5.1

git-graph
mhu
1.30.0

prettify-json
moh
0.0.3

vscode-duplicate
mrm
1.2.1

vscode-docker
ms-
1.25.1

remote-containers
ms-
0.292.0

postman-for-vscode
Pos
0.1.1

ruby
reb
0.28.1

slim
sia
0.1.2

sorbet-vscode-extension
sor
0.3.19

sass-indented
syl
1.8.26

code-gpt-va
vai
0.0.6

vscode-ruby
win
0.28.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30610678
2e4cg342:30602488
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
a9j8j154:30646983
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
19089226:30706075
57b77579:30736110
pythonfmttext:30731395
fixhidewlkth:30730051
showsbindicator:30730056
pythongtdpathcf:30739705

Recording video:
https://github.com/microsoft/vscode/assets/5206527/fa9d71a2-f11a-4e79-a0cd-3f42c72f04c6

/confirmPlease

Thanks for creating this issue! To help narrow it down, please take the time to follow below steps. Thanks a lot and happy coding 🙏
Is the issue caused by an Extension?
Stop VS Code and from the command line (NOT the integrated terminal in Code), execute: code --disable-extensions and try your steps again to see if it reproduces. If it still reproduces, jump to the next chapter.
If you see it is an issue with the extension, please run the command Start Extension Bisect and follow the instructions to find the extension that is causing this issue.

Please report the issue to the extension causing this.
Is the issue caused by VS Code Core?
If the issue is not caused by an extension, it is maybe caused by your configuration. Try to run code --disable-extensions --user-data-dir <directory> where <directory> is an empty folder. This will ensure Code is starting with a fresh data directory, e.g. no specific settings and without any extensions running.
If it still reproduces, it was maybe fixed already in our nightly release. Please download from https://code.visualstudio.com/insiders/ and see if you can reproduce the issue.
Finally, to help us narrow down what change caused the issue, please run the following command to find the build. It will start previous releases of VS Code insiders to see where the regression was introduced:

npx --yes vscode-bisect@latest
follow the instructions until you found the offending build
report back the commit range in this issue

Thanks!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

I have same issue when turn on CodeWhisperer Auto-Suggestions. when turn it off. everything is good.


reimbursableExpenses Ui implementation

Implementation Working video
https://github.com/fylein/fyle-integrations-settings-app/assets/55541808/a4a06f23-170d-4211-9a00-581f786dea39

all these comments are in export-setting-save PR https://github.com/fylein/fyle-integrations-settings-app/pull/233/commits/a5bd3992ec40ca0f145151bb965386d5f82ac7f9


RW-8970 Add alerting for large object names cron

Where are these alerts sent to?

Where are these alerts sent to?

There are sent to slack channels, for example: https://github.com/all-of-us/workbench-devops/blob/main/terraform-live/modules/prod/main.tf#L100


Add more description on project structure

What this PR does

Add more description on project structure
Re-format template

/cc @kubesphere-sigs/sig-devops

/ping @kubesphere-sigs/sig-devops


feat(lib): allow users to use custom conventions

@vyp came up with the great point of allowing users to define custom conventions that can be consumed by clog.
This issue should help us discussing this feature and how we want to implement it. It's clear that we need an AST in order to understand any convention, but therefore we also need a lexer/tokenizer that consumes a context free grammar that is defined by the user.
I'll think about what it could look like from a user perspective when hooking up a custom convention. Any thoughts regarding this implementation are very welcome!

The current functionality almost allows this. We're pretty liberal with what the posibilities are already. For example (and this isn't directed at @PascalPrecht , just in general :) ), our current commit message is pretty lose:
<SECTION_TRIGGER>[(COMPONENT)]<:><MESSAGE>

[BODY]

[Closes|breaks][#]

Where there are only three required items, section_trigger (aka alias, which determines which "section" the commit applies to), a : to separate the section_trigger (and optional component) from the commit message, and a message.
We already allow completely custom sections, etc.
Because of the way changelogs are written/coupled with finding appropriate commits, I'm not sure of how to allow pure custom conventions outside of the "rules" we've set. I.e. We use those rules to translate a commit message to and write:
<SECTION>
    * [COMPONENT [\n\t * ]] <MESSAGE> (<commit> [, [Closes #] ])

In order to define a truly custom grammar, the user would have to specify not only a regex to grab commits, but also some sort of closure to take those commits and write them out as well...which is almost like providing a custom re-write of clog. Not saying it's not doable, but I guess I'd need a more concrete example of what exactly someone would like to be able to do that they can't with the current "rules."

Sorry for the late reply guys. I think @PascalPrecht may have misunderstood my comment. I was talking about supporting the idea of an internal ast, or JSON output (or both) so that it can probably be pretty easy to render various "formats". So formats being markdown (already done), html, the gnu changelog thing I was talking about, etc etc.
When I quoted "conventional" in that comment in the last line, I was just referring to the "conventional" commit messages we are already using. i.e feat(scope): one line summary. Not that I wanted a fully flexible way of doing a whole different type of commit message. And as @kbknapp, the current functionality is already pretty flexible because @kbknapp implemented the custom sections that can be user defined.
In fact, I think we should stick to this "convention" of commit messages, not only does it make the code much simpler, but it's just overall less confusing to only support one single commit message format.
So to clarify again, I'm good with one single commit message format (i.e. what you type in git commit), but I'm also good with having various output formats (i.e. the format of the final changelogs themselves markdown, html, json etc.) as suggested in #57.
I'm quoting "convention" everywhere as a reference to the angular js git commit message guidelines, in assumption of the fact that they were the ones who invented the whole type(scope): subject thing right? But also to acknowledge that I understand that it isn't exactly much of a "convention" yet (i.e. widely used).

In order to define a truly custom grammar, the user would have to specify not only a regex to grab commits, but also some sort of closure to take those commits and write them out as well...which is almost like providing a custom re-write of clog.

I feel the same about this issue. I think an internal AST would be great to support different output formats but I don't think we can easily support different input formats.


Does class-validator work well with sequelize-typescript?

I haven't tried using class-validator with sequelize-typescript, but I'd like to ask first, does class-validator work well with sequelize-typescript?

Why would the built-in validations work for this? Those are really extensive as well, and so far have suited all my needs.

@MagicLegend
How to validate more than one columns value with sequelize-typescript?

@gzlock What do you mean? You can add multiple annotations to a single column if desired. E.g. (from the docs):
@IsDate
@IsBefore('2017-02-27')
@Column
producedAt: Date;

Make sure @Column is the last entry though, otherwise ST will start erroring out about it.


fix: keep original error message when transformation fails

Q
A

Bug fix?
Y

New feature?
N

BC breaks?
N

Deprecations?
N

Fixed tickets
N

When this transformer is in play, if the base validation of the DTO object fails, the error message of the transformer hijacked the error message of the base validation in the form.

@dameert what about this?

Hotfix, did I push it directly into master first? (Not sure)

Closed and created a ticket https://github.com/ems-project/EMSFormBundle/issues/273


Handle escape inside the find widget to prevent closing peek widget

fix #160621
The problem was that prior to this PR, the KeybindingsRegistry was handling the closing of the find widget when the escape key was pressed, regardless of whether the keypress occurred inside or outside of the widget. This caused the peek widget, which has a higher priority, to also be closed.
This PR maintains the registered keybinding for the escape key, but also adds additional handling within the find widget to close the widget when the escape key is pressed. This allows for the escape key to be handled by the widget itself, preventing it from propagating to the keybinding handler and affecting the peek widget.

Thank you for the contribution. The issue is a bit more complex as it needs to ensure no regression for https://github.com/microsoft/vscode/issues/88232 and also allow users to override the keybindings to escape the find widget so we can't hard code escape in the keydown handler. An alternative fix is https://github.com/microsoft/vscode/pull/203746.


Out of order event processing in ODE

Seems to happen randomly in rare cases. Working on a test case.

I just noticed that the commit for this issue did not include an update to the ode changelog.

Thanks for catching that Phil. I've updated the change log.


[fix] change the uniq id for an event.

The event id sent by the api is actually not uniq. Using now uid which is the hash of id and partitionDate.
Fix a small issue on bookingOpensOn.

Yes, strange conception.. They seem to use a combo of id/partitionDate to obtain a key. Nobody builds a database like that.