Error 1002 while trying to create a TCP monitor

Terraform version

Affected resource(s)

Terraform configuration files
resource "cloudflare_load_balancer_monitor" "tcp_monitor" {
  type = "tcp"
  method = "connection_established"
  timeout = 7
  interval = 60
  retries = 5
  description = "example tcp load balancer"
}

Debug output
https://gist.github.com/thecodeassassin/0aa038f39e96562046ab22f22a6eb83f
Panic output

Expected behavior
Monitor should have been created
Actual behavior
Error: error creating load balancer monitor: error from makeRequest: HTTP status 400: content "{\n  \"result\": null,\n  \"success\": false,\n  \"errors\": [\n    {\n      \"code\": 1002,\n      \"message\": \"interval is not in range [0, 0]: validation failed\"\n    }\n  ],\n  \"messages\": []\n}\n"

Steps to reproduce

Make a clean TF file like this:

provider "cloudflare" {
  email   = ""
  api_key = ""
}

resource "cloudflare_load_balancer_monitor" "tcp_monitor" {
  type = "tcp"
  method = "connection_established"
  timeout = 7
  interval = 60
  retries = 5
  description = "example tcp load balancer"
}

^^^ This comes from the documentation here => https://registry.terraform.io/providers/cloudflare/cloudflare/latest/docs/resources/load_balancer_monitor
terraform init
terraform plan
terraform apply

@thecodeassassin Are you able to provide a full reproduction test case as well as a full TF_LOG=DEBUG output? The resource and reproduction case you've provided don't match up. One is for cloudflare_load_balancer_monitor but the gist is for a failed spectrum application.

@jacobbednarz I updated the gist, these are logs from a clean terraform file as described above. All I did was copy paste the example and I was able to reproduce the issue.

Looking at that gist, the API response seems to erroneous as the validation field is looking for a range of 0,0 despite the API documentation allowing integers. Would you mind opening a support ticket and posting the # back here to track it?

@jacobbednarz the ticket number is  #2029359.

@jacobbednarz
The bug is fixed when you specify "account_id" in the provider config.
Also port is required but it's not mentioned in the docs at all.

The bug is fixed when you specify "account_id" in the provider config.

Oh, that is very surprising! The userBase method is responsible for this one and handles it behind the scenes if the account ID is defined.

Also port is required but it's not mentioned in the docs at all.

Do you mean the Cloudflare API docs? Or the Terraform resource docs? If it's the latter, you're welcome to raise a PR to address it.

The bug is fixed when you specify "account_id" in the provider config.

Oh, that is very surprising! The userBase method is responsible for this one and handles it behind the scenes if the account ID is defined (which is news to me as well!).

Also port is required but it's not mentioned in the docs at all.

Do you mean the Cloudflare API docs? Or the Terraform resource docs? If it's the latter, you're welcome to raise a PR to address it.

The Terraform resource docs, I'll make a PR to address it.
The CF docs:
Port number to connect to for the health check. Required for TCP checks. HTTP and HTTPS checks should only define the port when using a non-standard port (HTTP: default 80, HTTPS: default 443).

@thecodeassassin are you still interested in putting up a PR for this one?

@jacobbednarz I opened a PR to address this

For anyone who finds this issue in the future:
This is strange, but this error also can be produced if you don't have a Load Balancer feature in your account subscription.

For anyone who finds this issue in the future:
This is strange, but this error also can be produced if you don't have a Load Balancer feature in your account subscription.

Had this error message when trying to create a Load Balancer related resources, setting the optional account_id on these resources fixed the error.
Note:
When creating a monitor, you have to pass `account_id `to the provider configuration in order to create account level resources. Otherwise, by default, it will be a user level resource.


Do not show arrow if summary and expanded stats have same content

Type

[ ] Bug fix
[ ] New feature
[x] Improvement to existing feature

Related JIRA Issue(s)
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-932
Description
If the content of the collapsed section is the same as the content of the expanded section, there is no point in expanding the stats section, and therefore we shouldn't be showing the up/down arrow in its right corner.
Deployment URL
http://remove-stats-arrow.review.ensembl.org/
Views affected
Species stats page

@jyo @msmanoj what should be the algorithm, in pseudocode, to decide whether a section has expanded content? I was reading through the getExpandedContent function, but got confused.
Is it just as easy as: "if there is more than one group or if the first group has more members than the summary stats, then it means we have expanded content.
If so, wouldn't it be a good idea to extract this in a dedicated function:
const hasExpandedContent = () => {
  const { groups, summaryStats } = statsSection;

  const [firstGroup] = groups;

  return groups.length > 1 || summaryStats.length === firstGroup.length
}

@jyo @msmanoj what should be the algorithm, in pseudocode, to decide whether a section has expanded content? I was reading through the getExpandedContent function, but got confused.
Is it just as easy as: "if there is more than one group or if the first group has more members than the summary stats, then it means we have expanded content"?
If so, wouldn't it be a good idea to extract this in a dedicated function:
const hasExpandedContent = () => {
  const { groups, summaryStats } = props.statsSection;

  const [firstGroup] = groups;

  return groups.length > 1 || summaryStats.length === firstGroup.length
}

What if there are 2 groups eaach with one entry and we have both of those entries shown in the summary?

What if there are 2 groups eaach with one entry and we have both of those entries shown in the summary?

Will we ever? Can the summary contain combined content from several groups?

Will we ever? Can the summary contain combined content from several groups?

I would think it can.
I wonder if we can apply this filter within the speciesGeneralHelper file without having to do a lot of processing inside the UI component. I have created a draft PR which will actually compare all the summary and expanded stats and hide the hides the expanded stats if they both match: https://github.com/Ensembl/ensembl-client/pull/462/files.

Will we ever? Can the summary contain combined content from several groups?

I would think it can.
I wonder if we can apply this filter within the speciesGeneralHelper file without having to do a lot of processing inside the UI component. I have created a draft PR which will actually compare all the summary and expanded stats and hide the hides the expanded stats if they both match: https://github.com/Ensembl/ensembl-client/pull/462/files.

Yes thats a very good idea. Not giving back the expanded result would get rid off all the calculations in the UI :-) thanks @msmanoj  If you could get this merged I can change the code accordingly in this component


Troubleshoot connections on the new postgres box (pg13)

Honeybadger

please resolve https://app.honeybadger.io/projects/54497/faults/80884987 when closing this ticket.

Background
In response to https://github.com/pulibrary/bibdata/issues/1514 and https://github.com/pulibrary/bibdata/issues/1531 we moved bibdata alma boxes to point to the old postgres machine. This was to buy us time to figure out the reason we we losing connections. We want to move back to the new postgres box.
Process
We'd like to write a script that opens a database connection, sleeps for an increasing amount of time, and then writes to the connection. We can use this to pinpoint exactly when the connection is timing out, and try it under various conditions / environments.
Theory
Discussions on the ops team yielded the following theory:

VM locations:

bibdata-alma-staging1 is in New South
bibdata-alma-worker-staging1 is in Forrestal
bibdata-alma1 is in New South
bibdata-alma2 is in Forrestal
bibdata-alma-worker1 is in Forrestal
bibdata-alma-worker2 is in Forrestal
bibdata-alma-worker3 is in New South
lib-postgres3 (pg10) machine is in Forrestal
lib-postgres1 (pg13) machine is in is in New South
lib-postgres-staging1 (pg13) staging machine is in Forrestal

The difference in physical location between the 2 postgres machines could have resulted in greater latency to the postgres 13 machine.

Based on the error we got during indexing
For prod: The errors in are coming from bibdata-alma-worker{1, 2}.  bibdata-alma-worker3 was created 7/7, which is before these honeybadger errors. So that does support this theory since that one wasn't erroring.
For staging: The errors are coming from bibdata-alma-worker-staging1 which also supports this theory.
Another interesting pattern is that the error always came from the worker boxes. Weren't all the boxes running sidekiq at the time? Why would the web server boxes not have hit this error?
Note that the staging db was later moved down to lib-postgres3, but prod was not.
Based on the error we got while downloading the scsb files
We got lost connections from both bibdata-alma-staging1 and bibdata-alma-worker-staging1. The database was switched between these two errors. So these errors also support the theory.

The firewall idle timeout was updated this afternoon from 12 to 24 hours.

We'll pursue turning that firewall timeout off for traffic from our VMs

This still happening: https://app.honeybadger.io/projects/54497/faults/80884987#comment_15950891

Still happening as on 8/5/2022 https://app.honeybadger.io/projects/54497/faults/80884987#comment_15950891.

Do we still want to do this? I believe we've moved a new environment this past year.


Support the decimal type

float64 is a little questionable for decimal type but can't do better in stdlib afaik. If someone needs it later, I suppose they could send a PR using the de facto decimal library (shopspring/decimal)
I love how your PRs are ready to merge!


Filesystem configuration with nested disks config

Laravel Version
10.34.1
PHP Version
8.3.8
Database Driver & Version
No response
Description
I have filesystem disks config like
`'disks' => [
    'local' => [
        'driver' => 'local',
        'root' => storage_path('app'),
        'throw' => false,
    ],

    'public' => [
        'driver' => 'local',
        'root' => storage_path('app/public'),
        'url' => env('APP_URL').'/storage',
        'visibility' => 'public',
        'throw' => false,
    ],

    's3' => [
        'watermarks' => [
            'driver' => 's3',
            'key' => env('AWS_ACCESS_KEY_ID'),
            'secret' => env('AWS_SECRET_ACCESS_KEY'),
            'region' => env('AWS_DEFAULT_REGION'),
            'bucket' => env('AWS_WATERMARKS_BUCKET'),
            'url' => env('AWS_WATERMARKS_BUCKET_URL'),
            'endpoint' => env('AWS_ENDPOINT'),
            'use_path_style_endpoint' => env('AWS_USE_PATH_STYLE_ENDPOINT', false),
        ],
        'media-library' => [
            'driver' => 's3',
            'key' => env('AWS_ACCESS_KEY_ID'),
            'secret' => env('AWS_SECRET_ACCESS_KEY'),
            'region' => env('AWS_DEFAULT_REGION'),
            'bucket' => env('AWS_MEDIA_LIBRARY_BUCKET'),
            'url' => env('AWS_MEDIA_LIBRARY_BUCKET_URL'),
            'endpoint' => env('AWS_ENDPOINT'),
            'use_path_style_endpoint' => env('AWS_USE_PATH_STYLE_ENDPOINT', false),
        ],
        'attachments' => [
            'driver' => 's3',
            'key' => env('AWS_ACCESS_KEY_ID'),
            'secret' => env('AWS_SECRET_ACCESS_KEY'),
            'region' => env('AWS_DEFAULT_REGION'),
            'bucket' => env('AWS_ATTACHMENTS_BUCKET'),
            'url' => env('AWS_ATTACHMENTS_BUCKET_URL'),
            'endpoint' => env('AWS_ENDPOINT'),
            'use_path_style_endpoint' => env('AWS_USE_PATH_STYLE_ENDPOINT', false),
        ],
],

I use laravel/framework version 11.22.0 and all works fine. But after change laravel/framework version 11.34.1 and runcomposer upgrade` I got error

