text:
0.8.11 crash

Hello, using 0.8.11 (installed via cargo on MacOS) in --stdio mode via Emacs 29, I'm getting the following crash:
[stderr] thread 'main' panicked at /Users/anon/.cargo/registry/src/index.crates.io-6f17d22bba15001f/neocmakelsp-0.8.11/src/utils/findpackage/packagemac.rs:50:18:
[stderr] called `Option::unwrap()` on a `None` value
[stderr] stack backtrace:
[stderr]    0: _rust_begin_unwind
[stderr]    1: core::panicking::panic_fmt
[stderr]    2: core::panicking::panic
[stderr]    3: core::option::unwrap_failed
[stderr]    4: neocmakelsp::utils::findpackage::packagemac::get_cmake_message_with_prefixes
[stderr]    5: neocmakelsp::utils::findpackage::get_cmake_message
[stderr]    6: std::sync::once::Once::call_once::{{closure}}
[stderr]    7: std::sys::sync::once::queue::Once::call
[stderr]    8: neocmakelsp::utils::findpackage::get_cmake_packages_withkeys
[stderr]    9: std::sync::once::Once::call_once::{{closure}}
[stderr]   10: std::sys::sync::once::queue::Once::call
[stderr]   11: neocmakelsp::complete::get_cmake_package_complete
[stderr]   12: neocmakelsp::complete::getsubcomplete
[stderr]   13: neocmakelsp::complete::update_cache::{{closure}}
[stderr]   14: neocmakelsp::languageserver::<impl tower_lsp::LanguageServer for neocmakelsp::Backend>::did_open::{{closure}}
[stderr]   15: tower_lsp::jsonrpc::router::Router<S,E>::method::{{closure}}::{{closure}}::{{closure}}
[stderr]   16: <futures_util::future::future::Map<Fut,F> as core::future::future::Future>::poll
[stderr]   17: <tower_lsp::service::LspService<S> as tower_service::Service<tower_lsp::jsonrpc::request::Request>>::call::{{closure}}
[stderr]   18: <futures_util::future::poll_fn::PollFn<F> as core::future::future::Future>::poll
[stderr]   19: neocmakelsp::main::{{closure}}.11192
[stderr]   20: neocmakelsp::main
[stderr] note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Full eglot event log

[internal] Mon Oct 21 12:15:23 2024:
(:message "Running language server: neocmakelsp --stdio")
[client-request] (id:1) Mon Oct 21 12:15:23 2024:
(:jsonrpc "2.0" :id 1 :method "initialize" :params
          (:processId 19452 :rootPath "/Users/anon/Dev/SimAndSize/" :rootUri "file:///Users/anon/Dev/SimAndSize" :initializationOptions #s(hash-table size 1 test eql rehash-size 1.5 rehash-threshold 0.8125 data
                                                                                ())
                      :capabilities
                      (:workspace
                       (:applyEdit t :executeCommand
                                   (:dynamicRegistration :json-false)
                                   :workspaceEdit
                                   (:documentChanges t)
                                   :didChangeWatchedFiles
                                   (:dynamicRegistration t)
                                   :symbol
                                   (:dynamicRegistration :json-false)
                                   :configuration t :workspaceFolders t)
                       :textDocument
                       (:synchronization
                        (:dynamicRegistration :json-false :willSave t :willSaveWaitUntil t :didSave t)
                        :completion
                        (:dynamicRegistration :json-false :completionItem
                                              (:snippetSupport :json-false :deprecatedSupport t :tagSupport
                                                               (:valueSet
                                                                [1]))
                                              :contextSupport t)
                        :hover
                        (:dynamicRegistration :json-false :contentFormat
                                              ["markdown" "plaintext"])
                        :signatureHelp
                        (:dynamicRegistration :json-false :signatureInformation
                                              (:parameterInformation
                                               (:labelOffsetSupport t)
                                               :activeParameterSupport t))
                        :references
                        (:dynamicRegistration :json-false)
                        :definition
                        (:dynamicRegistration :json-false :linkSupport t)
                        :declaration
                        (:dynamicRegistration :json-false :linkSupport t)
                        :implementation
                        (:dynamicRegistration :json-false :linkSupport t)
                        :typeDefinition
                        (:dynamicRegistration :json-false :linkSupport t)
                        :documentSymbol
                        (:dynamicRegistration :json-false :hierarchicalDocumentSymbolSupport t :symbolKind
                                              (:valueSet
                                               [1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26]))
                        :documentHighlight
                        (:dynamicRegistration :json-false)
                        :codeAction
                        (:dynamicRegistration :json-false :codeActionLiteralSupport
                                              (:codeActionKind
                                               (:valueSet
                                                ["quickfix" "refactor" "refactor.extract" "refactor.inline" "refactor.rewrite" "source" "source.organizeImports"]))
                                              :isPreferredSupport t)
                        :formatting
                        (:dynamicRegistration :json-false)
                        :rangeFormatting
                        (:dynamicRegistration :json-false)
                        :rename
                        (:dynamicRegistration :json-false)
                        :publishDiagnostics
                        (:relatedInformation :json-false :codeDescriptionSupport :json-false :tagSupport
                                             (:valueSet
                                              [1 2])))
                       :experimental #s(hash-table size 1 test eql rehash-size 1.5 rehash-threshold 0.8125 data
                                                   ()))
                      :workspaceFolders
                      [(:uri "file:///Users/anon/Dev/SimAndSize" :name "~/Dev/SimAndSize/")]))
[server-reply] (id:1) Mon Oct 21 12:15:23 2024:
(:jsonrpc "2.0" :result
          (:capabilities
           (:completionProvider
            (:resolveProvider :json-false)
            :definitionProvider t :documentFormattingProvider t :documentLinkProvider
            (:resolveProvider t :workDoneProgress :json-false)
            :documentSymbolProvider t :hoverProvider t :referencesProvider t :textDocumentSync
            (:change 1 :openClose t :save t :willSave :json-false :willSaveWaitUntil :json-false)
            :workspace
            (:workspaceFolders
             (:changeNotifications t :supported t)))
           :serverInfo
           (:name "neocmakelsp" :version "0.8.11"))
          :id 1)
[client-notification] Mon Oct 21 12:15:23 2024:
(:jsonrpc "2.0" :method "initialized" :params #s(hash-table size 1 test eql rehash-size 1.5 rehash-threshold 0.8125 data
                                                            ()))
[client-notification] Mon Oct 21 12:15:23 2024:
(:jsonrpc "2.0" :method "textDocument/didOpen" :params
          (:textDocument
           (:uri "file:///Users/anon/Dev/SimAndSize/CMakeLists.txt" :version 0 :languageId "cmake" :text "cmake_minimum_required(VERSION 3.19 FATAL_ERROR)\nproject(SimAndSize VERSION 2.0.1 LANGUAGES CXX)\n\n#...")))
[client-notification] Mon Oct 21 12:15:23 2024:
(:jsonrpc "2.0" :method "workspace/didChangeConfiguration" :params
          (:settings #s(hash-table size 1 test eql rehash-size 1.5 rehash-threshold 0.8125 data
                                   ())))
[server-request] (id:0) Mon Oct 21 12:15:24 2024:
(:jsonrpc "2.0" :method "client/registerCapability" :params
          (:registrations
           [(:id "CMakeCacheWatcher" :method "workspace/didChangeWatchedFiles" :registerOptions
                 (:watchers
                  [(:globPattern "**/CMakeCache.txt" :kind 7)
                   (:globPattern "**/.cmake/api/v1/reply/*.json" :kind 7)
                   (:globPattern "**/CMakeLists.txt" :kind 5)]))])
          :id 0)
[client-reply] (id:0) Mon Oct 21 12:15:24 2024:
(:jsonrpc "2.0" :id 0 :result nil)
[stderr] thread 'main' panicked at /Users/anon/.cargo/registry/src/index.crates.io-6f17d22bba15001f/neocmakelsp-0.8.11/src/utils/findpackage/packagemac.rs:50:18:
[stderr] called `Option::unwrap()` on a `None` value
[stderr] stack backtrace:
[stderr]    0: _rust_begin_unwind
[stderr]    1: core::panicking::panic_fmt
[stderr]    2: core::panicking::panic
[stderr]    3: core::option::unwrap_failed
[stderr]    4: neocmakelsp::utils::findpackage::packagemac::get_cmake_message_with_prefixes
[stderr]    5: neocmakelsp::utils::findpackage::get_cmake_message
[stderr]    6: std::sync::once::Once::call_once::{{closure}}
[stderr]    7: std::sys::sync::once::queue::Once::call
[stderr]    8: neocmakelsp::utils::findpackage::get_cmake_packages_withkeys
[stderr]    9: std::sync::once::Once::call_once::{{closure}}
[stderr]   10: std::sys::sync::once::queue::Once::call
[stderr]   11: neocmakelsp::complete::get_cmake_package_complete
[stderr]   12: neocmakelsp::complete::getsubcomplete
[stderr]   13: neocmakelsp::complete::update_cache::{{closure}}
[stderr]   14: neocmakelsp::languageserver::::did_open::{{closure}}
[stderr]   15: tower_lsp::jsonrpc::router::Router::method::{{closure}}::{{closure}}::{{closure}}
[stderr]   16:  as core::future::future::Future>::poll
[stderr]   17:  as tower_service::Service>::call::{{closure}}
[stderr]   18:  as core::future::future::Future>::poll
[stderr]   19: neocmakelsp::main::{{closure}}.11192
[stderr]   20: neocmakelsp::main
[stderr] note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
[internal] Mon Oct 21 12:15:26 2024:
(:message "Connection state changed" :change "exited abnormally with code 101\n")
----------b---y---e---b---y---e----------
[stderr]
[stderr]
[stderr] nil
[stderr] nil
[stderr] Process EGLOT (SimAndSize/(cmake-mode)) stderr finished

Fixed, thanks! it is an serious bug, so I think I should publish a new release for it

Wow, you work fast! Thanks!

authors: Decodetalkers ILoveGoulash

repo: neocmakelsp/neocmakelsp


text:
Redesign options page

