Nextjs Problem with new updates from github don't render immediately

I have succesfully deployed a nextjs app, this is my config:
but if i make change to the main github repository and run dploy deploy main, i don't see the updated version but always the same version. In the [current directory] if i inspect code, is the newest version, why?  if i restart the server, it serve the newest version correctly
project:
git_repository: 'git@xxx'
deploy:
directory: '/home/xxx/htdocs/xxx'
shared_directories:
#- ''
before_commands: # Before symlink switch
- 'find {release_directory} -type d -exec chmod 0770 {} ; && find {release_directory} -type f -exec chmod 660 {} ;'
- 'cd {release_directory}/xxx && npm install'
- 'cd {release_directory}/xxx  && npm run build'
after_commands: # After symlink switch
- 'bash clpctl varnish-cache:purge --purge=all' # Varnish Cache Purge
- 'timeout 10 pkill -f nodejs &'
- 'cd {release_directory}/xxx && nohup npm start > /dev/null 2>&1 &'

Resolved this is the script that render the deploy of my js app using pm2 for the start.
It clones the github repository, install dependencies and build run & start. The script has been placed in crontab for automatic restart in case of restart of server.
For continuos deployment, a github action on push 'main' make a ssh connection to the server and start the script again, rendering the last version on github.
project:
git_repository: 'git@github.com:xxx/xxx.git'
deploy:
directory: '/home/xxxx/htdocs/xxxx'
shared_directories:
#- ''
before_commands: # Before symlink switch
- 'find {release_directory} -type d -exec chmod 0770 {} ; && find {release_directory} -type f -exec chmod 660 {} ;'
- 'cd {release_directory}/xxx && npm install'
- 'cd {release_directory}/xxx  && npm run build'
after_commands: # After symlink switch
- 'bash clpctl varnish-cache:purge --purge=all' # Varnish Cache Purge
- 'pm2 stop xxx || :'
#- 'cd {release_directory}/xxx && nohup npm start > /dev/null 2>&1 &'
-  'cd {release_directory}/xxx && pm2 start npm --name xxx -- start --watch'
this is the github action:
name: Node.js CI
on:
push:
branches: [ "main" ]
pull_request:
branches: [ "main" ]
jobs:
build:
runs-on: ubuntu-latest

strategy:
  matrix:
    node-version: [18.x]
    # See supported Node.js release schedule at https://nodejs.org/en/about/releases/

steps:
- uses: actions/checkout@v3
- name: Install xxx
  run: |
      sudo apt-get update
- name: Use Node.js ${{ matrix.node-version }}
  uses: actions/setup-node@v3
  with:
    node-version: ${{ matrix.node-version }}
- run: cd kosmos && npm install
- run: cd kosmos && npm run build --if-present
- name: ls -a via OPEN SSH Private Key
  uses: nekiro/ssh-job@main
  with:
      host: ${{ secrets.HOST }}
      user: xxx
      password: ${{ secrets.PASSWORD}}
      command: nvm use 18 && dploy deploy main
  env:
      CI: false
      node-version: ${{ matrix.node-version }}

I hope it help someone.
I have to understand because the documentation isn't helpful where to put env variables. if someone can help me


permissible_values - attribute text value (growth_facility) does not match key (field)

poetry run enum_annotator \
	--modelfile target/soil_biosample_modular.yaml \
	--requested_enum_name fao_class_enum \
	--ontology_string ENVO \
	--max_cosine 0.1  > target/temp1.yaml 2>> target/annotation.log

only when schema.enums[suffixed_name] = temp_enum in linkml_round_trips/modular_gd.py

  growth_facility_enum:
    name: growth_facility_enum
    permissible_values:
      field:
        text: growth_facility
      field_incubation:
        text: growth_facility
      growth_chamber:
        text: growth_facility
      open_top_chamber:
        text: growth_facility
      greenhouse:
        text: growth_facility
      experimental_garden:
        text: growth_facility
      lab_incubation:
        text: growth_facility
      glasshouse:
        text: growth_facility
      other:
        text: growth_facility


do not allow new column with same name as an existing one

@mbbui @nhanitvn please review

Tom, you need to merge with master again.

Test failed

@mbbui you mean merge with head of ddf-sparksql-adatao?

retest this please

retest this please

@mbbui @Huandao0812 unit tests fixed. This should have the correct behaviors.


Add the definition for external api

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/cc @khanhtc1202

/lgtm
plz resolve lint comments 👀

/approve
🎮


Workspace setting not detected

The extension works only if the configuration is added in the users settings not the workspace settings, but the documentation says the configuration should be added in the workspace setting.
It should work with both the configuration but workspace settings should override the other.

Yes, it's so

I'm also experiencing a similar problem when adding the configuration to the settings block for a development container. If we add the settings at the user level, the command runs as-expected; if we add the settings at the container level, the command does not run.


[PowerToys Run] finds only file explorer .ink

Environment
Windows build number: Microsoft Windows [Version 10.0.18363.778]
PowerToys version: 0.18.2
PowerToy module for which you are reporting the bug (if applicable): Run

Steps to reproduce
start writing explorer or file explorer in PowerToys Run
relates to https://github.com/microsoft/PowerToys/issues/3223
Expected behavior
File Explorer is listed
Actual behavior
Only "File Explorer.Ink" ist listed.
Also "Windows Software Development Kit" shows up when searching only for "explor"
And "File Explorer.Ink" is vanishing when entereing "explorer"
"
Screenshots

@olovsson can you please delete everything in this folder %localappdata%/Microsoft/PowerToys/PowerToys Run and see if it fixes the issue?

This solution didn't work for me. I fixed the issue by adding  C:\ProgramData\Microsoft\Windows\Start Menu\Programs path to excluded folders in windows search.


File Upload Format

Hi Volkan,
Please in uploading a file,how do I save only the original name of the file to the database so that I can create a readable hyperlink for users. The automatic file format for files are not friendly to reading.

Search for OriginalNameProperty in issues


UnhandledPromiseRejectionWarning: Unauthorized (401)

I keep getting the following error below while integrating Sendgrid SMTP
I've tried changing the API keys and sender emails severally, still the same.
(node:12324) UnhandledPromiseRejectionWarning: Unauthorized (401)
Permission denied, wrong credentials
null
null
at Request.http [as _callback] (/Users/brahwan/Desktop/LendCare-Frontline 2 2 2/node_modules/@sendgrid/client/src/classes/client.js:124:25)
at Request.self.callback (/Users/brahwan/Desktop/LendCare-Frontline 2 2 2/node_modules/request/request.js:185:22)
at Request.emit (events.js:198:13)
at Request.EventEmitter.emit (domain.js:448:20)
at Request. (/Users/brahwan/Desktop/LendCare-Frontline 2 2 2/node_modules/request/request.js:1161:10)
at Request.emit (events.js:198:13)
at Request.EventEmitter.emit (domain.js:448:20)
at IncomingMessage. (/Users/brahwan/Desktop/LendCare-Frontline 2 2 2/node_modules/request/request.js:1083:12)
at Object.onceWrapper (events.js:286:20)
at IncomingMessage.emit (events.js:203:15)
at IncomingMessage.EventEmitter.emit (domain.js:448:20)
at endReadableNT (_stream_readable.js:1143:12)
at process._tickCallback (internal/process/next_tick.js:63:19)
(node:12324) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:12324) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
*.env file:
<EMAIL_ADDRESS>SENDGRID_KEY=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

Node.js Version: v10.17.0

Hey @bsherhrh1, to get a clear idea what the error is, try adding it in a try catch block and awaiting the sgMail.send().
Meanwhile, how are you setting your SENDGRID_KEY ? 🤔

Closing as duplicate of #970
See my comment there to follow-up.


All check icons on bottom row

http://runescape.wikia.com/wiki/Money_making_guide

This is fixed in the pull request (#18) i made a while ago but @MidasLamb hasn't merged it yet.


resources.PostProcess breaks fingerprinting

Version 0.71.1.

Using example code straight from the documentation (first code example) ...
{{ $css := resources.Get "css/main.css" }}
{{ $css = $css | resources.PostCSS | minify | fingerprint | resources.PostProcess }}
<link rel="stylesheet" href="{{ $css.RelPermalink }}">

... fails with:
[...] execute of template failed: template: partials/head.html:31:25: executing "partials/head.html" at : error calling fingerprint: *postpub.PostPublishResource can not be transformed

Using example code straight from

I'm pretty sure that is not the case. PostProcess needs to be the last step in the chain. I use the above construct all the time without issues.

Thanks. Then I have to investigate closer.

Got it. Two mistakes at once.

My build script stashes automatically for production builds.
fingerprint was placed after PostProcess.

Sorry for the noise.


请问db_nemo_impl.h:58:7: error: ‘PinnableSlice’ has not been declared该怎么解决

In file included from /home/terark/workspace/pika/third/nemo/3rdparty/nemo-rocksdb/src/db_nemo_impl.cc:6:0:
./include/db_nemo_impl.h:58:7: error: ‘PinnableSlice’ has not been declared
PinnableSlice* value) override;

it looks like that nemo-rocksdb is not compatible with rocksdb, which release version of rocksdb do you use?

这是哪个版本的pika，执行下submodule update将submodule更新到相应版本试试


[WIP] Time series forecast v0.2.0 update

add support for multivariate forecasting
misc. bugfixes

@jereze PR ready for review!
In this new minor version, I have added support of external regressors for Neural Network, Prophet and ARIMA models. Note that this requires availability of future values of regressors when forecasting. The implementation has been tested on multiple datasets with several configurations.
I have also revamped the README to reflect these changes and harmonize the format. Could you use it to update the plugin page on our website? I have noted some minor inconsistencies, mostly outdated screenshots of the recipes.

@jereze I have made the requested change to the README. I will also make a PR to update the website doc.

Thanks @jereze! @du-phan can you give it a try? I have tested on my side, but it never hurts to be sure 🌞

ping @du-phan if you have some time to test the new version?

Hey sure I will test it later this week!

thanks! let me know when you deem it ready to merge!

I test it on local and everything looks good to me! @jereze we can push it to the store if you wish!


Support Geo URIs

A Geo URI specifies a geographic location in a URI-like string. When Android or Android apps want to open a map of a location, sometimes they start an ACTION_VIEW intent with a Geo URI as the data. Squad Leader should support the ACTION_VIEW intent when a Geo URI is provided.
Sometimes a Geo URI has a latitude and longitude. Other times, it has 0,0 followed by "?q=380+New+York+St" or something similar to provide a URL-encoded search string that specifies the location.
Some of the basics are found in TODO sections in AndroidManifest.xml and SquadLeaderActivity.java. Uncomment those, and then use the intentData Uri in SquadLeaderActivity.onCreate to pan the map to the specified location.

Backlog item - https://app.asana.com/0/23019535413944/23192032216712


Category and Cartesian instances for polymorphic predicates

This is a new category in which the objects are predicates (wrapped in a PRED record implicitly containing the underlying type and the predicate) and the morphisms map between these objects. This differs from the Preds instance in Categories.Category.Instance.Preds in that the morphisms can change the underlying type as well as the predicate. I'm tentatively referring to this as 'polymorphic' to distinguish it from the existing instance.
I'm not certain that this fits into this library, and if it does then maybe I need to break it apart into separate files for PRED, Preds and PredsCartesian? I'd be happy to adapt it if you're interested, or just to get feedback on any of it as I'm fairly new to Agda and dependent types.