Steps To Reproduce
Write filesystem disks nested config and
change laravel/framework version  from 11.22.0 or older to 11.34.1 and run composer upgrade

Laravel Framework doesn't accept nested configuration as suggested above and was not designed to work with such configurations.

Laravel Framework doesn't accept nested configuration as suggested above and was not designed to work with such configurations.

Thank you for response, but everything worked before, but after the update it stopped. I thought it was some kind of bug.

but everything worked before, but after the update it stopped. I thought it was some kind of bug.

If you are using 3rd party packages that accept nested configuration you need to report this issue to the correct repository as we don't consider changes to cause above error as a bug as we never support nested configuration.

@crynobone I use laravel/medialibrary v11.10.1 for work with files and code like below works file
$file_768_path = Storage::disk('s3.banners')->put($filename . '_768', $request->file('file_768'));   $img_link_768 = Storage::disk('s3.banners')->url($file_768_path);
I think it doesn't so matter, cause i got error when laravel parse config files.
For parsing config files I don't use 3rd party packages.


Timeline

I think I need something like https://popmotion.io/api/timeline/
Do you think there is a way to fit this in @vueuse/motion ?
I'd like to help and make a PR but I am not sure how to get started.
Thanks

Hello!
This is one of the features I would like to implement!
Do you have any ideas about how you would implement the API?


Implement internal_type_traits

Separated from #14.
I'm not sure other modules use this in what way. We want 0 or MAX in many modules.

I guess having max_value and min_value for Integral trait may be useful.

I will merge this PR according to this.


[Rollchains] Integrate Private Shannon TestNet with RollChains

Objective
Integrate with rollchains to post Shannon TestNet blob data on Celestia.
Origin Document
With the migration to a Validator based Cosmos SDK network, we still need to integrate with Celestia even if we're not a rollup.
Goals

Posting blob data from Private Shannon TestNet to Celestia Mocha TestNet

Deliverables

[ ] Full integration with rollchains
[ ] Using Grove's endpoints to post the blob on Celestia DA
[ ] Updating dev.poktroll.com to reflect these changes and visibility into how this is being posted.

Non-goals / Non-deliverables

Integrating with spawn
Any new on-chain functionality
Going overboard with the implementation / integration
Posting to any other DA layer

General deliverables

[ ] Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
[ ] Testing: Add new tests (unit and/or E2E) to the test suite.
[ ] Makefile: Add new targets to the Makefile to make the new functionality easier to use.
[ ] Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @Olshansk
Co-Owners: ???

Here's the PR: https://github.com/rollchains/tiablob/pull/2
And here's proof of us posting to Mocha testnet from a rollchains testnet: https://mocha.celenium.io/namespace/00000000000000000000000000000000000000000072635f64656d6f

https://github.com/rollchains/tiablob/blob/main/docs/integration.md

See this link for reference on what we should do: https://github.com/rollchains/tiablob/pull/2#event-12582817416


Tex value error when handling compilation issue

Description of bug / unexpected behavior

I had some missing tex dependencies that are resolved now, but when manim was gathering the details to report the error it failed as env_markers_indices in tex_file_writing > print_tex_error method was empty. It instead through the value error below in the logs that creates some noise hiding the real issue.
Expected behavior

Expected behaviour is that it should be able to handle empty env_markers_indices if there are cases where it will be empty and report the reason for the latex compilation error and not throw a ValueError
How to reproduce the issue

In my case I was missing latex dependencies that triggered this issue in ubuntu and ran the test
pytest manim/tests/test_graphical_units/test_axes.py::test_axes

Code for reproducing the problem
Paste your code here.

 This code finds the next such line after the error current error message
        line_of_tex_error = (
            int(
                [
                    log_line
                    for log_line in tex_compilation_log[error_start_index:]
                    if log_line.startswith("l.")
                ][0]
                .split(" ")[0]
                .split(".")[1],
            )
            - 1
        )
        # our tex error may be on a line outside our user input because of post-processing
        if line_of_tex_error >= len(tex_source):
            return None

        # all lines numbers containing '\begin{' or '\end{' - except the Manim added center and document tags
        env_markers_indices = [
            idx
            for idx, log_line in enumerate(tex_source)
            if any(marker in log_line for marker in [r"\begin{", r"\end{"])
        ][2:-2]

        context = "Context of error:\n\n"
        if line_of_tex_error in env_markers_indices:
            context += "".join(tex_source[line_of_tex_error - 1 : line_of_tex_error + 1])
        else:
>           marker_before_error = max(
                idx for idx in env_markers_indices if idx < line_of_tex_error
            )
E           ValueError: max() arg is an empty sequence

../../manim/utils/tex_file_writing.py:298: ValueError

Coverage.py warning: Module manim was previously imported, but not measured (module-not-measured)

============================== 1 failed in 2.80s ===============================
Manim Community v0.11.0

Process finished with exit code 1

System specifications
System Details

OS (with version, e.g Windows 10 v2004 or macOS 10.15 (Catalina)): Ubuntu 21.10
RAM: 16gb
Python version (python/py/python3 --version): 3.9.7
Installed modules (provide output from pip list):

PASTE HERE

LaTeX details

LaTeX distribution (e.g. TeX Live 2020): Version 3.14159265 (TeX Live 2020/Debian) (preloaded format=tex)
Installed LaTeX packages:

FFMPEG
Output of ffmpeg -version:
PASTE HERE

Additional comments

This is one of the most-reported errors in our Discord, it would be great to improve the error message.


[zh]sync-setup-3

This one supersedes #33625.
Co-Authored-By: @wlp1153468871
related:  #33475

/approve
/lgtm
Thanks!


Issue/2919 datetime aware

Description

Makes everything internal timezone-aware
API input accepts both timezone-aware and naive timestamps. Naive timestamps are assumed to be in UTC. Same for Python client input.
API output returns naive timestamps in UTC.

closes #2919
closes #2896
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[x] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Pull request rejected by merge tool. Failed to parse change entry file: Change entry /tmp/tmp0_3xd2xw/changelogs/unreleased/2919-datetime-aware.yml does not match pydantic schema: 1 validation error for ChangeEntry
issue
extra fields not permitted (type=value_error.extra)

Processing this pull request

Merged into branches master in 2a8c1999f9a74a3511112fc5b7ce9a8bfcdcc1ac

Processing #2967.


Diagnostic:Line Too Long runs on source files even if it is disabled

I have disabled the Diagnostic:Line Too Long but it is still running and reporting on each new source file I open.

I changed the configuration property name to "showCommandLogs" to avoid misunderstanding and added the new configuration property "enableDiagnostics" to enable/disable diagnostics.


[Infra] Create local testnet

We should have a docker-compose that allows us to have a local testnet, with an already working full node (e.g. prysm+geth) and a testing full node (e.g. lambda ethereum consensus + reth), with an empty blockchain. This would allow us to quickly test gossiping, propagation of blocks, and others in a controlled manner, and with low resources (i.e. we won't need to save the 2TB full mainnet blockchain locally). While this is not a full-proof method, it allows us to iterate quickly and is perfectly valid to test integrations.

Expanding on this, we'd need to add the following functionality:

import a genesis state (which corresponds to phase0) and update it to the latest spec
flags to set all ports (P2P, discovery, Beacon API)
load a custom config from a yaml

@Arkenan, If I understand this issues correctly, Kurtosis is used today to accomplish this. This issue doesn't refer to smoketests but manual test right? This is tracked in #812 as:

Add scripts and instructions for setting up/tearing down a local testnet

Which have been added as make task alongside documentation in #1197

Solved by #1197


⚠ Dovecot has degraded performance

In f0a320b, Dovecot (https://en.wikipedia.org) experienced degraded performance:

HTTP code: 503
Response time: 864 ms

Resolved: Dovecot performance has improved in d03c798 after 5 minutes.


Modifying RegExp prototype

Do you think it would be better if this library just returned a function?
var randexp = require('randexp');
randexp(/hello+ (world|to you)/); // hellooooooooooooooooooo world

Or:
 var RandRegExp = require('randexp')
   , rand = new RandRegExp(/hello+ (world|to you)/);

rand.gen();

I realise the API you've created using the getter on RegExp has a certain stylistic elegance, but most Node libraries don't work this way and people will get upset about modifying the built in prototype.

When I first started working on this I was going to make it into an exported function. But then I discovered randexp for Ruby and noticed the .gen syntax. That library seemed to be liked the way it is even though it modifies the RegExp class. Although I've never used Ruby so for all I know modifying a class in Ruby could be different from modifying a prototype in Javascript.
It can be optional. An exported function that modifies the RegExp prototype can be called, something like require('randexp').global().

I like the idea of it being optional


[INTEL MKL] Add primitive cache for mkl concat

Add support for primitive reuse in mkl concat with all the inputs as TF tensors.
Files modified
tensorflow/core/kernels/mkl_concat_op.cc
Please review

@penpornk  Thank you for your valuable review comments. I will address the review comments and submit the patch ASAP

@penpornk Sorry for the delay. Addressed all the review comments, please review it


[widclk] String artifact when changing number of digits showing

Affected hardware version
Bangle 2
Your firmware version
0.06
The bug
The widget was displaying "12:59." When the time changed to 1:00, the display was "1:009," I assume due to the difference in number of characters (therefore not overwriting the 9).
Installed apps
No response

@dormio does #3074 fix this?
(to try it out, you can paste the new widclk into the IDE and upload it as widclk.js)

Thanks for the report!
We were both working on this! - I've just pushed a fix to the development app loader.
I think probably when it was originally developed a 24 hour clock was used, so everything stayed the same width - it was only on 12 hour clock where this showed issues


Breadcrumb improvements

As outlined in this blog post, we have an opportunity to improve the accessibility for the breadcrumb snippet. There are some extra definitions we can add, notably "aria-current".

This won't be required with the new version.


synchronize schema spec

What
APM agent json schema automatic sync
Why
Changeset

https://github.com/elastic/apm-server/commit/e004300a9 dropped_spans_stats: Remove type, subtype fields (https://github.com/elastic/apm-server/pull/6268)

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-10-04T03:11:48.052+0000

Duration: 43 min 14 sec

Commit: b06548002cd15b1b08344bca2aa716a0fc3e2ac1

Test stats :test_tube:

Test
Results

Failed
0

Passed
9603

Skipped
8836

Total
18439

:green_heart: Flaky test report
Tests succeeded.
:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


BIO_lookup_ex: use AI_ADDRCONFIG only if explicit host name is given [1.1.1]

Backport of #16033 to the 1.1.1 branch.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged to 1.1.1.  Thanks for the backport.


feat: add xlswriter 1.5.3 release

update xlswriter with release 1.5.3

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


pip install falcon==0.x.x

Hello
I have bash code that fail because  I am doing: pip install falcon==0.2.0
but:
#pip list|grep falcon
falcon (0.2)
function pip_install_version(){
    set +e
    local install_package="$1"
    local install_version="$2"
    echo "${SUDO} ${PIP} install --upgrade ${install_package}==${install_version}"
    ${SUDO} ${PIP} install --upgrade "${install_package}==${install_version}" #--force-reinstall
    set -e
    ${SUDO} ${PIP} list 2> /dev/null|grep "${install_package}" |grep "${install_version}" &> /dev/null
    # fail on falcon==0.2.0 but install falcon 0.2
}
    case "$build_falcon_version" in
        0.1.10) pip_install_version ${FALCON} 0.1.10;;
        0.2.0) pip_install_version ${FALCON} 0.2.0;;
        0.3.0) pip_install_version ${FALCON} 0.3.0;;
        latest) pip_install_latest ${FALCON};;
        *) echo "You didn't setup propertly variable build_falcon_version"; exit 1;;
    esac