As the title suggests... The new look should coexist with the style and layout of Chrome's internal option pages (chrome://settings). Help sections being triggered by "Learn more" links and tabs controlled by the left navigation. No search box should exist.
Resources could possibly be looked up the same way as the internal option pages (e.g. chrome://resources/css/button.css). I'm only guessing this but it would be really useful if it could.
Finally, it might be useful if a better framework could be in use for i18n replacement. Chrome's internal pages use the attribute i18n-content to contain the name of the message to be looked up used to replace the element's content. However, consideration should be given to how attributes and messages with arguments should be handled, but this a good start.
All screenshots showing the options page should be updated afterwards.

See comments on iOrder for notes on how this was implemented there.

Also see the new comment on iOrder for information of how it was finally implemented.

Seems to be working fine. Currently I'm not planning on redesigning the options page but perhaps in the future if Chrome adds some kind of new framework and/or L&F for this.
Therefore this only includes the simplification of i18n within the options page and all other areas of the extension along with some other minor tweaks.

There's currently a cosmetic issue with facebox and the templates guide where the close button is outside of the boundaries. I thought I previously fixed this but it seems that fix now breaks the close button when the guide is displayed. It's something to do with the display type but I think the following workaround will fix this;
facebox = $ '#facebox > .popup > .content'
$(document).bind 'reveal.facebox', ->
  margin  = if facebox.find('> .template').length then '0' else ''
  facebox.css 'margin-right', margin

Now works fine.

Now looking at Twitter's bootstrap framework for redesigning the page.

Completely redesigned the options page and I'm extremely happy with the results. I'm now considering this closed but expect there may be minor changes to this page still before full release that relate to outstanding issues.

source: gharchive/issue

repo: neocotic/template

license: MIT

license_type: permissive

license_source: github-api


text:
SoftInputMode="adjustResize" doesn't work with MyNavigationDrawerTheme

Hi!
I noticed that Android can't resize the activity when SoftKeyboard is opened if I'm using your DrawerTheme. All works fine with Lollipop's default theme (applied in styles.xml).
Activity with default theme:

Activity with MyNavigationDrawerTheme:

EditText is below the Soft Keyboard, so the activity wasn't resized.
Best regards,
Paolo

could you share your drawer layout please?

Sure, I uploaded the sources on GitHub: https://github.com/numixproject/org.numixproject.hermes
I'm having this issue with basically all activities especially with ConversationActivity and AddServerActivity.
Thanks again!

Well, I didn't understood very well the problem...
When the keybard appears in landscape mode, the layout can't be resized so the edit text required remains under the keyboard?

Exactly. We're trying to make an IRC client and it's impossible to chat if you don't see the text you're typing. With default theme I've the same behaviour of Hangout or WhatsApp. It just doesn't work with your custom theme.
Thanks again for the help,
Paolo

Update: it happens both in landscape and portrait mode.

Well, sincerily I don't know how "adjustResize" works, I will investigate on that

UPDATE: It works well with adjustPan (scrolls up ALL the view, even the ActionBar):
Keyboard closed:

adjustPan on:

adjustResize on:

Actually I need Resize, because users should be able to see all messages and ActionBar.

I think it's an Android bug: https://code.google.com/p/android/issues/detail?id=5497&q=fullscreen&colspec=ID Type Status Owner Summary Stars
AdjustResize doesn't work when activity is full screen. Is MaterialNavigationDrawer using Full Screen activities?

no, materialnavigationdrawer use normal actionbar activity without action bar

Same problem here. Did you find a work arround PaoloRotolo?

I found a solution: https://stackoverflow.com/questions/21092888/windowsoftinputmode-adjustresize-not-working-with-translucent-action-navbar/22266717#22266717

@neokree i know this library is similar to my library. But i think it is always good to help each other. The issue you have here is because as soon as you have the FullScreen flag (which is required to get the Drawer below the StatusBar) the adjustResize feature won't work anymore. (Starting with API Level 19)
I have created a small util to get it working in my MaterialDrawer library. You can find an answer about this here: http://stackoverflow.com/a/30019136/325479

Great, I knew it was something related to Fullscreen activities. :)

Same problem here. What exactly solves it? I have a LinearLayout inside of a Scrollview.
Also I set android:windowSoftInputMode="adjustPan" in Manifest. Still, the Keyboard overlays parts of my view.

@mindrunner do you have the KeyboardUtil enabled?

@mikepenz  Don't really know. Must have a look at the code first. I am not actively maintaining it at the moment. Can you quickly point me to KeyboartUtil? How to activate it?

@mindrunner oops sorry ignore my comment. I was notified and thought it was an issue on my drawer lib. sorry

authors: PaoloRotolo josefigueredo mikepenz mindrunner neokree

repo: neokree/MaterialNavigationDrawer

license_source: bigquery


text:
neon-storage-controller: add params for compute hook, readiness hook

Intentionally not bumping the version because this can stay at 1.0.0 util it is stabilized enough to go into prod use.

Fix jwtToken variable handling
Add controlPlaneJwtToken, computeHookUrl
Use recently added /ready endpoint as readiness check.

Intentionally not bumping the version because this can stay at 1.0.0
Lint checks require version bumps though. For other Helm apps we have pinned versions, we can adopt that approach for this one as well:

https://github.com/neondatabase/aws/blob/main/.github/workflows/deploy-prod.yml#L4-L6
Also documentation lint is failing because there are two new config keys.

Lint checks require version bumps though.

Ah well, in that case I'll just bump the version.  Added missing keys to the README as well.

source: gharchive/pull-request

authors: fcdm jcsp

repo: neondatabase/helm-charts


text:
Can't connect from local machine

Steps to reproduce
Try to connect to a neon.tech database, e.g. with psql or prisma.
psql $DATABASE_URL
Expected result
Connect successfully
Actual result
psql hangs for about 130 seconds, then times out. prisma segfaults the first time it runs a query. weirdly, prisma db push succeeds.
Environment
Windows 11 with WSL (Ubuntu 20.04).
$ psql --version
psql (PostgreSQL) 12.14 (Ubuntu 12.14-0ubuntu0.20.04.1)

$ pnpm prisma -v
Environment variables loaded from .env
prisma                  : 4.12.0
@prisma/client          : 4.12.0
Current platform        : debian-openssl-1.1.x
Query Engine (Node-API) : libquery-engine 659ef412370fa3b41cd7bf6e94587c1dfb7f67e7 (at node_modules/.pnpm/@prisma+engines@4.12.0/node_modules/@prisma/engines/libquery_engine-debian-openssl-1.1.x.so.node)
Migration Engine        : migration-engine-cli 659ef412370fa3b41cd7bf6e94587c1dfb7f67e7 (at node_modules/.pnpm/@prisma+engines@4.12.0/node_modules/@prisma/engines/migration-engine-debian-openssl-1.1.x)
Format Wasm             : @prisma/prisma-fmt-wasm 4.12.0-67.659ef412370fa3b41cd7bf6e94587c1dfb7f67e7
Default Engines Hash    : 659ef412370fa3b41cd7bf6e94587c1dfb7f67e7
Studio                  : 0.483.0

Logs, links

Hi,
Could you show $DATABASE_URL and psql output?

The trail seems to have gone cold on this issue, closing.

I'm pretty sure this was fixed at some point 👍 sorry for not responding

authors: Crazytieguy arssher hlinnaka

repo: neondatabase/neon


text:
Release 2024-05-06

Release 2024-05-06
Please merge this Pull Request using 'Create a merge commit' button

Reviewed for changelog.

authors: danieltprice vipvap


text:
Spoon screenshot not showing up on the report?

Hi,
Thanks for the great work.
Just wondering did you #ever get Spoon screenshot working with Cucumber + Espresso, I can see the screenshot being taken and pulled to build/spoon/images folder, but can't get the images showing on the Spoon report.
Thanks.

Hi,
Yes, this line below is where I use cucumber embed method to embed all the image directly into the html report:
https://github.com/neoranga55/espresso-cucumber/blob/master/espresso-test-lib/EspressoInstrumentation/src/com/tomtom/espresso/test/CucumberHelperTestSteps.java#L86

Thanks for the replay, does this embed method adds the images to the Cucumber HTML report or the Spoon HTML report?
Thanks

This attaches the screenshot to the cucumber report which is the runner executing the tests.
I assume that if you want the Spoon report you'll need to switch to the Spoon runner and he will generate the report. So you have to make a choice between running the test in parallel on multiple devices or using the cucumber BDD language.

That's interesting, because I have configured Cucumber to run with Spoon. Basically if you do "./gradlew spoon", it deploys the tests to all connected devices and generates a Spoon reports at the end. Anyway, I have got the screenshot display in the Spoon report :), here is a sample project I made.
Cheers,

Awesome, I haven't used Spoon for multi-device execution but it's fantastic to know they can work together so well :+1:


text:
Name cleanup

Draft: Closes #64. server_configs.lua could probably use more cleanup. Currently fails docgen due to the renaming.

@justinmk Of course! Got bored and took advantage of the lua LSP for some quick refactoring :P

Which lua LSP server are you using?

I'm also curious. My sumneko one hasn't been working for a bit. I'm guessing he broke it. This is what is driving me to fork lua-lsp and start using treesitter to improve it. Sometime very soon.

@justinmk I'm using sumneko, but my version is fairly old and I haven't updated it, so maybe I'm not experiencing the same problems as @norcalli?

@mjlbach yeah don't update it. that was my mistake, lol

One nice emerging trend in nix/emacs(I use doom with straight.el) is the pinning and versioning all of the libraries which ensures these regressions never hit users. Looking forward to trying out your language server :) Minor quibble, but we should fix the fact that vim is always detected as an undefined global since that's brought in by neovim

authors: justinmk mjlbach norcalli

repo: neovim/nvim-lsp


text:
refactor: use vim.lsp.start instead of vim.lsp.start_client

This PR replaces the usage of vim.lsp.start_client by vim.lsp.start, delegating the logic of client reuse to the editor.
Also adding support for a silent option to prevent error notifications on init.

yes. i remember wrong version just tried on 0.9

yes. i remember wrong version just tried on 0.9

Thanks for checking!

Thank you, each incremental code reduction like this gets us closer to upstreaming to core. Nice naming improvements too.

That’s the goal indeed! And the name changes were mostly because I had to stare at the code for a while to understand what it was doing 😅

authors: MariaSolOs glepnir

repo: neovim/nvim-lspconfig


text:
readme: Fix lsp_document_highlight augroup to work for buffers

Without this the augroup will be removed from old buffers when a new language server is attached. Found the solution here: https://stackoverflow.com/a/31822230/1862923

Wow, faster than Lucky Luke 😛

I was just clicking refresh until you fixed this :P

authors: lithammer mjlbach


text:
Finalize repos for Nephio Packages

Right now most of the packages are in nephio-example-packages repo. Before the release we need to make sure core Nephio packages are hosted in the right repo.

So, here's one thing about #258. It applies ONLY to our sandbox, not to bring-your-own-cluster. So, I think we probably want (conceptually, names can be debated)