Note that I'm away at a conference right now, so won't be able to review in detail for another few days.
Because PRED is essentially isomorphic to 'containers' which is also known as 'polynomial functors', we definitely want this. Though doing it atop 'containers' is likely the way to go.

A better name for this would be Fam(Set); it is precisely the total category of the family fibration on the category of sets.
Containers/Poly would have the maps going "backwards", EG:
record _⇒_ (A B : PRED) : Set o where
  constructor mk⇒
  open PRED
  field
    {f} : U A → U B
    imp : ∀ {u} → P B (f u) → P A u

It might be worth doing Fam in it's generality though; this would give you Fam(Set), Fam(Setoids), and Fam(Setoids^op) all in one go.

Thanks for catching that I missed the orientation reversal. And yes, it probably is worth doing Fam instead. Do you know what that means @AJChapman  ?

I don't yet know what it means. My category theory knowledge is quite limited. I'll read up on what a 'family fibration' is at the 1lab link @TOTBWF gave. I see that 1lab already has this implemented, so I can use their code as a guide.


pip does not install extras_requires for console_scripts

This seems like a bug in pip, however it manifests itself in a broken pex tool if you 'pip install pex' into an empty virtualenv:
mba=pex=; virtualenv dist/pex_ve
New python executable in dist/pex_ve/bin/python2.7
Also creating executable in dist/pex_ve/bin/python
Installing setuptools, pip...done.

mba=pex=; source dist/pex_ve/bin/activate

(pex_ve)mba=pex=; pip install dist/pex-0.8.7-dev0.tar.gz
Unpacking ./dist/pex-0.8.7-dev0.tar.gz
  Running setup.py (path:/var/folders/rd/_tjz8zts3g14md1kmf38z6w80000gn/T/pip-hYkm9F-build/setup.py) egg_info for package from file:///Users/wickman/clients/pex/dist/pex-0.8.7-dev0.tar.gz

    warning: no files found matching 'docs/*.md'
Requirement already satisfied (use --upgrade to upgrade): setuptools>=2.2,<8 in ./dist/pex_ve/lib/python2.7/site-packages (from pex==0.8.7-dev0)
Installing collected packages: pex
  Running setup.py install for pex

    warning: no files found matching 'docs/*.md'
    Installing pex script to /Users/wickman/clients/pex/dist/pex_ve/bin
Successfully installed pex
Cleaning up...

(pex_ve)mba=pex=; pip list
pex (0.8.7-dev0)
pip (1.5.6)
setuptools (3.6)
wsgiref (0.1.2)

(pex_ve)mba=pex=; pex --help
Traceback (most recent call last):
  File "/Users/wickman/clients/pex/dist/pex_ve/bin/pex", line 9, in <module>
    load_entry_point('pex==0.8.7-dev0', 'console_scripts', 'pex')()
  File "/Users/wickman/clients/pex/dist/pex_ve/lib/python2.7/site-packages/pkg_resources.py", line 356, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/Users/wickman/clients/pex/dist/pex_ve/lib/python2.7/site-packages/pkg_resources.py", line 2431, in load_entry_point
    return ep.load()
  File "/Users/wickman/clients/pex/dist/pex_ve/lib/python2.7/site-packages/pkg_resources.py", line 2145, in load
    if require: self.require(env, installer)
  File "/Users/wickman/clients/pex/dist/pex_ve/lib/python2.7/site-packages/pkg_resources.py", line 2159, in require
    items = working_set.resolve(reqs, env, installer)
  File "/Users/wickman/clients/pex/dist/pex_ve/lib/python2.7/site-packages/pkg_resources.py", line 639, in resolve
    raise DistributionNotFound(req)
pkg_resources.DistributionNotFound: wheel>=0.24.0,<0.25.0

In the meantime we should remove the extras_requires from pex.  This means in order to build pexes with wheels (the default option) we will need an interpreter cache #47.

There is a related issue to this: https://gist.github.com/mitsuhiko/9ee3c30defba57bf04bf

I suppose if you want to keep wheels optional it makes sense to imply the -no-wheel option.

I think the untranslateable thing might be due to a timeout in the UrllibContext which is the default if requests is not linked into the pex.  The RequestsContext has http retries built-in whereas the UrllibContext does not, which causes no end of grief for users in general.
My inclination is to make wheel/requests optional but on by default (in other words you can disable them but you have to try), and use the interpreter cache to resolve them if they're not present on the system / in the virtualenv.

Fixed by e4a577ef0


Function for resetting glossary entry behavior

First of all, thank you for providing this very great package :)
In a document I am currently writing, I would like to reset the behavior of the referenced glossary entries once a new chapter begins so that entries are initially printed as they have not been used before. For that purpose, it would be great to have a function that resets the state that keeps track of whether a glossary entry has already been used before.
Would it be possible to add such a function to the package? Or is that already possible to achieve with what the package is exposing?

I also have this specific feature request. I made a fork which implements this for me, although I'm sure in general the design needs to be thought out more:
https://github.com/hbierlee/glossarium/tree/feature/reset-glossary

I also have this specific feature request. I made a fork which implements this for me, although I'm sure in general the design needs to be thought out more:
https://github.com/hbierlee/glossarium/tree/feature/reset-glossary

thanks, that works out fine for this specific chapter behaviour, but probably need to implemented differently if the user needs more control over when/how the state resets


Show contract address in output for --contract option

This fixes a spelling mistake ("deterministic") and also adds contract address to the output when the --contract parameter is supplied.
The change to the package.json file occurred during npm install and I suspect will happen every time.
Tested by running with and without --contract.

$ node index.js --contract -i 61
✔ {"address":"0x0460cdc242fde29f85cb529a28c4c07cf4f64459","privKey":"2724ff8eb4044e9a24cc558ce47ccddaa169e83280da0d02dbc2ece0a9af1c45","contract":"0x612e36c45d03eb701da71f84ecea0a812e7412dc"}

$ node index.js -i 61
✔ {"address":"0x615c767c933170df8a8b491eb8b16334e368d399","privKey":"4bbfb39b73a77af12ab7c21bae0c85a335210bb755c827626a10a43a6f0ab43c"}

@kvhnuke ptal

Please release a new package version that includes this change.

This change is included in 1.2.0.


Fix: validate discovery arguments & allow to build unsigned macOS artifact

Thanks @piersy for discovering the problem with the wrong argument:

spacemesh --discovery http://localhost:8080/networks.json -- does not work
spacemesh --discovery=http://localhost:8080/networks.json -- works
Now, if the User set the --discovery flag it will validate for having some string there. If it is not presented, a most likely argument was specified wrongly, as a consequence there is an empty string in the flag and we're exiting with an error.
Also updated README.md