@weldpua2008 Do you still need help on this one? I ran the script on my Arch Linux box and it seemed to work OK:
FALCON=falcon PIP=pip build_falcon_version=0.2.0 bash the_script

I'm seeing still this:
#pip list|grep falcon
falcon (0.2)

@weldpua2008 did you ever get this sorted out?

Please reopen if this is still an issue.


Confirmation

Selection Sort at line 35, should the max equal to i?

Thank you for raising the concern about the selection sort at line 35. However, it seems there might be a misunderstanding. In the selection sort algorithm, the variable "max" is not required to be equal to "i."  The purpose of max = 0 is to swap largest elements at the back of the array.  I believe you are referring to the function SelectionSortFront at line 73.

Thanks for the clarification!


Problem with useNativeDriver and componentWillReceiveProps

Issue
Two warnings
Animated: useNativeDriver was not specified. This is a required option and must be explicitly set to true or false
and
Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-unsafe-component-lifecycles for details.
Expected Behavior
Code
<DatePicker
   customStyles={{
      dateInput: {flexDirection: 'row', borderWidth: 0, justifyContent: 'flex-end', paddingTop: 3},
   }}
   minDate="2020-10-01"
   maxDate={Dates().today}
   confirmBtnText="Apply"
   cancelBtnText="Close"
   onDateChange={date => setDay(new Date(date).toLocaleDateString())}
/>

Environment

react-native -v: 39.0.0
node -v:  14.2.0
npm -v:  6.14.4
target platform:  Android  and  iOS
operating system:  mac os

+1

+1

+1

+1

+1

+1

+1 too

+1 too

+1

+1

@nihil-pro @dnlmzw @MahmutGundogduKizilay @ixez @dibyopra @Wassyk @JoaquinBeceiro @appseugene
try this: https://github.com/xgfe/react-native-datepicker/pull/441/commits/db1b18ecd9dcffdfe6c4668fa15afbcd6041b97b

+1

+1


Out Of Memory

I have azure function which returning data from Cosmos Db.
When the data is larger than 5MB, azure function call crashed on OutOfMemory Exception.
This happening just in Azure. On localhost it works ok.
I tried streamimg of response, but it is not supported.
public IAsyncEnumerable<Requirement> GetRequirements([HttpTrigger(AuthorizationLevel.Anonymous, "get", Route = null)] HttpRequestData request, FunctionContext functionContext)
        {
            var username = functionContext.GetUsername();
            return _inboxDbContext.Set<Requirement>().Where(p => p.Username == username && !p.Downloaded).AsAsyncEnumerable();
        }

Can you help me?

Here is a code which fails locally as well, writing directly to Body of a response.
Here is a code which fails locally as well:
public class ComplexObject
{
    public string Data { get; set; }
    public ICollection<ComplexObject> ComplexObjects { get; set; }
}

public class Function
{
    [Function("Test")]
    public async Task<HttpResponseData> GetRequirements([HttpTrigger(AuthorizationLevel.Anonymous, "get", Route = null)] HttpRequestData req, FunctionContext functionContext)
    {
        HttpResponseData response = req.CreateResponse();
        response.StatusCode = HttpStatusCode.OK;
        response.Headers.Add("Content-Type", "application/json");
        IAsyncEnumerable<ComplexObject> data = GetData();
        await JsonSerializer.SerializeAsync(response.Body, data, options: new JsonSerializerOptions
        {
            Encoder = JavaScriptEncoder.UnsafeRelaxedJsonEscaping
        });
        return response;
    }

    private async IAsyncEnumerable<ComplexObject> GetData()
    {
        await Task.CompletedTask;
        for (int i = 0; i < 100; i++)
        {
            yield return GetObject(0);
        }
    }

    private ComplexObject GetObject(int depth = 0)
    {
        ComplexObject complexObject = new ComplexObject();
        complexObject.Data = BigData;
        complexObject.ComplexObjects = new List<ComplexObject>();
        if (depth < 5)
        {
            for (int i = 0; i < 10; i++)
            {
                complexObject.ComplexObjects.Add(GetObject(depth + 1));
            }
        }
        return complexObject;
    }

    private const string BigData = "Lorem ipsum dolor sit amet, consectetuer adipiscing elit. Integer imperdiet lectus quis justo. Maecenas libero. Temporibus autem quibusdam et aut officiis debitis aut rerum necessitatibus saepe eveniet ut et voluptates repudiandae sint et molestiae non recusandae. Aliquam in lorem sit amet leo accumsan lacinia. Class aptent taciti sociosqu ad litora torquent per conubia nostra, per inceptos hymenaeos. Nunc dapibus tortor vel mi dapibus sollicitudin. Suspendisse nisl. Maecenas lorem. Morbi scelerisque luctus velit. Aliquam ante. Aliquam erat volutpat. Integer malesuada. In rutrum. Aenean id metus id velit ullamcorper pulvinar.";
}

@jerry2007 Just to confirm, are you sure that it works when the data size is smaller, and you use IAsyncEnumerable<T> as the return type? Are you getting a collection of Requirements when executing this function?

@kshyju nope...
IAsyncEnumerable<T> is not supported completly
But when the data is bigger then 5MB, deployed in Azure and return type is IEnumerable<T>, it crash on Out Of Memory Exception.

@jerry2007 Thank for confirming. Yes. Async stream support is not available today. This issue will be tracking the progress of that work. We will track the OOM issue here with IEnumerable<T> type. May I ask you to update the original code snippet to use IEnumerable<T> to keep the context of this issue clean?

@vvopenka Thanks! I tried your repro code. I am not getting any OutOfMemoryException. I get the "System.IO.IOException: Stream was too long" on the await JsonSerializer.SerializeAsync line which is inside your function. Here is the top portion of the stack trace I am seeing. Were you getting the same exception or something different?

System.IO.IOException: Stream was too long.
[2022-01-10T22:25:41.145Z]    at System.IO.MemoryStream.Write(Byte[] buffer, Int32 offset, Int32 count)
[2022-01-10T22:25:41.149Z]    at System.IO.MemoryStream.WriteAsync(ReadOnlyMemory`1 buffer, CancellationToken cancellationToken)
[2022-01-10T22:25:41.152Z] --- End of stack trace from previous location ---
[2022-01-10T22:25:41.153Z]    at System.Text.Json.JsonSerializer.WriteStreamAsync[TValue](Stream utf8Json, TValue value, JsonTypeInfo jsonTypeInfo, CancellationToken cancellationToken)
[2022-01-10T22:25:41.154Z]    at System.Text.Json.JsonSerializer.WriteStreamAsync[TValue](Stream utf8Json, TValue value, JsonTypeInfo jsonTypeInfo, CancellationToken cancellationToken)
[2022-01-10T22:25:41.156Z]    at System.Text.Json.JsonSerializer.WriteStreamAsync[TValue](Stream utf8Json, TValue value, JsonTypeInfo jsonTypeInfo, CancellationToken cancellationToken)
[2022-01-10T22:25:41.157Z]    at _761_OutOfMemory.Function.GetRequirements(HttpRequestData req, FunctionContext functionContext) in C:\Dev\Temp\GHIssues\761-OutOfMemory\Function.cs:line 26

The stack trace suggests the exception is being thrown from the serializer, not from the functions infrastructure/runtime code.
Another thing I noticed while executing your code is the spike in memory usage (just a single execution). It went from under 30 MB to 4 GB.
.
OOM exceptions are usually caused by a large object graph/large allocations than what the runtime can handle. Could you take a look at this recommendations to prevent OOM and see any of this makes sense/can be adopted in your case?

@kshyju Sorry, OutOfMemory exception is thrown when you deploy the function to Azure Function. The stack trace there looks like this:

Exception: System.AggregateException: One or more errors occurred. (Exception of type 'System.OutOfMemoryException' was thrown.)
---> System.OutOfMemoryException: Exception of type 'System.OutOfMemoryException' was thrown.
at System.IO.MemoryStream.set_Capacity(Int32 value)
at System.IO.MemoryStream.EnsureCapacity(Int32 value)
at System.IO.MemoryStream.Write(Byte[] buffer, Int32 offset, Int32 count)
at System.IO.MemoryStream.WriteAsync(ReadOnlyMemory`1 buffer, CancellationToken cancellationToken)

Locally it does indeed IOException. I think it's connected to the memory limit of the environment it runs on.
Just to be clear, we do not insist on returning IAsyncEnumerable. We just need to stream a big amount of data out of our Function which currently fails due to the output stream being a MemoryStream and so we are hitting the memory limits.

@jerry2007 Thank for confirming. Yes. Async stream support is not available today. This issue will be tracking the progress of that work. We will track the OOM issue here with IEnumerable<T> type.
I tried to repro this issue by returning a 10 MB string and I do not see any issues. Could you please share a minimal repro of OOM issue using IEnumerable<T> and the stack trace of the exception?

Here is Minimal project...
InboxMin.zip
When this project is deployed to azure (consuption mode) you can try these requests:
https://solitea-inbox-staging.azurewebsites.net/api/Sample1?maxlevel=5&itemscount=4 - OK
https://solitea-inbox-staging.azurewebsites.net/api/Sample1?maxlevel=5&itemscount=5 - Out Of Memory
https://solitea-inbox-staging.azurewebsites.net/api/Sample1?maxlevel=10&itemscount=10 - Out Of Memory

@jerry2007 Thanks for sharing the repro. When you use ?maxlevel=10&itemscount=10 for the request, you are building a very huge object graph with your code. You will be able to observe this in the diagnostics tools window in VS when you locally debug this code.

The consumption hosting plan has a max memory threshold of 1.5 GB. Your code is building a large object which goes beyond this threshold, causing the OOM exception.
You could consider changing to a different hosting plan or revisit your implementation to produce less data in memory.
https://docs.microsoft.com/en-us/dotnet/api/system.outofmemoryexception?view=net-6.0#remarks
Hope this helps.

Ok, I realized that my repro is not the best for my situation.
But it is complicated to create repro with my use case...
The usecase is that I have Azure function, which getting data from cosmosDb.
When I run it locally it is ok, return 200 and data. memory consuption in diagnotic tools window is on 668MB.
When I deploy this code to azure and try same request it crash on Out Of Memory Exception:

`HTTP/1.1 500 Internal Server Error
Request-Context: appId=cid-v1:87983616-9030-46bd-8e4f-8ecfcecd5850
Date: Thu, 13 Jan 2022 16:12:00 GMT
Content-Length: 85
AggregateException
Exception of type 'System.OutOfMemoryException' was thrown.
`

The function code:
public async Task<IEnumerable<Requirement>> GetRequirements([HttpTrigger(AuthorizationLevel.Anonymous, "get", Route = null)] HttpRequestData req, FunctionContext functionContext)         {             var username = functionContext.GetUsername();             _logger.LogTrace("Getting requirements for username = {UserName}", username);             var data = await _inboxDbContext.Set<Requirement>().Where(p => p.Username == username && !p.Downloaded).ToListAsync();             return data;         }
Where _inboxDbContext is DbContext of Entity Framework core 6.0.1

Is your production data different (larger object graph) than the one you are using for local development? What is the memory usage you are observing on production where you are getting this OOM exception? Can you try deploying a different hosting plan and see that help with the consumption plan limits?
Without a local repro, it is hard to point out what is causing it. But in general, OOM exception is thrown when you try to deal with huge amount of data (Most of the time, the environment limits are main causes than the infra code). I recommend reviewing the document linked in the previous comment and see what can be done in your app logic to reduce the data size you are handling.

Production data is same, in fact, i am trying to get only production data in same CosmosDb
But when I trying your advices I founded out, that new Azure function service works but my old one not...
When I create new Azure Function Service in Consuption mode in Azure and deploy same code it works.
But on Azure function Service, which was crated 1 year ago, same code not work and raising Out Of Memory Exception...
Is this ok? Should I just recrete Azure functions Service in Azure?

@jerry2007 Can you double check your old function app uses same settings (isolated version, function runtime version, hosting type etc.) as the new one?

I did that check and I maybe found that problem...
It looks like 64/32bit problem...
On old one service I had 64bit version on the new one 32bit. After set old one to 64bit it started work.
And one more problem was deployed as Self-Contained. When I deployd app as Self-Contained even on 32bit version it crashed also on OOM. After deploying as Framewokr-dependent it works.
So thank you for help and I will look forward for IAsyncEnumerable support :-)


Icons affect button height

When an icon is added to a Button the height of the icon can change the height of the button.
For a single button this may be ok, but when using multiple buttons or a button group the difference is obvious.

FWIW, we'll definitely need to make sure the buttons are all the same height, regardless of if they're standalone or now.


Generated input model is empty

Hi, I'm having trouble understanding how to use SPQR. I've use a similar project for node https://typegraphql.ml/docs/introduction.html before with much success, which shares a lot of concepts with this.
Questions:
~1. How to correctly generate input types for object classes?~
~2. How to control the required tribute of a schema object of field, eg (stockSize: Long!)?~
3. How can I generate a schema minus the extra annotations for use with a client or for documentation?
~4. @GraphQLId does not produce a schema field that uses the type ID?~
~5. How would I exclude a generated field like Id from an input type but not the output type, does this require multiple classes?~
Simple example
Dependency versions:
Java: 1.8
spring-boot-starter-parent: 2.1.1.RELEASE
io.leangen.graphql.spqr: 0.9.9

Here we have a simple POJO User.java
@Data // lombok
public class User {
    private final @GraphQLId Integer id;
    private String email;
    private final String username;
    private String phoneNumber;
    private final Boolean isActive;
}

And a resolver that references the user in a query and a mutation UserResolver.java
@Component
public class UserResolver {

    private UserService userService;

    @Autowired
    UserResolver(
        UserService userService
    ) {
        this.userService = userService;
    }

    @GraphQLQuery(name = "getUsers")
    public ArrayList<User> getUsers(){
        return this.userService.getUsers();
    }

    @GraphQLMutation(name = "createUser")
    public User createUser(@GraphQLArgument(name = "User") User user){
        return this.userService.creatUser(user);
    }

}

When the schema is rendered with new SchemaPrinter().print(schema); I get the following
#Mutation root
type Mutation {
  createUser(User: UserInput): User @_mappedOperation(operation : "__internal__")
}

#Query root
type Query {
  getUsers: [User] @_mappedOperation(operation : "__internal__")
}

type User @_mappedType(type : "__internal__") {
  email: String @_mappedOperation(operation : "__internal__")
  id: Int @_mappedOperation(operation : "__internal__")
  isActive: Boolean @_mappedOperation(operation : "__internal__")
  phoneNumber: String @_mappedOperation(operation : "__internal__")
  username: String @_mappedOperation(operation : "__internal__")
}

input UserInput @_mappedType(type : "__internal__") {
}

Solved:

How to correctly generate input types for object classes.

make sure withBasePackages is pointing to the root package of your application.
GraphQLSchema schema = new GraphQLSchemaGenerator()
            .withBasePackages("io.asx.dsp.graphql.poc")
            .withOperationsFromSingletons(userResolver)
            .generate();

My expectation was that properties that are final / don't have setters should not become part of the input model.
If I create another class called UserInput which does not include the id property like so:
@Data
@AllArgsConstructor
public class User {

    private final UUID id = UUID.randomUUID();
    private String email;
    private String username;
    private String phoneNumber;
    private boolean isActive;

    static public User fromInput(UserInput input) {
        return new User(
            input.getEmail(),
            input.getUsername(),
            input.getPhoneNumber(),
            input.isActive()
        );
    }
}

Then I have a resolver like this, where the GraphQLArgument name is UserInput or User or even empty string.
@GraphQLMutation(name = "createUser")
public User createUser(@GraphQLArgument(name = "UserInput") UserInput user){
    return this.userService.creatUser(user);
}

The schema output is this: UserInputInput is a rather undesirable name.
#Mutation root
type Mutation {
  createUser(user: UserInputInput): User @_mappedOperation(operation : "__internal__")
}

#Query root
type Query {
  getUsers: [User] @_mappedOperation(operation : "__internal__")
}

type User @_mappedType(type : "__internal__") {
  active: Boolean! @_mappedOperation(operation : "__internal__")
  email: String @_mappedOperation(operation : "__internal__")
  id: UUID @_mappedOperation(operation : "__internal__")
  phoneNumber: String @_mappedOperation(operation : "__internal__")
  username: String @_mappedOperation(operation : "__internal__")
}

input UserInputInput @_mappedType(type : "__internal__") {
  active: Boolean!
  email: String
  phoneNumber: String
  username: String
}

My expectation was that properties that are final / don't have setters should not become part of the input model.

Yeah, the lack of a setter should do it, rather than creating a separate UserInput class. In case it makes a difference, we pass the same User class as input in the @GraphQLQuery et al. methods and they automagically become UserInput in the schema.
I think maybe Lombok @Data is generating a setId() despite the instance var being final; either suppress it with @Setter(AccessLevel.NONE) or if you need the field to be settable in the User but not UserInput try annotating the generated method with:
@Setter(onMethod = @__({@GraphQLIgnore}))
private final UUID id = UUID.randomUUID();

I'm also wondering if SchemaPrinter().print(schema) is giving you misleading output -- we didn't need the basePackage trickery at all.

Worth trying:
@Getter(onMethod = @__({@GraphQLId }))

... to annotate the generated getter rather than the instance variable itself. Lombok doesn't copy annotations from vars to generated methods.

I'm also wondering if SchemaPrinter().print(schema) is giving you misleading output -- we didn't need the basePackage trickery at all.

I have checked that it matches the results of the introspective API.

Using @Setter(onMethod = @__({@GraphQLIgnore})) is experamental so ill stick with writing the getters and setter myself. For futre reference.
Model
@GraphQLType(name = "User") @GraphQLNonNull
public class User {

    protected String email;

    // applies to GraphQL output model
    @GraphQLNonNull
    public String getMobileNumber() {
        return this.email;
    }

    // applies to GraphQL input model
    public void setEmail(@GraphQLNonNull String value) {
        this.email = value;
    }

}

Resolver
@Component
public class UserResolver {

    private UserService userService;

    @Autowired
    UserResolver(
        UserService userService
    ) {
        this.userService = userService;
    }

    @GraphQLQuery(name = "getUsers")
    @GraphQLNonNull
    public ArrayList<User> getUsers(){
        ArrayList<UserEntity> entities = this.userService.getUsers();
        return User.toUserList(entities);
    }

    @GraphQLMutation(name = "createUser")
    @GraphQLNonNull
    public User createUser(@GraphQLArgument(name = "User") User user){
        return User.toUser(
            this.userService.creatUser(user)
        );
    }

}

will result in the types
type Mutation {
  createUser(User: UserInput!): User!)
}

type Query {
  getUsers: [User!]!
}

type User {
  email: String!
}

input UserInput {
  email: String!
}


Show available IP addresses

The current IPAM views only display addresses added to the system. It does not display available addresses within a subnet. It would be helpful to see all addresses within a subnet, noting each address' status. This could be similar to the "status" column on the /ipam/prefixes/ page, indicating "available", "reserved", or "active". Being able to filter the list based on status, and showing all addresses on a page rather than paginated would also be helpful.

I think is one of missing features, for example if is some Virtual System or not Racked System but have one IP. is more like IP Management combined with VLAN.
Great plataform!!!

Any IP address not listed should be considered available, since it has not been defined. What benefit would explicitly listing every IP address have? And how would you approach this for IPv6?

When allocating IP addresses to systems, seeing what addresses are available is helpful. This is especially true when deploying several systems at once that we'd like to be consecutively addressed.  With our current IPAM (GestioIP) I can clearly see which addresses are unused, and how large the range is:

With NetBox we have to look closely to see what ranges are available by figuring out where the breaks are:

The ability to quickly identify reserved addresses versus used and unused would be helpful.

An idea could be to group together IPs by block in order not to list all IPs (especially for IPv6). When aggregation is not possible, then list by smaller blocks or at the end individual IPs.
For my ISP usage, this would be a very very appreciated feature, in order to make choices when assigning IPs to customers.

So maybe between, say, .42 and .50, we'd inject a row reading "7 IPs available?" I think that would work fine for IPv4. We could employ a similar approach for IPv6 but I'll have to give it more thought.

Or maybe aggregate at the most.
For example (.42 and .50 are used but free between the both):
<IP_ADDRESS>/32: used
<IP_ADDRESS>/32: free
<IP_ADDRESS>/30: free
<IP_ADDRESS>/31: free
<IP_ADDRESS>/32: used
Don't know if it could work...

I think I'd prefer to just state the range, since unlike prefix aggregates IP addresses are atomic units. Generating aggregates to cover the space takes up more room on the page and can also make it more difficult to determine how much free space there is.

So maybe between, say, .42 and .50, we'd inject a row reading "7 IPs available?"
I think this is a reasonable solution and reduces clutter.