nephio-packages: base nephio components that apply to all installations (porch, nephio-controllers, nephio-webui, configsync, nephio-topology-controller, maybe some others?)
nephio-demo-sandbox-packages: versions (derivatives) of the nephio-packages with any edits needed for our sandbox environment. For example, we could clone nephio-webui, and the load balancer configuration to the service. This is also where all gitea, metallb, etc. packages would live
free5gc-packages: all the specific packages for free5gc including the operator and the individual function packages
example-packages: maybe we want this with examples of other workloads?

if you look at it I would go for 1 repo with all packages.
Now we can divide in different ways. Personally I like to split config from base SW.
E.g. we split rootsync from configsync because of this as they can be used in a decoupled matter.
SW we could split into:

core nephio
reference SW packages, like capi, metallb, resource backend, I also see free5gc here

Config would a reference configs. The fact they run on sandbox or other should not matter.

Better to defer this after R1.

Here is my proposal. For past discussion on this, see also #10 .
Conventions for Package Repositories

Repository names are lowercase, hyphen-delimited segments that match Kubernetes naming conventions so that the Porch name for the repository match the Git name for the repository.
The first segment is blueprints or deploy, to indicate whether they contain DRY or WET packages.
For deployment repositories, the second segment is the cluster name.

Q: Once #335 is implemented, we may have deployment repositories that are shared across clusters. So, maybe we should set a convention now that instead, the second second is cluster or group, and the third segment is the name of the cluster or group?

For blueprint repositories the second segment depends on if the repository is expected to be consumed as an external, organizational, or team repository.
For organizational and team blueprint repositories

The second segment should be org or team respectively.
The third section should be the team identifier for the team repositories.
The rest of the name should follow the same convention as for external repositories, just shifted over.

For external blueprint repositories (that is, repositories published by Nephio or a vendor for wide consumption), the second segment is infra or nf or app, indicating whether the packages in the repository are for creating infrastructure, network functions, or general purpose applications.

Q: Maybe we get rid of nf and just treat them as apps?
Q: Should we add a category for application config, appconfig?
Q: Should we further split infra? For example, cloud for things that are for provisioning the cloud layer, versus things like the vlanindex package?

For external blueprint repositories, the third segment is vendor or other similar grouping.
For external blueprint repositories, there may be a fourth segment that can refine the contents. For example, sometimes there are tweaks made for a particular opinionated installation. Those would have a fourth segment, indicating the opinionated installation name.

Nephio Repositories
Based on these conventions, Nephio has the following repositories in the nephio-project organization.

blueprints-app-nephio-core - contains packages for Nephio components themselves, including:

Porch
ConfigSync
Nephio Controllers
Nephio WebUI

blueprints-app-nephio-opt - contains optional packages used across multiple opinionated installations

Resource Backend
cert-manager
Gitea
Flux Helm Controllers

blueprints-app-nephio-sandbox - contains derivatives of the core and opt packages, as well as its own sandbox-specific packages
blueprints-app-nephio-gcp - contains derivatives of the core and opt packages, as well as its own gcp-specific packages
blueprints-infra-capi - contains packages for provisioning clusters with CAPI
blueprints-infra-gcp - contains packages for provisioning infrastructure on GCP
blueprints-infra-openshift - contains packages for provisioning infrastructure on OpenShift
blueprints-nf-free5gc - contains the free5gc operator and basic NF packages
blueprints-nf-free5gc-examples - for example architectures using free5gc
blueprints-nf-oai - contains the OAI operator and basic NF packages
blueprints-nf-oai-examples - for example architectures using OAI
blueprints-nf-mixed-examples - for examples mixing different vendor functions

Based on the meeting yesterday, we will instead use a single repo with subdirectories for most of this. There may be some additional repos, but we will start with a single one and directories so we can utilize the prow tooling more easily. Porch repos can be scoped to a directory.

Ok, based on the meeting, here is what I am proposing, please comment on this PR:
https://github.com/nephio-project/catalog/pull/2

authors: gvbalaji henderiw johnbelamaric

repo: nephio-project/nephio


text:
Documentation improvement

Is there an existing issue for this?

[x] I have searched the existing issues

Current Behavior
I ran into some issues to run Yace locally using docker command in the document and they took me some time to solve.

Couldn't read /tmp/config.yml: open /tmp/config.yml: permission denied
EnvAccessKeyNotFound: failed to find credentials in the environment

Solutions:

add the option --user "$(id -u):$(id -g)" in the docker run command
use the options in the docker run command instead of the bind mount of the credentials file
-e AWS_ACCESS_KEY_ID='' -e AWS_SECRET_ACCESS_KEY='' -e AWS_DEFAULT_REGION=''

Expected Behavior
Add these error cases and solutions in the docker run section in the document or may update the docker command if these are common cases.
Steps To Reproduce
No response
Anything else?
Accidentally labeled this as a Bug. It seems not able to be removed. Please help remove the bug label.

Closing this as it seems to refer to a really old version of yace. If you can still reproduce the issue with the latest realease please feel free to reopen.

authors: cristiangreco frankguo-dyedurham


text:
Expose GUI ordering API

We move away from setting order base on time.time(), which I imagine would fail in Windows. In Windows, time.time() uses 16ms increments; we've replaced this with a global counter.

We expose the .order property for GUI handles and the order= argument for "add GUI" methods. These will make it easier to, as an example, add a new GUI element between two existing ones or in place of an existing one.

To add an element between one you would have to manually update the orders of all elements right? What happens if you accidentally specify the same order

Like between two existing GUI elements?
If you have gui_a and gui_b, you could create a new GUI element at order=(gui_a.order + gui_b.order) / 2.0. The global counter's an integer but the order argument will take floats.

What happens if you accidentally specify the same order

I'm fairly confident that elements will just follow insertion order, but I think we can consider this undefined behavior.

authors: brentyi kerrj

repo: nerfstudio-project/viser


text:
Add multiplatform support

Use compose multiplatform to compile to iOS and Android

compose multiplatform is currently not production ready for iOS
we are planning to develop a native iOS app


text:
Support: does axon support track_block method?

use for block reward
track_block method supported by parity, but not supported by geth
https://www.quicknode.com/docs/ethereum/trace_block

It has not supported yet.

authors: KaoImin Naupio

repo: nervosnetwork/axon


text:
Post Processing Support

So, I have been working on adding a feature that can "clean up" the converted header that will compile. After working on this problem for a while, I discovered that, when you manually modify the header, you will be adding, removing, and changing lines. So, wrote some code that will allow you to do this easily and automate the process.
Next, I added a simple scripting interface over these four routines, and this forms the basis of this modification. I got it integrated tonight and it seems to be working. Yay! I'm gonna tweak it a bit more then figure out how to do a PR, LOL.

@jarroddavis68 So no PR then?

I'm reworking it for now, making it more efficient and easier to integrate. Something is going wrong each time I try to add your changes to mine, there are conflicts. I need to figure out how to make the integration so that this will not happen.

repo: neslib/Chet


text:
[sandbox] University scraping example using Faust

Hopefully you should be able to run this from my usage instructions - let me know if you have any problems.
Only gotcha: in config.yaml change joel to something-else
Absolutely no deadline on this!

Works great!

Works great!

authors: Jack-Vines jaklinger

repo: nestauk/faust-scraper


text:
Use RxJS instead of promises

Any chance you'd mind if I make a PR to switch the interceptor to using RxJS methods instead of switching the Observable to a promise?

Feel free!

It would be great for this PR to be included in a release.
I'm facing the same issue

@dsebastien fixed in 1.0.4

authors: dsebastien jmcdo29 johnbiundo

repo: nestjsplus/cookies


text:
Docker --net=host fails running in sysbox

Starting with Docker v24.0, running --net=host fails if Docker is run inside a sysbox container.
I've raised a Docker issue, and bisected the problem to this Docker commit.
The outer Docker version doesn't seem to matter, and I've verified the problem on Sysbox CE 0.5.2 and 0.6.2.
The new version of Docker attempts to bind-mount /proc/self/task/<tid>/ns/net and this fails with "permission denied".  Curiously, if I modify the Docker code to bind-mount /proc/thread-self/ns/net this succeeds.  However, this is not a viable solution for Docker, because /proc/thread-self is unsupported on older kernels that Docker still supports.
Below is a minimal Go program that can test bind-mounting in the way Docker does:
package main

import (
	"runtime"
	"fmt"
	"syscall"
	"os"
)

func main() {
	runtime.LockOSThread()
	fmt.Printf("uid: %d\n", syscall.Getuid())
	basePath := "/proc/self/task/%d/ns/net", syscall.Gettid())
	fmt.Println(basePath)
	lnPath := "/var/namespace/test"
	f, err := os.Create(lnPath)
	if err != nil {
		fmt.Printf("failed to create %s: %s\n", lnPath, err)
		os.Exit(1)
	}
	f.Close()
	err = syscall.Mount(basePath, lnPath, "bind", syscall.MS_BIND, "")
	if err != nil {
		fmt.Printf("failed to bind-mount %s: %s\n", basePath, err)
		os.Exit(1)
	}
	runtime.UnlockOSThread()
}

When I run it as root inside a sysbox 0.6.2 container, it fails with "permission denied."  If I switch basePath to be /proc/thread-self/ns/net it succeeds.
It also succeeds if I run it in a combination of user and mount namespaces, e.g. unshare --user --mount --map-root-user ./bmount.  Using unshare both the /proc/self/task/<tid> and /proc/thread-self variants succeed.  However, if I unshare with only a user namespace (no mount namespace), both variants fail with permission denied.
Sysbox is the only environment I've found where /proc/thread-self succeeds, and /proc/self/task/<tid> fails.  So, something more subtle than just user namespace isolation is going on.

@spikecurtis, thanks for the detailed description, that helps.
At first glance, I don't have an explanation for what you're seeing, especially since Sysbox is doing nothing with these bind-mounts. Right, Sysbox traps mount() syscalls through seccomp-bpf but it only handles the /proc/sys hierarchy (as well as certain portions of /sys). For all other paths, Sysbox lets the kernel be the one handling the mount syscalls.
Need to think about this one in more detail.

I can confirm for Sysbox 0.6.2, Docker 24.0.5 on Debian 11. Docker daemon on our gitlab-runner hosts and docker client within our build images is 24.0.5. Our temporary workaround is using Docker 23.0.6 for the dind rather than 24.0.5.

I can also confirm Sysbox 0.6.2, Docker 24.0.X on Ubuntu 20.04.
We use sysbox for creating github runner pods using the Actions runner controller. We are also sticking on temporary workaround of using Docker 23.0.