After some recent dependency updates it seems electron-builder tries to use CSC_LINK or Certificate from the keychain and then it fails (since I don't have it on my local machine). To make it possible to build unsigned binaries, I had to specify identity: null property to explicitly avoid signing. So now it works again with just DONT_SIGN_APP=1 env variable ;)

Signed binaries
In case you want to ensure that it does not break the release workflow, I've built these ones from test-macos-build branch, which has one commit with only a version bump on top of this PR.

Windows: x64
macOS: x64 arm64
Linux

AppImage x64 arm64
deb x64 arm64


Flask-Cache: CACHE_TYPE is set to null

Hi there,
I have been attempting to install Caravel for the last 4 hours with miserable results - my Linux is rather rusty so perhaps the issue is user orientated...
The issue I am facing is around the creation of a admin user, I keep being returned the following error:
/usr/lib/python2.6/site-packages/flask_cache/__init__.py:152: UserWarning: Flask-Cache: CACHE_TYPE is set to null, caching is effectively disabled.   warnings.warn("Flask-Cache: CACHE_TYPE is set to null, " Was unable to import caravel Error: invalid syntax (views.py, line 880)
Now I know that python2.6 is not supported, however I installed 2.7:
[oracle@bigdatalite ~]$ python Python 2.7.3 (default, Sep 18 2016, 21:11:17)  [GCC 4.4.7 20120313 (Red Hat 4.4.7-17)] on linux2
So it seems odd to me that the packages are being installed under the /usr/bin/python2.6/site-packages directory in python?
I have attempted to pip uninstall the various flask packages and retrieve previous versions which didn't help either.
Any help would be much appreciated.
T.

Better install caravel inside a virtualenv created with python3.4+ . If you can't do that python2.7 should be fine. Don't mess install python libraries in system python.


Bug/importer exporter

Fix exporting to use a dynamically created model so that we don't include dropout or anything like that. This addresses #25 and some conversation from #34.
NOTE: In frozengraph export, we no longer "clear devices". I think there's a way to do this using tf.train.saver, but I don't use frozengraphs so i don't know how to test that. opening that piece up to the community if someone has more experience there

Hi @mattfeury ,
This doesn't fix the issue with frozen model right ? It fixes the issue with savemodel.  Let me know if I'm wrong.
Thanks

theoretically it should fix both, because we are no longer importing the model from the checkpoint but using the dynamically created Model from python. however, like i mentioned i have not tested frozen_model so feel free to.

Sorry my bad. I didn't see the code in your second commit.

Looks good to me, merging. Will test #25 and #34 now, and if you can provide your reports as well, that'd be very much welcome!


Add link resolver to graph nodes

Closes Enterprise issue
This allows graph nodes to link to non-Kind-enum objects in enterprise.
Vid from enterprise:
https://user-images.githubusercontent.com/65822698/207640334-5cb18438-32b4-469a-904c-2bc4ceaed112.mov

Based on the video, the highlighted navigation object on the left isn't changing. Code LGTM (although it appears to need a prettier run).

Enterprise PR to fix this bug here

I am not sure if this is related to this PR and what expected behavior of the sidebar selection is, but I think there might be some minor issues with it.

When I first load the app in tilt (yarn start does not work for me) and click Applications > terraform, the section is removed from Applications. Only after I reload the app and click Applications > terraform after that, the selection is preserved on Applications in the sidebar. Attached the screen recording.

https://user-images.githubusercontent.com/8824708/207667276-fa3f16dd-149b-4877-a235-e63be6ed2b40.mov

If I start the app in tilt, go to Applications > terraform (the selection disappears from Applications), reload the page (to make the selection appear on Applications) and click on Clusters, then both Clusters and Applications become selected, Attached a screenshot.

@joshri

Enterprise PR to fix this bug weaveworks/weave-gitops-enterprise#2096

So, the behavior I noticed is probably a related issue?

Yep - looks like it's bigger than just the terraform page.

Code LGTM (although it appears to need a prettier run).

I ran make ui-prettify-format and there are no changes


[feature]: The width of the dialog should be defined according to its children

Current Behavior
The dialog width is being set to a default  width and creates a scrollbar to display the children.
Expected Behavior
Increase the width of the dialog according to its children, limited to the view width.
Version
0.1.6
What browsers are you seeing the problem on?
chrome
Anything else?
No response

Remove max-width property from default MUI dialog style


Use optimized sprintf

Use optimized version of sprintf. See https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/pull/8092.
It also fixes CI since the php-cs-fixer rule is enabled by default.

Thank you @Jean-Beru!


feat: add useSafeArea to showCustomDialog method in DialogService

@ferrarafer can you apply the dart format fix update here too please.

@saileshbro please, run the following command and push the changes, then I will merge the PR. Thanks for the collaboration 🔥
dart format --fix .

@ferrarafer fixed lint issues.

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

v1.2.0
GitHub release

Your semantic-release bot :package::rocket:


Errors with Three 124

Its great to see this in Three.js! Have an issue using in Typescript with latest Three.js<EMAIL_ADDRESS> Stage.init(renderer)
    Stage.scene = r.scene

Argument of type 'WebGLRenderer' is not assignable to parameter of type 'Renderer'.
  Types of property 'render' are incompatible.
    Type '(scene: Object3D, camera: Camera) => void' is not assignable to type '(scene: Scene, camera: Camera) => void'.
      Types of parameters 'scene' and 'scene' are incompatible.
        Type 'Scene' is missing the following properties from type 'Object3D': animations, clear
Property 'dispose' is missing in type 'THREE.Scene' but required in type

If I revert back to 116.1 it appears to be ok, but could you update to latest?

Hi, I try 0.126.1 but can not reproduce the issue.


Documentation: amd_media & mv_deploy

readme me files missing in AMD_MEDIA & MV_Deploy

need usage
need pre-reqs

@rrawther any update on this bug?

PR #255 fixed this issue


Ensure all TODO entries point to a bug.

In some cases simply fix the issue, it was easier than creating
the bug.  This fixes #41.

PTAL.  Yes, I took some shortcuts in the fixing of those TODOs.

@coryan — the Travis build failed, can you take a look, please?

Let me separate this into multiple PRs and resubmit.

Once PR #113 is merged, just rebase this PR on master and it should be good to go.

OK, PR #113 is merged; do you want to reopen and rebase this PR?


Adding ogv example html

I can work on making this a bit prettier if there is time- we'll need to host these ogv files somewhere so that they're not needed in this lib.

Closing and moving this to the other repo.


Add support for scripting on Linux

As of now, Ducktape uses the Window.h library for loading dlls for Native Scripting. This means that it would not work on Linux, and it'd be great if someone added Linux support using the builtin linux libraries (that contain dlopen(), etc), and used the platform based #ifdef guards to use the Linux or Windows library appropriately like:
#if defined(WINDOWS_MACRO)
#include <windowsLibrary>
#elif defined(LINUX_MACRO)
#include <linuxLibrary>
#endif

and did the same for the rest of the Dll loading functions.
The platform dependant functions are here and here.

i'll take a stab at this

Fixed in #109.


⚠ Processors has degraded performance

In c76f2d1, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: Processors performance has improved in 0ae8d77.


Search::Elasticsearch::Async needs a new release

The version on CPAN has a S::E::Cxn::AEHTTP that consumes S::E::Cxn::HTTP. That role has been removed in the CPAN version of Search::Elasticsearch, so S::E::A fails:
$ perl -MSearch::Elasticsearch::Async -E 'Search::Elasticsearch::Async->new'
Can't locate Search/Elasticsearch/Role/Cxn/HTTP.pm in @INC (you may need to install the Search::Elasticsearch::Role::Cxn::HTTP module) (@INC contains: /etc/perl /usr/local/lib/x86_64-linux-gnu/perl/5.20.2 /usr/local/share/perl/5.20.2 /usr/lib/x86_64-linux-gnu/perl5/5.20 /usr/share/perl5 /usr/lib/x86_64-linux-gnu/perl/5.20 /usr/share/perl/5.20 /usr/local/lib/site_perl .) at /usr/share/perl5/Module/Runtime.pm line 317.
Compilation failed in require at /usr/share/perl5/Module/Runtime.pm line 317.

I see the change has been in git for some time; a new release would be greatly appreciated.

That's weird - I did release v5.0.0 but for some reason PAUSE wasn't happy with it.
See https://metacpan.org/pod/release/DRTECH/Search-Elasticsearch-Async-5.00/lib/Search/Elasticsearch/Async.pm
I had some issues when releasing 5.0.0 - I think PAUSE was struggling to keep up and didn't adjust permissions properly.  I'll try a new release shortly.

There is still something not ok regarding the CPAN index:

[ahartmai@ahartmai-nb:~/perl5/git/NAC-Logging$ (master)]$ dzil listdeps --missing | cpanm
--> Working on Message::Passing::Output::Search::Elasticsearch
Fetching http://www.cpan.org/authors/id/A/AB/ABRAXXA/Message-Passing-Output-Search-Elasticsearch-0.003.tar.gz ... OK
Configuring Message-Passing-Output-Search-Elasticsearch-0.003 ... OK
==> Found dependencies: Search::Elasticsearch::Async
! Finding Search::Elasticsearch::Async on cpanmetadb failed.
! Finding Search::Elasticsearch::Async (5.00) on mirror http://www.cpan.org failed.
! Couldn't find module or a distribution Search::Elasticsearch::Async (5.00)
! Installing the dependencies failed: Module 'Search::Elasticsearch::Async' is not installed
! Bailing out the installation for Message-Passing-Output-Search-Elasticsearch-0.003.

All other version of Search::Elasticsearch::Async then 2.03, including 5.00 are only in BackPAN. Is it  possible to release the next version? Thank you.

Sorry it's taken a while, but Search::Elasticsearch::Async should now be on 5.02.  Please let me know if you see any issues

metacpan finde 5.02 when searching for Search::Elasticsearch::Async and cpanm --look Search::Elasticsearch::Async downloads and unpacks the correct tarball as well, so can confirm that it's fixed.
Thanks Clinton!

@abraxxa great - thanks for letting me know, and sorry it took so long


It's impossible to reset modules when using ES6 import syntax

If you are using ES6 import syntax, there is no way to reset modules between tests. The only workaround is to put tests in different files, or switch to non-standard require syntax.
For example, the following test suite will fail due to shared state:
import { five } from 'aModule';

beforeEach() {
  jest.resetModules();
}

describe('testSuite', () => {
  test('firstTest', () => {
    five++;
    expect(five).toBe(6);
  });

  test('secondTest', () => {
    five + 2;
    expect(five).toBe(7);
  });
});

Event using the resetModules option will not reset five because the reference has already been created.
There should be an option to run each test in complete isolation.

Resetting the module registry doesn't do any good if you don't re-import or re-require the module afterward. See examples at https://facebook.github.io/jest/docs/jest-object.html#jestresetmodules. I think your options are to use require or the fancy new import() syntax.

Just as @jwbay said (thanks ❤). Use require(), it's definitely not "non-standard".

Just in case someone else is also lost a bit after trying above's solution but using import: following is a fix for it.
describe('MyTests', () => {
  let MyModule;

  beforeEach(() => {
    return import('./index').then(module => {
      MyModule = module;
      jest.resetModules();
    });
  });
});

Requisites: to have the babel-plugin-dynamic-import-node configured.
Source: https://jestjs.io/docs/en/jest-object#jestdomockmodulename-factory-options


copying files too slow

What happened:
when use sealer run copying files too slow , need 10 min or more to copy file to destation host.
What you expected to happen:
copy file fast
How to reproduce it (as minimally and precisely as possible):
[root@internal ~]# sealer run kubernetes:v1.19.9 --masters <IP_ADDRESS> --passwd 123456
2021-09-27 11:09:30 [INFO] [default_image.go:55] image kubernetes:v1.19.9 already exists
copying files to <IP_ADDRESS>: 184/318

Anything else we need to know?:
Environment:

sealer version (use sealer version):

[root@internal ~]# sealer version
{"gitVersion":"v0.4.0","gitCommit":"7341634","buildDate":"2021-08-31 10:45:26","goVersion":"go1.14.15","compiler":"gc","platform":"linux/amd64"}
[root@internal ~]#

Cloud provider or hardware configuration:

vmware workstation 16

OS (e.g: cat /etc/os-release):

[root@internal ~]# cat /etc/redhat-release
CentOS Linux release 7.9.2009 (Core)

Kernel (e.g. uname -a):

[root@internal ~]# uname -a
Linux internal.registry.com 3.10.0-1160.6.1.el7.x86_64 #1 SMP Tue Nov 17 13:59:11 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
[root@internal ~]#

Install tools:

sealer

Others:

Change UseDNS yes in the /etc/ssh/sshd_config configuration to no，
Change the value of "GSSAPIAuthentication" to "no" (if there is no such configuration option, add the configuration).

so ridiculous! today I tried this,the same problem exists still.


A sample using Angular, oidc-client and Keycloak

A sample using Angular, oidc-client and Keycloak:

References:

https://robferguson.org/blog/2019/12/29/angular-openid-connect-keycloak/
https://github.com/Robinyo/serendipity

Ok, added to the wiki. Thanks.


no reliance on ... xcode?

Honestly I have no idea why brew needs xcode? seems like brew could replace the need for xcode by supplying more or less what it supplies, replacing it with gcc and the like? I don't really know what it supplies to be honest, or why this choice was made. I would love to know though.

Please always fill out the issue template. Our issue tracker is not for general support requests or questions.

Get help in GitHub Discussions
Have a question? Not sure if your issue affects everyone reproducibly? The quickest way to get help is on Homebrew's GitHub Discussions!
https://github.com/Homebrew/discussions/discussions


Update cats-helper to 2.7.6

Updates com.evolutiongaming:cats-helper from 2.7.2 to 2.7.6.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.evolutiongaming", artifactId = "cats-helper" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #216.


Commiter author should be gogs username, as on Github.

Gogs version (or commit ref): <IP_ADDRESS>8
Git version: 2.8.1
Operating system: Linux SMP Debian 4.3.5-1 (2016-02-06) x86_64 GNU/Linux
Database:

[x] PostgreSQL
[ ] MySQL
[ ] SQLite

Can you reproduce the bug at http://try.gogs.io:

[x] Yes (provide example URL): https://try.gogs.io/misuzu/username-test
[ ] No
[ ] Not relevant

Log gist:

Description
Commiter author should be gogs username, as on Github.

I do not want this change, or at lest not as the @misuzu asked for.
Reason: I use a lot of systems and want to know from what system I made the changes on. AKA: dasoren-ctl, dasoren-eps, dasoren-eff. The ctl, eps, eff are 3 of my systems. Those are the names setup in my .gitconfig file on each system.
When I do a push, I see that as the name on the commit and want to see it that way.
I understand why you want this change and can see why others would want this as well.
To who ever looks into this "issue/bug" please keep an option to use the name from the commiter as listed in the git commit message, rather than the gogs username.

So... what now?

How about another user-setting? Either show full name OR username

@bkcsoft pre-user?

Duplicated to https://github.com/gogits/gogs/issues/2923?

IMO yes :)

When you say "full name" you mean the name set in the git repo or some other name that is part of the user's data?
If this is going to be a thing, I would like an option to use the name set as the name in the git repo. Thanks.

Please follow on #2923


Device Details: H/S Module Card

Overview
This ticket outlines the frontend side of the heater shaker module card in device details.
Implementation details
The module card should be composed out of a generic ModuleCard component that can be shared by all other modules
Conditionally render the heater shaker module card inside of app/src/pages/Devices/DeviceDetails/index.tsx if there is/are heater module(s) attached.
Design
@howisthisnamenottakenyet please insert designs here when finalized
Acceptance criteria
Module card should display...

[ ] A rendering of the heater shaker module
[ ] The port the module is connected in
[ ] The heater shaker icon icon
[ ] The module's display name
[ ] The module's heater status (idle, heating, heated(??)) @anthonywilleee are these the correct statuses?
[ ] The modules target temperature (in Celcius)
[ ] The module current temperature (in Celcius)
[ ] The module's shaker status (idle, shaking)
[ ] The module's target RPM
[ ] The module's current RPM
[ ] Labware latch status (open or closed)
[ ] When clicking on the settings button, it should display 4 dropdown buttons that say Change temperature, Set shake speed, Close labware latch, and About module. For now, just console.log when each button is pressed

@shlokamin those are the correct heater statuses.  The dropdown menu may also have selection for set up and test shake, TBD based on user testing this week

Dup with 9288. Closing


APPLICATION ERROR #401 using announce plugin

Running MantisBT 2.13.1 with most up to date Announce plugin.
Just tested it. I added an options for each role as per the screenshot:

Steps to reproduce:

click the + column
add a rule for each user role
repeat step 1 & 2 for each role
When done press 'Edit' to finalize

I get this error:
APPLICATION ERROR #401

Database query failed. Error received from database was #1062: Duplicate entry '2-0-header' for key 'idx_plugin_announce_context' for the query: INSERT INTO mantis_plugin_Announce_context_table
(
message_id,
project_id,
location,
access,
ttl,
dismissable
) VALUES (
?,
?,
?,
?,
?,
?
).
Please use the "Back" button in your web browser to return to the previous page. There you can correct whatever problems were identified in this error or select another action. You can also click an option from the menu bar to go directly to a new section.

Edit:
This error also occurs when submitting more than 1 entry

I was able to reproduce this on my dev box. Will look into it.

@luzpaz there are 2 distinct issues here

The plugin should not crash with an SQL error due to user input
It was not designed to have multiple entries with distinct access levels for a single location

You probably misunderstood the purpose of the context concept, which was meant to allow display of the same announcement in different ways, even though at the moment only Page Header is implemented.
Your intended usage is therefore not supported, and I would even say not fixable without important modifications. Even if the unique key were changed to include the access level, it would still not work as you intend, because of the way this is handled by MantisBT: specifying e.g. Reporter always implies "or higher". So in your setup, a Developer would see all the announcements for Viewer, Reporter, Updater and Reporter (and would not be able to dismiss the first two...)
I will fix 1, but I'm afraid there is nothing I can do for 2 at the moment.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.10.46",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ba8a26a7ce8617f9f3d6230de34b2302df086b41"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ba8a26a7ce8617f9f3d6230de34b2302df086b41",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "regulator: rt4801: Fix NULL pointer dereference if priv->enable_gpios is NULL\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.46 by commit ba8a26a7ce8617f9f3d6230de34b2302df086b41. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1000994


Fix overrides syntax in useFilenamingConvention example

Adds the missing rules key
Summary
The overrides syntax has a rules key that was missing in this example.

Thank you @venables , but this document is auto-generated from source code


Error: Module version mismatch. Expected 48, got 47.

I installed slap using the command:
curl -sL https://raw.githubusercontent.com/slap-editor/slap/master/install.sh | sh
and when I run the command slap, this is what happened: http://pastebin.com/NQTj73Vx
It seems as if there is something wrong with my installation of nodejs. I don't know much about nodejs or javascript, I just wanted to use the slap-editor. What should I do to fix this problem?

Hi @paarthri,
This usually happens when you upgrade Node after installing slap. Try to reinstall slap for your current Node version:
$ npm i -g slap

I did that and it still does not work. Same output.

A simple update to node fixed the problem. Thanks

Glad to hear it @Paarthri. Thanks for the update!


feat: add public actor scrapers e2e tests

Added the tests currently present on the platform. There are some things I wanted to discuss/confirm:

Web scraper dev mode - it needs APIFY_CONTAINER_URL and APIFY_CONTAINER_PORT to start with dev mode on - I added localhost:8000 in order for the test to actually work. But locally there is kinda no container. Should we leave it like this? I am thinking maybe to add a condition that it will check whether test is running locally or not. Or maybe even simple process.env.APIFY_CONTAINER_PORT = process.env.APIFY_CONTAINER_PORT || '8000'; would work, what do you think?
Web scraper proxy rotation - it needs access to the proxy (i.e. Apify token/proxy password). How do you think it's better to handle it here? Just add the password/token for each npm run test:e2e call? As we could not simply hardcode it there.
ignore ssl errors enabled/disabled - for web scraper and puppeteer it works for different number of pages (when ignored -there are more successfully finished requests), while for cheerio this number is the same in both cases. Same on the platform. So basically test works the same locally, but I don't think it should be this way generally speaking, or am I missing something related to got-scraping?
Cheerio scraper xml content type - there's a test adding additionalMimeTypes: ['text/xml'], but I think it works the same with/without it - so this test does not make any sense really. I guess 'text/xml' type was added at some point as the type supported by default (not 100% about that). But if that's the case - we should probably ditch this test (or update it t some other content-type).

Otherwise all of them finish successfully.

@B4nan 1 and 2 are not relevant anymore (double-checked - tests pass, so loading the token from local auth file works just fine).


Fixing PIT flaky tests

Signed-off-by: Bharathwaj G<EMAIL_ADDRESS>Description
Fixing couple of flaky tests
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #4632 (e424cca) into main (ebbddd3) will decrease coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head e424cca differs from pull request most recent head 07b55c5. Consider uploading reports for the commit 07b55c5 to get more accurate results

@@             Coverage Diff              @@
##               main    #4632      +/-   ##
============================================
- Coverage     70.56%   70.54%   -0.02%
- Complexity    57516    57530      +14
============================================
  Files          4642     4642
  Lines        276786   276786
  Branches      40501    40501
============================================
- Hits         195311   195264      -47
- Misses        65107    65163      +56
+ Partials      16368    16359       -9

Impacted Files
Coverage Δ

...n/indices/forcemerge/ForceMergeRequestBuilder.java
0.00% <0.00%> (-75.00%)
:arrow_down:

...pensearch/indices/breaker/CircuitBreakerStats.java
27.77% <0.00%> (-41.67%)
:arrow_down:

...h/action/ingest/SimulateDocumentVerboseResult.java
60.71% <0.00%> (-39.29%)
:arrow_down:

...ations/bucket/terms/heuristic/PercentageScore.java
17.39% <0.00%> (-34.79%)
:arrow_down:

.../indices/forcemerge/TransportForceMergeAction.java
25.00% <0.00%> (-33.34%)
:arrow_down:

...opensearch/index/reindex/BulkByScrollResponse.java
48.38% <0.00%> (-29.04%)
:arrow_down:

.../search/aggregations/pipeline/HoltLinearModel.java
20.33% <0.00%> (-25.43%)
:arrow_down:

...java/org/opensearch/common/MacAddressProvider.java
40.62% <0.00%> (-25.00%)
:arrow_down:

...a/org/opensearch/index/mapper/MapperException.java
75.00% <0.00%> (-25.00%)
:arrow_down:

...ensearch/client/indices/DetailAnalyzeResponse.java
20.54% <0.00%> (-24.66%)
:arrow_down:

... and 461 more

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

In the description, please explain what makes the tests flaky and what changes are being made to fix the flakiness.

Use imperative mood while writing PR titles.

Thanks for fixing these tests.

@reta @Bukhtawar can you please review and merge ?


wasm: update benchmark to use url instead of path

Signed-off-by: Edoardo Vacchi<EMAIL_ADDRESS>Description
https://github.com/dapr/components-contrib/pull/2817/ updated to use url instead of path, however the companion benchmark was not update. Related: https://github.com/dapr/samples/pull/154
Issue reference
I haven't opened an issue since the change is trivial, but will do if requested.
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: (n/a)

thanks! Can you please check this one too? https://github.com/dapr/samples/pull/154


Create northeastern.txt

Adding top-level domain for Northeastern University (USA)

@david-m-rosen First stage of verification passed, official domain is northeastern.edu as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@david-m-rosen Pull request merged. Thank you.


go-sdk: Add some go sdk examples

OCP on RHV uses go sdk. These examples helps on
simulating the installer or debugging.
Signed-off-by: Douglas Schilling Landgraf<EMAIL_ADDRESS>

@gekorob @Gal-Zaidman @eslutsky

@dougsland , why do we need some  generic examples in this repo?
there are already examples in https://github.com/oVirt/ovirt-engine-sdk-go/tree/master/sdk/examples
we can use git submodule to link to the original sdk example , and to document how to use them in context of ocp-on-rhv installer.

@dougsland , why do we need some generic examples in this repo?
there are already examples in https://github.com/oVirt/ovirt-engine-sdk-go/tree/master/sdk/examples
we can use git submodule to link to the original sdk example , and to document how to use them in context of ocp-on-rhv installer.

@eslutsky great call. Agreed. Want me to do that?

done


Add suport for disallowing class properties

Hi, i would like to disallow class properties.
For example, in the following code, i wouls like the class property to be linted as an error :
    class MyClass {
        myProp = 5; // <- i would like this property to be linted as error or warning
        constructor() {
            // Do something
        }
    }

But it looks like "classProperties" plugin is harcoded in parser.js.
Am i wrong ? is there a way to ask babel-eslint to lint class properties ?
Regards,

In my case, it's the opposite. I want to enable the class properties, I installed the babel plugin but it reports all class properties as no-undef error.
package.json
{
  "devDependencies": {
    "@babel/cli": "^7.0.0-beta.51",
    "@babel/core": "^7.0.0-beta.51",
    "@babel/plugin-proposal-class-properties": "^7.0.0-beta.51",
    "babel-eslint": "^8.2.3",
    "eslint": "^4.19.1",
    "eslint-plugin-react": "^6.10.3",
  }
}

.babelrc
{
  "presets": [
    "@babel/preset-es2015",
    "@babel/preset-react",
    "@babel/preset-flow"
  ],
  "plugins": [
    "@babel/plugin-proposal-object-rest-spread",
    "@babel/plugin-proposal-class-properties",
    "@babel/plugin-proposal-export-default-from"
  ]
}

.eslintrc
{
  "parser": "babel-eslint",

  "parserOptions": {
    "ecmaVersion": 6,
    "sourceType": "module"
  },

  "env": {
    "browser": true,
    "node": true,
    "es6": true
  },

  "ecmaFeatures": {
    "classes": true
  }
}

I would like to continue using the Babel 7.

Thank you for the issue. Now that @babel/eslint-parser has been released, we are making this repository read-only. If this is a change you would still like to advocate for, please reopen this in the babel/babel monorepo.


Remove support for Identity v2 API as its deprecated

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind feature

What happened:
Identity v2 API support to be removed from provider as it is deprecated as of Queens release of Openstack.
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

openstack-cloud-controller-manager version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/assign

/remove-lifecycle-stale

/remove-lifecycle rotten

/remove-lifecycle stale

/remove-lifecycle rotten

As still some users still depend on v2 auth , https://github.com/kubernetes/cloud-provider-openstack/issues/803 , may be we should keep the support for the same for a while.
So closing the issue for now.
/close


docs(configuration): use --stats-colors

--colors is no longer valid. --stats-colors is the latest option.

We need fix in out CLI 😄

I think --stats-colors & --no-stats-colors works as expected,

but CLI logger does not respect stats.colors.red which it should but I am afraid stats.colors.red will not be accessible everywhere we use logger.

but CLI logger does not respect stats.colors.red which it should but I am afraid stats.colors.red will not be accessible everywhere we use logger.

ideally only --stats-colors should supports colors, other logger can ignore these, we need to add test


A feature request for access Kafka write & read metric counters

Hello,
I'm using K6 for testing Kafka clusters before migration plan.
And I need to be able to set a limit to the test based on one of the of the k6 Kafka metrics that are used in the summary report.
For example setting up threshold when kafka.writer.message,bytes = 1GB, enabling an access to these metrics will help me make it happened.

With the changes in #31 and the usage of the new k6 metrics API, you can use thresholds on the exported metrics, as shown below and in this commit.
https://github.com/mostafa/xk6-kafka/blob/e1a810d52112f05d7a66c12740d9885ebb64897e/scripts/test_json.js#L21-L27
So I consider this solved. Also, please keep an eye on #23 if you want to access the rest of the metrics published by kafka-go.


Allow to switch between lightwalletd servers

Is your feature request related to a problem? Please describe.
Right now the lightwalletd is hardcoded into the SDK but we want to let users of SDK switch between other servers or input a custom one.
Please implement:

a way how to setup several lightwalletd servers
a way how to switch between them
a way how to input a custom one

For the list of alternatives use (mainnet only!)

Location
Host
Port

North America
na.lightwalletd.com
443

South America
sa.lightwalletd.com
443

Europe & Africa
eu.lightwalletd.com
443

Asia & Oceania
ai.lightwalletd.com
443

Describe the solution you'd like
We'll possibly need to provide the network-type object through the persistable wallet object. In most cases, we use ZcashNetwork.fromResources(application) API to get the correct network object, which is a wrapper for the host and port setting. An important place to look is the LightWalletEndpointExt.kt component.
Additional context
There is a requirement to have this feature available to public API, so the wallet clients can leverage it. If we plan to allow server switching while we communicate with the previously set server, then we need to stop all related components and reset their state.

After examining the topic of changing the lightwalletd server via a client app more in detail, we should share this information:

A client app can change the lightwalletd server when establishing a connection with Synchronizer. Please, see the convenient way how to create a new Synchronizer. The important parameter is the lightWalletEndpoint: LightWalletEndpoint, which is a wrapper class for the host and port values.
The view-based Demo app uses this approach to create and use Synchronizer for communication to the lightwalletd server in SharedViewModel, although there is another way to create Synchronizer, and it's via WalletCoordinator which we use in the Secant wallet. We could consider tweaking the WalletCoordinator API to provide a way of setting the lightwalletd server in a parameter.
Then there one more way, which is even more customizable, and its via creating an instance of LightWalletClient, into which anyone can inject their own LightWalletEndpoint object.

I think 3rd option is good where we can inject any custom instance of LightWalletClient and  give option via any api to set in  WalletCoordinator.
Will it be requiring to restart the app to start sync again with new server, because now in Secant app we are persisting the WalletCoordinator?

Is this what you're referring to?
https://github.com/zcash/zcash-android-wallet-sdk/blob/ee28e4cedb2a8225c01f20f5e992614bae6de63a/sdk-lib/src/main/java/cash/z/ecc/android/sdk/model/LightWalletEndpointExt.kt#L29
If so, I believe that constant is only used by defaultForNetwork:
https://github.com/zcash/zcash-android-wallet-sdk/blob/ee28e4cedb2a8225c01f20f5e992614bae6de63a/sdk-lib/src/main/java/cash/z/ecc/android/sdk/model/LightWalletEndpointExt.kt#L11-L17
and that it is possible to just not use defaultForNetwork. Is there some other issue with that?

@daira Now we are using WalletCoordinator in app to get synchronizer and there we don't have a way to pass the custom LightWalletEndPoint. So if there is will be a way or api to pass the custom object from client with exiting implementation that will be good. I think that will be minimal changes at client and sdk both.


capture table DDL in meteor bigquery extractor

Is your feature request related to a problem? Please describe.
Meteor captures everything like table schema, column types, time partitioning field etc. But at the moment misses out on certain things like range partitioning or clustering options. But it would be nice to capture these along with capturing table DDL also.

#472 - feat(bigquery): add clustering fields and extra fields for partition


driver should trickle deals to controller

currently there's a fixed set of tasks. the controller on the testnet should get a dribble of ~5 tasks per days to keep it active

in #102 we now have a number of concurrent tasks able to be specified by a worker.


Bias in the accuracy

For figures except for the Wasserstein figure, the accuracy reaches 90% tops.

@shak360 you mentioned that the seed is set constant. Let's try changing that.


dataframe.with_column_renamed() does not rename properly

Describe the bug
The function Dataframe.with_column_renamed() cannot change uppercase single words to lower case words.
For instance,
df.with_column_renamed("Package", "package")?; // Produces "Package". Should be "package".
df.with_column_renamed("City Name", "city_name")?; // Produces "city_name". OK.

To Reproduce
I have the following CSV table

City Name
Package
Variety
Date
Low Price
High Price
Mostly Low

BALTIMORE
24 inch bins
HOWDEN TYPE
9/24/16
160
160
160

BALTIMORE
24 inch bins
HOWDEN TYPE
9/24/16
160
160
160

BALTIMORE
24 inch bins
HOWDEN TYPE
11/5/16
90
100
90

I load the CVS data with this code:
let mut config = SessionConfig::new();
 config = config.with_information_schema(true);

let csv_options = CsvReadOptions::new()
  .has_header(true)
  .schema_infer_max_records(2000);

let mut df = context.read_csv("data/pumpkins.csv", csv_options).await?;

Then, I try to change the column names from uppercase to lower case and replacing white spaces with underscores using this code:
let field_names: Vec<String> = df.schema().fields().iter().map(|f| f.name().to_string()).collect::<Vec<String>>();
println!("{:?}", field_names);

let re = Regex::new(r"\s+").unwrap();

for field in field_names {
  let new_field = field.clone().trim().to_lowercase();
  let new_field = re.replace_all(&new_field, " ");
  let new_field: String = new_field.to_lowercase().trim().replace(" ", "_");

  println!("{} -> {}", field, new_field);

  df = df.with_column_renamed(field, &new_field)?;
}

df = df.with_column_renamed("Package", "package")?;

let field_names: Vec<String> = df.schema().fields().iter().map(|f| f.name().to_string()).collect::<Vec<String>>();

println!("{:?}", field_names);

And I get this results:
["City Name", "Package", "Variety", "Date", "Low Price", "High Price", "Mostly Low"]
City Name -> city_name
Package -> package
Variety -> variety
Date -> date
Low Price -> low_price
High Price -> high_price
Mostly Low -> mostly_low
["city_name", "Package", "Variety", "Date", "low_price", "high_price", "mostly_low"]

As you can see, the function Dataframe.with_column_renamed() does not change the columns with an initial uppercase letter, even calling outside the for loop.
df = df.with_column_renamed("Package", "package")?;

Expected behavior
I would like the function changes the column name regardless of the case
df = df.with_column_renamed("Package", "package")?;      // produces ‘package’
df = df.with_column_renamed("PACKAGE", "package")?; // produces ‘package’

I had to use the format function and escaping quotation marks in the for loop in order to get my desired output from the Dataframe.with_column_renamed() function.
// (...)
for field in field_names {
  //(...)
  df = df.with_column_renamed(format!("\"{}\"", field), &new_field)?;
}
// (...)

The dataframe.with_column_renamed() should change the column name properly. In my case, I want his output:
["City Name", "Package", "Variety", "Date", "Low Price", "High Price", "Mostly Low"]
City Name -> city_name
Package -> package
Variety -> variety
Date -> date
Low Price -> low_price
High Price -> high_price
Mostly Low -> mostly_low
["city_name", "package", "variety", "date", "low_price", "high_price", "mostly_low"]

city_name
package
variety
date
low_price
high_price
mostly_low

BALTIMORE
24 inch bins
HOWDEN TYPE
9/24/16
160
160
160

BALTIMORE
24 inch bins
HOWDEN TYPE
9/24/16
160
160
160

BALTIMORE
24 inch bins
HOWDEN TYPE
11/5/16
90
100
90

I think it would be also useful to add a function to normalize all columns names to lower case and underscores
dataframe.to_lowercase() // produces ["city_name", "date", "low_price", "high_price"]
CsvReadOptions::new()::to_lowercase(true) // produces ["city_name", "date", "low_price", "high_price"]

Additional context
No response

Hi @edandresvan
you can achieve the same with SQL API
❯ select 1 as "Package";
+---------+
| Package |
+---------+
| 1       |
+---------+

As for Dataframe API you I have checked how Spark does and it renames the column as per your suggestion
scala> sql("select 1 a").withColumnRenamed("a", "Packe").printSchema
root
 |-- Packe: integer (nullable = false)

Seems root cause is how when searching for the column to rename, Package, you need to ensure the string you passed in is properly quoted as noted in the original issue:
.with_column_renamed("\"Package\"", "package")

Since otherwise it'll look for column package which doesn't exist.
https://github.com/apache/arrow-datafusion/blob/d6985b0fc1e4362b80beea059857c6bde65b4d04/datafusion/core/src/dataframe.rs#L1015-L1029
Maybe make the search for the field on line 1022, field_from_column(...) more smarter by allowing to do case insensitive match as fallback if can't find an exact match?

     #[tokio::test]
    async fn with_column_renamed_case_sensitive() -> Result<()> {
        let df = test_table()
            .await?
            .filter(col("c2").eq(lit(3)).and(col("c1").eq(lit("a"))))?
            .limit(0, Some(1))?
            .sort(vec![
                // make the test deterministic
                col("c1").sort(true, true),
                col("c2").sort(true, true),
                col("c3").sort(true, true),
            ])?
            .select_columns(&["c1"])?;

        let df_renamed = df
            .clone()
            .with_column_renamed("c1", "CoLuMn1")?
            .collect()
            .await?;

        assert_batches_sorted_eq!(
            vec![
                "+---------+",
                "| CoLuMn1 |",
                "+---------+",
                "| a       |",
                "+---------+",
            ],
            &df_renamed
        );

        let df_renamed = df
        .with_column_renamed("CoLuMn1", "c1")?
        .collect()
        .await?;

        assert_batches_sorted_eq!(
            vec![
                "+----+",
                "| c1 |",
                "+----+",
                "| a  |",
                "+----+",
            ],
            &df_renamed
        );

        Ok(())
    }

Looks like it works on latest DF. Please verify @edandresvan

@comphead I think your example is flawed as you're using the original DF at the end which doesn't have a column named CoLuMn1, instead of the renamed one

@Jefffrey ouch, you are right. I'm adding valid test case
 #[tokio::test]
    async fn with_column_renamed_case_sensitive() -> Result<()> {
        let df = test_table()
            .await?
            .filter(col("c2").eq(lit(3)).and(col("c1").eq(lit("a"))))?
            .limit(0, Some(1))?
            .sort(vec![
                // make the test deterministic
                col("c1").sort(true, true),
                col("c2").sort(true, true),
                col("c3").sort(true, true),
            ])?
            .select_columns(&["c1"])?;

        let df_renamed = df
            .clone()
            .with_column_renamed("c1", "CoLuMn1")?;

        let res = &df_renamed.clone().collect().await?;

        assert_batches_sorted_eq!(
            vec![
            "+---------+",
            "| CoLuMn1 |",
            "+---------+",
            "| a       |",
            "+---------+",
            ],
            res
        );

        let df_renamed = df_renamed
        .with_column_renamed("CoLuMn1", "c1")?
        .collect()
        .await?;

        assert_batches_sorted_eq!(
            vec![
                "+----+",
                "| c1 |",
                "+----+",
                "| a  |",
                "+----+",
            ],
            &df_renamed
        );

        Ok(())
    }

So as I confused everyone I will try to fix the issue then :)

