Establish TEST environment data

Describe the task
Following first PROD release, we need to determine what data is reflected in the test environment and set up a process for refreshing data.
Restoring an exact copy of the prod data in a test environment is a security flaw, as it increases the attack surface of our production system.
Using an anonymized export of the prod data, with Postgresql Anonymizer, would solve that issue.
Even with an anonymization technique, copying data should be done with caution.
Tools to generate fake data should be considered as well before making a decision.
https://www.thoughtworks.com/radar/techniques/production-data-in-test-environments

We continue to perceive production data in test environments as an area for concern. Firstly, many examples of this have resulted in reputational damage, for example, where an incorrect alert has been sent from a test system to an entire client population. Secondly, the level of security, specifically around protection of private data, tends to be less for test systems. There is little point in having elaborate controls around access to production data if that data is copied to a test database that can be accessed by every developer and QA. Although you can obfuscate the data, this tends to be applied only to specific fields, for example, credit card numbers. Finally, copying production data to test systems can break privacy laws, for example, where test systems are hosted or accessed from a different country or region. This last scenario is especially problematic with complex cloud deployments. Fake data is a safer approach, and tools exist to help in its creation. We do recognize there are reasons for specific elements of production data to be copied, for example, in the reproduction of bugs or for training of specific ML models. Here our advice is to proceed with caution.

Acceptance Criteria

[ ] Determine what data should be reflected in the test environment
[ ] Design process for replicating data from PROD to TEST

Additional context

Add any other context about the task here.
Or here

Results of the exploration:

[ ] Create a list of PII columns to anonymize
[ ] To investigate: an automated test whether each column is to be anonymized or not
[ ] Process:

pgdump prod into a new, standalone database in the -test namespace
run the anonymize function on that database (static masking)
pgdump into the -test database

[ ] What do we do with the form_change records?
good example here: https://gitlab.com/dalibo/postgresql_anonymizer/-/issues/266#note_817261637

First option: anonymized data
Second option: fake data (essentially, the dev data)
Third option: No data (just barebone contacts and operators)

We will want to address this issue once we are in PROD

Hi @pbastia - are you able to reach out to JK regarding the above comment?

Helm path to switch between real prod data in test (for debugging purposes) and anonymized data

https://gitlab.com/dalibo/postgresql_anonymizer/-/issues/266#note_817261637

Max

An example of an issue that happened where a more robust test dataset may have helped us:

It was reported on Jan 20th after a new release had just been deployed that projects could not be updated
After debugging, it was found that the error was because we'd dropped a column in a table and our commit function was trying to insert data into that column
This was because the form_data objects created before the drop contained a key/value pair for the dropped column, and that key/value pair was still part of the insert on commit flow

Just an example of something we could maybe catch in test before it makes it to prod if we create a good test dataset.

Another example happened on a prod deployment on March 14

a released failed on March 14 after some existing records were left behind by a migration script
that was because a couple of stale project revisions existed in the system, which were incorrectly created earlier and never caught.

Ideas floated around were:

restore from prod and run migrations over that data before erasing the database again and re-deploying the dev data
run a separate job that would restore from prod and run migrations in an isolated database

cc @dleard @LindsayMacfarlane

Using PGO doesn't really work for this scenario - we'll need to do this with our current pgdump - pgrestore strategy

Card description needs to be updated to reflect the results of the exploration and the action items:

deploying to -test loads the production data, runs migrations, then erases the data and redeploys the schema
enhancement idea: flag (or shipit task?) to keep prod data in test, temporarily?


Switch Composable Gets Clipped When Using Bootstrap CSS

YOUR BUG DESCRIPTION HERE
If relevant, specify:

OS: M1 Mac
Kobweb libs version: "0.20.0"
Kobweb CLI version:  0.9.18

Steps to Reproduce

Import the Bootstrap 5 CSS in the module level build.gradle.kts.

<link
    rel="stylesheet"
<EMAIL_ADDRESS>/>

Use the Silk Switch Composable

Box(Modifier.fillMaxSize(), contentAlignment = Alignment.Center) {
        //Text("THIS PAGE INTENTIONALLY LEFT BLANK")
        Switch(
            checked = true,
            onCheckedChange = {},
            size = SwitchSize.LG,
        )
    }

Observe the size of the Switch.

Expected Behavior
The Switch should render correctly at its default size of 46px × 24px as shown when Bootstrap is not included.
(the SwitchSize.LG is used, and the default text size of my Chrome browser is 16.px)

Actual Behavior
The size of the Switch gets clipped down to 39.984px × 17.984px when Bootstrap CSS is included, as attached screenshot below.

Additional Context
I have prepared the basic project that can reproduce the above issue, so please kindly refer to it if required.
https://github.com/masah517/SwitchKobwebDemo

The Chrome browser version is :  131.0.6778.140

Thanks for the detailed report!
Bootstrap's styling is fairly opinionated and unfortunately by default takes precedence over some of Silk's styles. However, you can give the Bootstrap styles lower precedence by importing it within a CSS layer, as described in this section of the README. With that change, Switch works as expected.
I'd recommend importing the Bootstrap styles in a layer regardless, but we could also fix this specific issue by removing Switch's questionable use of box-sizing: content-box, which is the property causing the styling to break.

While using a CSS layer works well for me, I was wondering if there are any plans to remove the use of
box-sizing: content-box in the future?
I agree that using layers is a better solution overall, but I’m curious if this specific property might be revisited down the line.


Ability to limit bandwidth from the Docker container?

Until https://github.com/alexjustesen/speedtest-tracker/issues/687 (if ever) is implemented, I'm really only interested in tracking latency and making sure my bandwidth hasn't dropped below a very low threshold, like from 1Gbit → 100Mbit for example.
So I've set my alerts accordingly, but the ST docker container is still using a lot of bandwidth, and I think occasionally the tests themselves interfere with my VoIP calls, streaming etc.
I've been looking into ways to artifically limit/cap the bandwidth of the container itself, using tc. I had to first edit my Docker compose file to include the required NET_ADMIN capability, e.g.:
services:
  speedtest:
    ...
    cap_add:
       - NET_ADMIN

But, even after I did that and rebuilt the containers, it just kills the container's network stack as soon as I set up the limiter. For example, I was trying to execute the following from inside the container itself:
docker exec -it speedtest sh

then
tc qdisc add dev eth0 root handle 1: htb
tc class add dev eth0 parent 1: classid 1:1 htb rate 100Mbit
tc filter add dev eth0 parent 1: protocol ip prio 1 u32 match ip src <IP_ADDRESS>/0 flowid 1:1

Is this is a dead-end? Does anyone know of a solution to cap bandwidth like this?

As far as I know it's not possible with Docker networking.
It's hacky but you could always (depending on your network setup) limit it at the router/switch or do it physically and make a custom cat5 cable with only two of the pairs.

The problem is that I run it on my NAS, so I can't use the switch or the physical cable to limit the bandwidth otherwise everything on the NAS would be capped, including file sharing, NFS etc.
Usinc tc seemed promising but I just can't get it to work. Maybe someone will figure out a way.
E.g. https://github.com/moby/moby/issues/4763#issuecomment-1774218714

I'll tinker with it later this week but tbh it's not going to be a priority.

Did some digging and couldn't find a good way to limit bandwidth at the container level. I'll keep this in mind but at this time not something I'm planning on working on.


Add some metrics to scan profile

Enhancement
I will add segment num/rowset number/ total column page number to scan profile

@DeepThinker666 Has this issue been finished yet?


[BEAM-4166] Invoke @Setup in FnApiDoFnRunner

Don't invoke Teardown because there seems to be no appropriate or necessary place to invoke it all the way up to the main() of FnHarness.
R: @bsidhom
CC: @tgroh @angoenka

Thanks! As for tracking Teardown - I think the current JIRA https://issues.apache.org/jira/browse/BEAM-4166 is an appropriate place to track it. The current PR addresses the Setup side of things; the Teardown side can be addressed either by another PR adding Teardown, or by closing the JIRA as "won't fix" with an explanation as to why calling Teardown here is unnecessary.


site permission requested every time I open a markdown note

Every time I run Open Note in Obsidian, I see the popup below.  Is there a way I can stop this?

I'm on Windows 7, Zotero 7, and am using the version of zotero-markdb-connect, as of of 8/17/24

Yeah, that's the case on other platforms too. I have been presuming that it's a Zotero feature in beta state - I presume that the stable version of Zotero will remember link permissions. (If someone knows otherwise, please let me know). So hopefully this will resolve itself upstream, but if it doesn't then I'll definitely look for a solution.

I'm using only one Obsidian Vault, and when I select System's Default Editor instead of Obsidian for Open Markdown Files using: in the MarkDB-Connect Setting, the popup doesn't appear.

For the moment, you can turn off the security feature that's raising the notification by setting security.external_protocol_requires_permission to false.
Zotero settings > Advanced tab > Config Editor" > Accept Risk and Continue > search for security.external_protocol_requires_permission > double click the string to change value to false.


nested choices/autofilter columns