Same here. Mentioned here, a way to replicate (https://github.com/moby/moby/issues/45681)
What is clear is that version 24 does not work with net=host in sysbox containers and DinD

Hi @spikecurtis, thank you again for reporting this issue and your effort in helping root cause it.
The problem was a bug in the way Sysbox was intercepting mount system calls when the mount path has the form /proc/self/task/<tid>/....
I've fixed it via this commit, so the fix will be present in the upcoming v0.6.3 release (1->2 weeks).
Closing this ticket now.

hey @ctalledo! much thanks for fixing this issue. When can we expect v0.6.3 to be released?

Hi @vaibhav-shah,

When can we expect v0.6.3 to be released?

Should be ready before the weekend, unless we hit an unforeseen problem.

Hi @vaibhav-shah,

When can we expect v0.6.3 to be released?

Should be ready before the weekend, unless we hit an unforeseen problem.

@ctalledo will the release cover the fix for this issue?

Hi @DekusDenial,

will the release cover the fix for this issue?

Let's discuss in that issue please to avoid polluting this one.

Hi @vaibhav-shah,

When can we expect v0.6.3 to be released?

Sysbox v0.6.3 has been released; let us know if you hit any issues please. Thanks!

authors: DekusDenial anshulpatel25 cruizba ctalledo nudgegoonies rodnymolina spikecurtis vaibhav-shah

repo: nestybox/sysbox


text:
remove my deadname

Today I saw one of the tutorials on the netcup Twitter and I saw that the tutorials I wrote still include my deadname, because of this it would be nice if that gets changed

Certainly, I have just approved your request and looked through all remaining tutorials, ensuring that the name has been changed. Thank you very much for participating in the community and do not hesitate to contact us for any changes.

Thanks! But, will the name on https://community.netcup.com/en/tutorials be automatically updated?

I have informed the responsible department to trigger a rebuild of the website as soon as possible. Once that has been done, the name change will be applied side wide.

repo: netcup-community/community-tutorials


text:
qlkube: parametrize exposed and subscription-enabled APIs

Description
This PR introduces the possibility to parametrize through helm the set of APIs exposed through graphql and the ones for which subscriptions are enabled.
How Has This Been Tested?

[x] Staging environment

/deploy-staging

/deploy-staging

/rebase

Are we waiting for the other qlkube PR to be merged prior to testing this one?

Are we waiting for the other qlkube PR to be merged prior to testing this one?

Yep, otherwise it would be crashing.

/rebase

/deploy-staging

/rebase

/rebase

/merge

repo: netgroup-polito/CrownLabs


text:
test: tryout jest as test framework

🎉 Thanks for submitting a pull request! 🎉
Summary
As discussed in our node guild sync I've tried out jest as test runner to see if it is less flaky on the CI.
one thing I already see is that tests are getting more readable through the expectation syntax of jest. IMHO
Furthermore the snapshot capability is better with jest as well and having the possibility to nest tests in describes with dedicated beforeEach has some advantages as well.
The true power though is the easiness of the mocking capabilities that jest provides. Which exceeds the capabilities with sinon by far.
As the mocking is built into the test framework you don't need to mock something first before a file is imported which is a killer feature IMO.

For us to review and ship your PR efficiently, please perform the following steps:

[ ] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or something that`s on fire 🔥 (e.g. incident related), you can skip this step.
[x] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[x] Update or add tests (if any source code was changed or added) 🧪
[x] Update or add documentation (if features were changed or added) 📝
[x] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

It did not solve our issue with the flaky test doing this migration.
Basically I was using jest-codemods that can automatic migrate ava to jest: https://github.com/skovhus/jest-codemods
If needed in the future this can be picked up cc @erezrokah

repo: netlify/cli


text:
chore: remove unused gh-release dependency

We don't it anymore :)

Added another commit to remove auto-changelog

repo: netlify/framework-info


text:
what

Describe the bug

To Reproduce

Expected behavior

Screenshots

Applicable Versions:

Netlify CMS version: [e.g. 2.0.4]
Git provider: [e.g. GitHub, BitBucket]
OS: [e.g. Windows 7]
Browser version [e.g. chrome 22, safari 11]

Node.JS version:

CMS configuration

Additional context

Closing, please re-open with the issue template

authors: erezrokah ifyaa

repo: netlify/netlify-cms


text:
Load other collection in registerPreviewTemplate

Hi all,
Is it possible to load other collocation (like header configuration) inside registerPreviewTemplate
?

Hi @PetroPavlenko, can you please follow the issue template? It is hard for me to understand what is your use case.

Sure. Sorry it's more like feature than issue, I'll fill issue template.

@erezrokah here it is
Describe the problem
Access different collection inside CMS.registerPreviewTemplate. I would expect it to be like this:
So if we have blog collection and header collection, it would be nice to have something like this
export const BlogPagePreview: ComponentType> = ({ entry, loadOtherEntry }) => (
  <BlogPageTemplate
    header={loadOtherEntry("header")}
    body={entry.getIn(["data", "body"])}
    title={entry.getIn(["data", "title"])}
  />
);

CMS.registerPreviewTemplate("blog", BlogPagePreview);

If you link those via a relation widget you can use:
https://www.netlifycms.org/docs/customization/#accessing-metadata

Thanks, I'll try that

authors: PetroPavlenko erezrokah


text:
add Antora

Add Antora to the catalog. See https://antora.org.

Is there anything else I can do to make this merge ready?

Nope, sorry for the wait - thanks for adding!

Thanks @erquhart!

It looks like the link to GitLab doesn't work. What's the correct syntax? It seems that staticgen is assuming the project is on GitHub.
https://www.staticgen.com/antora

The assumption seems to be here: https://github.com/netlify/staticgen/blob/master/src/Project/Project.js#L100-L102. Not all projects live on GitHub ;)

Argh, forgot we were holding off on this: https://github.com/netlify/staticgen/pull/429

Thanks for the pointer. To give the issue more visibility, I've created a dedicated issue for GitLab support. See https://github.com/netlify/staticgen/issues/510.

authors: erquhart mojavelinux

repo: netlify/staticgen

license: mit


text:
NETOBSERV-116: Fix namespace deployment

Changed route between frontend and backend since the backend now use proxy alias to be namespace agnostic.

/hold
To coordinate both PR merge

/unhold

/approve


text:
ThreadDeathWatcher causes custom classLoader script memory leaks

Expected behavior
Actual behavior
Steps to reproduce
`package net.jueb.redAlert.script.game.serverScript.sys;
import io.netty.buffer.ByteBufAllocator;
import net.jueb.redAlert.core.base.message.AppByteMessage;
import net.jueb.redAlert.core.base.message.MsgCode;
import net.jueb.redAlert.game.ServerConfig;
import net.jueb.redAlert.game.ServerMain;
import net.jueb.redAlert.game.factory.ScriptFactory;
import net.jueb.util4j.hotSwap.annationScriptFactory.annations.AnnationScript;
/**

调试脚本

@author Administrator
*/
@AnnationScript(id = MsgCode.Game_Sys_Debug)
public class DebugScript extends AbstractSysScript{
@Override
public void action() {
ClassLoader customLoader=ServerConfig.classProvider.getClassLoader();
if(getClass().getClassLoader()==customLoader)
{//this is hotswap script
//test1
ByteBufAllocator.DEFAULT.ioBuffer();// init ThreadDeathWatcher watcher thread
//or test2 ,use netty client write msg to init ThreadDeathWatcher
ServerMain.getInstance().getLogClient().sendMessage(new AppByteMessage(AppByteMessage.Heart_Req));
//so ,this script started netty thread
}
}
public static void main(String[] args) {
//run DebugScript action method
ScriptFactory.getInstance().buildAction(MsgCode.Game_Sys_Debug).run();
}
}`

Minimal yet complete reproducer code (or URL to code)

Netty version
4.1.6
JVM version (e.g. java -version)
OS version (e.g. uname -a)

I hope you can initialize manually
Or pre initialization ThreadDeathWatcher

@juebanlin sorry but I not understand what exactly is the problem and how to reproduce. Can you elaborate ?

@normanmaurer
There is a custom classloader loaded script, if script created PooledByteBuf, may trigger to create a thread, see ThreadDeathWatcher schedule method script classloader, and then lead to the Unrecyclable, finally lead to the hotswap system is a memory leak

@juebanlin I still not understand it completely... Can you provide a reproducer ?

@normanmaurer
I'm not good at English.I paste the code
Maven Dependencies
<dependency>
    <groupId>net.jueb</groupId>
    <artifactId>util4j</artifactId>
    <version>VERSION</version>
</dependency>

demo code:
public class Demo {
	public interface Code{
		public static final int Event_OnConnection=1001;
		public static final int Msg_Login=1002;
	}
	public interface Script extends Runnable{

		public int code();
		public void run();
		public void setParams(Object ...params);
		public Object[] getParams();
	}
	public abstract class AbstractScript implements Script{
		Object[] params;
		@Override
		public void setParams(Object... params) {
			this.params=params;
		}
		@Override
		public Object[] getParams() {
			return params;
		}
	}
	public class HotSwapScriptPrivder extends MapClassProvider<Script,Integer>{
		protected HotSwapScriptPrivder(IClassProvider classProvider) {
			super(Script.class, classProvider);
		}
		@Override
		protected Integer findKey(Script instance) {
			return instance.code();
		}
	}

	ScheduledExecutorService exec=Executors.newScheduledThreadPool(4);
	QueueGroupExecutor ge=buildByMpMc(2, 10, 1000);
	protected QueueGroupExecutor buildByMpMc(int min,int max,int maxPendingTask)
	{
		int maxQueueCount=maxPendingTask;
		//多生产多消费者队列(线程竞争队列)
		Queue<Runnable> bossQueue=new MpmcAtomicArrayQueue<>(maxQueueCount);
		QueueFactory qf=new QueueFactory() {
			@Override
			public RunnableQueue buildQueue() {
				//多生产单消费者队列(PS:bossQueue决定了一个队列只能同时被一个线程处理)
				Queue<Runnable> queue=MpscLinkedQueue.newMpscLinkedQueue();
				return new RunnableQueueWrapper(queue);
			}
		};
		IndexQueueGroupManager iqm=new DefaultIndexQueueManager(qf,false);
		KeyQueueGroupManager kqm=new DefaultKeyQueueManager(qf);
		DefaultQueueGroupExecutor.Builder b=new DefaultQueueGroupExecutor.Builder();
		b.setAssistExecutor(Executors.newSingleThreadExecutor());
		return b.setMaxPoolSize(max).setCorePoolSize(min).setBossQueue(bossQueue).setIndexQueueGroupManager(iqm).setKeyQueueGroupManagerr(kqm).build();
	}

	public void execute(String queueIndex,Runnable task)
	{
		ge.execute(queueIndex, task);
	}

	public void testStart() throws Exception
	{
		//step1 init hotswap system,it auto update class.
		String scriptLib="d:/demoLib/";
		DefaultClassSource classSource=new DefaultClassSource();
		classSource.addJarDir(new URL(scriptLib).toURI());
		classSource.updateAttach(exec, TimeUnit.SECONDS, 10);
		IClassProvider classPrivder=new DynamicClassProvider(classSource);
		classPrivder.setAutoReload(true);
		final HotSwapScriptPrivder testClassPrivder=new HotSwapScriptPrivder(classPrivder);
		//step2 start nettClient
		String demoHost="127.0.0.1";
		int demoPort=1001;
        EventLoopGroup group = new NioEventLoopGroup();
        try {
            Bootstrap b = new Bootstrap();
            b.group(group)
             .channel(NioSocketChannel.class)
             .option(ChannelOption.TCP_NODELAY, true)
             .handler(new ChannelInitializer<SocketChannel>() {
                 @Override
                 public void initChannel(SocketChannel ch) throws Exception {
                     ChannelPipeline p = ch.pipeline();
                     p.addLast(new ChannelInboundHandlerAdapter() {
                    	 @Override
                    	public void channelActive(ChannelHandlerContext ctx) throws Exception {
                    		Script script= testClassPrivder.buildInstance(Code.Event_OnConnection,ctx);
                    		execute(ctx.channel().id().asLongText(), script);//step3 handle logic
                    	}
                     });
                 }
             });
            ChannelFuture f = b.connect(demoHost, demoPort).sync();
            f.channel().closeFuture().sync();
        } finally {
            group.shutdownGracefully();
        }
	}

	public class ConnectionOpenScript extends AbstractScript{
		@Override
		public int code() {
			return Code.Event_OnConnection;
		}
		@Override
		public void run() {
			//step4
			ChannelHandlerContext ctx=(ChannelHandlerContext) getParams()[0];
			ByteBuf loginMsg=ctx.alloc().buffer();
			//....build Msg Content
			ctx.write(loginMsg);
			/**
			 * it may trigger the creation of a thread ==> ThreadDeathWatcher's schedule method
			 * and the watcher Thread holds classLoader reference
			 */
		}
	}
	public static void main(String[] args) throws Exception {
		Demo t=new Demo();
		t.testStart();
	}
}

current fix
If I'm going to fix this classLoader memory leak problem, I have to initialize watcherThread in advance
public void testStart() throws Exception
	{
		PooledByteBufAllocator.DEFAULT.ioBuffer();//Initialization Watcher Thread
		//step1 init hotswap system,it auto update class.
		//...
	}

@juebanlin can you check https://github.com/netty/netty/pull/7493 ?

Fixed by https://github.com/netty/netty/pull/7493

authors: juebanlin normanmaurer

repo: netty/netty


text:
Update compression dependencies

NOTE: we have to check that it works well with JDK 6, if we need to support this version.
For example, lz4-java hasn't officially support JDK 6 since 1.3.0, but its developer thinks that all should work well.
https://github.com/jpountz/lz4-java/commit/1ac84bbb5af5255985b905251a0e8c06e237ae61

AUTOMATED MESSAGE for ADMINS:Please verify and accept the pull request.To accept, say @netkins acceptTo build again, say @netkins buildTo whitelist the author, say @netkins whitelist

@netkins build

@normanmaurer please, restart build

@netkins accept

@normanmaurer do you know why it is failed?

@idelpivnitskiy All PR have the same problem: there seems to be a ByteBuf leak in the HTTP/2 module, which cause all builds to fail.

@idelpivnitskiy @slandelle @Scottmitch is working on it.

@idelpivnitskiy thanks!
Cherry-picked into 4.1 (cbbcdaa249b95c35a18bc2f64eeda44258943056)  and master (9f8e68b10f83660f9a6f810bfbea4b8a93757d4f).

@normanmaurer I've tested it on JDK 7 & 8, I wonder only about JDK 6

Should be fine with 6 as well

Am 11.05.2015 um 22:30 schrieb Idel Pivnitskiy notifications@github.com:
@normanmaurer I've tested it on JDK 7 & 8, I wonder only about JDK 6
—
Reply to this email directly or view it on GitHub.

authors: idelpivnitskiy netkins normanmaurer slandelle


text:
Upgrade to netty-tcnative 1.1.33.Fork7

Motivation:
Upgrade to latest netty-tcnative release and use latest os-maven-plugin.
Modifications:

Upgrade netty-tcnative
Upgrade os-maven-plugin

Result:
Latest releases are used.

@normanmaurer - Is this superseded by https://github.com/netty/netty/pull/4236?

authors: Scottmitch normanmaurer


text:
Return status 431 on header size errors

Motivation:
Currently clients attempting to send headers that are too large recieve
a RST frame. This makes it harder than needed for implementations on top
of netty to handle this in a graceful way.
Modifications:
When the Decoder throws a StreamError of type FRAME_SIZE_ERROR, the
Http2ConnectionHandler will now attempt to send an Http2Header with
status 431 and endOfStream=true
Result:
Implementations now do not have to subclass parts of netty to handle
431s

@mosesn It returns afters ending the headers, so should do either or

Can you clarify how this accounts for the following cases:

Clients shouldn't respond with a 431
if we have already sent the headers we should also not attempt to send this response
if create stream fails for a reason which means we aren't allowed to send a RST (e.g. stream may have already existed)

@Scottmitch

Good point, should be an easy enough fix

Will look into this. Not sure how this would happen though unless I misunderstand something. If we are a server (fixing that), we should never send out our own headers before we have read headers on this stream from the client?

Looking at the ways that createStream can fail, they all indicate that we cannot send anything on that stream id (unless it was because it already exist, but we just checked that). But any state we can be in where this would be the case, would the existing code not call resetStream anyway?

Updated to only send 431 headers if server + a bit more testing

If we are a server (fixing that), we should never send out our own headers before we have read headers on this stream from the client?

FRAME_SIZE_ERROR is not restricted to reading header frames at the beginning of a stream.

But any state we can be in where this would be the case, would the existing code not call resetStream anyway?

Good point ... looks like this is what resetStream does if there is no stream ... I thought we were being a bit more conservative there.

@Scottmitch

FRAME_SIZE_ERROR is not restricted to reading header frames at the beginning of a stream.

This one is a little more tricky...
What we have previously done as a nasty workaround in an implementation was look for "Header size exceeded" in the exception message but even that is not really enough, as Http2CodecUtil.headerListSizeExceeded is called by both Decoder and Encoder and FrameReader so this could be caused by trying to encode headers that are too large as well.
It seems like the nicest (but also somewhat intrusive) thing to to would be to add another enum to Http2Exception that would allow for specifying additional purely internal info that can be used when deciding how to handle an exception?
Or a bit less intrusive but also less generic, I could create a few StreamError subclasses for the cases of exceeding header list size when encoding/decoding respectively.
I'm happy to do the work, just want to make sure I don't start off in a bad direction.

Added a more specific exception for trying to decode headers that are too large. 431s should only be sent now when a server is trying to decode headers that are too large.

@nmittler @louiscryan PTAL as well

cc @ejona86

Or a bit less intrusive but also less generic, I could create a few StreamError subclasses for the cases of exceeding header list size when encoding/decoding respectively.

sgtm. We know when this condition occurs (you already found Http2CodecUtil#headerListSizeExceeded) so we should can be explicit when this condition occurs so we have a clean/reliable way to trigger this behavior.

Tried to address all comments/suggestions. Let me know what I'm still missing

Hmmm.... Seeing lots of test failures here now, but "mvn test" is all green locally

Two cases don't seem to be handled well, but... meh?:

If a client sends too-large trailers after the server has already responded with headers, the server sends trailers with a :status which is invalid (Pseudo-header fields MUST NOT appear in trailers).
If the server half-closes the stream before the client (without a following RST_STREAM with NO_ERROR... or does that even matter for the Netty-state checking?), then the client could send too-large trailers. At that point it is invalid for the server to send HEADERS for the half-closed (local) stream.

In both cases, RST_STREAM is the only real option. Client-provided trailers are pretty rare, but thought I should at least mention it.

@ejona86 I guess we can handle these cases a followup... @Scottmitch ok to pull in ?

@ejona86 - I share the same concerns. I think your item 1 maps to my item 2 in https://github.com/netty/netty/pull/6060#issuecomment-262644087. AFAIK we don't have a reliable way to check "have headers been sent for stream X". In general we rely upon users to distinguish headers from trailers (and not push pseudo headers in the trailers). I think we should address these concerns before we pull the PR in ... otherwise we may violate the RFC without any explicit user interaction.

Ok... That is a bit of extra work. I'll have a look. I'm pretty loaded with other stuff for the next few days, so it might be a little while before I have any updates. I guess I can track this in a similar way to how we track whether a GoAway frame has already been sent (don't have the code right here but seem to remember that is a thing)

Updated with the best way I could find to ensure that it is safe to send headers with a status code and default to the reset if not.

@nmittler @ejona86 @Scottmitch PTAL again

great work @nhnFreespirit and thanks for your patience!
4.1 (cd458f1)

Great: Thanks for sticking with this! :)
Status on this now says "Closed with unmerged commits". Is the anything I need to do?

Nothing to do...

We cherry-pick on the command line instead of use the merge button. I think now GitHub supports cherry-picking so this maybe just for historical reasons.

authors: Scottmitch ejona86 nhnFreespirit nmittler normanmaurer


text:
use of MAX_PACKET_BYTES vs MAX_PAYLOAD_BYTES in receive_packet functions

A question:
        *packet_bytes = packet->payload_bytes;
        netcode_assert( *packet_bytes >= 0 );
        netcode_assert( *packet_bytes <= NETCODE_MAX_PACKET_BYTES );

Since packet_bytes is actually packet->payload_bytes, shouldn't the latter netcode_assert be comparing to NETCODE_MAX_PAYLOAD_BYTES?
Found in the following locations:
https://github.com/networkprotocol/netcode.io/blob/master/netcode.c#L3359
https://github.com/networkprotocol/netcode.io/blob/master/netcode.c#L4858

(in debug build obviously) :)

authors: Kingsquee gafferongames


text:
Initial GKE enabled build

Signed-off-by: Andrey Sobolev haiodo@xored.com
GKE enabled building & testing.
Description
GKE enabled building & testing.
Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

WiP I guess. Yet please align the test targets with the parametrised 9nes we have for packet. Please check master for reference.

authors: haiodo nickolaev

repo: networkservicemesh/networkservicemesh


text:
Lecture 3 code level design

Here is Lecture 3.   No activities yet; need to find some good anti-examples.

Slides look good, but I agree that they would benefit from additional examples + activity

We have the weather station activity.   Do you want something more?

authors: jon-bell mwand


text:
Not so satisfied with the result of my own dataset. Display code request for debugging.

Hello Yiming,
Thanks again for sharing this wonderful work! While I was testing on my own data. The result came out to be not so satisfying. Is it possible that you can share the display code for incremental reconstruction? So I can try the locate my problem. If not, any help for me to start with will be much appreciatted.

I didn't write the code for online visualization. I refer you to https://github.com/zju3dv/NeuralRecon/blob/master/utils.py#L246
If you'd like to save incremental reconstruction mesh, you need to call this function
https://github.com/neu-vi/PlanarRecon/blob/main/utils.py#L434 in each iteration.

Reference in new

I didn't write the code for online visualization. I refer you to https://github.com/zju3dv/NeuralRecon/blob/master/utils.py#L246
If you'd like to save incremental reconstruction mesh, you need to call this function https://github.com/neu-vi/PlanarRecon/blob/main/utils.py#L434 in each iteration.

Thanks a lot!

@ChiyuanFeng Hi, I am a student from China. I am also using planarrecon to test my data recently, can I add your WeChat to have a discuss? Best of luck with your research！

authors: ChiyuanFeng Xuduoteng ymingxie

repo: neu-vi/PlanarRecon


text:
Add tutorial page

closes #555
This PR adds a tutorial page, mainly for two actions

upload a system
view detailed analysis of your outputs

@neubig I created the tutorial using the prod version and forgot that this tutorial would only be deployed with the current master branch. I have updated the examples accordingly, thanks! I'll make the pairwise analysis as a separate issue then


text:
Build docker image from clean context

Ensure the ci build process for the public docker image starts from a clean context (e.g. separate job)

Right now we build the docker image in the same job that we deploy the static site to GH pages from: https://github.com/neurobagel/query-tool/blob/8cdc5f2a2e62d8b98f352807f253b9c22fe03cf8/.github/workflows/cd.yaml#L37-L52
That's no good, because the .env file from the static site build gets baked into our public Docker image - and we don't want that. We will fix this in https://github.com/neurobagel/query-tool/issues/210. But we also should also make sure that we use a fresh context to build the image. Either by building in a separate workflow, or at least a separate job.
Originally posted by @surchs in https://github.com/neurobagel/api/pull/154#discussion_r1316470086

We will split these into two workflow files, maybe one build and one deploy

authors: alyssadai surchs

repo: neurobagel/query-tool


text:
Fixed installation error caused by old SSL module

This PR fixes an issue which happens on Python 2 platforms that have an outdated ssl module. These older ssl modules can cause some insecure requests to succeed where they should fail and secure requests to fail where they should succeed.
Source: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
Fixes #2222

getting this on installation:
Check if urllib3[secure] is installed...............[FAIL]

repo: neuropoly/spinalcordtoolbox


text:
[MAINT] Exclude only nilearn 0.10.3

Closes #872

Changes proposed in this pull request:

Thanks! I think this PR will be rolled with #873 and pulled in with that, I'll do that.

okidoki: will let you handle it (close, merge...) Ping me if you need anything.

authors: Remi-Gau jdkent

repo: neurostuff/NiMARE


text:
sound: fix noise between two playbacks for mp3/aac

I built xrdp with --enable-mp3lame or --enable-fdkaac option, and playback on Microsoft RDP client mstsc.exe.
Both mp3 and aac works.
However, an issue was found for both mp3 and aac.
Reproduction step is as follows:

playback an audio file using VLC.
stop playback after a couple of seconds.
playback the same audio file again.

I hear "short sound" first then the audio file's sound at step 3.
The "short sound" seems the chunk just after stop at step 2.
I guess:

The chunk was sent the mstsc.exe, but not decoded at step 2.
The chunk remains in the buffer of mstsc.exe.
Then it was decoded at step 3.

I thought, it might fix the issue to send silence data to mstsc.exe just before sound_send_close() to push out remaining chunk.
And I changed g_bbuf_size for mp3.
1152*4 is samples per frame (1152 bytes) x 16bit(2 Bytes) x 2ch.
This works fine.
I think the cause might be an issue of mstsc.exe.
This workaround is helpful to avoid this issue.

Thank you for your contribution. Could you format your code and squash commits? Run scripts/run_astyle.sh.

Also, could you address failing CI tests?
BTW, I think this is worth backporting to v0.9. Will cherry-pick to v0.9 after merge.

I found an issue that sound tends to be stuttered for sound with video.
It might be caused by changing g_bbuf_size value.
I need more investigation,

I confirmed your changeset fixes the issue. Thank you very much for your contribution!

@Hiero32 Do you have any further changes for this?

Sorry for waiting.
I fixed a couple of issues which I found during test.

sometimes sound stuttered during playback
sound stuttered on skip operation of vlc
There still exists sound stuttered a bit on skip operation of vlc.
However, it occurs without this patch.
I do not found other issue so far.
If necessary, I will squash.

Thanks, I'll test your additional patch soon. I'd appreciate if you squash commits then we cherry-pick just one commit when backporting your fix to older version.

Hmm, after some tests on my side, the first patch (with additional fix) occurs less stutter for me.

I think I found a condition which causes stutter.
If sound data is sent within 1000mS after SNDC_CLOSE is sent, stutter occurs.
This seems the characteristic of mstsc.exe..
Committed patch handles it.
One side effects is it takes 1000mS that sound comes out after skip operation.
I feel it is better than stuttered sound.
I hope this is the last patch.

I still have a doubt if 1000ms is a valid value (can be shortened?) but your fix gains significant improvement. Let's fix later if there's another glitch.

I tested 600mS through 1000mS.
The result was at least 1000mS is necessary.
Anyway, this patch is a workaround for an issue of mstsc.exe.
Following text is described in 3.2.5.2.1.7 Processing a Close PDU of [MS-RDPEA].

The client finishes any audio that arrived before this PDU and that remains to be rendered.

If mstsc.exe conformed to this spec, this patch was not necessary.

repo: neutrinolabs/xrdp


text:
Run a external python script

Hi, I have a external python script, which i want to run before every request. How to use execSync in javascript to run it?
This script just fetches the Kerberos token that i need for my authorization header which I will set in request object.
import socket
import sys

from requests_kerberos import OPTIONAL, HTTPKerberosAuth

url = sys.argv[1]
http_auth = HTTPKerberosAuth(mutual_authentication=OPTIONAL)
token = http_auth.generate_request_header(response=None, host=url, is_preemptive=True)
print(token)

resolved on own

repo: newhoo/RestfulBox


text:
Add Redis Enterprise Quickstart documentation

Summary
A concise description of the changes being introduced. If you are submitting a PR for a
new quickstart please review the pre merge checklist section to validate this PR is ready for review.

Pre merge checklist

[x] Did you check you NRQL syntax? - Does it work?
[x] Did you check your dashboard image quality? -  Do they look good?
[ ] Did you check that your alerts actually work?
[x] Did you include an InstallPlan and Documentation reference?
[x] Did you check your descriptive content for voice, tone, spelling and grammar errors?
[x] Did you attach images of your dashboards to the PR so we can see them working?

Screenshots
Attach images of any visual changes, such as a dashboard here.

@jcountsNR Thanks for the valuable feedback.  I've made the suggested changes but guessed a bit around the installPlan and the install sections.  Please advise

@zstix @josephgregoryii I did a first pass at this and I think it is close to ready. I'm not sure why the dashboard unique test is failing, they look like unique names to me and I did not find any others with the same name.

Hey @jcountsNR! Thanks for reaching out.
So it appears to me that the reason the test is failing is because there seems to be 3 dashboards under 1 directory (active_active, cluster, and database).
Each dashboards should live in their own directory, respectively, where the directory name maps to the name of the dashboard json.
Example:
dashboards/active_active/active_active.json
dashboards/active_active/active_active01.png
dashboards/cluster/cluster.json
dashboards/cluster/cluster01.png
dashboards/database/database.json
dashboards/database/database01.png
I also see that the error the dashboard uniqueness test is displaying doesn't give much information. I'll create a ticket for the team to take a look at so we can enhance our testing to log some more information.
I hope this makes sense! Feel free to reach out again if you have any information 😄

@josephgregoryii  Aha! Thanks for the help. @maguec , can you create a separate directory for each of the dashboards like Joseph mentioned? Then we can run the test again and see if it passes.

@jcountsNR - Moved each to its own directory

@josephgregoryii @maguec  looks great! Thanks for the help on this one, I've approved on my end.

authors: jcountsNR josephgregoryii maguec

repo: newrelic/newrelic-quickstarts


text:
Add event_harvest_config.harvest_limits.log_event_data to config

Add event_harvest_config.harvest_limits.log_event_data as a configuration option.
This option is not public and has a default value_of(:'application_logging.forwarding.max_samples_stored')
This is not required for the EAP.

Closed by #1009

repo: newrelic/newrelic-ruby-agent


text:
Include Entity in Optimization candidate view

Hopefully this commit fixes #20

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @StephenGoodall, looks good!
Also apologies for getting around to this one late as well.

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

authors: Kav91 StephenGoodall claassistantio devfreddy


text:
Memory charts: don’t divide by 1MB

Let the chart Y axis determine “M”, “G”, etc.  Otherwise you get “2K MB” for 2GB.

It looks like the platform addressed these issues and that this function handles the table content https://github.com/newrelic/nr1-top/blob/master/nerdlets/common/bytes-to-size.js

authors: cirne tangollama


text:
Update tessen, add OneTrustId

Summary

updated tessen to latest version (1.14.0)
added in OneTrustId for testing

@roadlittledawn Would it makes sense to get that integrations config value updated in this PR?

authors: aswanson-nr nr-kkenney

repo: newrelic/opensource-website


text:
Set finalizers on Map and Program

Leaking a Map or Program currently also leaks the kernel space resource. For a long running process
this can be lethal. Use the runtime.SetFinalizer machinery to handle things a little bit more
gracefully. This also means that it's not safe to unmarshal into an existing Map anymore, but
that was probably a bad idea from the start.

Codecov Report

Merging #52 into master will decrease coverage by 0.16%.
The diff coverage is 78.57%.

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
- Coverage   58.18%   58.01%   -0.17%
==========================================
  Files          10       10
  Lines        1387     1391       +4
==========================================
  Hits          807      807
- Misses        465      467       +2
- Partials      115      117       +2

Impacted Files
Coverage Δ

prog.go
62.72% <100%> (+0.69%)
:arrow_up:

map.go
58.73% <76.92%> (-1.56%)
:arrow_down:

Continue to review full report at Codecov.

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

authors: codecov-io lmb

repo: newtools/ebpf


text:
Failed to run improver vulnerabilities.importers.nginx.NginxBasicImprover.

On running
docker-compose exec vulnerablecode ./manage.py improve --all
Output:
Improving data using vulnerabilities.improvers.default.DefaultImprover
Successfully improved data using vulnerabilities.improvers.default.DefaultImprover
Improving data using vulnerabilities.importers.nginx.NginxBasicImprover
Cannot call GitHub API without a token set in the GH_TOKEN environment variable.
Traceback (most recent call last):
  File "/app/vulnerabilities/management/commands/improve.py", line 59, in improve_data
    ImproveRunner(improver).run()
  File "/app/vulnerabilities/improve_runner.py", line 47, in run
    process_inferences(
  File "/usr/local/lib/python3.9/contextlib.py", line 79, in inner
    return func(*args, **kwds)
  File "/app/vulnerabilities/improve_runner.py", line 71, in process_inferences
    for inference in inferences:
  File "/app/vulnerabilities/importers/nginx.py", line 239, in get_inferences
    all_versions = list(self.fetch_nginx_version_from_git_tags())
  File "/app/vulnerabilities/importers/nginx.py", line 294, in fetch_nginx_version_from_git_tags
    for version in nginx_versions:
  File "/app/vulnerabilities/package_managers.py", line 469, in fetch
    for node in self.fetch_tag_nodes(pkg):
  File "/app/vulnerabilities/package_managers.py", line 518, in fetch_tag_nodes
    response = utils.fetch_github_graphql_query(graphql_query)
  File "/app/vulnerabilities/utils.py", line 243, in fetch_github_graphql_query
    raise GitHubTokenError(msg)
vulnerabilities.utils.GitHubTokenError: Cannot call GitHub API without a token set in the GH_TOKEN environment variable.
Failed to run improver vulnerabilities.importers.nginx.NginxBasicImprover. Continuing...
Improving data using vulnerabilities.importers.github.GitHubBasicImprover
Successfully improved data using vulnerabilities.importers.github.GitHubBasicImprover
Improving data using vulnerabilities.importers.debian.DebianBasicImprover
Successfully improved data using vulnerabilities.importers.debian.DebianBasicImprover
Improving data using vulnerabilities.importers.gitlab.GitLabBasicImprover
Successfully improved data using vulnerabilities.importers.gitlab.GitLabBasicImprover
Improving data using vulnerabilities.importers.istio.IstioImprover
Successfully improved data using vulnerabilities.importers.istio.IstioImprover
Improving data using vulnerabilities.improvers.oval.DebianOvalBasicImprover
Successfully improved data using vulnerabilities.improvers.oval.DebianOvalBasicImprover
Improving data using vulnerabilities.improvers.oval.UbuntuOvalBasicImprover
Successfully improved data using vulnerabilities.improvers.oval.UbuntuOvalBasicImprover
Improving data using vulnerabilities.importers.apache_httpd.ApacheHTTPDImprover
Successfully improved data using vulnerabilities.importers.apache_httpd.ApacheHTTPDImprover
CommandError: 1 failed!: vulnerabilities.importers.nginx.NginxBasicImprover

@wiredhikari do you have a valid github token added in your env variables ?

repo: nexB/vulnerablecode


text:
分段函数的问题

是我的语法有问题吗？还是，很不理解，很影响使用体验

部分 Markdown 渲染插件会将 \\ 转义为 \，因此想要输出 \\ 需要写 \\\\

repo: next-theme/hexo-filter-mathjax


text:
Non bot token

Summary
Login as a non bot user
What is the feature request for?
The core library
The Problem
I'm developing a command line discord client. I want to port from discord.py 1.6.0 to nextcord but the client.run(bot=False) is missing. If I understand well, it's just a way of prefixing Bot in the Authorization header, see: https://github.com/Rapptz/discord.py/blob/v1.6.0/discord/http.py#L152
The Ideal Solution
Add client.run(bot=False) feature.
The Current Solution
No response
Additional Context
No response

Nextcord will not be adding the ability to interact as a non-bot user as that violates discord's terms of service assuming that's what you mean.
But I will let Oliver or Epic answer definitively upon their interpretation

This violates ToS, discord does not want libraries to do this. We would be removed from the community resources page. Discord's viewpoint (and mine too) is you just use a bot account for automation, you don't need users for anything. If you need a user, take a think on if what you're doing is okay privacy-wise and safety-wise.

authors: Skelmis fensoft ooliver1

repo: nextcord/nextcord


text:
document bot.dispatch

Summary
title
What is the feature request for?
The documentation
The Problem
sometimes i develop exts and want to utilise the event decorator but idk howto, sometimes in bot either.
The Ideal Solution
document bot.dispatch so i can know about its existance
The Current Solution
No response
Additional Context
No response

I'm not too sure about this, documenting it would give version guarantees which is good and bad in a sense that it currently is designed as an internal method, can be changed but need other's opinions on this.


text:
🛑 /getMusicFeed - Get music feed by URL or ID is down

In 6d2fef8, /getMusicFeed - Get music feed by URL or ID (https://nextpost.tech/tiktok-api/getMusicFeed?query=7129970471625362182&cache_timeout=0&license_key=$API_KEY) was down:

HTTP code: 400
Response time: 5704 ms

Resolved: /getMusicFeed - Get music feed by URL or ID is back up in d1f3351.

repo: nextpost-tech/tiktok-status


text:
🛑 /getUsernameByID - Get username by TikTok user ID is down

In 7bd7ff2, /getUsernameByID - Get username by TikTok user ID (https://nextpost.tech/tiktok-api/getUsernameByID?user_id=6809782522139870213&cache_timeout=0&license_key=$API_KEY) was down:

HTTP code: 400
Response time: 1445 ms

Resolved: /getUsernameByID - Get username by TikTok user ID is back up in 5cccfa4.


text:
Fix #149

[LEVEL]/[COMPONENT]
TASK: New docs UI sidebar glitches (overlays the navbar) when scrolling down
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Only
[x] Refactor

Description, Motivation and Context

Changed from position: sticky to position:fixed.  The issue was happening because the sticky positioning is treated as relative in normal context and as fixed when scrolling. Once the parent container would reach its end of height, it would just push all the content upwards.
Corrected offset to match the previous position.
Renamed Sticky to Fixed.
Screenshots - Animations
The before can be seen in Issue #149 .
After:
https://user-images.githubusercontent.com/73541683/150660378-b7338a66-150e-4f06-8a3c-2d819c95709d.mp4
Please let me know if there is anything that needs to be corrected/adjusted.
PS: The video is cropped because of the file size limit.

Thank you! @mariusflorescu 🚀

Codecov Report

Merging #153 (a446514) into main (3b941f1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #153   +/-   ##
=======================================
  Coverage   76.01%   76.01%
=======================================
  Files         140      140
  Lines        2264     2264
  Branches      680      680
=======================================
  Hits         1721     1721
  Misses        538      538
  Partials        5        5

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 3b941f1...a446514. Read the comment docs.

repo: nextui-org/nextui


text:
Prisma template should include prisma generate postinstall

It's pretty common to run prisma generate on postinstall. Would help keep this setup relatively automatic
// package.json

{
"scripts": {
  "start": "...",
  "postinstall": "prisma generate",
  ...
  },
...
}

implemented in #32

authors: TheoBr nexxeln


text:
fix path of .Rprofile and .Renviron

Thanks @mashehu I actually made these changes yesterday but didnt create a PR!
https://github.com/drpatelh/chipseq/commit/ade0db8a8f103a94088fdbbe7392985d220b3ffc
Ill merge this in favour of that 👍

authors: drpatelh mashehu

repo: nf-core/chipseq


text:
CurveCurator module

Description of feature
Implement module for calling curvecurator

@picciama

repo: nf-core/drugresponseeval


text:
Add parameter to specify filter field for cosmic data

Is your feature request related to a problem? Please describe
Currently, cosmic mutations are filtered by Primary site. Although py-pgatka has a parameter that can be set to a field name in the input file but the Primary site` is hard coded for cosmic mutations.
Describe the solution you'd like
In order to allow users to use different field names Histology it is good to have an additional parameter for the field name.
The default value for the new parameter can still be Primary site
Additional context
The same thing can be done for filtering cosmic cell lines, in which case yet another variable need to be added to specify the field name.  The current value is Sample name for cell lines mutation data that can be set as the default value for the new parameter.

Actually, for now I think the most common use case is to set the default parameter for cosmic mutations to Histology subtype 1 since filtering by cancer type is most frequent. done in commit: 49a3c71119b00a35fde5bfd1723aceb464c21985
Also, Sample name can be used as it is for cell lines mutation data to allow filtering by cell line names.
Thus, we can leave this request until there is a need for it. I will close until someone else asks for it.

repo: nf-core/pgdb


text:
UnRegister Accessory from Platform Plugin

I'm working on the Wink Platform Plugin for homebridge and want to remove/unpublish accessories  that have lost their connection with the Wink Hub without restarting homebridge.
Does anyone know if there is currently a way to do this?

Try talking to @devbobo If I am understanding what you are wanting to do he is doing something similar for the WeMo plugin he is working on.

This is Discussion #237

authors: AppleTechy pdlove

repo: nfarina/homebridge


text:
[NET] ARCtrl.Template.Web.getTemplates fails

This is the code i am running:
#r "nuget: ARCtrl, 2.0.0-alpha.6.swate"

let templateUrl = @"https://github.com/nfdi4plants/Swate-templates/releases/download/latest/templates_v2.0.0.json"

ARCtrl.Template.Web.getTemplates (Some templateUrl)
|> Async.RunSynchronously
|> fun (x: Map<string,ARCtrl.Template>) -> x.Values |> Seq.item (x.Count-1)

It fails, but the error message is so big, the console is not able to display the upper end (i scroll all the way up and it still shows json string):

[!NOTE]
Not sure why or when, but it now returns Map<name, template> which is strange?

Working on it. Problem seems to be that the decoder expects a json object but the encoder codes to an array.

Maybe kill this map logic? i would have also expected an array

Yup

Seems like we did actually write Objects (Dictionaries) in ARCtrl v1.x.x:
Here as an example the Templates.json File from the Swate-Templates release

As you can see, instead of an array, we have a dictionary where the key is the path to the template.

Maybe the idea was, that we wanted to improve loadtime but first offering a list of names, then parsing and reading the specific file. But this was postponed. So maybe just return an array and write an array

Also we now don't have the path information at all.

authors: Freymaurer HLWeil


text:
Suggests to depends

Closes #135 and updates the author list. Removed the cph as I don't think its accurate?

Removed the cph as I don't think its accurate?

It isn't - tanks for spotting.
That said, maintaining this could become annoying (what is the relationship of the R package to the course content? has everybody who has contributed to the course contributed to the package? etc). Perhaps we should just have a generic "NFIDD course contributors" author or the like? Counterpoint being that it might be useful to link the ORCID?

Given that we're only using this for dependency management and data loading it really is an abuse of R package infrastructure and probably could be solved more elegantly at a later stage.

"NFIDD course contributors" author or the like? Counterpoint being that it might be useful to link the ORCID?

Yeah I think this is the right call

authors: sbfnk seabbs

repo: nfidd/nfidd


text:
Format of item data in list

Hey, can I display only value without an id of an item? If I have [{id: 1, name: 'name'}], and I need only to display name without (id), what should I do? Thank you.

there is list-formatter as yo see in README.md
list-formatter, function variable name, custom list formatting function.e.g. 'myListFormatter', not 'myListFormatter()'

Oh, ok. Thank you.

authors: VitaliyKovtun allenhwkim


text:
some data lost after restart

TPCC loaded 1000 warehouses, after a restart, some data is lost.
The split size has changed during restart, right after the restart, there is a lot of splits.
Maybe it's a good way to reproduce it.
mysql> select count(*) cnt, c_w_id from customer group by (c_w_id) having cnt != 30000;
+-------+--------+
| cnt   | c_w_id |
+-------+--------+
| 27000 |    126 |
| 27000 |    127 |
| 24000 |    124 |
+-------+--------+
3 rows in set (12.18 sec)

Never reproduce.
There are several recovery-related bugs fixed recently, probably already fixed.

repo: ngaut/unistore


text:
Add nil check for controller when agent is shutting down

Proposed changes
Add nil check for controller when agent is shutting down to prevent the follow error from happening.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x10087d12a]