@Jefffrey DF works correctly with quota style scenario
df.withColumnRenamed("c1", "`CoLuMn1`").withColumnRenamed("`CoLuMn1`", "c1")

Do we really want to make it the same behaviuor as spark? this may be confusing for the system

@comphead
Yeah I believe it's because Spark's query analyzer is case insensitive by default:
https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala#L889-L895
So selecting with a column name of wrong case will still find correct column.
Whereas for DataFusion, I guess for now it is case sensitive. Could introduce config like Spark does to toggle the case sensitivity for analyzer/planner, but that would require more work.

Whereas for DataFusion, I guess for now it is case sensitive. Could introduce config like Spark does to toggle the case sensitivity for analyzer/planner, but that would require more work.

https://arrow.apache.org/datafusion/user-guide/configs.html
This one might work:

datafusion.sql_parser.enable_ident_normalization
true
When set to true, SQL parser will normalize ident (convert ident to lowercase when not quoted)


Add in release github action

Added release document explaining how releases can be done
Github Action that will append the binaries to a release created via the github UI

There are many ways of doing releases, but this is a simple yet effective approach that is used in other Fabric related projects.
Signed-off-by: Matthew B White<EMAIL_ADDRESS>

@ale-linux thanks for the comments... I've update the text based on both.

Would it not make more sense to move the steps that generates binaries and create a release to a separate workflow that is triggered tag pushes or by dispatch? Then you have the original workflow that runs checks / tests on each push / PR and another workflow that can create binaries and release when needed. I'm not sure I understand why we would want to generate binaries on each push to main.