Any suggestions how one would implemented nested choices/autofilter columns? I have a table with a number of columns, which can be filtered by choices, but if you select a filter for one column, then the choices for the other columns would need to be reduced based on the selection for the first column

Hello, I'm closing this issue now, but I'm willing to accept PR on this


Depending on locally defined packages

Since buildIdris is now exposed to consumer flakes, it must be possible to add custom packages to a project (via an overlay (?)), that are then callable in a TOML file. To be completely honest, I'm not exactly certain the steps to do so, but here's exactly what I'd like to do:
# myproject/other-packages/some-package.toml
name = "some-package"
[ source ]
# etc...

# myproject/myproject.toml
[ depends ]
idrisLibs = [ "some-package" ]

# myproject/flake.nix
# ???

Am I overthinking this?

On a relatively different vein, I just noticed that an idrisPackages set exists on nixpkgs, which contains (one of) the packages I want to depend on (effects). However, it doesn't work when I add it to my TOML depends.idrisLibs. Why doesn't idris2-pkgs channel from these derivations as well?

I have figured out to use callNix on an attrset containing the package. I cannot used callTOML because, well, it's not exposed :). So... I'm stuck on two points:

Should we expose callTOML so that I can write local packages in TOML instead of Nix?
How do I add my new, local package to the list of packages that idris2-pkgs reads in idrisLibs? i.e., the following doesn't work:

# myproject/myproject.toml
[ depends ]
idrisLibs = [ "some-package" ] # err: some-package doesn't exist

I have tried adding some-package to my pkgs.idrisPackages via an overlay, but that doesn't help the situation. Thoughts?

As an update, I managed to create an overlay that adds some-package to pkgs.idris2.packages, but it seems the call is internal to idris2-pkgs, because this doesn't help, either. Thoughts @claymager?

callTOML and idris2.packages (or ipkgs, as it is/was called in utils/callToml.nix) are mutually dependent. There's no way to inject new packages as it is currently defined.
I've started sketching in my callToml-tweaks branch, but haven't had the chance to do more than basic tests with it yet.

callToml-tweaks looks really good! My only issue is that local doesn't work, pointing to the utils source directory instead of the project directory. Apart from that, thank you for the time you put in for this!

This feature is built into the new idrisPackage.

documentation
example


isInt and isFloat

Hi,
For me, the validation always fails with isInt and isFloat validators even if the value is an integer or a float.
"mongoose": "4.11.5",
"mongoose-validator": "1.3.2"

Hi - validator.js only validates against strings, so if you've got a schema type as a number then it'll fail. Changing it to a string should allow it to pass. However this might not suit your schema design. You could write your own isFloat/isInt custom validators to get around validator.js limitation and use other data structures.

Thank you @leepowellcouk !


Different Healthcheck to determine epinio app (k8 pods) is ready

Is your feature request related to a problem? Please describe.
Is there a way to configure epinio push app where by we can specify when an app is healthy by:

http
port
process

very similar to CF: https://docs.cloudfoundry.org/devguide/deploy-apps/healthchecks.html#types
I think this will be a useful feature especially, the application might be doing some short init during boot up and if that task a few seconds, there will be timeout or down time when K8 just shutdown the previous app after the push.
Describe the solution you'd like
eg: epinio push --name abc-app --health-check=http
eg: epinio push --name abc-app --health-check=process
eg: epinio push --name abc-app --health-check=port:8080
Describe alternatives you've considered
nil
Additional context
This request is related to this I think:
https://github.com/epinio/epinio/issues/900