goroutine 45 [running]:
main.handleSignals.func1()
        /Users/d.hurley/go/src/github.com/nginx/agent/main.go:141 +0x2ca
created by main.handleSignals
        /Users/d.hurley/go/src/github.com/nginx/agent/main.go:126 +0x1f8
exit status 2
make: *** [run] Error 1

Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING document
[x] I have run make install-tools and have attached any dependency changes to this pull request
[x] If applicable, I have added tests that prove my fix is effective or that my feature works
[x] If applicable, I have checked that any relevant tests pass after adding my changes
[x] If applicable, I have updated any relevant documentation (README.md)
[x] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

Codecov Report
Patch coverage has no change and project coverage change: +0.01 :tada:

Comparison is base (4716d2a) 67.31% compared to head (1fad6e8) 67.33%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #357      +/-   ##
==========================================
+ Coverage   67.31%   67.33%   +0.01%
==========================================
  Files         113      113
  Lines       12731    12738       +7
==========================================
+ Hits         8570     8577       +7
  Misses       3588     3588
  Partials      573      573

see 6 files with indirect coverage changes

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

authors: codecov-commenter dhurley

repo: nginx/agent


text:
fix: issues with digital ocean and required environment variable versus pulumi value

Describe the bug
This is related to https://github.com/nginxinc/kic-reference-architectures/issues/207 but slightly different. It appears that in order for the Pulumi provider to pull the token from the Pulumi config it needs to be both:

In the digitalocean namespace, as digitalocean:token
Encrypted.

I have made the first change, but in order to make the second change we are going to need to do a bit more work because of the way we inherit the provider from the superclass for the do module.
As a stopgap, I've noted in the logic when prompting for a key that it should be in the environment.
This - and #207 - both really need to be made a bit more sane given the fact we try and auth three different ways in order to make this work.
To Reproduce
Steps to reproduce the behavior:

Deploy on DO w/o setting the DIGITALOCEAN_TOKEN environment variable.

Expected behavior
It should, uh, work.
Your environment
n/a
Additional context
None

Note - this is also causing the failure of DO Jenkins build jobs.

repo: nginxinc/kic-reference-architectures


text:
NGF Blog Work

Issue to capture work done on blogs for NGF blogs on the Sprint Board.

Update: Have submitted my blog work to Iliya.

repo: nginxinc/nginx-gateway-fabric


text:
CRDs are not installed on Operator installation

Describe the bug
Since a couple of weeks the operator does not install its respective CRDs (except for nginxingresses.charts.nginx.org) on our OpenShift clusters anymore, I am able to reproduce this with our internal tooling (that is setting up the CatalogSource, Subscription and so on..) and also with the manual installation described here.
For the manual installation that's the log:
[ydixken@silence.local:redacted/redacted/nginx-ingress-helm-operator] » make deploy IMG=nginx/nginx-ingress-operator:1.5.0