Would it not make more sense to move the steps that generate binaries and create a release to a separate workflow that is triggered tag pushes or by dispatch? Then you have the original workflow that runs checks / tests on each push / PR and another workflow that can create binaries and release when needed.

It's perfectly reasonable to approach this way, and potentially with a larger and complex workflow it would make even more sense. There is the question of how much should be shared between them. i.e. a release workflow may well wish to re run tests to ensure that the binaries are ok. With a simple workflow like this I think the arguments are more finally balanced.

I'm not sure I understand why we would want to generate binaries on each push to main.
Well ideally the repo should have branch protection to stop anything being pushed to main, save for when the PRs are merged.
As is the case on other Fabric related repos.

Some CI systems actually need to have the push to main for other reasons - don't think github actions is one though unless @jt-nti knows otherwise (the approach I'm using is copied from the approach he's used on other fabric repos)

I see, if this is how the repo intends to handle releases then everything LGTM.

I did some testing with the softprops/action-gh-release action and it is using a deprecated command which causes a warning during the workflow run. The action works currently but GitHub says the command will soon be unsupported so I recommend switching over to the ncipollo/release-action as it has offers the same functionality but doesn't generate any warnings.

On the subject of a separate workflow, keeping as much as the release process in the main workflow as possible is useful to avoid (as many) surprises during a release build. (You could still have a separate workflow and have it run on pushes as well, but I'm not sure that's worth it for a simple project.)