I would also like to have this feature. And I agree with the @jeremystretch 's range proposal, it will work both on IPv4 and IPv6. And this should be showed in a slightly different colour, light green or similar.
I also think this feature should be extended to VLANs within the group to show available VLANs as ranges. (btw, I belive the sites with no VLAN group should have an implicit default group, even if the user is not creating one).
Otherwise great work with the project Jeremy, I think it really is adding features other IPAMs don't have, even though a few more features could be welcomed.

phpipam might be a source of inspiration. I think they do a great job with IPAM in general. Seems like netbox does a better job with DCIM and phpipam does a better job with IPAM, while they each try to cover both needs. It will be interesting to see where the two projects end up long term.

Agreed, phpipam does handle that very well.

@alexjhart @anthraxau Could you elaborate on what you like about phpIPAM's view?

IMO this is a confusing way to show IP addresses. Why does the free range appear in the hostname column? I was thinking of something like this, but more similar to how we show unallocated child prefixes.

The block display, while visually appealing with small IPv4 networks, isn't very useful for prefixes larger than a /24 (and not at all with IPv6).

In the regards to the first image I just like how the dhcp range is grouped. You're right other than that it behaves incorrectly.
The 2nd image is nice to see what's available at a quick glance but I agree that on a /22 or larger it would be painful I didn't even think of IPv6 :(
I've been using excel for a /22 and a few other prefixes so I'm feeling abit blessed with what you have currently.
I'm not sure if other people do this but I tend to group similar devices in groups on the prefix for example routers .250-254 core switches .240-249 VMware stuff .100-.149 etc and I usually colour code them for easy navigation of my spreadsheet so that I can quickly glance other it and see any gaps that are available. I've noticed you do this for rack devices. Would be nice to be able to enable this for the ipam

I never considered the ranges to be under the hostname column. My eye accepted that as a range of available IPs and find the offset helpful in establishing the difference.  More important than which column data shows up under is the display and grouping of unused space. Having a place holder with a different row color and a range displayed of the available space is adequate. I also like similar types of ranges being grouped, like the dhcp range shown here, though that is less important to me than showing available space. It just plays into the argument you had for the next screenshot where doing things that help with large subnets is important. This could streamline the display of a lot of larger subnets.

I could care less about this visual display and agree it isn't practical for larger subnets. If you did want to have something like this, maybe you would set a limit for max prefix size it is displayed for.

I think you can take a similar approach to what you do with the child prefixes table. Here you have available subnets (though I didn't realize this at first, so maybe consider a different row color). Those are hyperlinked to a create new subnet form with the range autofilled, which I like. Though the workflow might be nicer if you used a popup rather than directing the user to a new page?

Anyway, maybe something like this. I imagine that range there is hyperlinked and takes you to a new IP address form for the first IP in that range. By the way, it would be nice to be able to add a range of IPs, not just one at a time. Again, I think it would flow better if you handle the addition in a popup (similar to phpipam).
I like @anthraxau's idea of having color coded ranges. Maybe this can be accomplished with a tag #132  I imagine something similar to gmail labels where you have have multiple tags shown in the table which can have colors assigned to them, to make different things stand out more. That idea is also off topic though.

How about this?

Clicking one of the "free IPs" buttons takes the user to the IP adress creation form, pre-populated with the first available IP in that range.

Great.
On Wednesday, 3 August 2016, Jeremy Stretch<EMAIL_ADDRESS>wrote:

How about this?
[image: netbox_ip_rows]
https://cloud.githubusercontent.com/assets/13487278/17369606/fb21969a-5966-11e6-9003-7115f435c3ce.png
Clicking one of the "free IPs" buttons takes the user to the IP adress
creation form, pre-populated with the first available IP in that range.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/digitalocean/netbox/issues/289#issuecomment-237257185,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANLscxU8TJb4f073j-4UQ9OREj8zTEwoks5qcLTugaJpZM4JL12t
.

--

Arafat M. Bique
Security Analyst
CISA | GCFE | GWAPT | GPEN | CISSP | CHFI | CEH | CCNP | CCNA SECURITY |
CCAI | MCSE
Esta mensagem foi enviada a partir de um dispositivo móvel, pelo que poderá
conter erros de acentuação e informação incompleta.
This message was sent from a mobile device so some features and information
might not be completed or available.

Looks good. Only improvement I can see would be to add the ranges like I had in my mockup.

Being able to "add an IP" from the free block would be great

@bellwood The "free IPs" button is indeed a link to add an IP, initialized to the first available IP in the range.

Please try out v1.4.1 and provide feedback.


ValidityRange is not encoded into CBOR when Balancing Transactions

Summary
This bug occurs when the PAB tries to encode a transaction, it “forgets” to insert the required fields to denote the ValidityRange. Then when the wallet receives this CBOR it can’t find the fields and so sets the Range to always. This means that any validator using this field to ensure the timing of a transaction will fail if a value different than always is expected.
Steps to reproduce the behavior
Using a validator with the following checks:
{-# INLINABLE mkContractValidator #-}
mkContractValidator
    :: Parameter
    -> MyContractDatum
    -> MyContractRedeemer
    -> ScriptContext
    -> Bool
mkContractValidator _ _ _ ctx =
    traceIfFalse
        "This should only happen when not using the mustValidateIn Constraint"
        (txRange /= always)
  where
    info :: TxInfo
    info = scriptContextTxInfo ctx

    txRange :: POSIXTimeRange
    txRange = txInfoValidRange info

And building the transaction with the following constraint, where ct is the current time:
Constraints.mustValidateIn (interval ct (ct+1500000))

Actual Result
The wallet returns the following error
[pab:Warning:263] [2022-03-07 17:08:01.38 UTC] WalletClientError "FailureResponse (Request {requestPath = (BaseUrl {baseUrlScheme = Http, baseUrlHost = \\"localhost\\", baseUrlPort = 8090, baseUrlPath = \\"\\"},\\"/v2/wallets/3b3bde62a1012e9e9ee0914af0b1ff10faef0ba8/transactions-balance\\"), requestQueryString = fromList [], requestBody = Just ((),application/json;charset=utf-8), requestAccept = fromList [application/json;charset=utf-8,application/json], requestHeaders = fromList []), requestHttpVersion = HTTP/1.1, requestMethod = \\"POST\\"} (Response {responseStatusCode = Status {statusCode = 400, statusMessage = \\"Bad Request\\"}, responseHeaders = fromList [(\\"Transfer-Encoding\\",\\"chunked\\"),(\\"Date\\",\\"Mon, 07 Mar 2022 17:08:01 GMT\\"),(\\"Server\\",\\"Warp/3.3.17\\"),(\\"Content-Type\\",\\"application/json;charset=utf-8\\")], responseHttpVersion = HTTP/1.1, responseBody = \\"{\\\\\\"message\\\\\\":\\\\\\"I was unable to assign execution units to one of your redeemers: spending(1st f3a23879); Its execution is failing with the following error: ValidationFailedV1 (CekError An error has occurred:  User error: The provided Plutus code called 'error'.) [\\\\\\\\\\\\\\"This should only happen when not using the mustValidateIn Constraint\\\\\\\\\\\\\\",\\\\\\\\\\\\\\"PT5\\\\\\\\\\\\\\"].\\\\\\",\\\\\\"code\\\\\\":\\\\\\"redeemer_script_failure\\\\\\"}\\"})"

Showing the error ["This should only happen when not using the mustValidateIn Constraint","PT5"] which is traced in the on-chain validator shown above.
We found out this was a PAB issue, and not a wallet one, by capturing packets that the PAB sent to the wallet. Inside these packets we found the transaction CBOR, but after decoding it we couldn’t find the expected fields in the transaction_body part of the CBOR (namely, the third and eighth fields, as described here).
Here is the transactions part of the CBOR for the transaction generated and sent by the PAB:
[{0: [[a22bb475acd036f9af74f08de16203fdee13af90ff5f3f882de9a8d8c403c073,
       0]],
  1: [[709d5501676afa4dea1e0c8ae52a5763a6cbb56030387a8920d9451780,
       2000000,
       ed16b7c053ffb878113c84e81d79a5ecfe86be57fe30941dd05405a67c7bb180]],
  2: 0,
  13: [],
  14: []},
 {3: [590bd601000033233322233223322323233322232333222323333333322222222323332223233332222323233223233322232333222323233223322323233...],
  4: [CBORTag(121, [CBORTag(121, [0])])],
  5: [[0, 0, CBORTag(121, []), [0, 0]]]},
 True,
 None]

Expected Result
The wallet should receive the entire CBOR and run the validator using the correct ScriptContext with the ValidityRange set correctly.
Describe the approach you would take to fix this
No response
System info
This bug was tested in 2 different settings, namely Arch Linux (version: rolling) and Ubuntu (version: 21.10). In both cases, we used the following tags:

plutus: 2721c59fd2302b75c4138456c29fd5b509e8340a
plutus-apps: c330fc60d0cd174743c95de71e5c404b5008f186

Can you check #285 if it's relevant? If so can you try bumping to HEAD of main, or this specific fix?
https://github.com/input-output-hk/plutus-apps/commit/36a74175e64e4e044739cb909cc1cadff3c2cd47

It seems that it's been solved in that fix. Thanks for helping us.


Unable to launch after installation on Win11

missing wpcap.dll
When I try to install WinPcap.exe, I reopen it and receive a new error message

I suggest you to use npcap instead of winpcap, as described in the "Required dependencies section" of Sniffnet README.
Moreover you also have to install npcap SDK and configure your LIB environment variable:

thanks


An exported experiment should always be inactive

Describe the bug
Currently, if an enrolling experiment is exported, the exported JSON file contains "state":"enrolling". So if the file is imported, the experiment status is set to "enrolling" making it not possible to edit or schedule the experiment. Therefore, I believe an exported experiment's status should always be "inactive".
To Reproduce
Steps to reproduce the behavior:

Create an experiment in UpGrade
Set the experiment status to "enrolling"
Export the experiment file in JSON
Delete the experiment and try importing the exported file
You will see the imported experiment is set to "enrolling"

Expected behavior
An exported experiment's status should always be "inactive".
Video
https://user-images.githubusercontent.com/90279765/205711101-85a999a2-91e0-47e7-a395-3ef7c95ea80b.mov

I would allow the export (you might want to document the experiment design or duplicate and modify it for some purpose), but I'd ignore "state" when importing and always set the imported experiment to "inactive".


Replication / init_replica with an already build schema

Hey @the4thdoctor - thanks for your work on this.
Migrating a large app db from maria10.2 to postgres12 using chameleon!
Is your feature request related to a problem? Please describe.
I want to migrate our new production from the schema of our web app and just have it match and sync columns assuming the with the schema I have already initialized from our web app code.
Describe the solution you'd like
What I would like is to be able to init_replica/start replication without having pg_chameleon build the schema in my destination postgres. We are using phoenix + https://github.com/elixir-ecto/ecto . It names indexes, constraints, differently to pg chameleon.
Describe alternatives you've considered
About to hop into the source code and see if i can change init_replica not to do any schema alteration at the beginning.

Added to 2.0.13


refactor: SwappingShelfs Component

Refactor Request
Describe the Refactor Request
Required:
#45 ( Search Bar )
Tab.tsx

Definition of done:

[ ] Refactor the SwappingShelfs component

What we are looking for:

Style:

Layout:

width: Fixed (600px)
height: Fill (460px)
border-radius: 16px
border: 1px

Properties:

Describe Preferred Solution
1.The Tabs will have this dimensions:

Tab Properties:

Layout:
width: Hug (114px)
height: Hug (48px)
padding: 16px, 20px, 16px, 20px
border: 0px, 0px, 2px, 0px

2.The Header of the SwappingShelfs will have these dimensions:

Properties:

Layout:
width: Fixed (600px)
height: Hug (48px)
padding: 0px, 8px, 0px, 0px
border: 0px, 0px, 1px, 0px
justify: space-between

3. The Search-Bar will have this dimensions:
#45
4. The body of the Swapping Shelfs will have this dimensions:

Properties

Layout:
width: Fill (600px)
height: Fill (412px)
padding: 20px
border-radius: 12px
gap: 20px

give to me @heronlancellot

give to me @heronlancellot

Hey @gabrieljolvm I'm assigning this to you. If you find any difficulties or lack any material from the design, let me know so I can help you out. And I'm sorry that took so long to respond!

hey yo , like we talked in discord before @gabrieljolvm , I solved this issue in the PR #166 :)


Some infrastructure pcd file missing in the cooperative label_world

When using this dataset, I found that in ./cooperative-vehicle-infrastructure/cooperative/data_info.json file, the pcd file in "infrastructure_pointcloud_path" attribute  sometimes doesn't exist at all. Is this a problem of the dataset?

When using this dataset, I found that in ./cooperative-vehicle-infrastructure/cooperative/data_info.json file, the pcd file in "infrastructure_pointcloud_path" attribute sometimes doesn't exist at all. Is this a problem of the dataset?

Can you provide more details? For example, which idx of the infrastructure_pointcloud_path.

This is normal. We associate vehicle-side and infrastructure-side frames according to a certain time difference constraint. There are some vehicle-side frames at the edge of the clip without corresponding infrastructure-side frames


Minor: Improve PruningPredicate docstrings more

Which issue does this PR close?
Related to https://github.com/apache/arrow-datafusion/issues/7869
Rationale for this change
As we have extended the PruningPredicate's features and scope, we should also improve its documentation to match.
What changes are included in this PR?
Improve the PruningPredicate doc strings to reflect its ongoing changes
Are these changes tested?
via doctests
Are there any user-facing changes?
Doc strings only. No code changes

fyi @my-vegetable-has-exploded and @yahoNanJing

👍 Nice write up.

Thank you @Ted-Jiang !


InvalidCastException, object of type 'LiteDB.EmptyPage' to type 'LiteDB.HeaderPage', on collection Insert()

Hello,
I get the following error when I'm trying to add a new object to an existing collection:
System.InvalidCastException: Unable to cast object of type 'LiteDB.EmptyPage' to type 'LiteDB.HeaderPage'.
   in LiteDB.PageService.GetPage[T](UInt32 pageID)
   in LiteDB.TransactionService.Commit()
   in LiteDB.LiteEngine.Commit()
   in LiteDB.LiteEngine.Transaction[T](String collection, Boolean addIfNotExists, Func`2 action)
   in LiteDB.LiteEngine.Insert(String collection, BsonDocument doc)
   in LiteDB.LiteCollection`1.Insert(T document)

I'm not able at the moment to provide a method to replicate this exception, because I'm not able to identify a fixed event that triggers the error. By now the only consideration I can do is that I noticed this exception occurs when the collection exceeds a number of elements, but I'm not sure about this.
Do you have any idea about this?
Actually I'm using v3.1.4 but I used to encounter this error on previous 3.x versions too.

Hi @Grul-, this it's a v3 bug on concurrency environment. Please, consider upgrade to v4 where this is fixed... or, if you want keep in v3 you will need wrap database instance with lockers..

Hi,
Upgrading to v4 solved that Exception.
But now I have another issue to solve, the objects I previously stored (with v3) don't have an Id property, but I used a property called Guid instead.
Now if I add a new Id property to my class I get an InvalidCastException during deserialization, on the other hand, if I don't add the Id property, I get a LiteDB.LiteException: 'Invalid BSON data type 'Null' on field '_id'.'. I've also tried to add the [BsonId] attribute to my Guid property but the cast is still not valid.
Is there a way to keep the existing property and to use it in the BsonDocument as Id?

Hi,
If (in v2) your class has no Id and you set Guid id in method, LiteDB just set in document (after deserialize your class to BsonDocumento) _id = yourGuid. You can check this on shell tool. In shell/json, Guid must me show as _id: { $guid: "00000000-0000-0000-0000-000000000000" }. Checks if is not a string value.
If you want add Id now, you need Id with Guid type. You can still using Insert without an Id, as here: https://github.com/mbdavid/LiteDB/blob/master/LiteDB/Database/Collections/Insert.cs#L33

I guess I found the problem. I used the shell tool to open the existing database file and (here it is!) the deserialized objects have their _id defined as "_id":{"$oid":"5a0571ac5083f01ee8b8161a"}.
So I guess the InvalidCastException is due to the _id type being $oid and not $guid. So I add the Id property, as I tried before, but I changed its type from System.Guid to LiteDB.ObjectId, now everything works fine.
Thank you for your answers!


Use API framework newly moved to cmomon with extension points

In https://github.com/hyperledger/firefly-common/pull/16 the API framework was made general for re-use in other microservices.
Specifically FFTM, which needs REST APIs for subscriptions and event streams.
The Swagger has changed a tiny bit in this:

Fixed a typo in a description
The formatting of the default request-timeout header is now 2m0s rather than 120s

Codecov Report

Merging #851 (51bb096) into main (698e8b3) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main      #851      +/-   ##
===========================================
+ Coverage   99.98%   100.00%   +0.01%
===========================================
  Files         300       297       -3
  Lines       19811     19423     -388
===========================================
- Hits        19809     19423     -386
+ Misses          1         0       -1
+ Partials        1         0       -1

Impacted Files
Coverage Δ

internal/data/data_manager.go
100.00% <ø> (ø)

internal/metrics/metrics.go
100.00% <ø> (ø)

pkg/core/batch.go
100.00% <ø> (ø)

pkg/core/contracts.go
100.00% <ø> (ø)

pkg/core/datatype.go
100.00% <ø> (ø)

pkg/core/event.go
100.00% <ø> (ø)

pkg/core/identity.go
100.00% <ø> (ø)

pkg/core/message.go
100.00% <ø> (ø)

pkg/core/namespace.go
100.00% <ø> (ø)

pkg/core/operation.go
100.00% <ø> (ø)

... and 204 more

Continue to review full report at Codecov.

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


Building Python Bindings

Hi, I'm trying to chase down a segfault and am rebuilding this without optimisations & with debug symbols, however it seems the included Makefile & CMake config doesn't build the python bindings. I have pointed CMake to the pybind11 directory as CMakeLists.txt asks, but I'm not sure what comes next, it doesn't look like pybind.cpp is included in any of the target rules (or if that's what I should be looking at at all).
How do I build and link the python bindings?
Thanks.

Figured it out, also needed to set the flag SDF_BUILD_PYTHON to true when calling CMake. This will generate a shared object that can go in your python environments site-packages directory.


Couldn't work well sbt compile using jar file

I want load model at scala to use jar file.
I confirmed that the scala code is moving with Docker.
However, an error will occur when reading with a jar file using sbt.
build.sbt
name := "mxnet_scala”
version := "1.0"

libraryDependencies ++= Seq(
  "ml.dmlc.mxnet" % "libmxnet-scala-osx-x86_64-cpu" % "0.9.3a",
  "ml.dmlc.mxnet" % "mxnet-full_2.11-osx-x86_64-cpu" % "0.9.3a"
)

code
import ml.dmlc.mxnet._
import ml.dmlc.mxnet.module.Module

val modelPrefix: String = "mx_mlp"
val modelEpoch: Int = 130

val mod = Module.loadCheckpoint(modelPrefix, modelEpoch, loadOptimizerStates = true)

error
java.lang.UnsatisfiedLinkError: /private/var/folders/6l/8ldfjblx7mjgcwf1k16yt1v80000gp/T/mxnet2627551133410770585/mxnet-scala: dlopen(/private/var/folders/6l/8ldfjblx7mjgcwf1k16yt1v80000gp/T/mxnet2627551133410770585/mxnet-scala, 1): Library not loaded: /usr/local/lib/libopencv_calib3d.2.4.dylib
  Referenced from: /private/var/folders/6l/8ldfjblx7mjgcwf1k16yt1v80000gp/T/mxnet2627551133410770585/mxnet-scala
  Reason: image not found

I examined it, but I could not find a similar problem.
Is this a problem with sbt compile rather than MXNet?

@reminisce : Please label this as : Scala, Question

well, have you installed opencv?
and you run it in docker? the OS is osx?

Just to follow up, are you still facing the same issue now? Currently we have changed the namespace into Apache org.

Hi @sugi1229 , with our 1.2 release, we have published the pre-built package on Maven, please try with our new Inference API, it's a lot easier.


Update bootstrap1.php

Scan submitted to Checkmarx

Scan submitted to Checkmarx

Checkmarx scan completed
Full Scan Details
Checkmarx Scan Summary

Severity
Count

High
0

Medium
1

Low
1

Informational
4

Violation Summary

Severity
Count

Low
1

Medium
1

Details

Lines
Severity
Category
File
Link

1
Low
Client_Insufficient_ClickJacking_Protection
app/bootstrap1.php
Checkmarx

1
Medium
Missing_HSTS_Header
app/bootstrap1.php
Checkmarx

Scan submitted to Checkmarx

Scan submitted to Checkmarx

Checkmarx scan completed
Full Scan Details
Checkmarx Scan Summary

Severity
Count

High
0

Medium
1

Low
1

Informational
4

Violation Summary

Severity
Count

Low
1

Medium
1

Details

Lines
Severity
Category
File
Link

1
Low
Client_Insufficient_ClickJacking_Protection
app/bootstrap1.php
Checkmarx

1
Medium
Missing_HSTS_Header
app/bootstrap1.php
Checkmarx

Scan submitted to Checkmarx

Checkmarx scan completed
Full Scan Details
Checkmarx Scan Summary

Severity
Count

High
0

Medium
1

Low
1

Informational
4

Violation Summary

Severity
Count

Low
1

Medium
1

Details

Lines
Severity
Category
File
Link

1
Low
Client_Insufficient_ClickJacking_Protection
app/bootstrap1.php
Checkmarx

1
Medium
Missing_HSTS_Header
app/bootstrap1.php
Checkmarx

Scan submitted to Checkmarx


More options, specially for the situation with i

Not every user of this addon is a english speaker or writer. I use it to write in romanian for example, where I dosn't mean what means in english and it gets capitalised automatically. Please give me the option to choose what I want to capitalise, I don't want to be forced to use I instead of what I need, the small i. Thanks, waiting for reply!

Instead of having a list of single letter words such as I to capitalise, a toggle/checkbox can be used instead of enable/disable capitalising I.
I can look at this as a feature request.

@McAbadBoy I've just uploaded a new version which allows you to disable autocapitalising 'i' - controlled by a new checkbox.
Let me know how it goes.

Hei, the "I" is making troubles again after the latest update, even tho the box is not checked.
On Thursday, October 1, 2020, 07:12:57 AM GMT+3, hrai<EMAIL_ADDRESS>wrote:
Closed #27 via #35.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@McAbadBoy I've pushed a new version with the fix.


Add njsscan config

This adds a github action for doing additional security scans across the runner code base
https://trello.com/c/gDswARc2/728-pentest-incident-action-find-a-vulnerability-scanner-for-javascript

Unable to use this adequately. Closing.


Add in Nightfall Image to parsed response

This adds in to activity search parsed response the image for that activity, such as the banner for this weeks nightfall: http://bungie.net/img/theme/destiny/bgs/pgcrs/shield_brothers.jpg
Also added to the skulls work @aergonaut did to pass an array of hashes with name, description and icon instead of just name.
I also added test cases for all possible uses of the activity search by advisors, and neatened up some things.

Latest run after adding in skull tests

This is really cool!
Thank you so much for taking the time out to add to this gem, it's getting really awesome to see people interested in this.
I'll go ahead and merge this request for you and update the version of the gem for everyone.
Seriously, this is some great stuff :D

Awesome thank you! I'm glad youre liking the additions, I'm adding things as I need them for my project but still trying to keep it generic enough for others to use. My nightfall use is pretty much wrapped up, so expect a xur request next!


Ensure cert file env var is string

As is, Rails.root.join('tmp', 'cacert.crt') will return a Pathname object [1].
In order to set ENV['SSL_CERT_FILE'], the value must be a string [2], meaning
we need to call .to_s [3] on the output of the Pathname in order for it to set
correctly, without erroring: TypeError: no implicit conversion of Pathname into String.
The initial PR to introduce this change [4] was extracted from a compliance PR [5]
where the value of ENV['SSL_CERT_FILE'] was being explicitly set to a string.
[1] https://ruby-doc.org/3.1.2/exts/pathname/Pathname.html
[2] https://ruby-doc.org/3.1.2/ENV.html
[3] https://ruby-doc.org/3.1.2/exts/pathname/Pathname.html#method-i-to_s
[4] https://github.com/RedHatInsights/clowder-common-ruby/pull/32
[5] https://github.com/RedHatInsights/compliance-backend/pull/1664/files#diff-274fbb104eb7e6e64d378ecb4f260a28440db9fc32916c19337dae8aa46cf85aL14

LGTM!

Removed test setup in https://github.com/RedHatInsights/clowder-common-ruby/compare/c3e4517176edb0a707702cf25db3a2745288428f..8ff0b212d7e7559a7a63659093238c6e285de7aa per @skateman


[Improvement] Support combine operation in map stage.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
When map send buffer to shuffle server,  we could reduce data transmission as much as possible through combine operations.
Because most data may be aggregated in the shuffle server, only part of the data can be combined on the map side. , so the proportion of merge may not be very high.
How should we improve?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Good idea. The combine could be supported on the different computing frameworks, like Spark/Tez/MR , right?


Significant state about forks is stored in ephemeral blockHistory

Rollbacks can only be initiated when the action reader sees them happening, with its occurrence stored in AbstractActionReader#blockHistory. If the indexState is on a forked block at startup, then the action reader will not have any way of knowing this and send a block whose blockHash does not match, and it is not trivial to determine how far the Handler needs to roll back.

It's been awhile since we discussed it, but we talked at one point about persisting blockHistory to Postgres. Would that be one potential solution here?

blockHistory is something that's in the ActionReader. Maybe we could make a abstract subclass of the AbstractActionReader called the AbstractPostgresActionReader or something, that connects to a postgres database for persisting its own state, but I think a cleaner solution would be to:

Persist lastIrreversibleBlock data to indexState
Detect this case when the ActionHandler processes its first block, and rolls back to the lastIrreversibleBlock number from indexState
Request the next block needed after the ActionHandler's rollback

Step 1 will be done in #132


Disassociate sql db from sql elastic pool

My request to disassociate sql db from sql elastic pool is successful. But when I get sql db, it still shows its elastic pool is as before.
Update a sql db:
PUT https://management.azure.com/subscriptions/{{subscriptionId}}/resourceGroups/yupsql2/providers/Microsoft.Sql/servers/yupsql2/databases/yupsql2db?api-version=2017-10-01-preview

{
    "properties": {
        "elasticPoolID":""
    },
    "location": "eastus"
}

Async Response: Successful
{
    "name": "2e026cf4-f2f8-40f3-83af-329b2bdacbcd",
    "status": "Succeeded",
    "startTime": "2020-07-03T08:50:49.287Z"
}

Get the sql db
Get https://management.azure.com/subscriptions/{{subscriptionId}}/resourceGroups/yupsql2/providers/Microsoft.Sql/servers/yupsql2/databases/yupsql2db?api-version=2017-10-01-preview
Response:
{
    "sku": {
        "name": "ElasticPool",
        "tier": "GeneralPurpose",
        "capacity": 0
    },
    "kind": "v12.0,user,vcore,pool",
    "properties": {
        "collation": "SQL_Latin1_General_CP1_CI_AS",
        "maxSizeBytes":<PHONE_NUMBER>8,
        "elasticPoolId": "/subscriptions/.../resourceGroups/yupsql2/providers/Microsoft.Sql/servers/yupsql2/elasticPools/yupsql2ep2",
        "status": "Online",
        "databaseId": "...",
        "creationDate": "2020-07-03T08:31:15.73Z",
        "currentServiceObjectiveName": "ElasticPool",
        "requestedServiceObjectiveName": "ElasticPool",
        "defaultSecondaryLocation": "westus",
        "catalogCollation": "SQL_Latin1_General_CP1_CI_AS",
        "zoneRedundant": false,
        "licenseType": "LicenseIncluded",
        "maxLogSizeBytes": 0,
        "earliestRestoreDate": "2020-07-03T08:32:36Z",
        "readScale": "Disabled",
        "readReplicaCount": 0,
        "currentSku": {
            "name": "ElasticPool",
            "tier": "GeneralPurpose",
            "capacity": 0
        }
    },
    "location": "eastus",
    "tags": {},
    "id": "/subscriptions/.../resourceGroups/yupsql2/providers/Microsoft.Sql/servers/yupsql2/databases/yupsql2db",
    "name": "yupsql2db",
    "type": "Microsoft.Sql/servers/databases"
}

I am not sure whether the call is correct. I think at least Sku need to be updated.

@weidongxu-microsoft Thanks, it works when sku_name is assigned in request


fix bug in optimizer epoch counter

this will fix the lr_scheduler not work problem

i created an issue to discuss it #625


DISC BOI

Live Game URL: https://dv-jackson.github.io/BaseGame/
Code Repository URL:https://github.com/DV-Jackson/BaseGame/deployments

Thank you for your submission! Unfortunately you were not selected as a winner.  But it was hard to determine that this was intentional as you randomized the aim, which makes the player feel they have less control over the game. You also had some overlapping text that should have been cleaned up.
Thank you for participating and we hope you enjoyed the competition!


fix(driver/kmod): always send fds to userspace in poll/ppoll syscall exit

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
This uniforms the kmod driver's behavior to that of bpf and mbpf drivers in poll/ppoll syscall
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: mrgian
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 1c6fa2e20775c864d01a935515b211fdaec51de2

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, mrgian
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

More standardization across the drivers is always welcome, thanks!

Me and @therealbobo were already investigating the failing test:

[  FAILED  ] SyscallExit.process_vm_readvX_failure

You should probably modify that test by changing
size_t res = syscall(__NR_process_vm_readv, getpid(), iov, iovcnt, iov, iovcnt, 0);
to
size_t res = syscall(__NR_process_vm_readv, getpid(), iov, iovcnt, iov, iovcnt, 1);
ie: passing a non-empty flag.

New changes are detected. LGTM label has been removed.

Looks like SyscallEnter.spliceE is failing too 🤔

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: 2d05938cb604d0dcb4c05ffb5163ca1d583853aa

/milestone next-driver

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, mrgian
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, mrgian
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Consider using JSON schema

Instead of the DSL implemented in schema.py, it would be nice to use JSON schema. This would make it easier to implement tuf in other languages, because the schemas could be easily reused.

Closing this issue as it was filed against (what is now known as) the legacy codebase: issue seems to not be relevant anymore. Please re-open or file a new issue if you feel that the issue is revelant to current python-tuf.
More details
Current source code (and upcoming 1.0 release) only contains the modern components

a low-level Metadata API (tuf.api) and
tuf.ngclient that implements the client workflow,

Legacy components (e.g. tuf.client, tuf.repository_tool, tuf.repository_lib as well as the repo and client scripts) are no longer included. See announcement and API reference for more details.

note that we're still not using json schemas so the feature request could still be valid ... it is just quite unlikely we'd go that way now.


Decide on templating language to use instead of Nunjucks and migrate existing Nunjucks templates

As highlighted by John, we may want to move away from Nunjucks and onto Liquid or WebC instead as a templating language.
Resources:
1, [2] (https://github.com/mozilla/nunjucks), [3] (https://, [4] (https://github.com/11ty/webc)

We are choosing to use Liquid as the template language.

The ADR is located in this issue https://github.com/kindsys/digital-services-coalition/issues/12


Provide ansible playbook for consul deployment

Provide an ansible playbook that deploys consul.
Purpose of this playbook is to replace the ant targets
startConsulServer
startConsulClient
startRegistrators

in services/consul/build.xml
Depending on how ansible is going to change the overall deployment, we might also include the importWhiskProperties task in the playbook.

investigate using consul_kv module instead of own script to load properties into kv store.
see http://docs.ansible.com/ansible/consul_kv_module.html


Simplify fingerprint retrieval in Certificates.create()

Codecov Report

Merging #507 (8d17021) into master (99c809f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #507   +/-   ##
=======================================
  Coverage   97.05%   97.05%
=======================================
  Files          56       56
  Lines        4106     4106
=======================================
  Hits         3985     3985
  Misses        121      121

Impacted Files
Coverage Δ

pylxd/tests/mock_lxd.py
100.00% <ø> (ø)

pylxd/models/certificate.py
100.00% <100.00%> (ø)

pylxd/tests/test_client.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

@stgraber finally ready to go.


release binaries in addition to container image

We should release binaries of individual guac components to support for non-container usage. e.g. ingesting docs directly at cmd line using guacone or guaccollect
We should use goreleaser to build the binaries and have it also package them into container image so we don't have to build the code twice.

Work completed in #918


Use UV_SYSTEM_PYTHON to allow the system Python interpreter

Hi!
I noticed this repository linked in https://github.com/astral-sh/uv/issues/3765 and see that you're having trouble with the breaking change we released. For some more context, you were using the VIRTUAL_ENV variable to point to a system installation which was a way to bypass our lack of system Python support. However, in the latest release we added stricter validation of virtual environments so if VIRTUAL_ENV points to a system Python interpreter, we will ignore it. Of course, in CI you don't care about virtual environments and it's totally fine to mutate the system Python — for that reason we added --system support a while ago.
I saw in #7309 you tried to thread through a --system flag to opt-in to using the system interpreter. That's the right idea, but it's easy to miss a case — which will cause uv to fail as it did there. Instead, you can use the UV_SYSTEM_PYTHON flag to opt-in without threading the argument anywhere. I think this should solve your problems, if it doesn't I'll definitely look into it. I've added an integration test for this pattern to be sure we have extra coverage of it https://github.com/astral-sh/uv/pull/3805. As a minor note, you could probably use UV_PYTHON with the path to the interpreter as well as we'll allow you to explicitly select the system interpreter that way.
In this pull request, I also add an upper bound to uv so you shouldn't encounter any breaking changes unless you opt-in to a later version. Note, you could avoid installing pip in CI at all and use our curl installer with a pinned uv version e.g. curl -LsSf https://astral.sh/uv/0.2.2/install.sh | sh but then you can't use version ranges so 🤷♀
Of course, feel free to disregard this if you're not interested.

@zanieb thanks for getting in touch, and submitting a potential fix for us :)

@zanieb looks like that has done the trick, many thanks :)

Feel free to ping me if something breaks in the future.

Thank you for the PR!


Use the same Pod labels across all three samples?

I'm trying to write some automation that works across all three of the samples (Ansible, Go, and Helm), and right now it seems Ansible and Go assign the label app=memcached to the Memcached Pods, which is nice... but the Helm operator assigns the label app.kubernetes.io/name=memcached.
Is there any particular reason for this? I have to add special cases in my code to account for the fact the Helm sample is unique—there are some other oddities too, but this is the main one that's been causing me pain.

(Additionally, and this may be a separate issue—the Ansible and Go operator examples use size in the spec to indicate how many instances of Memcached should be running for a given CR, whereas the Helm operator example uses replicaCount. Could we get that to be uniform across the three?)

hi @geerlingguy,
Also, see that the idea is to deprecate this one. More info: https://github.com/operator-framework/operator-sdk-samples/pull/163.
I am closing this one. However, please feel free to raise issues against the SDK repo if you need.


"Sample Permission Server" in readme

The link for "Sample Permission Server" in the README.md is a 404 link.

Thanks. Fixed.


New Client Code needs to be included in builds

at_client 3.0.4

[x] @ wavi
[x] @ buzz
[x] @ mospherpro
[x] @ rrive

@ mospherePro and @ rrive have been updated to at_client 3.0.4 and published.

@DivyaYadavareddy Please confirm on wavi and buzz.

@ wavi and @ buzz have been updated to at_client 3.0.6 and published.

Ticking off the last two and closing based on @DivyaYadavareddy update


Fix example of using ZSH integration manually

The ZSH integration file should be sourced so it's read into the current process, rather than executed in a separate ZSH process, in which case it doesn't have any impact on the current ZSH process.

I think this is a dup of #3332.

Actually this wasn't a dupe since it was incorrect in two different places (the other PR updated README.md, this updates the comments in the Zsh shell integration source code), but I myself forgot to check closed PRs for dupes when i created #3680 (I'll leave it to Mitchell to decide which PR to merge).

Actually this wasn't a dupe since it was incorrect in two different places

Oh, yup. My apologies, @Tenzer. 😔


Errors when testing the Thermostat tutorial problem

I am trying to test the Thermostat problem from the tutorial, but it fails with errors during precompilation. Is this a bug?
Reproducing the behavior

Create a Conda environment for Julia with micromamba.

micromamba env create --name julia
micromamba activate julia
micromamba install --channel conda-forge julia
Version 1.9.3 installed

Create a project environment for the tutorial with Julia:

pkg> activate Tutorial
pkg> add ReachabilityAnalysis, ModelingToolkit
ReachabilityAnalysis version: v0.22.1
ModelingToolkit version: v8.70.0

At this point the packages that fail to compile are:

SimpleNonlinearSolve
NonlinearSolve
OrdinaryDiffEq
ModelingToolkit

Error messages
If I try to run the tutorial problem, I get the following error message when running from the project directory:
$ julia --interactive --project=. Thermostat.jl                                                                                [373/1049]
ERROR: LoadError: UndefVarError: `NonlinearLeastSquaresProblem` not defined
Stacktrace:
 [1] top-level scope
   @ ~/micromamba/envs/julia/share/julia/packages/SimpleNonlinearSolve/nusbP/src/raphson.jl:66
 [2] include(mod::Module, _path::String)
   @ Base ./Base.jl:457
 [3] include(x::String)
   @ SimpleNonlinearSolve ~/micromamba/envs/julia/share/julia/packages/SimpleNonlinearSolve/nusbP/src/SimpleNonlinearSolve.jl:1
 [4] top-level scope
   @ ~/micromamba/envs/julia/share/julia/packages/SimpleNonlinearSolve/nusbP/src/SimpleNonlinearSolve.jl:30
 [5] include
   @ ./Base.jl:457 [inlined]
 [6] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::String)
   @ Base ./loading.jl:2049
 [7] top-level scope
   @ stdin:3
in expression starting at /home/gkaf/micromamba/envs/julia/share/julia/packages/SimpleNonlinearSolve/nusbP/src/raphson.jl:66
in expression starting at /home/gkaf/micromamba/envs/julia/share/julia/packages/SimpleNonlinearSolve/nusbP/src/SimpleNonlinearSolve.jl:1
in expression starting at stdin:3
ERROR: LoadError: Failed to precompile SimpleNonlinearSolve [727e6d20-b764-4bd8-a329-72de5adea6c7] to "/home/gkaf/micromamba/envs/julia/share/julia/compiled/v1.9/SimpleNonlinearSolve/jl_S3r7y0".

Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:35
 [2] compilecache(pkg::Base.PkgId, path::String, internal_stderr::IO, internal_stdout::IO, keep_loaded_modules::Bool)
   @ Base ./loading.jl:2300
 [3] compilecache
   @ ./loading.jl:2167 [inlined]
 [4] _require(pkg::Base.PkgId, env::String)
   @ Base ./loading.jl:1805
 [5] _require_prelocked(uuidkey::Base.PkgId, env::String)
   @ Base ./loading.jl:1660
 [6] macro expansion
   @ ./loading.jl:1648 [inlined]
 [7] macro expansion
   @ ./lock.jl:267 [inlined]
 [8] require(into::Module, mod::Symbol)
   @ Base ./loading.jl:1611
in expression starting at /home/gkaf/Documents/repositories/Research/gaussian_regression/Reach/Thermostat.jl:1

The content of the thermostat file is:
module thermostat

using ReachabilityAnalysis, ModelingToolkit

const A1 = 40.0
const A2 = 30.0
const B = 0.5
const t1 = 75
const t2 = 65

const var = @variables t

function thermostat_on()
    invariant = HalfSpace(t <= t1)
    @system(x' = -Bx + A1, x ∈ invariant)
end

function thermostat_off()
    invariant = HalfSpace(t >= t2)
    @system(x' = -Bx + A2, x ∈ invariant)
end

function thermostat_hybrid()
    automaton = GraphAutomaton(2)
    add_transition!(automaton, 1, 2, 1)
    add_transition!(automaton, 2, 1, 2)

    mode1 = thermostat_on()
    mode2 = thermostat_off()
    modes = [mode1, mode2]

    # transition on -> off
    guard = HalfSpace(t >= t1)
    trans1 = ConstrainedIdentityMap(1, guard)
    # transition off -> on
    guard = HalfSpace(t <= t2)
    trans2 = ConstrainedIdentityMap(1, guard)
    resetmaps = [trans1, trans2]

    return HybridSystem(automaton, modes, resetmaps, [AutonomousSwitching()])
end

function model(X0)
    H = thermostat_hybrid()
    return IVP(H, X0)
end

end  # module

As you say, these other packages fail to compile. We cannot do anything about that. NonlinearLeastSquaresProblem seems to have been added in https://github.com/SciML/SimpleNonlinearSolve.jl/pull/91. Please report the issue in that repository.

Thank you, I will monitor the issue in SciML and close the issue when it is resolved.

Hey @gkaf89 shall we close this, or is the issue still present?

No, tested again with the latest package versions, and the issue is still present. I will try to look into a solution as soon as I get the chance.

@gkaf89: I found that one of our dependencies held back some other central package, which was likely the reason why you had to use such an old version of SimpleNonlinearSolve. I just fixed that. Here is the result of up'ing the packages:
↑ SimpleNonlinearSolve v0.1.23 ⇒ v1.2.0

I suggest you try again now.

Hello, I have tested with version 1.2.0 of SimpleNonlinearSolve, and the tutorial loads and runs successfully.
Thank you very much!


Can I save the plots as pdf or png file with some customization?

Thanks for the libraries.
All the examples you have setup in the documentation directly output the plot. Is there any way I can save the plots directly as pdf or png file as in savefig of matplotlib. That would be really helpful. In addition, some option to customize the plot such as figure size or font size would be nice.

Thank you for your suggestions therimalaya. In the next release there should definitively be an input parameter in the main plot function such as saveFigure with options True or False and and additional parameters for plot format and plot size.

What I forgot to mention, there is a primitive workaround to saving the plots with a desired size and format. If you are using the Spyder editor you can run your script in an external console (Spyder main menu: "Run" -> "Configuration per file" -> In the window popping up in the section "Console"select "Execute in an external system terminal").
Running your script this way will produce plots one-by-one in own windows that can be resized and saved in the desired plot format.

Thank you Oliver for the suggestion and quick reply. I will go for this option if need to save the plots for now.

Your pull request has been merged into the master. Thank you for your contribution, Raju.


[v1.4] 2019-03-26 backports

PR #7414

This change is

test-me-please

test-missed-k8s

Hit VM download issue: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/11075/execution/node/57/log/

test-me-please

Hit VM download issue again: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/11077/

test-me-please

test-me-please

test-missed-k8s

test-me-please

test-me-please

test-missed-k8s

test-docs-please


Allow alternative shells more gracefully.

Right now it appears that this uses bash.exe. Please use wsl.exe instead, so that it opens using the default shell. There is a way to set the default shell in /tools but it's unnecessary. Unless I am mistaken, and it's more complicated.

See also https://github.com/rprichard/wslbridge/issues/17.

Waiting for new wslbridge releases.


Revert "Newsletter Subscribe"

Reverts kubevela/kubevela.github.io#1258

revert as we need to let mainters to be the account owner for compliance.