cd config/manager && /Users/ydixken/development/redacted/redacted/nginx-ingress-helm-operator/bin/kustomize edit set image controller=nginx/nginx-ingress-operator:1.5.0
/Users/ydixken/development/redacted/redacted/nginx-ingress-helm-operator/bin/kustomize build config/default | kubectl apply -f -
namespace/nginx-ingress-operator-system created
Warning: resource customresourcedefinitions/nginxingresses.charts.nginx.org is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
customresourcedefinition.apiextensions.k8s.io/nginxingresses.charts.nginx.org configured
serviceaccount/nginx-ingress-operator-controller-manager created
role.rbac.authorization.k8s.io/nginx-ingress-operator-leader-election-role created
clusterrole.rbac.authorization.k8s.io/nginx-ingress-operator-manager-role created
Warning: resource clusterroles/nginx-ingress-operator-metrics-reader is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
clusterrole.rbac.authorization.k8s.io/nginx-ingress-operator-metrics-reader configured
Warning: resource clusterroles/nginx-ingress-operator-nginx-ingress-admin is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
clusterrole.rbac.authorization.k8s.io/nginx-ingress-operator-nginx-ingress-admin configured
clusterrole.rbac.authorization.k8s.io/nginx-ingress-operator-proxy-role created
rolebinding.rbac.authorization.k8s.io/nginx-ingress-operator-leader-election-rolebinding created
clusterrolebinding.rbac.authorization.k8s.io/nginx-ingress-operator-manager-rolebinding created
clusterrolebinding.rbac.authorization.k8s.io/nginx-ingress-operator-proxy-rolebinding created
service/nginx-ingress-operator-controller-manager-metrics-service created
deployment.apps/nginx-ingress-operator-controller-manager created