After org name changed and build, still shows errors

Description:
When I change the org name during development still showing compilation issue even build is success

Can you try reloading the VSCode window? Currently toml changes are not reflected in Language server

Can you try reloading the VSCode window? Currently toml changes are not reflected in Language server

Yes .After reloading it worked.

Yes .After reloading it worked.


CookiesInterface unnecessary?

Since the Http\Cookies only contains static helper methods, why is there still an interface defined? The class does not even implement the interface. (see https://github.com/slimphp/Slim/blob/develop/Slim/Http/Cookies.php#L14)

I am plus one for dropping cookies out of core and putting them as middleware. Seems to have been following the trend ie sessions and lots of other things

Agreed on both counts. @geggleto I'm unsure if middleware or a service provider (or both?) would be more appropriate. What would your proposed middleware look like?

Actually I think your suggestion might be better. A service provider to parse the header and then middleware to interact with it. For the interaction we can use an interface so you can develop your own and plug in play for additional functionality

Right. So the service provider could provider Request Cookie parsing and Response Set-Cookie serialization, while the middleware is where these methods would be used.
What is the workflow for a developer to add a cookie to the Response? What does that look like, and where does that happen?

Bear in mind we should limit ourselves to PSR7 message interfaces.

Is there even any allowances for this via the psr7 interface? I am not up to speed on it yet. But it would seem that, our choices are going to depend on filling that requirement first .

Read this https://groups.google.com/forum/#!topic/php-fig/6zesbZWLY_Y

The clean solution appears to be this: creating new cookies means storing the cookie data in a separate, stand-alone object (via service provider). This object is in no way contained by or related to the response object. I imagine a shared Pimple service that looks like this:
$app['cookies']->set('name', ['value' => 'foo', 'domain' => 'example.com', 'expires' => '3 days']);

The middleware layer is responsible for 1) unserializing the Request Cookie header, and 2) serializing the cookie data from the service provider above into HTTP Set-Cookie headers. I don't like how this is separated from the Response object; it feels wrong, but I think it's the best solution.
Aside: For middleware and service providers that work together like this, do you still prefer adding them to the application separately (e.g., $app->add(middleware) and $app->register(provider)) or is there a way to add them both at once?

Disregard that last aside. I think we'll register the provider in the App constructor, and we may not even need a stand-alone middleware. We can instead deserialize cookies in the Request (for its getCookieParams() method), and serialize cookies in the App::run() method.

Reading that thread gave me chills. The whole point of a framework is to make things easier. I don't see how rawly encoding cookies into header fields does that.
Hmm I think this might be getting out of hand a bit. I am going to bow out for now so I can think on it having 2 more objects seems excessive and not very clean especially given the coupling they are likely to have.

Right. That's the predicament :( I'll think on this too. What is available now in the develop branch is definitely not final.

Another option is to just re-introduce first-class methods on the App class itself. App::getCookie() uses Request object's PSR7 methods to fetch a cookie value. App::setCookie() uses the built-in provider (hidden from end user) to store cookies that later serialized onto the Response object (again hidden from end user).

+1 for that last suggestion.

As someone who doesn't use cookies at all, I'd prefer that they were handled by a helper class. However, if they really are in the 80%-use case, then App methods are fine.

I personally don't use cookies at all, however I know there are valid use cases. I don't think they will be around forever so having them somewhat optional would be beneficial.

Pushed up some changes here https://github.com/slimphp/Slim/commit/fa6f24b88bdf8af4ae6a0d705d3807181d23f599
Usage:
$app->get('/set', function ($req, $res, $args) {
    $this['cookies']->set('foo', [
        'value' => 'bar',
        'domain' => 'example.com',
        'path' => '/sub',
        'expires' => '4 days'
    ]);
});

Or you can just do:
$app->get('/set', function ($req, $res, $args) {
    $this['cookies']->set('foo', 'bar');
});

Maybe I'm wrong, but the current implementation has 2 issues for me:

In the current state of this project, we can send and receive multiple request/response from the same app instance ( while testing for exemple or react the day they will implement psr7  ).
The only thing that will be share between the request-response is the cookie service.

if a middleware adds token by its own this will erase the cookie it may have set

I'm really not an expert but I don't understand why this is cleanest than the withCookie that was in the Response class, if anyone want to enlighten me he's welcome.

So the two best solutions are:

Keeping Response::withCookie($name, $props) as @lalop suggested. This keeps Cookie concerns with the Response object. However, this must immediately serialize the cookie into a Set-Cookie header.
Use the cookie service. This decouples cookies from the HTTP messages, and it lets a Slim app continue to have cookie management tools even if the PSR7 message objects are swapped out with an alternative implementation.

I'm honestly fine with either. Neither is perfect. What are everyone's thoughts?

In my opinion, the non-standard withCookie()method is cleaner. However, the cookies service is guaranteed to work regardless of PSR7 implementation. I understand the semantic argument against it, but I'm going to keep that solution.

Hi,
that 's fine for me, I wanted to point it.
I've seen that the same method is used on the httCache middleware so that can be a kind of convention.


got "control reaches end of non-void function -werror=return-type" when compiling GEM-5

I ran python2 /usr/bin/scons build/X86/gem5.opt -j5 in extern/network_backend/garnet/gem5_astra, and got an error message got control reaches end of non-void function -werror=return-type.
Finally I found that function Sys::get_priority(SchedulingPolicy pref_scheduling)  in extern/network_backend/garnet/gem5_astra/build/X86/mem/ruby/network/garnet2.0/astra-sim/system/Sys.cc ended without else return, so I changed it as:
int Sys::get_priority(SchedulingPolicy pref_scheduling) {
  if (pref_scheduling == SchedulingPolicy::LIFO) {
    return priority_counter++;
  } else if (pref_scheduling == SchedulingPolicy::FIFO) {
    return priority_counter--;
  } else if (pref_scheduling == SchedulingPolicy::HIGHEST) {
    return 100000000;
  } else if (pref_scheduling == SchedulingPolicy::None) {
    if (scheduling_policy == SchedulingPolicy::LIFO) {
      return priority_counter++;
    } else {
      return priority_counter--;
    }
    // added by Zhenlong Ma
  } else {
    return -1;
  }
}

I don't know if this will bring unexpected effect, please figure it out if I made it wrong, thanks!

Hi @ZhenlongMa ,
Thanks for your comment. In reality, it never reaches that else condition you added. So, the code runs as expected whether or not you add the else statement. We will fix this in the next update


Incorporate project descriptions and preliminary results from initial scientific demonstration explorations into the manuscript draft

Target date: July 31

Closing per conversation with Amanda and Bailey.


chore: update workflow nodejs version

Summary
https://app.clickup.com/t/86dtc32dt

Checklist

[ ] I checked my UI changes against the design and there are no notable differences
[ ] I checked my UI changes for any responsiveness issues
[ ] I checked my UI changes in light AND dark mode
[ ] I checked my (code) changes for obvious issues, debug statements and commented code
[ ] I opened a corresponding card on Clickup for any remaining TODOs in my code
[ ] I added a short description on how to test this PR (if necessary)
[ ] Documentation (if necessary)
[ ] Tests (if necessary)
[ ] Ready to be merged

Should be read now @ItsANameToo. I've added a separate card to handle the SVGO action as that's no longer maintained (as of tomorrow)


EKS Managed Node group IAM Role and instance profile have different names

Description