Hi @gitricko, just to understand a bit more, is this something related to the liveness/readiness probes?
Epinio uses a standard Helm chart to deploy its applications, and at the moment it doesn't support them. This can be solved customizing the chart (https://docs.epinio.io/next/references/customization/appcharts) but it's probably a common need that should be addressed.
Also this issue is probably related: https://github.com/epinio/epinio/issues/1792 from @olblak
While the port is an easy customization (it's just a value) the probes are a bit more complicated, because they could have different options (initialDelay, period, path). By the way HTTP is still the most common, and we could try to find a way to support it, maybe with parameters:

--health-check "http"
--health-check "http:8080/check"
--health-check "http:8080/?delay=5s&period=10s"
--health-check "tcp:8888"
--health-check "grcp:8000"

As @agracey mentioned this could be probably be templated with custom values, avoiding to have too many flags

-v/--chart-value health-check "tcp:8888"

I agree, we should default use http health check. In the meantime, any example how to do with appchart ? I think we can use a tutorial for how to leverage appchart

Here should be a guide on how to do that: https://docs.epinio.io/next/howtos/create_custom_appcharts
Once that the charts are available the developer can choose the one that better fits the needs of the application.

For reference on the kubernetes side: https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/#define-a-liveness-http-request
One thing for us to keep in mind is how many options there are. I like the templated string but how much logic should we add to map the templated values to all of the different options available?

I think it would be nice to cover at least some basic cases. If you need to cover more options you can still provide your own app chart.

After discussing this issue in Grooming it was decided to focus on other issues since currently exists a workaround using custom app chart.


Kendo? Grid?

Has anyone had any experience trying to integrate Kendo-UI?
I pretty well could do without the whole of Kendo except for the Kendo Grid to which there seems nothing in the same class.

The guys at Telerik are working on this. site.

Thanks, yes I'm following the kendo site.

I am going to have a go at doing this. If I work it out I will post a Wiki article about it. Regardless, we can close this issue now.


Usage of history.pushState(state, path)

    import { createHistory } from 'history'

    let history = createHistory()

...
    export default class Navbar extends React.Component {

...
        leftNavClick (event, selectedIndex, menuItem) {
            history.pushState(null, 'about')
        }

When I click the handler, the browser's address updates but the page remains the same. What is wrong?

Make sure you're using a router appropriately. If you have further problems:

For questions and support, please visit our channel on Reactiflux or Stack Overflow. The issue tracker is exclusively for bug reports and feature requests.


Maps API Javascript Files Not Found

I have an integration with a 5.1.3 version of GEE. There are a number of files available on the server that are not present in the same location for the latest release. The files expected are:
mapfiles/390/v3/geometry.js
mapfiles/390/v3/main.js
mapfiles/390/v3/common.js
mapfiles/390/v3/map.js
mapfiles/390/v3/overlay.js
mapfiles/390/v3/util.js
mapfiles/390/v3/infowindow.js
mapfiles/390/fusion_map_obj_v3.js
It would be nice to have these files moved to the expected location or links pointing to the new and updated version of those files.

The missing files are attached. They need to go under earth_enterprise/src/maps/mapfiles.
fusionmapsapis.zip

Saving @tst-ahernandez's original comment for posterity:
I have an integration with a 5.1.3 version of GEE. There are a number of files available on the server that are not present in the same location for the latest release. The files expected are:
mapfiles/390/v3/geometry.js
mapfiles/390/v3/main.js
mapfiles/390/v3/common.js
mapfiles/390/v3/map.js
mapfiles/390/v3/overlay.js
mapfiles/390/v3/util.js
mapfiles/390/v3/infowindow.js
mapfiles/390/fusion_map_obj_v3.js
It would be nice to have these files moved to the expected location or links pointing to the new and updated version of those files.


Fix first incorrect delegate call

Hey! Thanks for this great control.
Just came across a small bug, looks like observers don't run in the initializer for swift which causes a first delegate call with 0.0 (Factor's default value) instead of the value you can put in the initializer for selected section.
Let me know what you think 👍

Thanks, @jugutier.


Resize image

How I can resize image before upload?

Here's something on stackoverflow might help you.


Kali Linux Raspberry Pi Zero - Nothing Happens

Installed the deps, ran the script in configure and plugged in my USB drive which was a SanDisk Cruzer Fit, left it in while working, once done wanted to test if unplug2shutdown was working so pulled the USB drive and waited. 20min later still up and running so ran script again in case config files in root dir got deleted or something, removed USB again and still nothing. After another 15min i issues a shutdown and within 20 seconds shutdown.

Hello, this is a really old proof of concept which has never really been maintained.
Thanks for understanding,
Claudio


Flutter Video Player plugin installation steps update

For installation steps in Flutter Video Plugin , for Android the only  step mentioned is to add <uses-permission android:name="android.permission.INTERNET"/> to Manifest.
But, the plugin also requires Java 8. So it would nice if documentation was updated to reflect to add the following lines to build.gradle , as default Flutter project does not have it and it causes exception.
compileOptions {
    sourceCompatibility JavaVersion.VERSION_1_8
    targetCompatibility JavaVersion.VERSION_1_8
}

Hi, thank you for sharing this.
I'm having hard time to initialize video with it and I think it's worth for trying this.
Could you share your  build.gradle and which gradle is this?
I'm sorry for silly question since I'm from iOS land.

If you create a new Flutter project with flutter create foo this is already included in the Gradle file.
See https://github.com/flutter/flutter/wiki/Workarounds-for-common-issues#generated-project-files-outdated for how to update existing projects.
I'm closing because I don't think there is anything specific to do.
Feel free to add a comment if you disagree to have the issue reopened.


bugfix: bm register

这个 PR 实现什么功能/修复什么问题:
优化托管物理机的体验
是否需要 backport 到之前的 release 分支:
release/2.12
/area baremteal
/cc @swordqiu @zexi

Codecov Report

Merging #3645 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #3645      +/-   ##
=========================================
- Coverage    6.95%   6.95%   -0.01%
=========================================
  Files         499     499
  Lines       83709   83745      +36
=========================================
- Hits         5823    5821       -2
- Misses      77390   77428      +38
  Partials      496     496

Flag
Coverage Δ

#aFlag
6.95% <0%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/baremetal/manager.go
0.51% <0%> (-0.01%)
:arrow_down:

pkg/baremetal/tasks/bm_register.go
0% <0%> (ø)
:arrow_up:

pkg/cloudcommon/db/lockman/inmemory.go
64.63% <0%> (-2.44%)
:arrow_down:

Continue to review full report at Codecov.

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

/lgtm

/lgtm
/approve


Add Quay autobuild to support testing-in-docker

Validate requires a copy of womtool. To do this, we need to get our docker image into the cloud. @rexwangcc Are you up for this?

I will start working on this issue once I get permissions on quay.io

Done: https://quay.io/repository/broadinstitute/cromwell-tools?tab=tags

Done: https://quay.io/repository/broadinstitute/cromwell-tools?tab=tags


centurion: add package

Before adding new features and new modules, please go to issues to submit the relevant feature description first.
Write good commit messages and use the same coding conventions as the rest of the project.
Please commit code to dev branch and we will merge into master branch in feature
Ensure your edited codes with four spaces instead of TAB.

增加新特性和新模块之前，请先到issues提交相关特性说明，经过讨论评估确认后，再进行相应的代码提交，避免做无用工作。
编写友好可读的提交信息，并使用与工程代码相同的代码规范，代码请用4个空格字符代替tab缩进。
请提交代码到dev分支，如果通过，我们会在特定时间合并到master分支上。
为了规范化提交日志的格式，commit消息，不要用中文，请用英文描述。

what can I do about failing checks?

https://github.com/xmake-io/xmake-repo/blob/a1a5ca1ff583456ed9519703bdcdf00ce3fa2c0b/packages/l/libsdl/xmake.lua#L249-L255

you mean I should add SDL_MAIN_HANDLED to tests?

oh oops, I copied the function from some package

maybe add:
if is_plat("wasm") then
    add_configs("shared", {description = "Build shared library.", default = false, type = "boolean", readonly = true})
end

since the other libsdl packages have it

maybe add:
if is_plat("wasm") then
    add_configs("shared", {description = "Build shared library.", default = false, type = "boolean", readonly = true})
end

since the other libsdl packages have it

done

what does it want now

@c8ef https://github.com/xmake-io/xmake-repo/commit/e167e3a9cc6d83706c50908306f7227c1d47f119 break android ci :(

we can use io.replace or add_patches to fix it.


Add unicode mapping in the font cmap to have correct chars when printing in pdf (bug 1778484)

It aims to fix https://bugzilla.mozilla.org/show_bug.cgi?id=1778484.

/botio test

/botio browsertest

/botio browsertest

Interesting, it's almost what I just pushed, modulo the grammar nits I'll fix.
I replaced the Object.create(null) in favor of a Map because I considered that it was useless to convert keys into numbers (thanks to the x | 0 trick).
I was wondering if we should move from Object to Map in this file in order to avoid useless conversion (I suppose that the Object are here because they've been written before Map exists.

/botio test

What's up with the thousands of test failures on Linux?
There's also some movement on Windows, which doesn't look like clear improvements to me.

/botio test

We'd like to have this patch in Firefox but of course the overall rendering is a way more important that the issue this patch is supposed to fix.
So we want to merge it (except if something is really bad), test it on nightly (with the help of the nightly community) and if the feedback is negative then we'll back it out.
I looked at a lot of tests and I agree that sometimes, when keeping the t key pressed I can see some differences, but my feeling (I'm not sure that the "feeling" is a concept that we can truly rely on...) is that the user should see a small improvement (at least with screens with a low resolution) because there are a little more black pixels.

/botio makeref

/botio-linux makeref

@Snuffleupagus do you know why the lint is failing ?
About the timeout on linux I suspect a regression because of a patch around gpu acceleration.
Do you think we should pref off gfx.canvas.accelerated ?

do you know why the lint is failing ?

No idea, unfortunately :-(
Given that gulp lint works just fine locally, and when run in the GitHub Action, I'm at a loss here.
If this doesn't "magically" go away

About the timeout on linux I suspect a regression because of a patch around gpu acceleration.
Do you think we should pref off gfx.canvas.accelerated ?

That sounds reasonable for now, since I recall having seen issues (in Bugzilla) about that mode being slower.

/botio-windows lint

/botio-windows lint

/botio-windows lint

/botio-windows lint

/botio-windows lint

/botio-windows lint

/botio-windows lint

/botio-windows lint

/botio-windows lint

@Snuffleupagus do you have an idea on how to fix that:
http://<IP_ADDRESS>:8877/e90d7319a77005a/output.txt ?

My only idea would be to try what's suggested in the log, i.e. using npm install --force instead.

I think it's because of eslint-plugin-prettier:

https://github.com/mozilla/pdf.js/blob/41b2f52f70bba1c4df62ece070487d59d970f775/package.json#L28
https://searchfox.org/mozilla-central/rev/5ea9694f10703efbd2f8c25c107c096309fb8fbb/tools/lint/eslint/eslint-plugin-mozilla/package.json#41

With node 17.4.0 the dep mismatch is ok when it's a problem in 18.6.0

/botio-windows lint

Strange, I tried the LTS wednesday and it was failing but it's now working.

/botio-linux lint

/botio-linux lint

/botio-linux makeref

Both bots are now using node 16.6 (LTS) and to bypass the versions mismatch around eslint-plugin-prettier  I set this:
npm set legacy-peer-deps true -g

/botio-linux makeref

/botio-linux makeref


Unified vs Classic Interface regarding Knowledge Articles

Given that the command bar is different for Knowledge Articles in the Classic vs Unified Interface it's unclear how one can explicitly modify the command bar in one and not the other. How can one modify the Command bar in the Unified interface for Knowledge Articles?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 43f6a97f-ba86-4666-76c0-967b19cd236c
Version Independent ID: bc485d43-72c5-1e80-1999-b8c40cc8f82f
Content: Customize commands and the ribbon (Developer Guide for Dynamics 365 for Customer Engagement)
Content Source: ce/developer/customize-dev/customize-commands-ribbon.md
Service: crm-online
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@click-here If you want to add a button to the command bar for knowledge articles in Unified Interface, export the Customer Service app Sitemap and customize the command bar.

Thanks for contacting us. As we haven’t heard back from you, I am closing this issue. If you need further help, please reopen the issue or open a new issue.


Add Security Levels to CBOM

Currently the NIST security level and the classic security levels are not reported by the plugin.
Useful sources

https://eprint.iacr.org/2020/1322.pdf

Will not be implemented. The CBOM generated by the scanner should only repot static values. Security levels are debatable and not constant over time.


Add find image position function

Add find image position function since sometimes we just need the position and don't need to click it.
The function click_by_image has parameters "button='left', double_click= False" but not used. Update this function to implement.

@Abel-csd is it possible to make the second parameter not only image path but also allow to be a PIL image? You can judge if the type is a str first

I also would like to suggest that you add an optional parent PIL img parameter in find_image_position so that it can search a sub image in a larger parent image. If this parameter is not provided then we take a screenshot and use it as parent image. We can even add a dedicated find_image_on_screen method to invoke find_image_pisition with none parent image parameter

Thanks for your suggestions, I've updated the code.

For click_by_image, since you can only click on screen, it might be not necessary to have the parent image parameter


PathTooLongException again on 1.0.10 and 1.0.11 on windows

Verified that 1.0.9 works.

Verified workaround by installing 1.0.9 and update works. Bring up to 1.0.11

Can you post some more information, such as the whole error stack if you have it?

2015-09-03 09:38:22> Program: Starting Squirrel Updater: --install .
2015-09-03 09:38:22> Program: Starting install, writing to C:\Users\paloaltonetworkssupr\AppData\Local\SquirrelTemp
2015-09-03 09:38:22> Program: About to install to: C:\Users\paloaltonetworkssupr\AppData\Local\atom
2015-09-03 09:38:22> Program: Install path C:\Users\paloaltonetworkssupr\AppData\Local\atom already exists, burning it to the ground
2015-09-03 09:38:22> CheckForUpdateImpl: Failed to load local releases, starting from scratch: System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\paloaltonetworkssupr\AppData\Local\atom\packages\RELEASES'.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
at Squirrel.Utility.LoadLocalReleases(String localReleaseFile)
at Squirrel.UpdateManager.CheckForUpdateImpl.d__2.MoveNext()
2015-09-03 09:38:22> CheckForUpdateImpl: Reading RELEASES file from C:\Users\paloaltonetworkssupr\AppData\Local\SquirrelTemp
2015-09-03 09:38:22> CheckForUpdateImpl: First run or local directory is corrupt, starting from scratch
2015-09-03 09:38:23> ApplyReleasesImpl: Writing files to app directory: C:\Users\paloaltonetworkssupr\AppData\Local\atom\app-1.0.11
2015-09-03 09:38:28> IEnableLogger: Failed to install package to app dir: System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.
at System.IO.PathHelper.GetFullPathName()
at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
at System.IO.Path.GetFullPathInternal(String path)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractEntry(ZipEntry entry)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(Stream inputStream, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime, Boolean isStreamOwner)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(String zipFileName, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime)
at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClass6_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Squirrel.Utility.d__331.MoveNext() 2015-09-03 09:38:28> Unhandled exception: System.AggregateException: One or more errors occurred. ---> System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.    at System.IO.PathHelper.GetFullPathName()    at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)    at System.IO.Path.GetFullPathInternal(String path)    at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractEntry(ZipEntry entry)    at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(Stream inputStream, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime, Boolean isStreamOwner)    at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(String zipFileName, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime)    at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClass6_0.<<installPackageToAppDir>b__0>d.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Squirrel.Utility.<LogIfThrows>d__331.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Squirrel.UpdateManager.ApplyReleasesImpl.d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Squirrel.UpdateManager.d__10.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Squirrel.Update.Program.d__4.MoveNext()
--- End of inner exception stack trace ---
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at System.Threading.Tasks.Task.Wait()
at Squirrel.Update.Program.executeCommandLine(String[] args)
at Squirrel.Update.Program.main(String[] args)
---> (Inner Exception #0) System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.
at System.IO.PathHelper.GetFullPathName()
at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
at System.IO.Path.GetFullPathInternal(String path)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractEntry(ZipEntry entry)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(Stream inputStream, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime, Boolean isStreamOwner)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(String zipFileName, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime)
at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClass6_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Squirrel.Utility.d__33`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Squirrel.UpdateManager.ApplyReleasesImpl.d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Squirrel.UpdateManager.d__10.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Squirrel.Update.Program.d__4.MoveNext()<---

2015-09-03 09:43:40> Program: Starting Squirrel Updater: --install .
2015-09-03 09:43:40> Program: Starting install, writing to C:\Users\myveryverylonguserna\AppData\Local\SquirrelTemp
2015-09-03 09:43:40> Program: About to install to: C:\Users\myveryverylonguserna\AppData\Local\atom
2015-09-03 09:43:40> CheckForUpdateImpl: Failed to load local releases, starting from scratch: System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\myveryverylonguserna\AppData\Local\atom\packages\RELEASES'.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
at Squirrel.Utility.LoadLocalReleases(String localReleaseFile)
at Squirrel.UpdateManager.CheckForUpdateImpl.d__2.MoveNext()
2015-09-03 09:43:40> CheckForUpdateImpl: Reading RELEASES file from C:\Users\myveryverylonguserna\AppData\Local\SquirrelTemp
2015-09-03 09:43:40> CheckForUpdateImpl: First run or local directory is corrupt, starting from scratch
2015-09-03 09:43:40> ApplyReleasesImpl: Writing files to app directory: C:\Users\myveryverylonguserna\AppData\Local\atom\app-1.0.11
2015-09-03 09:43:47> IEnableLogger: Failed to install package to app dir: System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.
at System.IO.PathHelper.GetFullPathName()
at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
at System.IO.Path.GetFullPathInternal(String path)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractEntry(ZipEntry entry)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(Stream inputStream, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime, Boolean isStreamOwner)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(String zipFileName, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime)
at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClass6_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Squirrel.Utility.d__331.MoveNext() 2015-09-03 09:43:47> Unhandled exception: System.AggregateException: One or more errors occurred. ---> System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.    at System.IO.PathHelper.GetFullPathName()    at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)    at System.IO.Path.GetFullPathInternal(String path)    at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractEntry(ZipEntry entry)    at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(Stream inputStream, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime, Boolean isStreamOwner)    at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(String zipFileName, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime)    at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClass6_0.<<installPackageToAppDir>b__0>d.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Squirrel.Utility.<LogIfThrows>d__331.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Squirrel.UpdateManager.ApplyReleasesImpl.d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Squirrel.UpdateManager.d__10.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Squirrel.Update.Program.d__4.MoveNext()
--- End of inner exception stack trace ---
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at System.Threading.Tasks.Task.Wait()
at Squirrel.Update.Program.executeCommandLine(String[] args)
at Squirrel.Update.Program.main(String[] args)
---> (Inner Exception #0) System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.
at System.IO.PathHelper.GetFullPathName()
at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
at System.IO.Path.GetFullPathInternal(String path)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractEntry(ZipEntry entry)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(Stream inputStream, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime, Boolean isStreamOwner)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(String zipFileName, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime)
at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClass6_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Squirrel.Utility.d__33`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Squirrel.UpdateManager.ApplyReleasesImpl.d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Squirrel.UpdateManager.d__10.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Squirrel.Update.Program.d__4.MoveNext()<---

Duplicate of #5109, please subscribe there for updates.


Run processes serially on Buildkite

This is sad but it's not uncommon to run into issues with typical instance sizing at SEEK. This should also alleviate contention on the shared publishing agents.
If we want to be clever in future we could inspect BUILDKITE_AGENT_META_DATA_AWS_INSTANCE_TYPE.

Longer term I'd like to explore running these in-process rather than spawning child processes. Unfortunately these tools generally don't provide programmatic APIs that match the convenience of their CLIs.


use DNS srv format for istio sni names in mTLS connections

We were previously using cluster name as SNI, but cluster names
cannot be used in Gateway/virtual service routing.
Signed-off-by: Shriram Rajagopalan<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
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:

pilot/pkg/networking/OWNERS

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

cc @baodongli

@rshriram: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
e9e4e281a7ab8a4c4ee8a1f9cc99b19efd89479e
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
e9e4e281a7ab8a4c4ee8a1f9cc99b19efd89479e
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
e9e4e281a7ab8a4c4ee8a1f9cc99b19efd89479e
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
e9e4e281a7ab8a4c4ee8a1f9cc99b19efd89479e
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
e9e4e281a7ab8a4c4ee8a1f9cc99b19efd89479e
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
e9e4e281a7ab8a4c4ee8a1f9cc99b19efd89479e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
02e040f1136f2a9f75c6db5dae48a9ad845da079
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
02e040f1136f2a9f75c6db5dae48a9ad845da079
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
02e040f1136f2a9f75c6db5dae48a9ad845da079
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
02e040f1136f2a9f75c6db5dae48a9ad845da079
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
02e040f1136f2a9f75c6db5dae48a9ad845da079
link
/test istio-unit-tests

prow/istio-integ-local-tests.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
02e040f1136f2a9f75c6db5dae48a9ad845da079
link
/test istio-unit-tests

prow/istio-integ-local-tests.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test istio-integ-k8s-tests

prow/e2e-mixer-no_auth-mcp.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test istio-integ-k8s-tests

prow/e2e-mixer-no_auth-mcp.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test e2e-mixer-no_auth-mcp

prow/e2e-mixer-no_auth.sh
b7f4a2529e7c1b6658e0153c59e80dea7f6c2cbe
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/approve
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: GregHanson
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:

pilot/pkg/networking/OWNERS [GregHanson]

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

/test e2e-mixer-no_auth


[BUG] Back gesture in photo info page

Describe the bug
Not sure if it's a bug, but when in the picture info, I can't use the back gesture...
To Reproduce
Steps to reproduce the behavior:

Go to image
Slide up to show info
Try the back gesture

Expected behavior
I think it would be intuitive to have the info disappear with the back gesture
Screenshots
https://github.com/IacobIonut01/Gallery/assets/4184001/93d7fec0-7b88-450c-9f6c-5ba2b99d5683
Smartphone (please complete the following information):

Device: nothing phone 1
OS: android 13

Confirmed.
It's a compose bug and I'm still struggling to understand why and how to fix this.
Thanks


Addressing memory leaks

Related to https://github.com/mochajs/mocha/issues/1991
Is there any way we could reasonably expose the the runner (from mocha.run(..)) so that we can delete suite properties as mentioned in the linked issue?

+1 On this one. This hurts on large test suites.

@rcollette @tarjei is issue still actual?

@maksimr good question. It seems to me to at least have become better. I've done a number of tricks to improve test performance, and I'm not sure if the memory leaks are as big as they used to.
But, I think that is a very good idea to be able to access the runner for testsuite cleanups.
T

Thanks,
I think this problem could be solved on project(client) side.
You can simple put code below to your test-main.js
var orgMochaRun = mocha.run

mocha.run = function (fn) {
  var runner = orgMochaRun(fn)

  runner.on('suite end', function (suite) {
    delete suite.tests
    delete suite._beforeAll
    delete suite._beforeEach
    delete suite._afterEach
    delete suite.ctx
    delete suite._afterAll
  })

  return runner
}

@maksimr, you didn't give me much time to response.  It is absolutely still an issue and I have also tried the delete code above and it didn't work.  In fact, we finally wound up having to split our unit tests (4k+) into separate runs.   Kind of a problem when you have two groups and you want to use a .only as the group not containing the .only will still run all tests.

@rcollette as I understand memory leaks inside mocha's runner and you want get access to this runner inside your tests, yes?
Code above only show how get access to runner from test code
var orgMochaRun = mocha.run
var runner;

mocha.run = function (fn) {
  runner = orgMochaRun(fn)
  return runner
}

Thanks


Can't find words hyphenated across lines

Attach PDF file here: 1503.01817v2.pdf (from https://arxiv.org/pdf/1503.01817v2.pdf)
Configuration:

Web browser version: Chrome 80.0.3987.158
Operating system version: Chrome OS 80.0.3987.158
PDF.js version: 2.5.40 https://mozilla.github.io/pdf.js/web/viewer.html
Is a browser extension: no (but also reproduces on https://chrome.google.com/webstore/detail/pdf-viewer/oemmndcbldboiebfnladdacbdfmadadm version 2.3.164)

Steps to reproduce the problem:

Open the attached PDF.
Search for "caption".

What is the expected behavior?
There should be 2 matches, one on the line "captions extracted from the videos", and another straddling the two lines "boxes, segmentations of objects and faces, and image cap-" and "tions are not yet available for the YFCC100M at present,". This is what you get in Chrome's built-in PDF viewer.
What went wrong?
Only the first match is found, as you can see in this screenshot:

Analysis
This seems to happen because pdf.js doesn't join lines as well as Chrome's PDF viewer. If you copy the two lines using pdf.js, you get the following string, with an extraneous dash between "cap" and "tions":
boxes, segmentations of objects and faces, and image cap-tions  are  not  yet  available  for  the  YFCC100M  at  present,

whereas if you copy the same text using Chrome's PDF viewer, you get the following string without a hyphen:
boxes, segmentations of objects and faces, and image captions are not yet available for the YFCC100M at present,

(If you search for "cap-tion" with a hyphen in pdf.js, it does match, but that's not particularly useful.)
Possible solutions
The cleanest solution would be to remove line break hyphens in the middle of words when joining lines together, like Chrome's pdfium. This would fix both copy/paste and Find in Document.
A hackier alternative would be to just make Find in Document ignore hyphens (so "caption" would match "cap-tion"), but then there would still be extraneous hyphens when you copy text that was broken across lines midword.
Additional information
Related to #4742 (Can't search across lines in some PDFs) and its PR #5783 (Improve Copy/Paste).
The PDF was generated with standard software for scientific papers: [1.5 pdfTeX-1.40.12 / LaTeX with hyperref package]

Hi, I'd like to work on this.
Can anyone mentor me through this bug?


Getting latest changes from old fork

[x ] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[x ] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[ x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Hey, it looks like something went wrong when you created this PR.  Please take another look at the contribution guidelines.

This fork is way behind master.  It needs to be re-synced.


TypeError: Cannot set property 'runtimeChunk' of undefined

I do need to install react-app-rewired module in the project. However, the current version tunrned into conflict with webpack module. So I went to install the next version:
npm i<EMAIL_ADDRESS>
but when I run: react-app-rewired start I get the error:
yarn run v1.22.10
warning ..\..\..\package.json: No license field
$ react-app-rewired start
C:\Users\user1\Desktop\merge\editor\config-overrides.js:3
    config.optimization.runtimeChunk = false;
                                     ^

    TypeError: Cannot set property 'runtimeChunk' of undefined
        at Object.webpack (C:\Users\user1\Desktop\merge\editor\config-overrides.js:3:38)
        at Object.<anonymous> (C:\Users\user1\Desktop\merge\editor\node_modules\react-app-rewired\scripts\start.js:22:15)
        at Module._compile (internal/modules/cjs/loader.js:1068:30)
        at Object.Module._extensions..js (internal/modules/cjs/loader.js:1097:10)
        at Module.load (internal/modules/cjs/loader.js:933:32)
        at Function.Module._load (internal/modules/cjs/loader.js:774:14)
        at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
        at internal/main/run_main_module.js:17:47
    error Command failed with exit code 1.
    info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

the config-override.js looks like this:
module.exports = {
  webpack: (config, env) => {
    config.optimization.runtimeChunk = false;
    config.optimization.splitChunks = {
      cacheGroups: {
        default: false,
      },
    };

    config.output.filename = "static/js/[name].js";

    config.plugins[5].options.filename = "static/css/[name].css";
    config.plugins[5].options.moduleFilename = () => "static/css/main.css";
    return config;
  },
};

How do I fix this?

This looks like a config issue vs an issue with this repo - please update to the latest and we'll re-open as needed.


Script giving "Runtime error"

I have successfully installed the script and followed the command to merge all cbz into one but it gave this error

I have tried both in virtual and normal environment
PS- I have a alias "py" as python3

Hi! Thanks for bringing this to my attention. I managed to replicate that error by having a smaller number of files to merge than the number of cpu's detected by the program. I just uploaded a fix for this bug.
I also noticed that you are using a Mac, which I haven't been able to test the script on, so let me know if you run into any other problems. I would be happy to work them out with you.

Hi! Thanks for bringing this to my attention. I managed to replicate that error by having a smaller number of files to merge than the number of cpu's detected by the program. I just uploaded a fix for this bug.
I also noticed that you are using a Mac, which I haven't been able to test the script on, so let me know if you run into any other problems. I would be happy to work them out with you.

Hello, I tried this again, this is running but giving another error

Hi! Could you please share with me the full contents of your /User/nitchi/Desktop/manga directory? Some file in there must be causing the error, but I'm not sure how to go around it. Seeing what files are there would help me a great deal to troubleshoot the problem.

Hello, you were right, the problem was some content in the manga directory because when I tried this on different one, it merged it successfully. But I want to know what is this zipper folder mean. because it has the same content as the extracted manga.zip folder

I am so glad that you got it working! The zipper folder is meant to hold all the files in a single folder, without sub-directories, so that the script can merge them in the correct order. It is an intermediate step, so if you don't need it, feel free to delete it.
I'm really happy that you found my script useful!
I'll mark this as closed. If you encounter any further problems, please open a new issue to let me know.


Add Spanish i18n

Just a little pull request adding spanish (spain) translation for Telescope in es.js file

We're switching to LingoHub to help manage translations. If you'd like to be added as a translator to the project, could you join the #i18n channel on Telescope's Slack?


Update cobra package

Signed-off-by: Vijayan Balasubramanian<EMAIL_ADDRESS>Description
Update cobra to latest
Issues Resolved
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has documentation added

[ ] Commits are signed as per the DCO using --signoff

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

Codecov Report

Merging #62 (8e496d3) into main (6cc9916) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #62   +/-   ##
=======================================
  Coverage   67.39%   67.39%
=======================================
  Files          32       32
  Lines        1865     1865
=======================================
  Hits         1257     1257
  Misses        502      502
  Partials      106      106

Flag
Coverage Δ

opensearch-cli
67.39% <ø> (ø)

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


Building romana-1.3.1 failing: git.js no such file or directory

Hello,
I'm trying to build the project from the v1.3.1.tar.gz sources of the github releases page. After installing all necessary dependency the command sudo REAL_BUILD=y make dpkg failes with the following error:
...
Running "requirejs:dist" (requirejs) task
>> Error: Error: ENOENT, no such file or directory '/home/vagrant/romana-1.3.3/manage/app/scripts/git.js'
>> In module tree:
>>     main
>>       app
>>
>>     at Object.fs.openSync (fs.js:439:18)
Warning: Task "requirejs:dist" failed. Use --force to continue.
...

I'm trying to compile it on a vagrant ubuntu 12.04 VM. Building the sources from the master branch works, but from my understanding the version in the master branch is v1.2.2.
Moreover are the realeses on the github release page considerd to be "stable" releases - ready for use in production?
Best,
Martin

When I build the package from the git repository and to a checkout of the 1.3.1 tag, the build is successful. So I'm using this way to build my packages.


Java client: Improve fragile tests

Some tests were using constants instead of the generated bindings.

bors r+


Node.js  Loaders Team Meeting 2023-08-29

Time
UTC Tue 29-Aug-2023 18:00 (06:00 PM):

Timezone
Date/Time

US / Pacific
Tue 29-Aug-2023 11:00 (11:00 AM)

US / Mountain
Tue 29-Aug-2023 12:00 (12:00 PM)

US / Central
Tue 29-Aug-2023 13:00 (01:00 PM)

US / Eastern
Tue 29-Aug-2023 14:00 (02:00 PM)

EU / Western
Tue 29-Aug-2023 19:00 (07:00 PM)

EU / Central
Tue 29-Aug-2023 20:00 (08:00 PM)

EU / Eastern
Tue 29-Aug-2023 21:00 (09:00 PM)

Moscow
Tue 29-Aug-2023 21:00 (09:00 PM)

Chennai
Tue 29-Aug-2023 23:30 (11:30 PM)

Hangzhou
Wed 30-Aug-2023 02:00 (02:00 AM)

Tokyo
Wed 30-Aug-2023 03:00 (03:00 AM)

Sydney
Wed 30-Aug-2023 04:00 (04:00 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Loaders Team+Meeting+2023-08-29&iso=20230829T1800
or https://www.wolframalpha.com/input/?i=06PM+UTC%2C+Aug+29%2C+2023+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1rn7WKxmylE5fK2vO5TVvG7Bn3LVhnVAhncpUrYU6RR4/edit

Agenda
Extracted from loaders-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
Invited

Loaders team: @nodejs/loaders

Observers/Guests
Notes
The agenda comes from issues labelled with loaders-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting

link for participants: https://zoom.us/j/97506450082

For those who just want to watch: https://www.youtube.com/c/nodejs+foundation/live

So https://github.com/nodejs/node/pull/49144 is the last PR to finish Milestone 2, after which we will have finished everything that we need to finish to potentially go stable. Should we perhaps use this meeting time to screenshare and try to resolve any issues with that PR? @JakobJingleheimer @izaakschroeder?

That would be helpful! I think @aduh95 wrote the bit of code that seems relevant to the issue I'm experiencing.


AJAX / Duplicated CSS file

I have a page where some parts of the content is fetched via AJAX.
When all assets are minified, the ajax content includes link to a css file which has already been included in the minified file. This is causing an issue if there are customized style defined for a specific class, and that is overwritten by this css file.
Eg, the ajax content displays bootstrap Progress widget which requires bootstrap.css. This css file has already been minified. In the html output for the ajax section there's additional html <link href="/minify/(random string).css" rel="stylesheet">.
If concatCSS property is set to false, this line does NOT appear in the output and the page is displayed as intended. It seems somehow the ajax page knows that the file has already been included in the main page. Can this be fixed when concatCss property is set to true? Or is there a workaround solution not to minify asset files?

As I understand, you can exclude such assets from minification, by using the component's configuration property 'excludeFiles' or even exclude specific AssetBundle by using excludeBundles

Thanks @kwazaro . Using excludeFiles is good for now as workaround solution - the specific file is not included in the minify and hence not duplicate in the ajax content.
It'd be nice to permanently fix so that the file can be included in the minified file and not duplicated.


Adding maintainers and maintainers guidelines docs

Signed-off-by: Tihomir Surdilovic<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it:
Adds the maintainers info to this repo and maintainers guidelines. All SDK repos should have these.
Special notes for reviewers:
Additional information (if needed):

@ricardozanini fixed the copy/paste typo


ngrx entity

there is no use if the new library @ngrx/entity.
is it in puprpose?

Hi @azulay7 !
Thank you for your feedback!
It was not really on purpose, it was not used because there is no example which implements CRUD functionality so there was no need.
It would make sense to create such an example where we have a list of items which can be opened, edited and saved and use @ngrx/entity for it.
I will create issue for it and maybe somebody will contribute implementation.


Support listing/getting/creating/updating tables/datasets/jobs in other projects

This fixes #1123
@tswast feel free to have a look and comment whether this meets your expectations.
/cc @lesv

Coverage increased (+0.002%) to 85.197% when pulling d6576cef43f544255a6e506e9be66c44fdfdcc8c on mziccard:bigquery-project-id into 7cd60df2f29f6e9e10729125b4023106bcf1ab1f on GoogleCloudPlatform:master.

LGTM

LGTM


Change the uievents spec link to the living-draft

closes #2
@RByers I was not sure whether I can also update the reference link or not as it was automatically created. Was it possible?

This LGTM, I think the official reference has to be to a TR version.  But having the explicit  links be to the live version seems like a good idea to me.


Add docs about installation and Project Explorer features

@edmundreinhardt This docs PR is ready for review. Easiest way to check out the changes is to run npm run docs using this branch.


make weakly stable builds disableable

cc @eparis @lavalamp

should we do the same for the 'strong' stable builds?  (less likely to be 'empty')

but this UI for the next person who comes along to have to remember one place to set the weak stable and another place to set the strong stable jobs is off-putting.

also, FYI, I looked at doing something like this a while ago, but never finished doing it...
https://github.com/kubernetes/contrib/compare/master...eparis:zero-empty-string-slices

I'm confused, doesn't just passing an empty string disable it? or a ","?

OK. Merging.


Rotate my SSH key

Figured I might as well with the new laptop. (Also I wanted to update the comment.)

This change is

r? any of @Manishearth @edunham @metajack @larsbergstrom

:pushpin: Commit 0973572 has been approved by Manishearth

@bors-servo r+

:hourglass: Testing commit 0973572ac27484ae46f9aadccb348b5396b81a83 with merge 6600b3b41a9f6fe93fb34b400bdfb09aa5e33782...

Thanks @Manishearth, going to follow the rotation steps now. I don't think anything in this repo can get merged before #719 which works around a Travis change.

I've deployed this to all hosts.

:broken_heart: Test failed - status-travis

@bors-servo retry

#719 has landed

:hourglass: Testing commit 0973572ac27484ae46f9aadccb348b5396b81a83 with merge 51ef90e360c28168fb21c0abef43b061e27c0a92...

:sunny: Test successful - status-travis
Approved by: Manishearth
Pushing 51ef90e360c28168fb21c0abef43b061e27c0a92 to master...


Mac App Store, Update

My Setup

Operating System: Mac 10.14
App Version: 2.13.3
Installation type: App Store

[ ] I have tested with the latest version
[ ] I can simulate the issue easily

Description
The app store needs to be updated. The version installed by it is 2.13.3, current version is 2.14.4.
Current Behavior
App Store installs 2.13.3
Expected Behavior
App Store installs 2.14.4
Probably related to #284.

Any updates here?

@Maqsyo Yes. The latest binary was rejected by Mac App Store due to some bugs on https://open.rocket.chat. We are working on them.

Currently, the discrepancy is 2.17.9 in the store and 2.17.11 on Github. Especially when there are exploitable issues around that involve the client, (CVE-2020-15926) keeping users up to date via app store update is important.

The version is already 3.0.2, and in the Apple Store it is still 2.17.9 :(

I think this issue could now be closed in favor of #1931.

@tassoevan Do you suggest to use Rocket.Chat downloaded from https://github.com/RocketChat/Rocket.Chat.Electron/releases/ and not from the App Store?
It seems like you are not going to update this anytime soon.

@tassoevan Do you suggest to use Rocket.Chat downloaded from https://github.com/RocketChat/Rocket.Chat.Electron/releases/ and not from the App Store?
It seems like you are not going to update this anytime soon.

Yes I would download it from here vs the Mac App Store.

@Maqsyo Yes. The latest binary was rejected by Mac App Store due to some bugs on https://open.rocket.chat. We are working on them.

Is that issues resolved now?

thanks @ankar84 for pushing this :) Seems you have some influence here and can resolve this 👍

Seems you have some influence here and can resolve this

I wish I have ;)
But I will try to report this to proper team members.

Apple changed the requirements to be able to update the app on Apple Store.
We will be working on this in the next weeks.

The Apple Store version is updated, you can install it from there right now.


the text of window need to be copiable

the text "Could not establish connection to "<IP_ADDRESS>": Permission denied (publickey)" need to be copiable.

Native OS dialogs are out of our hands, but you can switch to the custom ones with "window.dialogStyle": "custom"


Please update Readme

Great library, it was exactly what I needed for a recent project!  But I ran into a few pain points due to the fact that that parts of the Readme don't seem to have been updated since forking from Resemble.js.  Specifically, what are the best practices for capturing a diff image and saving it off?  This is how I'm currently doing it, but I'm not sure if it's the best way:
resemble(image1).compareTo(image2).onComplete(function(data) {
    data.getDiffImage().pack().pipe(
        fs.createWriteStream(diffImagePath)
    );
});

Thanks!

Hi There,
Thanks for this library.
I tried to follow the above example but getting an empty diff image, tried the following code:
var imgData1 = 'C:\\Users\\Image1.png';
var imgData2 = 'C:\\Users\\Image2.png';

var diffImage = 'C:\\Users\\';

resemble(imgData1).compareTo(imgData2).ignoreNothing().onComplete(function (data) {

    data.getDiffImage().pack().pipe(fs.createWriteStream( diffImage + 'diff.png'));
    console.log(data);
})

Log shows that the 2 images are different:
{ isSameDimensions: true,
  dimensionDifference: { width: 0, height: 0 },
  misMatchPercentage: '0.73',
  analysisTime: 143,
  getDiffImage: [Function] }

Not sure what am I missing here, any help would be much appreciated


Updated the project details page to match the UX design

New details page:

Not part of the UX mocks - separated the links in a separate tab for now:

@lgalis is this still a draft?

@benthomasson - yes, still a draft - adding all the types and test setup here as well. I will ping you when this is ready.

@benthomasson - please review


Enable track test suite

After some experiments, I got a nice way for track testing without a complex shell(or python or whatever) script. Still need to figure out if it works on deployment (CI + docker and, of course, the site).

(Side note: I don't think Print Assumptions is necessary at all; equality tests and type checks should suffice. The command has been used for automatic checking, but if a student tries to use an axiom in any way, it's highly visible in the code itself.  Using Print Assumptions on them would give noisy outputs on the console, possibly confusing the student.
Btw, Codewars solved the axioms problem by creating a mini testing framework as an ML module, but I guess such a thing isn't a particularly good fit for Exercism.)

To-do list

[ ] Change the filenames on the exercises to *Defs.v, *.v, *Test.v, example/*.v
[ ] Add Makefile and _CoqProject to each exercise
[ ] Add make in the Travis script
[ ] Make sure the existing exercises have proper contents and pass the track tests
[ ] Update test_pattern, ignore_pattern, solution_pattern in config.json as per the docs
[ ] Check that the exercises are deployed correctly on the main site (is there a way to check this before actual deployment?)
[ ] Update README to reflect the track test command
[ ] Update the template for students view
[ ] Add the generation tool

Testing setup for single exercise
Directory structure
hello-world
- HelloWorldDefs.v : Preliminary definitions
- HelloWorld.v : Exercise tasks
- HelloWorldTest.v : Type checking scripts
- _CoqProject : Lets CoqIde/ProofGeneral detect dependencies
- Makefile : Build scripts
- example
  - HelloWorld.v : Reference solution

HelloWorldDefs.v
From Coq Require Export String.

HelloWorld.v
From HelloWorld Require Import HelloWorldDefs.

Definition hello_world : string. Admitted.

HelloWorldTest.v
From HelloWorld Require Import HelloWorldDefs HelloWorld.

Example test : hello_world = "Hello, World!"%string.
Proof. reflexivity. Qed.

example/HelloWorld.v
From HelloWorld Require Import HelloWorldDefs.

Definition hello_world : string := "Hello, World!".

_CoqProject
-R . HelloWorld

HelloWorldDefs.v
HelloWorld.v
example/HelloWorld.v
HelloWorldTest.v

Makefile
.PHONY: check track clean

NAME=HelloWorld
COQC=coqc -R . $(NAME)

check:
	$(COQC) $(NAME)Defs.v
	$(COQC) $(NAME).v
	$(COQC) $(NAME)Test.v

track:
	$(COQC) $(NAME)Defs.v
	$(COQC) example/$(NAME).v -o $(NAME).vo
	$(COQC) $(NAME)Test.v

clean:
	rm -f *.vo *.glob

Instructions for students

Run make check once before opening the IDE. Otherwise the *Defs module is not detected.
After completing the definitions/proofs, run make check to check for compilation errors.
If it compiles without any error, congratulations, you're good to go.
Run make clean to remove artifacts.

Track test instructions

Run make track to compile with reference solution.

Track test setup for multiple exercises
Directory structure
exercism
- Makefile
- exercises
  - hello-world
    - ... (as shown above, including its own makefile)
  - rna-transcription
    - ...

Top level Makefile
SUBDIRS := $(patsubst exercises/%/.,%,$(wildcard exercises/*/.))
.PHONY: all clean $(SUBDIRS)

all: $(SUBDIRS)
$(SUBDIRS):
	$(MAKE) -C exercises/$@ track

clean:
	find . -regextype posix-egrep -regex '.*\.(vo|glob)' -delete

Track test instructions

Run make to run (build) track tests for all exercises.
Run make <exercise-folder-name> to run the track test for a specific exercise.

For multiple exercises, make <ex1> <ex2>.

If make succeeds, the exercises are ready.
Run make clean to clean all build artifacts.

I'm planning to apply the changes in two phases:

Update existing exercises' filenames to meet the format above, and update test_pattern, ignore_pattern, solution_pattern in config.json as per the docs, and see if the exercises are deployed correctly on the main site
Add Makefiles and a make command in Travis script

This should do for the generation script.
#!/bin/bash
# $1 = folder name
# $2 = main module name

if [ $# == 2 ]; then
  cd exercises
  mkdir "$1"
  cd "$1"
  echo "(* TODO: Add preliminary definitions. *)" > "$2Defs.v"
  echo "From $2 Require Import $2Defs.

(* TODO: Add exercise skeletons. *)" > "$2.v"
  echo "From $2 Require Import $2Defs $2.

(* TODO: Add test cases. *)" > "$2Test.v"
  echo ".PHONY: check track clean

NAME=$2
COQC=coqc -R . \$(NAME)

check:
	\$(COQC) \$(NAME)Defs.v
	\$(COQC) \$(NAME).v
	\$(COQC) \$(NAME)Test.v

track:
	\$(COQC) \$(NAME)Defs.v
	\$(COQC) example/\$(NAME).v -o \$(NAME).vo
	\$(COQC) \$(NAME)Test.v

clean:
	rm -f *.vo *.glob" > Makefile
  echo "-R . $2

$2Defs.v
$2.v
example/$2.v
$2Test.v" > "_CoqProject"
  mkdir example
  cd example
  echo "From $2 Require Import $2Defs.

(* TODO: Add reference solution. *)" > "$2.v"
  cd ../../..
else
  echo "Usage: gen.sh <folder_name> <module_name>"
fi

I can help you with CI if that's a missing piece, I won't be able to do so before next weekend. Could you please write up some instructions how to get coq up and running on an Ubuntu Linux?
If docker is easier I can get that running two, but that would take me

@NobbZ The easiest way to use Coq would be to use the official Coq docker image.
Installing natively on Ubuntu had some problems around opam 2 and bubblewrap. My workaround was to install opam via apt-get (this installs opam 1.2) and follow the previous instruction for Coq 8.9.0. The caveat is that you only get 8.9.0 while the latest stable is 8.9.1.
Full instruction:

Run the following script

# I don't recall all of the necessary dependencies, but here are at least some of them
sudo apt-get install opam m4 libgtksourceview2.0-dev
export OPAMROOT=~/opam-coq.8.9.0
opam init -n --comp 4.02.3
opam repo add coq-released http://coq.inria.fr/opam/released
opam install coq.8.9.0 && opam pin add coq 8.9.0
opam install coqide # if you want an IDE

Add the following to .bashrc or equivalent

export OPAMROOT=~/opam-coq.8.9.0
eval `opam config env`

But in case you didn't notice, I already got the CI running with Coq docker, though Travis seems to fetch the image every time it runs (which takes over a minute, and IIRC the docs say we can't cache it).
The only question right now is, do the *_pattern options operate on full paths or individual filenames? (I saw the comments on Slack but forgot to ask this.)

Caching would be possible by some tricks, but that wouldn't change anything, perhaps even slow things down, as docker only downloads image, but if we were caching (using load and save of docker) we would add additional upload and compression steps for the cached artifact.
Also one minute of downloading the image is probably father than bootstrapping the VM each time from scratch.
Anyway I'm looking forward to see what you make from coq, as coq, Idris, agda and similar languages are interesting to me, but I rarely have possibility to use them.


[notify.py] Refactoring

What does this PR do?
This refactors the notify module by adding tasks.libs.notify package.
Tests have also been refactored and a test_job_executions context manager has been implemented to avoid changing version controlled test data.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


License

I wanted to ask if you can add a license to this repo.
I've included the code in a Docker image that I built for myself, but ideally I'd like to share it with other people as well. 🙂

Good point, added MIT license


Fix ReconnectionDelay being always the default value

Because the WebSocketClient was being created in the constructor, the value of ReconnectionDelay was always the default (30 seconds)
Since it was only modifiable in the accessor which is called later than the constructor, changing the value never did anything. So I decided to deprecate it and move it to ClientOptions
Code used to test constructor/accessor calls:
Forsen a = new()
{
    Property = "xD"
};

public class Forsen
{
    private string _field = "a";

    public string Property
    {
        init
        {
            Console.WriteLine("property init");
            _field = value;
        }
    }

    public Forsen()
    {
        Console.WriteLine("ctor");
    }
}

output:
ctor
property init

Works


python310Packages.aiopyarr: 22.2.2 -> 22.6.0

Description of changes
https://github.com/tkdrob/aiopyarr/releases/tag/22.6.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 175901 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.sonarr
python310Packages.aiopyarr
python39Packages.aiopyarr


ActionHistory.Cc_Csr and ActionHistory.Cc_Customer not deserialized from REST API results.

The data types for ActionHistory.Cc_Csr and ActionHistory.Cc_Customer are ArrayList; but the data in the REST API results are strings.
Changing these to string type in the ActionHistory class seems to fix the deserilaization.

Definite bug. Should be a simple fix.

Completed and fixed in master.


Need to be able to set expiry on a Live Object

I haven't been able to find a way to EXPIRE the key for a live object. I am working around this by extracting the underlying RMap field with reflection and calling expire on that.
public static RMap extractLiveObjectLiveMap(Object o) throws Exception {
    Field f = o.getClass().getDeclaredField("liveObjectLiveMap");
    f.setAccessible(true);
    return (RMap)f.get(o);
}

extractLiveObjectLiveMap(liveObject).expire(24, TimeUnit.HOURS)

Obviously thats not a great solution :)

A solution is coming shortly.

This should cover it.
https://github.com/mrniko/redisson/pull/609

Nice! Thanks a stack!


Changing import type resets changes guessing checkbox state

STR:

Open "connect dataset" modal
Click on "Dropbox" tab (or any other)

Expected result:
Checkbox remains unchecked.
Actual result:
Checkbox is checked again

@xavijam assigned you since you know these parts best

Doing it in create-view-fixes branch.


add option for silent boot

Issue description
I don't want to see the boot messages on my desktop systems. The Login manager should appear directly after the Grub menu (when there are no critical errors).
I tried several things like:
  # less verbose boot log
  boot.consoleLogLevel = 3;
  boot.kernelParams = [ "quiet" "udev.log_priority=3" ];

and
  boot.loader.grub.extraConfig = "GRUB_CMDLINE_LINUX_DEFAULT=\"quiet loglevel=3 vga=current\"";

It has no effect.
Source: https://wiki.archlinux.org/index.php/silent_boot
Steps to reproduce
Have NixOS system with Grub 2.

Technical details

system: "x86_64-linux"
host os: Linux 4.9.66, NixOS, 17.09.2281.b4a0c011e81 (Hummingbird)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 1.11.15
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

Ubuntu uses Plymouth.
It can get enabled in NixOS but looks broken (that is another issue).
  boot.plymouth.enable = true;

Should we enable it by default for any displayManager?

It's also the case that the default Plymouth theme is incredibly ugly (I'm reluctant to make it the default while it is so). I've been meaning to submit a PR to make the default a nicer one, I should get around to that.

A NixOS theme would be nice.
There are also others that can get packaged.
Tux:
https://www.gnome-look.org/p/1189328/
https://www.gnome-look.org/p/1000026/
this would be nice for apple devices: https://www.gnome-look.org/p/1009320/
Windows boot screens are funny:
https://www.gnome-look.org/p/1202357/
https://www.gnome-look.org/p/1202116/

I packaged a nice one from KDE-land that works with NixOS branding, just need to make it available by default.

https://github.com/NixOS/nixpkgs/pull/32578

That still doesn't do anything about the stage 1 messages, though.

Yep. /cc @abbradar Also https://github.com/NixOS/nixpkgs/issues/32556

We probably need to patch those scripts to not use echo but a logging function where the verbosity can be controlled.

That's what I did locally but I really shoudl clean that up and upstream it.

I tried to set up quiet boot but still got the stage1 logging, thankfully I quickly encountered this post :+1:
I found the patch to the initrd sh script in your nixpkgs fork @peterhoeg, thanks a lot. I'll be stealing that for the moment.
Should be enough to get lazy fbcon init to work, which would allow keeping the grub splash image until getty or the DM start.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/removing-persistent-boot-messages-for-a-silent-boot/14835/2

As mentioned in that Discourse thread, this is not yet fixed for the stage2 script, so reopening.

taking a look at the stage-1 script: how does the @verbose@ placeholder/flag work? I've never seen that syntax before.

not stale

im surprised this isnt an open to have a silent boot up in ones nixos config

For anyone new to nixos wondering how to do this check out my dotfiles: https://github.com/smravec/nixos-config (look in the ./nixos/silent-boot/* and in ./nixos/configuration.nix files, and essentially what I have done is disabled the scripts that dont respect the kernel params and imported back a modified version of them that has all the lines with echo removed )

This might be a silly question, but if I want those lines still being output/logged in case they're needed after the fact for troubleshooting or research or diagnosis or anything, and just want them not to be displayed at the time or (if this were possible) for the display of the animated boot screen, which already hides the later messages, to kick in sooner and hide even the earlier messages… Is that a reasonable goal, hiding the messages one way or another without actually removing them?

I dont think you can start plymouth before the stage 1 but you could instead of having them displayed redirect their output to some log file (which you could look at later in chroot if your system is not booting)

Note that with EFI, modern linux will show the BGRT logo (e.g. manufacturer logo) until something needs to be printed to the fbcon.
So, one way to solve/workaround the problem here is to use [ "console=tty2" ], or using [ "fbcon=vc:2-6" "console=tty0" ].
I haven't re-verified the former (using tty2) since a long while, but that will allow a normal fbcon to exist there with a getty, so that may not be sufficient.
As for the latter solution (using fbcon=vc:2-6), this is removing fbcon from VT1, so in other word ctrl+alt+f1 goes to a VT with nothing on it. So the BGRT will be present until something graphical takes over. My boots are entirely text-free. With the recently added grub changes, they are also flicker-free where possible (with boot.loader.grub.timeoutStyle = false; ).
Briefly: two solutions:

Don't print to tty1/default VT's fbcon
Don't have an fbcon on VT1 (default VT)

if I want those lines still being output/logged in case they're needed after the fact for troubleshooting or research or diagnosis or anything

Good news, they are!

https://github.com/NixOS/nixpkgs/blob/2501addf2d61a3052128659883fee33435edd669/nixos/modules/system/boot/stage-1-init.sh#L163-L178

You should be able to see them with e.g. journalctl -b somewhere after Run /init as init process, since it's part of the dmesg output by being redirected to kmsg.
I believe that with the new systemd-based stage-1, it will also log to the journal, through a different mechanism [citation needed].


feat(auth): enables OIDC Auth code flow

Provides an option for developers to specify the OAuth response type for their OIDC provider (either one of these can be set:):

id_token
code (if set, must also set the client secret)

@lsirac

Please rebase against the dev branch, and set the target branch to the same. We do not merge to the main branch in this repo.

I couldn't figure out how to update the PR to compare against a different branch, so I created a new PR
https://github.com/firebase/firebase-admin-go/pull/450


Remove Underscore [2 of 2]

As much as this will be terrible to review, I assure you it was even more terrible to put together. :'D

Reviewer: @kennyt

doing a search in the project for _. i see there are still a lot of underscores left

Same for _ and underscore to catch imports and requires

tests failing

conflicts

tests failing

Remaining comments addressed here: https://github.com/dcos/dcos-ui/pull/152


Troubleshoot puma9 issues

Workers are launched and then immediately fail without reporting that to the main process (so the main job keeps running forever).  Worker error logs end in
Error in dial(sock, url = url, autostart = asyncdial || NA, tls = tls,  :
  31 | Connection shutdown
Calls: <Anonymous> ... do.call -> <Anonymous> -> dial_and_sync_socket -> dial
Execution halted

Threads/comments that might help:

https://github.com/wlandau/crew.cluster/issues/1#issuecomment-1540135170

To do:

[x] Confirm not an OOM error
[x] Try specifying host with Sys.info()["nodename"]
[x] Try on elgato to confirm it is a puma9 issue and not a crew/crew.cluster issue
[ ] Simulate OOM error (small worker, some job that somehow uses known amount of RAM) to see if that's what is causing the above error on puma9
[ ] Try debugging with just mirai (clues here: https://github.com/wlandau/crew.cluster/issues/1#issuecomment-1540135170)

Confirmed not an OOM error—errors happen with lightweight targets that previously run on puma with no issue

adding host = Sys.info()["nodename"] to the crew controller did not help

Works fine on elgato, so this does seem to be a puma9 / RHEL9 issue

Ugh, nevermind! Now it works fine on puma9 again!  I swear I didn't do anything different than yesterday!

Ok, pretty confident this is not anything I can fix.  I'll bring it up to the HPC folks, but it also looks like there is a potential fix in mirai in the works.  More here: https://github.com/wlandau/crew.cluster/issues/50#issuecomment-2552043799


Using analytics.js on node server?

Greetings, I utilized the suggested "build my own analytics.js file" without all the un-needed integrations, however the built analytics.js file has a bunch of references to the window global, so i can't use it on the server...
am i to install the analytics-node package from segment.io in order to use this? And if so, then why do i need the okgrow package?
Thanks for helping.
my code is simple, using meteor.js; i have a methods.js file in a common folder (accessible both by client and server)
when the file is loaded, the server fails with:
ReferenceError: window is not defined
thanks for any help

@tdotholla This pkg is for providing a router agnostic client side analytics and was designed to only run in the browser, it's not intended to be used for server side analytics.
Check out the example folder to see how to use this pkg, if you move your code to the client you should be able to get it working.
If your looking for server side analytics then you will need to use a different pkg.


CLI: "Did you mean?" prompt if command doesn't exist

% meltano cloud invoke evidence:dev
Error. `meltano cloud invoke` is not a valid command. Did you mean `meltano invoke`?

% meltano status
Error. `meltano status` is not a valid command. Did you mean `meltano cloud status`?

% meltano login
Error. `meltano login` is not a valid command. Did you mean `meltano cloud login`?

This would still be great. How hard is something like this to implement @WillDaSilva ?

@tayloramurphy Pretty easy - so much so that responding with what would be required to implement this feature is hardly faster than just implementing it:

https://github.com/meltano/meltano/pull/7932

@WillDaSilva Love those kind of issues 🤣