Actual Behaviour
11:03 $ oc get crds | grep -e 'nginx'

nginxingresses.charts.nginx.org                                        2023-07-11T08:55:37Z

Expected after the installation:
11:17 $ oc get crds | grep -e 'f5\|nginx'

apdoslogconfs.appprotectdos.f5.com                                     2023-07-11T09:17:38Z
apdospolicies.appprotectdos.f5.com                                     2023-07-11T09:17:39Z
aplogconfs.appprotect.f5.com                                           2023-07-11T09:17:38Z
appolicies.appprotect.f5.com                                           2023-07-11T09:17:38Z
apusersigs.appprotect.f5.com                                           2023-07-11T09:17:38Z
dnsendpoints.externaldns.nginx.org                                     2023-07-11T09:17:39Z
dosprotectedresources.appprotectdos.f5.com                             2023-07-11T09:17:39Z
globalconfigurations.k8s.nginx.org                                     2023-07-11T09:17:39Z
nginxingresses.charts.nginx.org                                        2023-07-11T08:55:37Z
policies.k8s.nginx.org                                                 2023-07-11T09:17:39Z
transportservers.k8s.nginx.org                                         2023-07-11T09:17:39Z
virtualserverroutes.k8s.nginx.org                                      2023-07-11T09:17:39Z
virtualservers.k8s.nginx.org                                           2023-07-11T09:17:39Z

This behavior might have been triggered through a failed upgrade, that I might not be aware of, I'm currently working around this issue by installing the CRDs manually. From my point of view the operator installation is succeeding correctly, with a healthy CatalogSource and a healthy OP install.
Is this an already known behaviour or changed installation workflow, that I didn't realized yet?
I can happily provide logs / conf, if needed.
Expected behavior
I expect the operator to install the CRDs, as it was doing before.
Your environment

Version of the NGINX Ingress Operator - release version or a specific commit
1.5.0
Version of the Ingress Controller - release version or a specific commit
3.2.0
Version of Kubernetes
OpenShift 4.12.22
Kubernetes platform (e.g. Mini-kube or GCP)
AWS
Using NGINX or NGINX Plus
We're under NGINX Plus subscriptions.

Additional context
Add any other context about the problem here. Any log files you want to share.

Closing, found the issue on our end.

Would you mind sharing the resolution to help inform anyone that discovers this issue?

TL;DR: Please review your CRD dependencies.
We have a set of self-developed tools that set up our clusters with the necessary NGINX manifests such as Ingress, ClusterRoles, APPolicies, and more. These scripts ensure that the required CRDs are present before proceeding to the next set of manifests that rely on these CRDs.
However, we encountered a potential future issue when onboarding more features than just plain NGINX. Therefore we've introduced a check for 'f5' and 'appprotect' CRDs (aside from previously only 'nginx\.org') as a hook, and we made these CRDs mandatory for the Ingress (!). The problem is that the Ingress itself - or its chart (which only requires nginxingresses.charts.nginx.org to be set up) is the one responsible for adding the missing CRDs.
The issue started occurring after we purged the entire operator, namespace, and related resources, without removing the actual CRDs.
What we did:

Carefully reviewed our CRD dependencies to understand the root cause.
Placed the checks for 'f5' and 'appprotect' CRDs in the appropriate manifests.
Removed any unnecessary dependencies that were identified.

authors: brianehlert ydixken

repo: nginxinc/nginx-ingress-helm-operator


text:
Release 0.6

Issue to release 0.6.
Acceptance Criteria:

The release process doc is updated to mention testing our examples before release.
The release process doc is updated to mention how to update our conformance report with the Gateway API manually (or automated once 0.8 is released).

Full release process: https://github.com/nginxinc/nginx-kubernetes-gateway/blob/main/docs/release-process.md

Blocked by #761 and

authors: kate-osborn mpstefan

repo: nginxinc/nginx-kubernetes-gateway


text:
Reduce polling interval to 10 seconds

What
Describe what the change is solving
As desired by our product spec.
How
Describe the solution
Breaking Changes
Are there any breaking changes in this PR?

#491  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hjkatz and the rest of your teammates on  Graphite

I'm going to merge this for now (we would like it for testing and demos) and we can update the polling interval before release.

Merge activity

Nov 7, 10:39 AM EST: A user started a stack merge that includes this pull request via Graphite.

repo: ngrok/ngrok-operator