When we use terraform-aws-eks module for creating the cluster and eks managed node groups we use IAM Role for node groups as well.
But the Instance profile name and IAM role name  for node groups are different?
Is there a way we can achieve the same name?

If your request is for a new feature, please use the Feature request template.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]:

Terraform version:

Provider version(s):

Reproduction Code [Required]

Steps to reproduce the behavior:

Expected behavior

Actual behavior

Terminal Output Screenshot(s)

Additional context

EKS creates the instance profile with managed nodegroups so it's not controlled in this module


I can't pass options "margin"

Hello,
I've backup my server in new server Debian 11
Natively I've install last paquet of wkhtmltopdf and update my composer mikehaertl\wkhtmlto\Pdf
But when I run my script to generate PDF, PHP return an error :
--margin-bottom specified in incorrect location

I've remove and re-install a previous version of wkhtmltopdf but I've a same problem.
my version : wkhtmltopdf <IP_ADDRESS> (with patched qt)
my script :
$commandOptions = array(
//    'ignoreWarnings' => true,
    'disable-smart-shrinking',
    'enable-local-file-access' => true,
    'encoding' => 'UTF-8',
    'margin-bottom' => '10in',
    'margin-top' => '40in',
    'header-spacing' => 10,
    'user-style-sheet' => 'header.css',
    'header-html' => $header,
    'footer-html' => $footer
);
$pdf = new Pdf($commandOptions);
$pdf->setOptions($commandOptions);
// Add a page. To override above page defaults, you could add
// another $options array as second argument.
$pdf->getCommand();
$pdf->addPage($content, $commandOptions);
//    $pdf->send();
if (!$pdf->send()) {
    $error = $pdf->getError();
    // ... handle error here
    print $error;
}

Can you help me please ?

The error is created by wkhtmltopdf itself so we don't have much control here. --margin-bottom should actually be a global option but can you verify with wkhtmltopdf -H if this is really the case for your version?
You can also check the created command with print $pdf->getCommand()->getExecCommand() right before $pdf->send() (maybe comment that out).

Hello,
Thank's for yr comment, but not work.
I've try directly from bash :
root@sw1:/# wkhtmltopdf --margin-left 0 --margin-right 0 --margin-top 0 --javascript-delay 2000 http://google.fr /tmp/google.pdf
Loading pages (1/6)
Counting pages (2/6)
Resolving links (4/6)
Loading headers and footers (5/6)
Printing pages (6/6)
Done
root@sw1:/# wkhtmltopdf --margin-bottom 25 --margin-left 0 --margin-right 0 --margin-top 10 --javascript-delay 2000 http://google.fr /tmp/google.pdf
Loading pages (1/6)
Counting pages (2/6)
Resolving links (4/6)
Loading headers and footers (5/6)
Printing pages (6/6)
Done

Work Perfectly...
I've try this configuration, with yr recommendation :
$pdf = new Pdf($commandOptions);

$pdf->getCommand()->getExecCommand();

$pdf->setOptions($commandOptions);

// Add a page. To override above page defaults, you could add
// another $options array as second argument.
$pdf->addPage($content, $commandOptions);

//    $pdf->send();
if (!$pdf->send()) {
    $error = $pdf->getError();
    // ... handle error here
    print $error;
}

not work :-(
in Extended help I've this option...

Please check my comment again: You should print out the generated command, so that we can compare it:
// Put this instead of $pdf->send()
print $pdf->getCommand()->getExecCommand()

Oh excuses me !
Please see the result :
wkhtmltopdf '--allow' '/var/www/html/*****.fr/panel/manager/rq/***_template' '--encoding' 'UTF-8' '--margin-right' '0' '--margin-left' '0' '--margin-bottom' '10' '--margin-top' '40' '--header-spacing' '10' '--user-style-sheet' 'header.css' '--header-html' '/tmp/tmp_wkhtmlto_pdf_EHksHU.html' '--footer-html' '/tmp/tmp_wkhtmlto_pdf_ilM7kU.html' '/tmp/tmp_wkhtmlto_pdf_upaZTT.html'

Well, this is the command that is executed by our library.
You can try if this works on the command line - just replace the filenames with some real files (they are temporary files, that are automatically created).

yes it's works perfectly :
root@sw1:/var/www/html/****.fr/panel/manager/rq/****_template# wkhtmltopdf '--allow' '/var/www/html/*****.fr/panel/manager/rq/***_template' '--encoding' 'UTF-8' '--margin-right' '0' '--margin-left' '0' '--margin-bottom' '10' '--margin-top' '40' '--header-spacing' '10' '--user-style-sheet' 'header.css' '--header-html' 'Fact_Op.php' '--footer-html' 'Fact_Op.php' --load-error-handling 'ignore' hello.html test.pdf
Loading pages (1/6)
Counting pages (2/6)
Resolving links (4/6)
Loading headers and footers (5/6)
Warning: Failed loading page file:///var/www/html/****.fr/panel/manager/rq/*****_template/****_Op.php?page=1&section=&sitepage=1&title=&subsection=&frompage=1&subsubsection=&topage=1&doctitle=&sitepages=1&webpage=hello.html&time=3:55 PM&date=10/26/22 (ignored)
Warning: Failed loading page file:///var/www/html/*****.fr/panel/manager/rq/***_template/*****_Op.php?page=1&section=&sitepage=1&title=&subsection=&frompage=1&subsubsection=&topage=1&doctitle=&sitepages=1&webpage=hello.html&time=3:55 PM&date=10/26/22 (ignored)
Printing pages (6/6)
Done

I've just add --load-error-handling because my html file isn't completed.

If is important for you :
If I not use my margin options my document generated is bad.
BUT
I've Solved my problem 👍
Please see my configuration :
$commandOptions = array(
    'allow' => __DIR__,
    'encoding' => 'UTF-8',
    'header-spacing' => '5',
    'user-style-sheet' => 'header.css',
    'header-html' => $header,
    'footer-html' => $footer
);

$marginOptions = array(
    'margin-bottom' => 10,
    'margin-top' => 40
);

$pdf = new Pdf($commandOptions);
$pdf->setOptions($marginOptions);

$pdf->getCommand()->getExecCommand();

$pdf->addPage($content, $commandOptions);

if($_POST["send_mail"] == 0){
    if (!$pdf->send()) {
        $error = $pdf->getError();
        print $error;
    }
}

I generate my "html" options in setOptions not in the constructor New pdf();
wkhtmltopdf '--allow' '/var/www/html/.fr/panel/manager/rq/_template' '--encoding' 'UTF-8' '--header-spacing' '5' '--user-style-sheet' 'header.css' '--header-html' '/tmp/tmp_wkhtmlto_pdf_KjLvbe.html' '--footer-html' '/tmp/tmp_wkhtmlto_pdf_v6bFnf.html' '--margin-top' '40' '/tmp/tmp_wkhtmlto_pdf_Y2jyCf.html'

In the generated command my option is displayed just before the stdout and work perfectly.
I try with wkhtmltox <IP_ADDRESS>-2.buster : 🆗
And i've update to last version wkhtmltox <IP_ADDRESS>-2.bullseye : 🆗
Thank's for your help !

Alright, glad you solved it. I'll close this issue then.


Scan with failed status does not fire a AlertOnFailure alert

On 2.5-head - latest

Configure Monitoring and alerting on a cluster
Deploy CIS v2 app
Create a scan CR.
Enable AlertOnFailure alerts
Create a CIS permissive scan run and give in Scan state for "warn" results: Fail
The scan runs and the state of the scan is "Fail"
But no alerts are triggered.

Can we trigger alerts for "Failed" scans also ? Those that have no "Failed" tests but have a "failed" scan state?

The AlertOnFailure is not tied with the "failed" status but with "non-zero" test failures. Since there were no tests that failed, there will not be an alert which is triggered only when number of tests failed is non-zero.
Currently the alerts are tied with prometheus metrics that are exported by cis-operator. The operator increments the "cis_scan_num_tests_fail" metric when there are test failures which drives alertmanager to fire alert.
If we want to also drive alerts if there are tests with "warn" status, we need to add another metric and an option AlertOnWarn
cc @deniseschannon does this sound like a new addition we need?

@deniseschannon confirmed that we should send out an alert for AlertOnFailure as user has indicated that warn = fail

On 2.5-head - commit id: baf4867 with latest cis v2 charts: rancher-cis-benchmark:1.0.300

Create a CIS v2 scan CR with Alert on for "failures" and select rke-permissive cis 1.5 scan profile with "warn" as fail
Alerts are fired.


Uncaught TypeError: Cannot read property 'length' of undefined

[Enter steps to reproduce:]

...
...

Atom: 1.34.0 x64
Electron: 2.0.16
OS: Mac OS X 10.14.3
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'length' of undefined
At /Applications/Atom for cpp.app/Contents/Resources/app/static/<embedded>:14

TypeError: Cannot read property 'length' of undefined
    at ScreenLineBuilder.emitText (/Applications/Atom for cpp.app/Contents/Resources/app/static/<embedded>:14:1103304)
    at ScreenLineBuilder.buildScreenLines (/Applications/Atom for cpp.app/Contents/Resources/app/static/<embedded>:14:1099010)
    at DisplayLayer.getScreenLines (/Applications/Atom for cpp.app/Contents/Resources/app/static/<embedded>:14:89306)
    at TextEditorComponent.queryScreenLinesToRender (/Applications/Atom for cpp.app/Contents/Resources/app/static/<embedded>:11:79324)
    at TextEditorComponent.updateSyncBeforeMeasuringContent (/Applications/Atom for cpp.app/Contents/Resources/app/static/<embedded>:11:69977)
    at TextEditorComponent.updateSync (/Applications/Atom for cpp.app/Contents/Resources/app/static/<embedded>:11:67605)
    at TextEditorComponent.didMouseWheel (/Applications/Atom for cpp.app/Contents/Resources/app/static/<embedded>:11:92354)

Commands
     -1:02.3.0 core:copy (atom-notification.fatal.icon.icon-bug.native-key-bindings.has-detail.has-close.has-stack)

Non-Core Packages
ask-stack 2.2.0
atom-beautify 0.33.4
atom-material-ui 2.1.3
atom-prettify 0.1.3
atomic-chrome 0.3.3
auto-update-packages 1.0.1
autocomplete-paths 2.12.2
browser-plus 0.0.98
busy-signal 2.0.0
color-picker 2.3.0
emmet 2.4.3
expose 0.15.0
file-icons 2.1.30
git-plus 8.6.1
git-time-machine 2.1.0
gpp-compiler 3.0.7
highlight-selected 0.14.0
intentions 1.1.5
language-liquid 0.7.0
linter 2.3.0
linter-eslint 8.5.2
linter-rubocop 2.2.4
linter-scss-lint 3.2.1
linter-ui-default 1.7.1
minimap 4.29.9
open-in-browsers 0.0.30
open-recent 5.0.0
pane-layout-plus 0.7.4
pdf-view 0.71.0
pigments 0.40.2
project-manager 3.3.6
remote-edit 1.9.0
sync-settings 0.8.6
tabs-to-spaces 1.0.5
terminal-plus 0.14.5
vim-mode 0.66.0
web-view 3.0.0

Thanks for the report!  Can you confirm a few things for us?

What were you doing at the time of the error?
Is the error is reproducible?
If reproducible, do you see the error in safe mode (atom --safe)?

x-ref: https://github.com/atom/atom/issues/16892

Got the same issue with atom on ubuntu 18.04

This issue might be occurring due to the length of the file, smaller files have no issues.My file is around 500 lines. I stand to be corrected on this though.

i think so too, i was working on a file with 400 line but the moment i went over 500 i had this problem, which is very confusing because 500 is not that many line.

I am facing the same issue in one file, but i guess its nothing because of the number of lines, all my files exceed 500 lines and even 600 , and the issue happens only in one file. Not sure whats the problem.

Thanks for the extra details - does anyone have specific steps to reproduce along with a sample file?  And can you reproduce in safe mode?

I don't believe number of lines is the issue. I have a file with less than 200 lines triggering the error. The error pops up when I try to scroll:
TypeError: Cannot read property 'length' of undefined
    at ScreenLineBuilder.emitText (C:\Users\abell\AppData\Local\atom\app-1.35.1\resources\app\static\<embedded>:14:1104122)
    at ScreenLineBuilder.buildScreenLines (C:\Users\abell\AppData\Local\atom\app-1.35.1\resources\app\static\<embedded>:14:1099828)
    at DisplayLayer.getScreenLines (C:\Users\abell\AppData\Local\atom\app-1.35.1\resources\app\static\<embedded>:14:89305)
    at TextEditorComponent.queryScreenLinesToRender (C:\Users\abell\AppData\Local\atom\app-1.35.1\resources\app\static\<embedded>:11:79324)
    at TextEditorComponent.updateSyncBeforeMeasuringContent (C:\Users\abell\AppData\Local\atom\app-1.35.1\resources\app\static\<embedded>:11:69977)
    at TextEditorComponent.updateSync (C:\Users\abell\AppData\Local\atom\app-1.35.1\resources\app\static\<embedded>:11:67605)
    at TextEditorComponent.didMouseWheel (C:\Users\abell\AppData\Local\atom\app-1.35.1\resources\app\static\<embedded>:11:92354)

Makes editing the file impossible. I guess maybe I'll finally switch to Emacs.

It appeared for me when did a search in a directory and opened a file from the "Project Find Results" pane.  However, when I open the same file from project directory pane no errors occur.

I've been having this problem too, however, I've managed to overtake it by closing the file and reopening it. After 24 hours I continue without problems.

11

[Enter steps to reproduce:]

after install minimap 4.29.9
open this files:

https://github.com/shellphish/rex/blob/master/rex/crash.py
https://github.com/shellphish/rex/blob/master/tests/test_rex.py

Atom: 1.32.2 x64
Electron: 2.0.9
OS: Mac OS X 10.14.3
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'length' of undefined
At /Applications/Atom.app/Contents/Resources/app/static/<embedded>:14

TypeError: Cannot read property 'length' of undefined
    at ScreenLineBuilder.emitText (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1108203)
    at ScreenLineBuilder.buildScreenLines (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1103913)
    at DisplayLayer.getScreenLines (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:89276)
    at TextEditorComponent.queryScreenLinesToRender (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:79025)
    at TextEditorComponent.updateSyncBeforeMeasuringContent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:69678)
    at TextEditorComponent.updateSync (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:67306)
    at TextEditorComponent.didShow (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:90936)
    at TextEditorComponent.didFocus (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:91127)
    at HTMLElement.addEventListener (/app.asar/src/text-editor-element.js:36:67)
    at HTMLElement.activeItemChanged (/app.asar/src/pane-element.js:164:16)
    at Function.simpleDispatch (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1186984)
    at Emitter.emit (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1188425)
    at Pane.setActiveItem (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:452700)
    at Pane.activateItemAtIndex (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:454751)
    at Pane.activatePreviousItem (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:454248)
    at Pane.removeItem (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:456970)
    at Pane.destroyItem (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:458255)
    at <anonymous>

Non-Core Packages
atom-beautify 0.33.4
autocomplete-python 1.14.0
minimap 4.29.9
vim-mode-plus 1.36.0

Hi, I have the same issue. It seems to have to do something with the network drive.
The share I was working on is mapped to U:. When I browse to U:\Operations\Position and open position.py it gives this error. When I got to \\<domain>\shares\<project>\Operations\Position and open position.py it doesn't give the error. This behavior is repeatable.
Atom: 1.35.1 x64
Electron: 2.0.18
OS: Microsoft Windows 10 Enterprise
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'length' of undefined
At C:\Users\timmer\AppData\Local\atom\app-1.35.1\resources\app\static\<embedded>:14

TypeError: Cannot read property 'length' of undefined
    at ScreenLineBuilder.emitText (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:14:1104122)
    at ScreenLineBuilder.buildScreenLines (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:14:1099828)
    at DisplayLayer.getScreenLines (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:14:89305)
    at TextEditorComponent.queryScreenLinesToRender (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:11:79324)
    at TextEditorComponent.updateSyncBeforeMeasuringContent (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:11:69977)
    at TextEditorComponent.updateSync (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:11:67605)
    at visible.suppressUpdates.updatedSynchronously.updateScheduled.i.getScheduler.updateDocument (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:11:66633)
    at ViewRegistry.performDocumentUpdate (~/AppData/Local/atom/app-1.35.1/resources/app/static/<embedded>:11:326599)

Non-Core Packages
atom-runner 2.7.1
city-lights-syntax 1.1.8
city-lights-ui 1.5.3

hi, Since yesterday I have the same problem with a file. Is there any news about it? Anyway, I love this tool, but that problem generates a serious problem in my workflow.


cmd/compile: relocation target ... not defined while building targets for kubernetes with go tip

What version of Go are you using (go version)?

$ go version
go version devel +e25040d162 Fri Feb 26 02:52:33 2021 +0000 linux/ppc64le

What operating system and processor architecture are you using (go env)?
Operating system: Linux
Architecture: ppc64le and amd64
What did you do?
Building targets for kubernetes using the latest golang in main branch

What did you expect to see?
Kubernetes binaries build without error
What did you see instead?
Below error:
-bash: build-log.txt: command not found
^C[root@rajalakshmi-workspace kubernetes]# make cross 2>&1 | tee build-log.txt
+++ [0226 08:39:26] Building go targets for linux/ppc64le:
    ./vendor/k8s.io/code-generator/cmd/prerelease-lifecycle-gen
Generating prerelease lifecycle code for 24 targets
+++ [0226 08:39:32] Building go targets for linux/ppc64le:
    ./vendor/k8s.io/code-generator/cmd/deepcopy-gen
Generating deepcopy code for 226 targets
+++ [0226 08:39:42] Building go targets for linux/ppc64le:
    ./vendor/k8s.io/code-generator/cmd/defaulter-gen
Generating defaulter code for 88 targets
+++ [0226 08:39:56] Building go targets for linux/ppc64le:
    ./vendor/k8s.io/code-generator/cmd/conversion-gen
Generating conversion code for 123 targets
+++ [0226 08:40:23] Building go targets for linux/ppc64le:
    ./vendor/k8s.io/kube-openapi/cmd/openapi-gen
Generating openapi code for KUBE
Generating openapi code for AGGREGATOR
Generating openapi code for APIEXTENSIONS
Generating openapi code for CODEGEN
Generating openapi code for SAMPLEAPISERVER
+++ [0226 08:40:43] Building go targets for linux/ppc64le:
    ./vendor/github.com/go-bindata/go-bindata/go-bindata
+++ [0226 08:40:45] Building go targets for linux/ppc64le:
    cmd/kube-proxy
    cmd/kube-apiserver
    cmd/kube-controller-manager
    cmd/kubelet
    cmd/kubeadm
    cmd/kube-scheduler
    vendor/k8s.io/kube-aggregator
    vendor/k8s.io/apiextensions-apiserver
    cluster/gce/gci/mounter
+++ [0226 08:46:53] Building go targets for linux/ppc64le:
    cmd/kube-proxy
    cmd/kubeadm
    cmd/kubelet
+++ [0226 08:47:19] Building go targets for linux/ppc64le:
    cmd/kubectl
    cmd/kubectl-convert
+++ [0226 08:47:49] Building go targets for linux/ppc64le:
    cmd/gendocs
    cmd/genkubedocs
    cmd/genman
    cmd/genyaml
    cmd/genswaggertypedocs
    cmd/linkcheck
    vendor/github.com/onsi/ginkgo/ginkgo
    test/e2e/e2e.test
    cluster/images/conformance/go-runner
+++ [0226 08:51:42] Building go targets for linux/ppc64le:
    cmd/kubemark
    vendor/github.com/onsi/ginkgo/ginkgo
    test/e2e_node/e2e_node.test
# k8s.io/kubernetes/test/e2e_node.test
k8s.io/kubernetes/vendor/go.etcd.io/etcd/etcdserver/api/v3client.New.func1·f: relocation target k8s.io/kubernetes/vendor/go.etcd.io/etcd/etcdserver/api/v3client.New.func1 not defined
!!! [0226 08:52:30] Call tree:
!!! [0226 08:52:30]  1: /root/kubernetes/hack/lib/golang.sh:875 kube::golang::build_binaries_for_platform(...)
!!! [0226 08:52:30]  2: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)
!!! [0226 08:52:30] Call tree:
!!! [0226 08:52:30]  1: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)
!!! [0226 08:52:30] Call tree:
!!! [0226 08:52:30]  1: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)
make[1]: *** [Makefile:92: all] Error 1
make: *** [Makefile:508: cross] Error 1
[root@rajalakshmi-workspace kubernetes]#

@mkumatag @laboger ^^

@pmur

Simple test case:
bar/b.go:
package bar

type Bar struct {
        F func ()
}

func BuildBar() Bar {
        ret := Bar{ F: func() {} }
        return ret
}

main.go:
package main

import (
        "fmt"
        "example.com/local/foo"
)

func main() {
        fmt.Printf("%v\n",foo.New(1))
}

foo/a.go:
package foo

import (
        "example.com/local/bar"
)

type Foo struct {
        b bar.Bar
}

func New(a int) Foo {
        return Foo{b: bar.BuildBar() }
}

@pmur Your test case builds for me with GOARCH=ppc64le go build.  How are you building it, and what error do you see?  Thanks.

@ianlancetaylor Here are the other details I should have included too go version devel +a400eb3261 Mon Mar 1 15:24:01 2021 +0000 linux/ppc64le
>go build
main.main.func1·f: relocation target main.main.func1 not defined

go.mod:
module example.com/local

go 1.17

This might be a dup of #44330, which was fixed since the revision that you are testing.

@ianlancetaylor yes, this is a duplicate.  I verified my test case and kubernetes compile after cherry-picking the commit in #44330.  This issue can be closed.  Thank you for pointing out that issue.


Question regarding map_parser.inc

I loaded all my maps (76 to test) & the console even said the maps loaded correctly.. but nothing is shown ingame, no RemoveBuilding nor CreateDynamicObject

I didn't tested the map_parser so i was expecting bugs! I will do testing soon, could you send me an example map file i could use for testing.

This also bugs up with multiple SetDynamicObjectMaterial's on one CreateDynamicObject

Map parser is unstable and requires lot of work, and i don't have time for it, sorry.
map_parser.inc has been deleted from repo.