Public Cache API and test

Make Cache API public, and test capturing events.

This API doesn't address the handling of the unexplained runtime exceptions due to file corruption/race conditions/multi process etc.
But it raises a few things to consider

Should we have unit tests for corruption cases.
Should we build to provably recover or consider an API to report them? (corrupted count or onCorruption).
Should we provide a sample to write your own decorated filesystem that avoids the multi instance or process cases? e.g. a filesystem that writes out the process name and pid and clears up under normal uses and logs if there is a mismatch?

3 is a half baked idea, but how do we help clients be aware that something is broken?

I'd love to figure out the definitive cause here.
If it's multiple processes overlapping? I bet there's probably some patterns here we haven't considered to detect it, and then we'd need a strategy to recover if that happens. Perhaps not cache on the 2nd instance? Yuck.
If it's multiple instances in the same process? We should probably detect this through a static variable and throw an exception?

What about combining the processes and instances.  An Android gist with a FileSystem that tries to enforce uniqueness by soft-locking on "appname:instancecounter", and open and close write and delete the market file.  Report by default, but optional fail on startup during dev?

@swankjesse are you for exposing this in 5.0?

Yes!

OK, holding off for approval.  This change is just that minimal public API.
class Cache(
   directory: Path,
   maxSize: Long,
   fileSystem: FileSystem
)

And a useful test with example logging that should be easy to build new integrating cases off.

@swankjesse ping


The count must be a number.

Hello, I've got an error when the giveaway is supposed to end:
(node:15844) UnhandledPromiseRejectionWarning: TypeError: The count must be a number.     at Map.random (C:\Users\--\Desktop\--\node_modules\discord.js\src\util\Collection.js:134:42)     at Giveaway.roll (C:\Users\--\Desktop\--\node_modules\discord-giveaways\src\Giveaway.js:304:14)     at processTicksAndRejections (internal/process/task_queues.js:97:5)     at async C:\Users\--\Desktop\--\node_modules\discord-giveaways\src\Giveaway.js:354:27 (node:15844) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 9)

Try installing the latest discord-giveaways version and see if it fixes it

Try installing the latest discord-giveaways version and see if it fixes it


Optional method of better rendering ischrones for accessibility using nodes + edges

Regarding this example:
https://github.com/gboeing/osmnx-examples/blob/master/notebooks/13-isolines-isochrones.ipynb
I propose a new method, make_iso_polys, outlined below. This, rather than a convex hull, might best demonstrate coverage area while preserved nuances ("inlets" or areas of inaccessibility not captured when creating a convex hull around a point cloud).
Example of what improved plot looks like:

Ful function defintion.
def make_iso_polys(G, edge_buff=25, node_buff=50):
    isochrone_polys = []
    for trip_time in sorted(trip_times, reverse=True):
        subgraph = nx.ego_graph(G, center_node, radius=trip_time, distance='time')

        node_points = [Point((data['x'], data['y'])) for node, data in subgraph.nodes(data=True)]
        nodes_gdf = gpd.GeoDataFrame({'id': subgraph.nodes()}, geometry=node_points)
        nodes_gdf = nodes_gdf.set_index('id')

        edge_lines = []
        for n_fr, n_to in subgraph.edges():
            f = nodes_gdf.loc[n_fr].geometry
            t = nodes_gdf.loc[n_to].geometry
            edge_lines.append(LineString([f,t]))

        n = nodes_gdf.buffer(node_buff).geometry
        e = gpd.GeoSeries(edge_lines).buffer(edge_buff).geometry
        all_gs = list(n) + list(e)
        new_iso = gpd.GeoSeries(all_gs).unary_union
        isochrone_polys.append(new_iso)
    return isochrone_polys

Further elaboration available here: http://kuanbutts.com/2017/12/16/osmnx-isochrones/

Those are both buffered at 25 (meters).
In the final example at the bottom of http://kuanbutts.com/2017/12/16/osmnx-isochrones/
I use 25 on edges and 50 on nodes (the above defaults).

I like how the blocks are filled-in in your example image above. But, I'm not seeing the same results when I parameterize your sample code that way:
def make_iso_polys(G, edge_buff=25, node_buff=50):
    isochrone_polys = []
    for trip_time in sorted(trip_times, reverse=True):
        subgraph = nx.ego_graph(G, center_node, radius=trip_time, distance='time')

        node_points = [Point((data['x'], data['y'])) for node, data in subgraph.nodes(data=True)]
        nodes_gdf = gpd.GeoDataFrame({'id': subgraph.nodes()}, geometry=node_points)
        nodes_gdf = nodes_gdf.set_index('id')

        edge_lines = []
        for n_fr, n_to in subgraph.edges():
            f = nodes_gdf.loc[n_fr].geometry
            t = nodes_gdf.loc[n_to].geometry
            edge_lines.append(LineString([f,t]))

        n = nodes_gdf.buffer(node_buff).geometry
        e = gpd.GeoSeries(edge_lines).buffer(edge_buff).geometry
        all_gs = list(n) + list(e)
        new_iso = gpd.GeoSeries(all_gs).unary_union
        isochrone_polys.append(new_iso)
    return isochrone_polys

isochrone_polys = make_iso_polys(G, edge_buff=25, node_buff=25)
fig, ax = ox.plot_graph(G, fig_height=8, show=False, close=False, edge_color='k', edge_alpha=0.2, node_color='none')
for polygon, fc in zip(isochrone_polys, iso_colors):
    patch = PolygonPatch(polygon, fc=fc, ec='none', alpha=0.6, zorder=-1)
    ax.add_patch(patch)
plt.show()

Does it make sense for 75-meter buffers to fill in the blocks? They are much larger than 75x75 meters, so this output I'm seeing makes sense with those parameters.

So sorry and thanks for pointing this out. I updated the post and also am including the fixed script here. I forgot to add on step to the method. I acknowledge this won't always work if you have a "donut" situation. Would need to think about a better way to handle such situations...
def make_iso_polys(G, edge_buff=25, node_buff=50, infill=False):
    isochrone_polys = []
    for trip_time in sorted(trip_times, reverse=True):
        subgraph = nx.ego_graph(G, center_node, radius=trip_time, distance='time')

        node_points = [Point((data['x'], data['y'])) for node, data in subgraph.nodes(data=True)]
        nodes_gdf = gpd.GeoDataFrame({'id': subgraph.nodes()}, geometry=node_points)
        nodes_gdf = nodes_gdf.set_index('id')

        edge_lines = []
        for n_fr, n_to in subgraph.edges():
            f = nodes_gdf.loc[n_fr].geometry
            t = nodes_gdf.loc[n_to].geometry
            edge_lines.append(LineString([f,t]))

        n = nodes_gdf.buffer(node_buff).geometry
        e = gpd.GeoSeries(edge_lines).buffer(edge_buff).geometry
        all_gs = list(n) + list(e)
        new_iso = gpd.GeoSeries(all_gs).unary_union

        # If desired, try and "fill in" surrounded
        # areas so that shapes will appear solid and blocks
        # won't have white space inside of them
        if infill:
            new_iso = Polygon(new_iso.exterior)
        isochrone_polys.append(new_iso)
    return isochrone_polys

Here's the result of the following script:
isochrone_polys = make_iso_polys(G, 50, 0, True)

fig, ax = ox.plot_graph(G, fig_height=8, show=False, close=False, edge_color='k', edge_alpha=0.2, node_color='none')
for polygon, fc in zip(isochrone_polys, iso_colors):
    patch = PolygonPatch(polygon, fc=fc, ec='none', alpha=0.6, zorder=-1)
    ax.add_patch(patch)
plt.show()

thanks @kuanb I've added a demo of this to the notebook!


Timeout during instance update when recreating many instances

Describe the bug
When deploying CloudFoundry with a high number of VMs (i.e. diego-cells > 500) we
reproducibly are able to deploy a couple of the vms and then run into
Task 1788328 | 19:08:05 | Updating instance diego-cell: diego-cell/ba4535ec-221f-4e31-ae1b-f8430a8753d7 (793) (00:13:46)
                     L Error: Timed out sending 'update_settings' to f840b722-4daa-4bc4-a31b-4c22ac74309b after 45 seconds

for various different NATS messages, not only update_settings.
After running bosh deploy several times, the CloudFoundry deployment
eventually succeeds by successfully updating some instances each run.
This bug appeared only after we switched on bosh-dns.
To Reproduce
Steps to reproduce the behavior (example):

Deploy a bosh director on probably any IaaS
Deploy CloudFoundry with > 500 cells
Redeploy with --recreate

We are currently working on an easier way to reproduce it. Progress tracked in story #163348595. If you have an idea how to reproduce it easily please let us know.
Expected behavior
Cloudfoundry gets deployed without an error.
Versions (please complete the following information):

BOSH version: 268.x

Investigation
We instrumented the bosh director to log when messages are sent and
received here.
In addition, we enabled log output from NATS by editing the bpm.yml, adding "-DV" as argument to gnatsd.
A message resulting in a timeout

is published in the NATS client library, but no answer is received
is not received by the NATS server

In critical phases, the bosh director tries to send 15000 sync_dns requests
from which only a significantly smaller number receives an answer.
However, timed out sync_dns requests do not result in failing deployments.
Mitigation
We mitigated the problem by commenting out this line.
This leads to the director not sending sync_dns calls when a vm gets updated.
Instead it only relies on the 10s interval for bosh-dns synchronization.

We believe that this will be much better in the next release of BOSH. We have chosen to only send the sync_dns message to the instance that is being updated, rather than broadcasting it to every instance in the deployment.
I'm going to close this issue in the meantime. Please feel free to re-open if the next release does not fix your issue.

We are still facing the issue with BOSH version v269.0.1.
Current setup are 1400 diego cells resulting into a NATS timeouts. Setting max DB connections to 300 and increasing the AgentClient timeout for NATS message to 90 seconds doesn't resolve the issue. We were able to validate that the NATS server didn't receive the timed out messages.

@videlov Thank you for reopening this, and adding the extra context.

@miguelverissimo we are still working on the analysis of the issue and will update with more details soon.

Fixed with https://github.com/cloudfoundry/bosh/pull/2254


[Bug]: One Team Member Link Broken

Contact Details
<EMAIL_ADDRESS>What happened?
In Team Section Of Fosscu Community Website . The twitter Link of Shivam Yadav was not Working.
https://github.com/FOSS-Community/website-fossc/assets/92575432/8ac4cceb-97fc-4c79-ab57-4e27544b5d98
What browsers are you seeing the problem on?
No response
Relevant log output
No response
Code of Conduct

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

@codecShivam Please Assign this issue to me as i have resolved the same

go ahead!


Issue while installing on Kubesphere using Helm Charts

Error: parse error at (backstage/charts/postgresql/charts/common/templates/_labels.tpl:14): unclosed action
https://github.com/user-attachments/assets/c79d7fa6-a71d-4a04-a53f-a63642025624

That postgresql chart is being pulled in from bitnami/charts as a dependency, and it uses the bitnami/common chart as a dependency, so you would probably need to go log an issue on their repository https://github.com/bitnami/charts. There have also been many updates to the bitnami charts and there is a pull request #232 open to update to the newer versions, which would hopefully resolve your issue.


can't reproduce the result of only training composition-classification model

hi @bo-zhang-cs
i follow your instruction to train the composition-classification model, but find that the best model only get 80.8%, lower than your released model 88.4%.
can u do me a favor to figure out how to reproduce your result？
thanks

sorry, i make a mistake so the result is not good, now i fix it, anything goes right.


How can I get next to serve out it _next link resources when embedded in a Koa server

These resources are required:
GET /_next/1497539794531/page/about
GET /_next/1497539794531/page/_error.
GET /_next/1497539794531/manifest.js
GET /_next/1497539794531/commons.js
GET /_next/1497539794531/main.js
Is there another call besides the render call that I should be using when I detect a _next/ route

This problem is actually more general and happens when serving any static resources from next on a windows 10 platform, when next is embedded in another server. Should I re-log under a different heading?

@serle I'm not sure we handle these requests and it works well in express.
But you can manually handle them in _next/ too.
Could you send me a sample repo, where I can have a look at this?


Will DateTime.Parse be able to deal with nanoseconds?

Issue Title
DateTime can not deal with the fraction of second more than 7 digits.
General
Scientific applications often need to handle timestamp with precision of nanosecond. Although the DateTime datatype do not need to actually save the nanosecond precision, it could be very helpful if DateTime.ToString and DateTime.ParseExact (and other parser methods) could generate and recognize nanoseconds.
Just think about double.Parse , double.Parse("1.12345678901234567890123") will return 1.12345678901235, similarly, it is reasonable if DateTime.ParseExact("20190705 10:20:30.123456789", "yyyyMMdd HH:mm:ss.fffffffff", null) returns 2019-07-05 10:20:30.1234568

DateTime is represented by a 64bit long so it doesn't have the capacity to measure both the full date range and nanosecond precision; it only has 100-nanosecond precision.
https://docs.microsoft.com/en-us/dotnet/api/system.datetime?view=netcore-2.2#quick-links-to-remarks-topics

The DateTime value type represents dates and times with values ranging from 00:00:00 (midnight), January 1, 0001 Anno Domini (Common Era) through 11:59:59 P.M., December 31, 9999 A.D. (C.E.) in the Gregorian calendar.
Time values are measured in 100-nanosecond units called ticks. A particular date is the number of ticks since 12:00 midnight, January 1, 0001 A.D. (C.E.) in the GregorianCalendar calendar. The number excludes ticks that would be added by leap seconds. For example, a ticks value of<PHONE_NUMBER>0000000L represents the date Friday, January 01, 0100 12:00:00 midnight. A DateTime value is always expressed in the context of an explicit or default calendar.

TimeSpan also has a similar issue as it can measure the full interval between two dates; so is also at 100-nanosecond precision
NodaTime uses nanosecond precision which may be more suitable?
And I'm sure @jskeet would say its a better date+time for .NET 😉

@benaadams I know the 64bit limitation of DateTime, I'm not asking for a longer binary format for DateTime, it will be a disaster like the migration from python2 to python3 😈. What I need is a more user-friendly string representation/parsing, just like double. When using double.Parse, users do not care about how many digits of the string, and the result will be rounded properly. Similarly, DateTime.Parse should recognize nano-second, and round to 100-nano properly.

This request belongs to the dotnet/corefx repo. I'll transfer it there.


Fix incorrect warnings from Table function

Table function gives incorrect warnings in cases like
With({majors: Filter(DS, Age >= 18)},Table(majors,{Id:0, Name: \"Other\", Age:100}))

this PR addresses this issue.

✅ No public API change.


[Fill component] pass option to hide or disable opacity

when working with plain colors (hex value) the opacity is always 1 so we need to hide the option or disable it fixed to 1

the alpha is always 1! ^^^
/cc @urbanphes in case it needs a design proposal

😍


etcdctl: treat permission denied as healthy endpoint

Relaxes the permission expectations for endpoint health by noting:

permission denial on linearized reads is always through consensus
endpoint health means consensus with the cluster through the endpoint

So, there's no need to require permission on a health check key in order
to know whether the endpoint is healthy.
Fixes #7075
/cc @mitake

BTW the issue 7075 doesn't exist yet (maybe https://github.com/coreos/etcd/issues/7057 ?). I think the 7057 is already fixed in the latest master because the root role can access to any key.

But backporting this PR to 3.0.x will solve the issue more gracefully because it doesn't change the behavior of the state machine... thanks!

lgtm


fix: [skip ci] use Node v22 for the model updates

FYI the title should be fix: use Node v22 for the model updates [skip ci], not that it would matter much in here


Headers

Adding a check for Missing copyright headers.

Solves #738

Look at this build for an example of what happens when files are added without the copyright notification: https://travis-ci.org/Azure/azure-sdk-for-go/builds/275698423


Date instance on eve of the beginning of daylight savings returns hour behind (Android)

Environment - react-native info
  React Native Environment Info:
    System:
      OS: Windows 10
      CPU: x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
      Memory: 6.80 GB / 15.86 GB
    Binaries:
      npm: 5.6.0 - C:\Program Files\nodejs\npm.CMD
    IDEs:
      Android Studio: Version  <IP_ADDRESS> AI-173.4907809

Other Environment Details
React: 16.5.0
React-native: 0.57.2
Timezone: AEDT - Australian Eastern Daylight Time
Device: Android 8.0 (API 26). Running in emulator (Genymotion). Have briefly tested on device, while it wasn't as thorough, it did show same issue.
Description
Creating a Date instance on the eve of daylight savings starting in Australian Eastern (Daylight) Time returns the hour as an hour behind. Only tested on Android.
The following line is expected to print 20, however it shows 19.
new Date(2018, 9, 6, 20, 0).getHours()

In browser (latest Chrome) and Nodejs (8.11.4)  the above line returns 20.
I did further testing with the following timezones using the eve of the respective day light savings date and they did not show the same issue.

PDT (GMT-7)
EDT (GMT-4)

Reproducible Demo

Create new react app via react-native init DateIssueTest
(I had an issue with starting so did react-native start --reset-cache)
react-native run-android
Modified App.js to include:

        <Text style={styles.instructions}>{new Date(2018, 9, 5, 20, 0).getHours()}</Text>
        <Text style={styles.instructions}>{new Date(2018, 9, 6, 20, 0).getHours()}</Text>
        <Text style={styles.instructions}>{new Date(2018, 9, 7, 20, 0).getHours()}</Text>

The device timezone must be Australian Eastern (Daylight) Time (i.e. Sydney). So Settings > System > Date & time > Select time zone > Sydney
It's expected that each line of code above should show 20, however in my test, the middle line shows 19

Possibly related to this: https://bugs.webkit.org/show_bug.cgi?id=137003

No problem. Thanks @kelset. I'm using DayJS and it still shows the same problem, but I'll give Moment a shot.


⚠ Bofrost.at has degraded performance

In 533ae3d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 830 ms

Resolved: Bofrost.at performance has improved in cc35bb1 after 7 minutes.


Re-export RunError and PooledConnection from bb8

Re-export the RunError type from bb8. Required when handling errors from the pool.

Edit: Updated to include the specific PoolError in the type alias and to include PooledConnection.

Thanks for opening this PR. I potentially plan to relicense diesel-async in the future. Because of that I need an explicitly written consent that you transfer the ownership of this change to me. Please add that to the commit message.

Thanks for opening this PR. I potentially plan to relicense diesel-async in the future. Because of that I need an explicitly written consent that you transfer the ownership of this change to me. Please add that to the commit message.

Thank you, consent added!


Add spectator menu like lunar.gg

Is your feature request related to a problem?
comfortable for user
Describe the solution you'd like.

Click on hotbar
In Menu List all match playing
lore:

''
'UserA'
'vs'
'UserB'
''
'Match Details:'
'Map: XXXX'
'Duration: 11.11'
'Other if u want'
''

Describe alternatives you've considered.
for new customer to determine for purchase frost
Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] I have ensured the feature I'm requesting isn't already in the latest Frost.

Other
No response

like lunar.gg

Feature implemeted


context big scheduler array broken after update

after updating a bunch of nodes in comyfui it looks like they added some new scheduler types so the Context Big, which i use everywhere, is broken since the arrays of scheduler types don't match.
it does work with the ksampler though which is weird.
FaceDetailerPipe 275:

Return type mismatch between linked nodes: scheduler, ['normal', 'karras', 'exponential', 'sgm_uniform', 'simple', 'ddim_uniform'] != ['normal', 'karras', 'exponential', 'sgm_uniform', 'simple', 'ddim_uniform', 'AYS SDXL', 'AYS SD1', 'AYS SVD']

This looks like it's something specific in the FaceDetailerPipe. Those 'AYS SDXL', 'AYS SD1', 'AYS SVD' values do not exist in the schedules according to ComfyUI (For instance, KSampler's schedulers do not have those).
But really, it's just a bad handling of combos from within Comfy UI's widget handling, which is where the error is coming from... I actually don't think there's anything I can fix here, since the list I'm getting is what Comfy says are the available schedulers.

I remember how you can get around the restriction because ComfyUI doesn't check too hard when things get complex for it.
Place a reroute (either an rgthree reroute or a comfyui reroute) in between the two nodes. Connect the FaceDetailerPipe's scheduler input to the reroute's output first, then connect the ContextBig's scheduler output to the reroute input. ComfyUI should not stop this connection from what I've found.


Logo dark mode

I've noticed that the logo for Rubocop in Github's Dark Mode is difficult to see, so I created a dark mode logo and added some markdown to check the user's appearance mode and select the correct logo to display. I may need to change the url for the dark mode logo in production, I'd love to hear some feedback on a good place to store the image. All tests are passing.

. I may need to change the url for the dark mode logo in production, I'd love to hear some feedback on a good place to store the image. All tests are passing.

Just add it alongside the other logo files. It'd be nice if the dark existed for every logo variant (there are 3 of them, plus of a few different sizes for the pngs).

@GrahamHutchinso6275 ping :-)

If someone else picks this up again, there is a better way to mark up the README now:
See https://stackoverflow.com/questions/65413712/changing-readme-md-image-display-conditional-to-github-light-mode-dark-mode


vulnerability that editor user can change admin user's password

editor role user can change admin user's properties including password hash, salt and token.
let's say you have 2 users in db
id      name  isadmin    hash
1111 admin   true       aaaaaa
2222 editor   false      bbbbbb
editor user can use the postuser function to change his password. Attacker can use this function to change the id and other parameters. If editor changed the id to 1111 (post body)which belong to admin, then he can send another postuser request (set to 1111 in both post body and url) to overwrite admin's properties (since findone by id 1111 matches first row now) including hash, token and salt, also changed isadmin to false,
static postUser(req, res) {
    let id = req.params.id;
    let properties = req.body;

    UserController.authenticate(req.cookies.token, req.params.project)
    .then((user) => {
        let hasScope = user.hasScope(req.params.project, 'users');

        if(user.id == id || hasScope) {
            // If the current user does not have the "users" scope, revert any sensitive properties
            if(!hasScope) {
                properties.scopes = user.scopes;
                properties.isAdmin = false;
            }

            return Promise.resolve();
        }

        return Promise.reject(new Error('User "' + user.name + '" does not have scope "user"'));
    })
    .then(() => {
        HashBrown.Service.UserService.updateUserById(id, properties);  <<<<<< accept id, and password value

I was really hoping for someone like you to show up. Thank you so much for your inspection of the codebase.
I will prevent any changes to the user id. Anything else you think would be appropriate to do herw while I'm at it?

the username is dangerous as well. If the editor change the name to admin, the logoff. The logoff update by username function will copy editor user setting to overwirite admin. So there are 2 duplicate lines of editor in db now. I have concern about password part as well, but I can't find a way to use it to exploit without changing username and id.

Then maybe let's work backwards from the ideal solution instead of patching an insecure paradigm.
What would be your ideal approach to modifying user data that would prevent these exploits? These are the requirements:
Any user should be able to change their own username, full name, email and password
Admins should be able to change any field of any user
No one should be able to change the id of any user

no one can change id. before write new name to db make a check the new name is unique. You have this check when create new users but you did not use it in postuser.

@ufo009e the latest change implements both a duplicate username check and pruning of potentially disruptive field changes as discussed above.
Are you still able to perform the exploits you found with the latest code in develop?

looks like fixed this.

Great, thanks!


agent: send SIGKILL instead of SIGTERM to container init process

If container initProcess doesn't install any handler for SIGTERM,
it will ignore this signal, thus send it SIGKILL instead of SIGTERM
to terminate it.
Fixes:#525
Signed-off-by: lifupan<EMAIL_ADDRESS>

/test

/test

/test

Hi @bergwolf  @devimc  Updated according your inputs.

@teawater The CI failed  on lint tools

/test

/test

Nice work @lifupan.  Is it feasible to add tests for the new function?

Nice work @lifupan. Is it feasible to add tests for the new function?

@egernst  Of course.

/test

jenkins-ci-Power8-ubuntu-16-04-initrd  failed:
GitHub pull request #526 of commit 7fbd860aca8c5d6234f3efbf69da88602c99b6ce, no merge conflicts.
Started by user Fupan Li
Rebuilds build #94
Setting status of 7fbd860aca8c5d6234f3efbf69da88602c99b6ce to PENDING with url http://jenkins.katacontainers.io/job/kata-containers-agent-Power8-ubuntu-16-04-PR-initrd/95/ and message: 'Build running'
Using context: jenkins-ci-Power8-ubuntu-16-04-initrd
Building remotely on Power8_slave01 (power8-ubuntu-1604 power8_node) in workspace /home/jenkins/workspace/kata-containers-agent-Power8-ubuntu-16-04-PR-initrd
[WS-CLEANUP] Deleting project workspace...
[WS-CLEANUP] Deferred wipeout is disabled by the job configuration...
ERROR: [WS-CLEANUP] Cannot delete workspace: Unable to delete '/home/jenkins/workspace/kata-containers-agent-Power8-ubuntu-16-04-PR-initrd/go/pkg/linux_ppc64le/github.com/kata-containers/tests/vendor/github.com/BurntSushi/toml.a'. Tried 3 times (of a maximum of 3) waiting 0.1 sec between attempts.
ERROR: Cannot delete workspace: Unable to delete '/home/jenkins/workspace/kata-containers-agent-Power8-ubuntu-16-04-PR-initrd/go/pkg/linux_ppc64le/github.com/kata-containers/tests/vendor/github.com/BurntSushi/toml.a'. Tried 3 times (of a maximum of 3) waiting 0.1 sec between attempts.
Performing Post build task...
Match found for :.* : True
Logical operation result is TRUE
Running script  : #!/bin/bash

export GOPATH=$WORKSPACE/go
export GOROOT="/usr/local/go"
export PATH=${GOPATH}/bin:/usr/local/go/bin:/usr/sbin:/usr/local/bin:${PATH}

I think this PR does not relate to this fail.

Just curious about the behavior,  that make stop behavior not same as runc, so  why do this @lifupan  ?

Just curious about the behavior, that make stop behavior not same as runc, so why do this @lifupan ?

The problem for kata is that it will take much longer time for the container Init process to be started up successfully withed the 9p backed. During it's startup, it's SIGTERM handler maybe haven't installed, thus it would ignore the SIGTERM signal, as a result, the stop container operation would failed. For runc, in theory it also had this issue.


How to detect adult content with flutter

Im creating a sossial madia app. Users can share photo and text. Its greate but i need to detect adult content and block user.
Do you heve any idea.
How i can detect adult content with flutter?
İs there another way to solve this problem?

hello,
try asking in one of the community https://flutter.dev/community
i recommend slack: https://flutter.dev/community

I agree with @BothOfUsTogether2  , this is not a bug in flutter or an actionable feature request, so it is better suited to one of the above forums

sorry i deleting


Pretty up the query page

Current view:

I like the center justified layout and the new controls.  So mainly confirming that #19 seems to have been a good improvement.
You comment in #19 that it is "not pretty", so this issue is a place holder for ideas about making it prettier.

[x] "Get" should probably be more specific -- Perhaps "make plot"
[x] Perhaps "Get" and "Download Data" should be in lower left and right corners?
[x] Should country(ies) get the same center justified treatment?  (I think so, but I'd like to see how it looks.)  Also the singular plural is a bit weird because of English.
[x] "including NEU" -> "include NEU"
[x] a tough of space between "last stage" and the drop down?

requested enhancements made

I'm not seeing all of these changes on the public-facing site.  Have they been deployed?


Update index.html

@whysogreen pls do follow me i will resolve the issues in your repo and it will be merged


Add Application Load Balancer collector

Add support for Application Load Balancer namespace

@msummers-nr If you can bump https://github.com/newrelic-platform/newrelic_aws_cloudwatch_plugin/blob/master/lib/newrelic_aws/version.rb to next minor dot number (3.3.5) and add an entry in https://github.com/newrelic-platform/newrelic_aws_cloudwatch_plugin/blob/master/CHANGELOG.md we can merge this in.

@chaupt Done

Thanks! :shipit:


Longitude in path is truncated to integer

From one of the unit tests (logs can be found here), it is clear that the digits after the decimal are truncated when a decimal is passed in the HTTP path:
...
00:24:54.242 [main] DEBUG org.springframework.web.servlet.mvc.method.annotation.RequestMappingHandlerMapping - Looking up handler method for path /ghost/95837.39821/12345.54321
00:24:54.252 [main] DEBUG org.springframework.web.servlet.mvc.method.annotation.RequestMappingHandlerMapping - Returning handler method [public com.pm.server.response.IdResponse com.pm.server.controller.GhostController.createGhost(double,double,javax.servlet.http.HttpServletResponse) throws com.pm.server.exceptionhttp.ConflictException]
00:24:54.272 [main] DEBUG org.springframework.beans.factory.support.DefaultListableBeanFactory - Returning cached instance of singleton bean 'ghostController'
00:24:54.294 [main] DEBUG com.pm.server.controller.GhostController - Mapped POST /ghost/95837.39821/12345.0
...

My guess is that this is because the longitude is the last value in the path, and so the digits after the period are treated as a file extension or something similar.
To fix this, the Pacman and Ghost createPlayer and setPlayerLocation mappings should no longer have the coordinates in the path, but rather in the request body. This requires a change to the API, but is absolutely necessary.

Working on this on the branch remove-path-decimals.

Issue addressed with merge request 31.

Thank you. API documentation has been updated accordingly.


Speed-up time-based samplers by 20X and index-based by 1.5X

Fixes https://github.com/pytorch/torchcodec/issues/256
We now let the samplers rely on our C++ "sort and dedup" logic, instead of the less efficient Python ones. This has a few benefits:

we can avoid extra copies
samplers can now return a 5D FrameBatch instead of a list of 4D FrameBatch. The 5D FrameBatch output is a "batch" of clips. Its data is of shape (num_clips,  num_frames_per_clips, C, H, W) (or HWC).
the dedup logic now works efficicently for time based samplers (i.e. this fixes https://github.com/pytorch/torchcodec/issues/256)

Running our samplers benchmark:
On main:
----------
num_clips = 1
clips_at_random_indices     med = 19.09ms +- 4.24
clips_at_regular_indices    med = 10.75ms +- 1.10
clips_at_random_timestamps  med = 17.47ms +- 4.45
clips_at_regular_timestamps med = 17.29ms +- 4.10
----------
num_clips = 50
clips_at_random_indices     med = 144.86ms +- 10.09
clips_at_regular_indices    med = 162.97ms +- 40.31
clips_at_random_timestamps  med = 2332.83ms +- 426.51
clips_at_regular_timestamps med = 1871.10ms +- 351.33

This PR:
----------
num_clips = 1
clips_at_random_indices     med = 15.27ms +- 4.07
clips_at_regular_indices    med = 8.70ms +- 1.33
clips_at_random_timestamps  med = 16.34ms +- 4.40
clips_at_regular_timestamps med = 9.57ms +- 3.69
----------
num_clips = 50
clips_at_random_indices     med = 97.06ms +- 3.90
clips_at_regular_indices    med = 107.23ms +- 2.73
clips_at_random_timestamps  med = 104.52ms +- 3.49
clips_at_regular_timestamps med = 117.30ms +- 5.80

:rocket:

Thanks for the review! I'll follow-up with the tests you're mentioning soon


feat: support flat config

docs: https://eslint.org/docs/latest/user-guide/configuring/configuration-files-new
closes #122

@sxzz I'm not sure about the term legacy 🤔
Is it coming from ESLint? Or did you came up with this term?

@Shinigami92 According to the blog, I think flat config is the future. Actually, I'm not sure, but it has to get a name. What name do you prefer?
legacy, nested, or something else?

@Shinigami92 According to the blog, I think flat config is the future. Actually, I'm not sure, but it has to get a name. What name do you prefer? legacy, nested, or something else?

Ah yeah, then I'm against legacy as it indicates now as it would be deprecated, but it is not. The new FlatConfig is just the new way for now.
For now I wouldn't change it at all and just stick with config and flat-config and use this terminology the whole way down.
Like ESLintConfig and FlatESLintConfig and so on.
... And oh yeah, please use FlatESLintConfig instead of ESLintFlatConfig as this is what they call it:

@Shinigami92 I agree with you. I will change it

I have some Meetings and Company Work today, if I find the time this evening, I will release your (welcome 🤗) additions 👍
(I'm in German TimeZone)

@Shinigami92 OK thanks~


[POTENTIAL BUG]

I am working on this excellent source code recently and there seems to be a potential error:
There seems a slip of the pen in the class "class Registry" which is inherited from the template class
AbstractRegistry<T_Ptr, std::unordered_map<T_Key, T_Ptr*>> whose template arguement is "std::unordered_map<T_Key, T_Ptr*>".
Howerer one of the constructor of "Registry" is implemented like this:
Registry(const Registry& sr) : AbstractRegistry<T_Ptr, std::vector<T_Ptr*>>() {
...
}
the std::vector<T_Ptr*> is used as the template arguement of AbstractRegistry which may cause an error when the constructor of Registry is called.

reflect the problem by pulling a request


Create Event Registration Endpoint

Description
Users need to be able to register for events. We need an endpoint that is given a session and

check if the session has room
registers the user if appropriate

Possible Implementation

Out of scope

Requirements / Restrictions

Hey team! Please add your planning poker estimate with ZenHub @bcobo341 @JasonGadoury @m-triassi @RobertoNittolo


Target multiple breakpoints

Hey, hope this isn't a dumb question. I'd really love to be able to do:
$mobile: new-breakpoint(...)
$custom: new-breakpoint(..)
@include media($mobile,$custom) {
//some styles
}
Is there a way to achieve this currently? I looked at the docs but it doesn't appear to be possible. Thanks so much for your time. Bourbon Neat is the best thing since sliced bread! xoxo.

what about @samueljseay request, any updates?

+1 I'm in the same boat, the current neat library doesn't seem to address this in a way that also supports new-breakpoint. I'll have to keep 12 columns across breakpoints and define the column span in individual locations instead.

+1 here

+1, that should REALLY be implemented!

We'll be accepting PR's if anyone wants to implement this.

https://github.com/postcss/postcss
On Wed, Jun 3, 2015 at 11:47 AM, Reda Lemeden<EMAIL_ADDRESS>wrote:

We'll be accepting PR's if anyone wants to implement this.
—
Reply to this email directly or view it on GitHub
https://github.com/thoughtbot/neat/issues/132#issuecomment-108250689.

@cmnstmntmn Not sure that's helpful.

Hey guys, still need to add column support and all the rest but would something like this be headed in the right direction?
http://sassmeister.com/gist/663615b7ddbd80965d84

+1

+1

Some great comments here. Since we are working on further changes, I'm going to close this for now.


typechecker+tests: Revert "ensure function block typecheck runs in all cases" and add test

Reverts SerenityOS/jakt#388
I've also added a test to check for proper scope id creation order in the presence of generic functions.

Hello!
One or more of the commit messages in this PR do not match the Jakt code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why.
Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.

(reviewed by @alimpfard)


CI3 and PHP8.1(ctype_digit(): Argument of type int will be interpreted as string in the future)

When trying to use PostgreSQL under the given conditions, the following error occurs:

A PHP Error was encountered
Severity: 8192
Message: ctype_digit(): Argument of type int will be interpreted as string in the future
Filename: postgre/postgre_driver.php
Line Number: 98

Should I modify the postgre/postgre_driver.php file to resolve this? If it’s acceptable, I can submit a PR.
Also, has anyone resolved the same issue using an override?
I attempted to create a custom class under the application/core directory to override it, but this error still persists.

Looks to me that the ctype part of that if statement should just be removed. As far as I can see so far none of the other drivers check anything other the value isn't empty before using it.

Removing the check might have unintended consequences so a better approach would be to simply cast the variable as a string which is done pretty much everywhere else in the framework.
Change line 98 in the file to read:
if ( ! empty($this->port) && ctype_digit((string) $this->port))

Removing the check might have unintended consequences so a better approach would be to simply cast the variable as a string which is done pretty much everywhere else in the framework.

It's inconsistently checked in each database driver. Most just check if the port is not empty. In this case, it's just to find out if the port number is not empty.

Correct. Also in DB_driver.php, DB_query_builder.php, Session.php, Form_validation.php and Image_lib.php

The thing is, in some of these cases the variable is expected to be a string and not an int, for example in the valid_url function. The lack of type hinting is causing a problem here.


Update sidebar navigator to use accordion menu.

I want to organize the page navigation in the side bar to use an accordion menu.
There will be grouped pages in the future and the idea is to make it easier for the user to navigate pages and keep the sidebar length to a minimum.
The side bar should match the style of the website and the accordion should open to the section that contains the current page.

Additionally, this menu should be upkept in a manner that is easy to update in the future. I am thinking that there is a config file that is the source of truth for the website directory.


simultaneous processing support with provided specific chunk size

request 1~5 + chunk size 3
job1 start
job2 start
job3 start
...
job2 finished
job1 finished
job3 finished

start next chunk
job3 start
job4 start
....
job4 finished
job3 finished

all done.

this would be great to have for faster processing!! i will donate if you have a place to!


Failed to read the 'frame' property from 'Window'

I have a simple Angular app that is loaded within an iframe within a Chrome Extension.
The app runs fine when loaded in an iframe outside of the extension, but when loaded via the unpacked extension, I get the following error:
Uncaught SecurityError: Failed to read the 'frame' property from 'Window':
Blocked a frame with origin "chrome-extension://xxx" from accessing a frame with origin "https://google.com".
The frame requesting access has a protocol of "chrome-extension", the frame being accessed has a protocol of "https". Protocols must match.

I'm starting to look into the Angular codebase, but can anyone suggest why the window.frame is being accessed (and if I can suppress this)?

This turns out to be a problem with systemjs not angular. Closed.


ES6 error property declarations

Copy issue by @zhonshu from https://github.com/Microsoft/vscode-react-native/issues/199
renderPhotosList = () => { if (this.state.showSelectPhotos) { } }
'property declarations' can only be used in a .ts file.
(property) MainPage. renderPhotosList: () => JSX.Element
in ES6 function bind this

Duplicate of https://github.com/Microsoft/TypeScript/issues/6997 also see this comment https://github.com/Microsoft/TypeScript/issues/6997#issuecomment-184509014.
If you are on the insiders build then the work around is to disable the JS diagnostics from Salsa and to use eslint https://github.com/Microsoft/TypeScript/issues/6997#issuecomment-201721683.
insiders build http://code.visualstudio.com/Download#insiders

thanks @egamma .
FYI, insiders download path is https://code.visualstudio.com/insiders

@mcMickJuice thanks, I've updated my comment. BTW, this change is now in stable since a while.


Add "config_dir" class method.

I'd like to add a classmethod, "config_dir":
@classmethod
def config_dir(cls, app_name):
    """
    Return a path object referencing a suitable config
    directory for the relevant platform. Honors the
    `XDG Base Directory Specification
    <http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html>`_
    on Unix and App Data conventions on Windows.
    Ensures that directory exists.
    """

Thoughts?

Seems reasonable to me.

As I start to dive in with an implementation, I see issues. For one, a 'config' dir isn't the only directory one might require. The XDG specification also gives a spec for data, cache, and runtime dirs. It feels improper to only implement one of those.

I've put together an implementation for consideration and feedback. There is one aspect of this implementation that I'd like to remove, and that's the implicitness of two things - that an app_name is always appended and that the target is always created. I think I want to change the implementation to only ever resolve the directories, and leave it to another function to handle what to do with the result.

In that case, you may want to implement all dirs. App dirs does this : https://pypi.python.org/pypi/appdirs/1.4.0.
Because path.py wants to be minimalist, we could choose the optional dependancy approach :

make it appear in "extra_requires=" (http://stackoverflow.com/questions/10572603/specifying-optional-dependencies-in-pypi-python-setup-py)  in setup.py so one can installed it with pip install path.py[sysdirs]
make a conditional import in path.py (try / except ImportErrror);
if the module is installed, expose a SysDir class with methods returning the dirs as Path objects.

 I certainly don't want to create a duplicate, divergent, and less-complete implementation. I like your suggestion.

  I got a very different approach though : https://github.com/sametmax/path.py/tree/sysdirs. I feel like my version has some advantages :
First, you can store the SysDirs object an reuse it without passing the params everytime. It will likely be done in a configuration stage, and avoid moving around a dictionary called app_dir_config just for that.
Secondly, it mimic the appdirs API dynamically, so not only the appdirs documentation will still be valid, but if appdirs add a new property, it will be picked up automatically.
Also it offers a clear NotImplementedError if you try to use it without appdirs being installed with an helpful error message.
Maybe I'm wrong but I feel like the current implementation doesn't allow to pass version, roaming, etc. Although know I think about it, mine has a bug with "multipath" (can be fixed though).
I like the fact that your version is a static method of Path, because it uses the current Path class.
Should we merge ?
Mine passes all the tox tests, but I need to add some tests for multipath apparently.

Maybe... the current implementation doesn't allow to pass [other parameters].

It does actually expose the full signature of the underlying appdirs functions.
I think a merge might be good. I haven't yet have a chance to review yours, but I will. The one aspect I liked most about my implementation was that the API provided a clear designation for the scope and class of the appdirs and in separate namespaces (not combining them into a single namespace as appdirs does). The other thing that is important to me in the implementation is that it should be easily to invoke in a single statement.
Does your implementation give usage examples? If not, would you add them so I can review the differences?

I've taken some time to review your implementation.
I wasn't aware of the AppDirs class in my initial review of it. Indeed, they don't reference AppDirs in the docs, and AppDirs docstring indicates that it's a "convenience wrapper".
I'm not compelled by the re-use case, though I do usually prefer class wrappers for encapsulation.
I'm pretty sure my implementation has issues with multipath as well. The implementation wraps the result in a single path class even though appdirs returns a pathsep-concatenated string. For compatibility, I recommend we wrap that result in another MultiPath class, which is again a str/unicode, but which when iterated over will yield Path instances.
One thing that strikes me about your implementation is the rename of AppDirs to SysDirs. Is there a reason not to defer to ActiveState/appdirs for the naming convention?

I've taken some time to review your implementation.

I wasn't aware of the AppDirs class in my initial review of it. Indeed, they don't reference AppDirs ithe docs, and AppDirs docstring indicates that it's a "convenience wrapper".

I'm not compelled by the re-use case, though I do usually prefer class wrappers for encapsulation.

I'm pretty sure my implementation has issues with multipath as well. The implementation wraps thresult in a single path class even though appdirs returns a pathsep-concatenated string. For compatibility, I recommend we wrap that result in another MultiPath class, which is again a str/unicode, but which when iterated over will yield Path instances.

:+1:

One thing that strikes me about your implementation is the rename of AppDirs to SysDirs. Is there reason not to defer to ActiveState/appdirs for the naming convention?

I choose SysDirs because I think it's easier to understand what it will do than AppDirs. AppDirs feels like it's organising the structure of your app dir tree, while SysDirs give you directory names matching the system conventions. When you write a script or a daemon, you may want to write a log, although you are not making an "app".
This has the additional benefit of allowing us add other directories later even if they are not related to app. E.G: download dir, music dir, video dir, document dir, desktop dir...
Thinking about it, we may want to rename it SysPath actually. Indeed, in the future, we may want to add path to important files such as the hosts file. So let's not close that door.

@sametmax I'm struggling to imagine an implementation that meets your goals (re-usable class) without losing the ability to invoke the functions with their parameters. For example, I want to support this syntax:
(for now I'm using the 'app dirs' name; I'll deal with naming later)
Path.app_dirs.site.config('My App', version="1.0")

And you would also like to support something like:
app_dirs = AppDirPaths(Path, 'My App', version="1.0")
app_dirs.site.config()

Where AppDirPaths serves a similar purpose to the convenience wrapper appdirs.AppDirs.
I tried using appdirs.AppDirs to capture the wrapper when constructing AppDirPaths, but its properties do not allow the parameters to customized when they're called.
On further consideration, maybe this indicates a different usage, where Path.app_dirs is a callable, not a property, and the properties on the result are not. Yes, I think perhaps that's better. So this would be the canonical usage:
Path.app_dirs('My App', version="1.0").site.config

I'll work on that.

Okay. I'm pretty happy with the latest implementation as found in the branch.
I believe all that's left to reconcile is the naming. I'm slightly inclined to stick with app_dirs because that's congruent with the upstream implementation (though doesn't concatenate words, following PEP-8 guidelines).
Even as SysDirs or SysPaths may be more precise names, that precision doesn't in my mind provide enough value to justify the disparity with the upstream naming.
I don't feel strongly about it, though, so I'll defer to @sametmax and others if there's a consensus. @msabramo What's your take? Looking at the changelog for some indication as to the usage, what attribute name would you prefer?

I see little value of following upstream convention as so few people knows about this lib and we are abstracting away anyway. However, if one day we want to add path to the trash directory or to the program files directory, where are we going to put it ? In app dir ? That would be weird. Yes, the current path are related to the app because it contains the app name, but the specificity of the path are definitely constraint by the OS, so SysPath makes more sense to me.
But, as you said, I can live with app_dir too. So I'll vote in favor in SysPath (explicitly stating it can be any path, not just a dir, and specific to the OS), but I app_dir is fine if needed.

It doesn't strike me as weird that AppDirs().trash or AppDirs().program_files would reflect those special paths. In looking around, it seems that Windows and Mac refer to these directories as "special folders", so maybe "special" is the right namespace for them.
What about Path.special().user.config(), ... where Path.special() returns a SpecialResolver instance?
Then, there could be something like Path.special().user.documents or Path.special().public.documents.
I realize one reason I'm shying away from SysPath is because it sounds an awful lot like sys.path, and when spoken is indistinguishable.
At this point, I'm convinced that keeping AppDirs isn't all that valuable, so I'm happy to drop it. And if the class is called a Resolver (rather than a Path, which it is not), then a SystemResolver class and .system method would provide a nice API.
@sametmax If you think about those options (SystemResolver and .system() vs. SpecialResolver and .special()), which do you prefer?

You are righ about the sys.path confusion. +1 for special;

Excellent. Thanks. Released as 7.7.

I may be too late in the discussion, but why isn't special a property ? These empty parens seem to add nothing of value here.

Well, I would use them to pass default values to "appname" and "username", is that the goal?

That's right - the user needs a way to pass configuration parameters that may affect the selection of the target path. For some special folders, that's unnecessary, in which case the parameters may be omitted.

OK, nevermind then.


Bump MetalLB commit-id

Bumping the commit-id to run our E2E tests
ci lane with the ebgp-multihop/containers changes.
Signed-off-by: Ori Braunshtein<EMAIL_ADDRESS>

LGTM


🎇 Created Website footer for issue  #8

Description
Created a website footer for both mobile and web view
fixes #8
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Reference the issue related to this PR as shown below. Every issue has it's own number you can find the issue numbers here.
Fixes # (issue) - Example: Fixes #10, #20 NOTE: GitHub issue numbers should be prefixed with # to aid with issue tracking.
Type of change
Please delete options that are not relevant.

[ ] New feature (non-breaking change which adds functionality)

Screenshot (optional)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Ran a live server to test the styles
Test Configuration: (optional)

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist

[ ] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Good attempt closing this pr as it is already done, please pick another issue


Use correct set handlers when updating passwordless message-texts

When updating passwordless message-texts, they were not persisted. When reloading the page, the changes were gone. Turns out they use get... method, although they should use the set... version.
Discord Thread: https://discord.com/channels/927474939156643850/1072546283639799839
### Definition of Ready
- [x] I am happy with the code
- [x] Short description of the feature/issue is added in the pr description
- [ ] PR is linked to the corresponding user story
- [ ] Acceptance criteria are met
- [ ] All open todos and follow ups are defined in a new ticket and justified
- [ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
- [x] No debug or dead code
- [ ] Critical parts are tested automatically
- [ ] Where possible E2E tests are implemented
- [ ] Documentation/examples are up-to-date
- [ ] All non-functional requirements are met
- [ ] Functionality of the acceptance criteria is checked manually on the dev system.

@peintnermax can you have a look at this?


Enhancement support hms event deserialization for HD…

…P/CDH Hive versions.
Proposed changes
Support to parse the gzip compressed message body of hms NotificationEvent from a HDP/CDH Hive cluster.
Problem summary
Some HDP/CDH Hive versions use gzip to compress the message body of hms NotificationEvent, so com.qihoo.finance.hms.event.MetastoreEventFactory can not transfer it rightly.
Checklist(Required)

[ ] Does it affect the original behavior
[x] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[x] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Add Steampunk Spotter

Adding Steampunk Spotter - an Ansible Playbook Scanning Tool that analyzes and offers recommendations for your playbooks.

[x] I have not changed the README.md directly.

@mre thanks for adding more resources and updating my changes, it looks better now.

You got it. 👍


License details

Hi,
Your script has come in really handy. Thanks.
I need to know what license you distribute the code under to be able to include it in my project but I can't find it.
Do you use a license for this code? It'd be great to have a free one.
Thanks again.
Chris.

Glad it was helpful.  Consider it MIT - I'll add the file when I get some time.

http://choosealicense.com/licenses/
http://choosealicense.com/licenses/mit/

The MIT License (MIT)

Copyright (c) 2015 [fullname]

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

 IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.

(Thanks!)

https://github.com/moagrius/isOnScreen/blob/master/LICENSE

Thanks!
On Jun 10, 2015 9:06 AM, "Mike"<EMAIL_ADDRESS>wrote:

https://github.com/moagrius/isOnScreen/blob/master/LICENSE
—
Reply to this email directly or view it on GitHub
https://github.com/moagrius/isOnScreen/issues/2#issuecomment-110767255.

yw!


Stop error being thrown if there was no matches in file

This stops gulp from throwing an error and exiting if a file contains 0 matches:
[18:19:05] Using gulpfile ~/Repositories/app/Gulpfile.js
[18:19:05] Starting 'default'...
[18:19:05] 'default' errored after 153 ms
[18:19:05] TypeError in plugin 'gulp-scan'
Message:
    Cannot read property 'forEach' of null

Thanks for the PR. Please run gulp and fix the linting error :)

All good to go

Thanks much!


Icon missing in 1.5.5

Dear all,
Does someone else have the same problem that the icon is no longer showing up in the recent release 1.5.5?

Warmly,
~Robert

Hi Robert, I just manually updated from the github release and Nylas threw an
error that the apm install task failed and it deactivated the plugin. After
restarting Nylas, I reactivated the plugin and everything appears to work. I
think there might be a timeout call, which causes the error. Let me know if it
is just that the plugin is deactivated and/or if there is an error in the
console
![](https://link.nylas.com/open/1y2nvi3n6gk2cgzejrks8jzeo/local-6410ee72-22f3?
r=cmVwbHkrMDAzOWJlOTM5NzRkNjdjODBhNDEzZDMyMjAwMjg2MGQzNWEyMWZlNmRmNzg2MDZlOTJj
ZjAwMDAwMDAxMTNiZGFjYTE5MmExNjljZTBhMWUyM2UzQHJlcGx5LmdpdGh1Yi5jb20=)
On Aug 6 2016, at 9:48 am, Robert Orzanna<EMAIL_ADDRESS>wrote:

Dear all,

Does someone else have the same problem that the icon is no longer showing
up in the recent release 1.5.5?

![image](https://cloud.githubusercontent.com/assets/170947/17457087
/2b0e741c-5bed-11e6-807f-a1fdd2035573.png)

Warmly,

~Robert

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, [view it on GitHub](https://github.com/colinking
/n1-unsubscribe/issues/51&r=cmVwbHkrMDAzOWJlOTM5NzRkNjdjODBhNDEzZDMyMjAwMjg2MG
QzNWEyMWZlNmRmNzg2MDZlOTJjZjAwMDAwMDAxMTNiZGFjYTE5MmExNjljZTBhMWUyM2UzQHJlcGx5
LmdpdGh1Yi5jb20=), or [mute the thread](https://github.com/notifications
/unsubscribe-auth/ADm-kzGseducHoMmdXNSS6iHDeJEtAfPks5qdJChgaJpZM4JeSF9&r=cmVwb
HkrMDAzOWJlOTM5NzRkNjdjODBhNDEzZDMyMjAwMjg2MGQzNWEyMWZlNmRmNzg2MDZlOTJjZjAwMDA
wMDAxMTNiZGFjYTE5MmExNjljZTBhMWUyM2UzQHJlcGx5LmdpdGh1Yi5jb20=).![](https://git
hub.com/notifications/beacon/ADm-
k0voyTwawBrfHtTwtwWiiG62xm3Jks5qdJChgaJpZM4JeSF9.gif)

Dear @KyleKing,
The plugin is enabled and the output of the error console is the following:

Weird, try uninstalling and reinstalling again. It couldn't install one the N1 npm dependencies, 'underscore'

On Aug 6, 2016, at 10:40 AM, Robert Orzanna<EMAIL_ADDRESS>wrote:
Dear @KyleKing,
The plugin is enabled and the output of the error console is the following:
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hmm, I have no idea how but the icon is there again.
Weird. I guess it's working again. Thanks for your advice, though!
Let's close this for now.
~Robert


Is there a way to use incremental learning for cnn face recognition model?

What I'm trying to do is to use incremental learning to dynamically train the existing model dlib_face_recognition_resnet_model_v1.dat with addidtional faces. Or if this is not possible, can I just remove it and use another one? If so, how can this be achieved and how can I create this .dat file containing my model? Thanks.

I guess it might be possible to fine tune the model if you know what you are doing, but it's an exercise beyond the scope of a github comment. You'd have to write custom code to do it and I'm not sure what kind of accuracy improvement you'd get (if any) with a small amount of training data. I'm not familiar enough with the details of the dlib CNN C++ api to know if it supports things like easily freezing certain layers during training, etc. But I'm sure it's possible with some work.
You can definitely just remove the included model here and train your own. In that case, you might just want to use dlib directly though since this library is really meant to make it easy to get started with a pre-trained model as easily possible. Check out this for a little more detail: https://github.com/ageitgey/face_recognition/wiki/Face-Recognition-Accuracy-Problems#question-can-i-re-train-the-face-encoding-model-to-make-it-more-accurate-for-my-images
The short answer though is that training a new face encoding model from scratch requires a huge data set and enough familiarity with CNNs that you could look at the dlib metric learning example here and figure out how to do it.

Thank you for your answer! Do you maybe know any other pre-trained models that are compatible with face_recognition and can be used instead of dlib_face_recognition_resnet_model_v1.dat?

Train a classifier on top like this:  https://github.com/ageitgey/face_recognition/blob/master/examples/face_recognition_knn.py


dataset

Is your dataset convenient for disclosure?

For the sorghum dataset, please see the source referenced in the paper. For the wheat dataset, please send a request via email.


lxqt: 0.12.0 -> 0.13.0

Motivation for this change
Update LXQt to version 0.13.0.
TODO

[x] Verify that the following change (from the release notes) works:

Upstream configuration defaults were moved to /usr/share/lxqt, and distribution’s overrides can be placed into /etc/lxqt.

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: pcmanfm-qt
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: pcmanfm-qt
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/0a83cd1by12mn8a5ihajh3g924g8370i-pcmanfm-qt-0.13.0
shrinking /nix/store/0a83cd1by12mn8a5ihajh3g924g8370i-pcmanfm-qt-0.13.0/bin/pcmanfm-qt
gzipping man pages under /nix/store/0a83cd1by12mn8a5ihajh3g924g8370i-pcmanfm-qt-0.13.0/share/man/
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0a83cd1by12mn8a5ihajh3g924g8370i-pcmanfm-qt-0.13.0/bin
patching script interpreter paths in /nix/store/0a83cd1by12mn8a5ihajh3g924g8370i-pcmanfm-qt-0.13.0
checking for references to /build in /nix/store/0a83cd1by12mn8a5ihajh3g924g8370i-pcmanfm-qt-0.13.0...
postPatchMkspecs
postPatchMkspecs
/nix/store/0a83cd1by12mn8a5ihajh3g924g8370i-pcmanfm-qt-0.13.0

@romildo is this ready to merge now? If yes, please resolve the conflict.

is this ready to merge now? If yes, please resolve the conflict.

@xeji Yes, it is ready to merge.

@GrahamcOfBorg build lxqt.lxqt-panel lxqt.lxqt-session pcmanfm-qt lxqt.lxqt-config

Success on x86_64-linux (full log)
Attempted: lxqt.lxqt-panel, lxqt.lxqt-session, pcmanfm-qt, lxqt.lxqt-config
Partial log (click to expand)
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/h6vkh1r2c19g96ya3kf0sdkz1s24bpj0-lxqt-panel-0.13.0/lib  /nix/store/h6vkh1r2c19g96ya3kf0sdkz1s24bpj0-lxqt-panel-0.13.0/bin
patching script interpreter paths in /nix/store/h6vkh1r2c19g96ya3kf0sdkz1s24bpj0-lxqt-panel-0.13.0
checking for references to /build in /nix/store/h6vkh1r2c19g96ya3kf0sdkz1s24bpj0-lxqt-panel-0.13.0...
postPatchMkspecs
postPatchMkspecs
/nix/store/h6vkh1r2c19g96ya3kf0sdkz1s24bpj0-lxqt-panel-0.13.0
/nix/store/2xmknl1arxlc10ns3xgxi491cqfn6h11-lxqt-session-0.13.0
/nix/store/h772jnpfi567zgpmfmasqm97g8ds96bv-pcmanfm-qt-0.13.0
/nix/store/y246m6grswd3g9gpxwkji6nvmlqj9wim-lxqt-config-0.13.0

Thank you!

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: lxqt.lxqt-panel, lxqt.lxqt-session, pcmanfm-qt, lxqt.lxqt-config
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.


Flakiness in numbers for complex_layout_scroll_perf_ios__timeline_summary 90th_percentile_frame_build_time_millis

Looks like it may be in https://github.com/flutter/flutter/pull/53701. Recent regression, though, and this benchmark is sometimes noisy, so I could be wrong.

cc @stuartmorgan @liyuqian and @gw280 (engine sheriff for the week)

Looking at the history of this benchmark, it's experienced such noise several times in the past.  Removing the TODAY label, and marking this as a flake.
FYI @godofredoc

#53701 changes some behavior of when the flutter tool downloads certain things. It has no effect on build products, and can't possibly affect layout perf.

This doesn't seem that actionable.


Trusted Service Usage Unclear

"Usage Scenarios" implies that these are an example, where in fact - in the case of Key Vault - this is the ONLY supported scenario. So, I can deploy a cert via keyvault to an appservice but I can't use secrets.
This document is highly ambiguous and has led to a lot of wasted work. Please update it to clearly reflect that the usage scenarios are the ONLY supported scenarios - maybe name the column "Supported Use Cases" or something similar?

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

ID: 45d20da7-23ff-7f4b-6689-be40e3b55028
Version Independent ID: 20d481b8-80bc-8249-7196-7d09834f61c2
Content: Virtual network service endpoints for Azure Key Vault - Azure Key Vault
Content Source: articles/key-vault/key-vault-overview-vnet-service-endpoints.md
Service: key-vault
GitHub Login: @amitbapat
Microsoft Alias: ambapat

This has been fixed. The document has been updated with the table header clearly stating "Supported Usage Scenarios" to indicated only the listed scenarios are supported. Please close this ticket.

@constructdian We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Navigating from a function (grouped nodes) adds an element to vector editor

Steps I did:

Clicked in a Main.func1
Navigated back to the project
Vector element was prepended to one of the nodes.

Screencast from 2023-05-04 18-36-43.webm

Oh wow, I'm so intrigued what is causing it!

To be checked if still reproducible after #6540

I am attaching one more screencast from today's book club that demonstrates that once you get outside the grouped nodes, all the vector editors prepend a default value
Screencast from 2023-05-12 20-58-14.webm

I don't reproduce it on the newest develop (08e6d21885497215cbef6bd358122c1eeec6ef3b). @Akirathan you can also check and reopen if you still do.

I don't reproduce it on the newest develop (https://github.com/enso-org/enso/commit/08e6d21885497215cbef6bd358122c1eeec6ef3b). @Akirathan you can also check and reopen if you still do.

Seems working on the latest develop. I was also not able to reproduce it anymore. Thx for the fix.


[hotfix] Fix ESLint reviewdog issue

/cib hotfix eslint reviewdog issue

Branch 8-hotfix-eslint-reviewdog-issue created!


Fix: Fixed issue where archive size wouldn't update

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #11580

Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[x] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[x] Are there any other steps that were used to validate these changes?

Create an archive with at least one item.
Observe that the size is correctly displayed., you don't need to Refresh the page to see the size correctly displayed.

Screenshots (optional)

Closing due to inactivity


static proxy with stub data doesnt work with yaml config

Describe the bug
I am trying to create an endpoint with static/stub data. When I use a yaml config file, the endpoint does not work.
➜  ~  ☞  curl -v http://localhost:8888/health
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 8888 (#0)
> GET /health HTTP/1.1
> Host: localhost:8888
> User-Agent: curl/7.64.1
> Accept: */*
>
< HTTP/1.1 500 Internal Server Error
< X-Krakend: Version 1.1.0
< X-Krakend-Completed: false
< Date: Mon, 09 Mar 2020 01:08:43 GMT
< Content-Length: 0
<
* Connection #0 to host localhost left intact
* Closing connection 0

[GIN] 2020/03/09 - 01:03:06 | 500 |      3.0527ms |      <IP_ADDRESS> | GET      /health
Error #01: Get "http://fake-backend/": dial tcp: lookup fake-backend on <IP_ADDRESS>:53: no such host

However, if I use a json config file it works as expected. I mostly followed the example here: https://www.krakend.io/docs/endpoints/health/
I tried experimenting with output_encoding: string and using a string in data: 'hello there' with no success.
Your configuration file
{
    "version": 2,
    "port": 8888,
    "endpoints": [
        {
            "endpoint": "/health",
            "extra_config": {
                "github.com/devopsfaith/krakend/proxy": {
                    "static": {
                        "data": {
                            "status": "OK"
                        },
                        "strategy": "always"
                    }
                }
            },
            "backend": [
                {
                    "url_pattern": "/",
                    "host": [
                        "http://fake-backend"
                    ]
                }
            ]
        }
    ]
}

version: 2
port: 8888
endpoints:
- endpoint: "/health"
  extra_config:
    github.com/devopsfaith/krakend/proxy:
      static:
        data:
          status: OK
        strategy: always
  backend:
  - url_pattern: "/"
    host:
    - http://fake-backend

Commands used
How did you start the software?
 docker run -p 8888:8888 -v "${PWD}:/etc/krakend/" devopsfaith/krakend run -d -c /etc/krakend/krakend.yaml

Expected behavior
A json config file should work the same as a yaml config file. I should be able to create a static proxy endpoint.
Thanks for your help and thanks for the amazing product, I really enjoy using KrakendD.

It seems like using YAML format is a lot of trouble. I have an issue with extra_config for the krakend-martian module. Config snippet below:
    backend:
      - host:
          - https://api.wit.ai
        url_pattern: "/message"
        extra_config:
          github.com/devopsfaith/krakend/http:
            return_error_details: wit.ai

          github.com/devopsfaith/krakend-martian:
            header.Modifier:
              scope:
                - request
              name: Authorization
              value: Bearer XXXX

Error in the log output:
[XXXXXX] 2020/06/22 - 11:33:09.408 ▶ ERROR {<nil> marshalling the extra config for the martian module} map[github.com/devopsfaith/krakend-martian:map[header.Modifier:map[name:Authorization scope:[request] value:Bearer XXXX]] github.com/devopsfaith/krakend/http:map[return_error_details:wit.ai]]

yaml support has some caveats with the extra_config sections (as pointed out in this issue).
we do not recommend using YAML with complex configurations because of that and due it's incompatibility with the flexible-config module.


Right clicking files in Open Editors view closes them before a context menu can be presented

Issue Type: Bug
Steps to reproduce:

Open one or more files

View these files in the Open Editors pane

Right click on one of the filenames

The file closes out of vscode instantly, and you'll wind up getting a context menu for whatever was under the mouse after the file you were trying to interact with closed.
If the file was not saved, you are presented with a "save before close" dialog box.

Expected outcome:
File would stay open and you could view it's context menu, with options such as "Select for Compare" and "Compare selected".
Temporary workaround
It is possible to instead select the file using left-click, and press the "context menu" button on an extended keyboard if you have one.
VS Code version: Code - Insiders 1.28.0-insider (79f79a2c971d6eab36b92d78505402b42ffcae93, 2018-09-24T05:19:32.981Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8400 CPU @ 2.80GHz (6 x 2808)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.82GB (9.46GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

overtype
ada
0.2.0

routeros-syntax
cpe
1.0.0

ini-for-vscode
Dav
0.0.4

gc-excelviewer
Gra
2.1.26

reg
ion
0.0.2

jackfranklin-elm
jac
0.23.0

vscode-lsl
vrt
0.1.1

@HappMacDonald thanks for filing this issue and for using insiders


Unable to provide multiple common_name entries for cloudflare_access_policy

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.0.2
on windows_amd64
Running with v2.25.0 of the cloudflare provider.
Affected resource(s)
cloudflare_access_policy
Terraform configuration files
resource cloudflare_access_policy mtls_allow {
  zone_id        = var.zone_id
  application_id = cloudflare_access_application.app.id
  name           = "mtls-allow"
  precedence     = 1
  decision       = "non_identity"

  require {
    # Require a valid certificate.
    certificate = true
  }

  include {
    common_name = "Foo Bar, Bar Foo"
  }
}

Debug output
Happy to provide this if need be though I'm not sure it's necessary for my issue.
Panic output
No response
Expected output
Providing a string of common names separated by commas would result in multiple common name entries in the cloudflare dashboard UI.
Actual output
When entering common names via the Cloudflare dashboard the UI allows you to enter multiple common name entries by separating them by commas. Via the terraform provider the common_name variable requires a string, entering a comma separated list of entries in this way simply produces a single entry when viewing the policy in the UI.
Steps to reproduce
Terraform apply my example above and view the policy in the Cloudflare dashboard. You'll see that it adds a single common name rather than splitting the entries into multiple common names.
Additional factoids
I also tried making the include block dynamic and creating multiple entries that way, though that just ended up creating a single entry.
References
No response

As it stands, the service only accepts a single common name to match against certificates against. If this is something useful, I'd recommend opening a support ticket (or chat to your account team) about filing a feature request. Once the service supports it, we can look at implementing it here.


Data import want to add a update option

First,thanks  for apply this free database tool.
When I use data import to import cvs data  ,I want set options likes :insert data,update data or both insert and update.
but now since the DBeaver 7.0.2 only can insert data，what I want to confirm is whether there is a problem with the way I use this tool or is it not supported temporarily?

repeat question


[fix] fix bug for issue#29

The parameters of BaseTransform's forward() funciton seem to be wrong, which leads to fail to visualize the seg results of camera-only and fusion models.
This PR is for issue #29.

Yes, that's a fix we forgot to merge to the main branch. Thanks for the contribution!


[operator] strange log message gzip: invalid

What steps did you take and what happened:
Getting the following error:
{"level":"error","ts":1642143772.741397,"logger":"reconciler.vulnerabilityreport","msg":"Scan job container","job":"starboard-operator/scan-vulnerabilityreport-d84c5c4dc","container":"liveness-probe","status.reason":"Error","status.message":"2022-01-14T07:02:52.076Z\t\u001b[31mFATAL\u001b[0m\terror in image scan: failed analysis: analyze error: failed to analyze layer: sha256:ad6d58c8e6c2d61b3d641d98eaea8f66d25626f3a2f9ba90e8eadff03ca56155 : unable to get uncompressed layer sha256:ad6d58c8e6c2d61b3d641d98eaea8f66d25626f3a2f9ba90e8eadff03ca56155: failed to get the layer content (sha256:ad6d58c8e6c2d61b3d641d98eaea8f66d25626f3a2f9ba90e8eadff03ca56155): gzip: invalid header\n","stacktrace":"sigs.k8s.io/controller-runtime/pkg/reconcile.Func.Reconcile\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.10.3/pkg/reconcile/reconcile.go:102\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.10.3/pkg/internal/controller/controller.go:114\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.10.3/pkg/internal/controller/controller.go:311\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.10.3/pkg/internal/controller/controller.go:266\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.10.3/pkg/internal/controller/controller.go:227"}

Running starboard operator with the following config on helm release 0.8.2
targetNamespaces: ""

trivy:
  mode: ClientServer
  severity: MEDIUM,HIGH,CRITICAL
  ignoreUnfixed: true
  serverURL: "http://trivy.trivy.svc.cluster.local:4954"

operator:
  configAuditScannerEnabled: false
  kubernetesBenchmarkEnabled: false

Running trivy helm release 0.4.9 with it's default settings.
What did you expect to happen:
Not to get this error. Don't know why I get it. I think it's because trivy can't download the image but I'm unsure.
Anything else you would like to add:
I can see that the vulnerabilityreports gets generated and it contain CVE information. I haven't been able to verify if i get the CVE information if I get the error.
In this example it's one of the following images and they are public and ready to download.
      image: mcr.azk8s.cn/oss/kubernetes-csi/csi-node-driver-registrar:v2.3.0
      image: mcr.microsoft.com/oss/kubernetes-csi/csi-node-driver-registrar:v2.3.0

I get this for a number of other containers as well and all of them are located on public repos.
[Miscellaneous information that will assist in solving the issue.]
Environment:

Starboard version (use starboard version): 0.13.2
tryvy version: 0.22.0

Are you sure mcr.azk8s.cn is public? I'm getting 403. I can pull from mcr.microsoft.com without credentials though.
$ docker pull mcr.azk8s.cn/oss/kubernetes-csi/csi-node-driver-registrar:v2.3.0
Error response from daemon: error parsing HTTP 403 response body: invalid character '<' looking for beginning of value: "\r\n<title>403 Forbidden</title>\r\n\r\n
403 Forbidden
\r\n
nginx/1.14.0 (Ubuntu)\r\n\r\n\r\n"

Hi @NissesSenap,
by looking at the error, failed to get the layer content --> gzip: invalid header,  I suspect that Trivy is if failing to inspect the layer due to following reasons:

Layer is not compressed with gz, a.k.a. tar.bzip (In this case, Trivy cannot scan the layer.)
Network issue, In this case, can you try to reproduce it again ?

Are you sure mcr.azk8s.cn is public? I'm getting 403. I can pull from mcr.microsoft.com without credentials though.
$ docker pull mcr.azk8s.cn/oss/kubernetes-csi/csi-node-driver-registrar:v2.3.0
Error response from daemon: error parsing HTTP 403 response body: invalid character '<' looking for beginning of value: "\r\n<title>403 Forbidden</title>\r\n\r\n
403 Forbidden
\r\n
nginx/1.14.0 (Ubuntu)\r\n\r\n\r\n"

My bad @danielpacak I honestly just assumed that it was public so didn't try for some reason. Why in the world would this very open-source project not be hosted in a public repo? I assume that it's some internal azure image that I can reach since I'm running AKS but it's very strange why azure would store behind a login.
@chen-keinan  I haven't looked at this part of the code but would it be hard to actually show a 403 error to make it super clear that me as a user actually don't have access to download it? I guess it depends on how trivy responds to these kind of errors.
I will try to play around with this a bit more during the week but sadly I don't have super much time right now.

@NissesSenap for the 403 error Trivy will return a different error :
`2022-01-17T10:30:42.106+0200	FATAL	scan error: unable to initialize a scanner: unable to initialize a docker scanner: 3 errors occurred:
* unable to inspect the image (mcr.azk8s.cn/oss/kubernetes-csi/csi-node-driver-registrar:v2.3.0): Error: No such image: mcr.azk8s.cn/oss/kubernetes-csi/csi-node-driver-registrar:v2.3.0
* unable to initialize Podman client: no podman socket found: stat podman/podman.sock: no such file or directory
* GET https://mcr.azk8s.cn/v2/: unexpected status code 403 Forbidden:
403 Forbidden

403 Forbidden
nginx/1.14.0 (Ubuntu)

`
I assume that  for the `gzip: invalid header` error, it is  related to layer compression not supported

Fair enough. Thanks for the quick response from both of you.
I will close the issue :)


TODOS:

Back-End Specifications
You’ll need to complete the following tasks in each of these back-end files:

[x] auth.js: Update the auth middleware function to work with the GraphQL API.

[x] server.js: Implement the Apollo Server and apply it to the Express server as middleware.

Schemas directory:

[x] index.js: Export your typeDefs and resolvers.

[x] resolvers.js: Define the query and mutation functionality to work with the Mongoose models.

Hint: Use the functionality in the user-controller.js as a guide.

[x] typeDefs.js: Define the necessary Query and Mutation types:

Query type:

[x] me: Which returns a User type.

Mutation type:

[x] login: Accepts an email and password as parameters; returns an Auth type.

[x] addUser: Accepts a username, email, and password as parameters; returns an Auth type.

[x] saveBook: Accepts a book author's array, description, title, bookId, image, and link as parameters; returns a User type. (Look into creating what's known as an input type to handle all of these parameters!)

[x] removeBook: Accepts a book's bookId as a parameter; returns a User type.

[x] User type:

_id

username

email

bookCount

savedBooks (This will be an array of the Book type.)

[x] Book type:

bookId (Not the _id, but the book's id value returned from Google's Book API.)

authors (An array of strings, as there may be more than one author.)

description

title

image

link

[x] Auth type:

token

user (References the User type.)

Front-End Specifications
You'll need to create the following front-end files:

[x] queries.js: This will hold the query GET_ME, which will execute the me query set up using Apollo Server.

[x] mutations.js:

[x] LOGIN_USER will execute the loginUser mutation set up using Apollo Server.

[x] ADD_USER will execute the addUser mutation.

[x] SAVE_BOOK will execute the saveBook mutation.

[x] REMOVE_BOOK will execute the removeBook mutation.

Additionally, you’ll need to complete the following tasks in each of these front-end files:

[x] App.js: Create an Apollo Provider to make every request work with the Apollo Server.

[x] SearchBooks.js:

[x] Use the Apollo useMutation() Hook to execute the SAVE_BOOK mutation in the handleSaveBook() function instead of the saveBook() function imported from the API file.

[x] Make sure you keep the logic for saving the book's ID to state in the try...catch block!

[x] SavedBooks.js:

[x] Remove the useEffect() Hook that sets the state for UserData.

[x] Instead, use the useQuery() Hook to execute the GET_ME query on load and save it to a variable named userData.

[x] Use the useMutation() Hook to execute the REMOVE_BOOK mutation in the handleDeleteBook() function instead of the deleteBook() function that's imported from API file. (Make sure you keep the removeBookId() function in place!)

[x] SignupForm.js: Replace the addUser() functionality imported from the API file with the ADD_USER mutation functionality.

[x] LoginForm.js: Replace the loginUser() functionality imported from the API file with the LOGIN_USER mutation functionality.

Completed all these TODOs, still having issues with making the requests.


run the same suite 2 times with different params

is there way to merge 2 commands into 1 command

protractor protractor.conf --suite create_buyer --params.buyerName=Buyer1
protractor protractor.conf --suite create_buyer --params.buyerName=Buyer2

like

protractor protractor.conf --suite create_buyer,create_buyer --params.suites[0].buyerName=Buyer1 --params.suites[1].buyerName=Buyer2

to make this commad work i need to know the current suite index
is it possible?
may there is better way!

Hi there!
Your question is better suited for StackOverflow or Gitter. Please ask a question there with the 'protractor' tag or post in the Gitter Channel to get help.
From the the getting help section of the README:

Please ask usage and debugging questions on StackOverflow (use the "protractor" tag) or in the Angular discussion group. (Please do not ask support questions here on Github.)

The parsing of --params is very simplistic; you are just going to get a stringified version of whatever you pass in, so references to other elements in your params will need to parsed elsewhere (someone with more knowledge correct me if I am wrong). This seems like something that should either be handled at the shell level (in bash/PowerShell) or in the onPrepare block of your tests.

Hi there!
Your question is better suited for StackOverflow or Gitter. Please ask a question there with the 'protractor' tag or post in the Gitter Channel to get help.
From the the getting help section of the README:

Please ask usage and debugging questions on StackOverflow (use the "protractor" tag) or in the Angular discussion group. (Please do not ask support questions here on Github.)

Params parsing is very simplistic, you are going to get a stringified version of whatever value you pass in. It seems like what you want to accomplish is better handled at the shell level or in in the onPrepare block of your config.


Acceptance tests fail for prod builds of pass-ui

Parts of the current acceptance tests use data-test-* attributes to select certain elements for inspection, but these data attributes are stripped from the HTML for production builds. This is by design in the Ember build process, as these attributes are intended to be used for Ember tests.
We'll need to rewrite those parts of the acceptance tests that utilize these selectors to use other selectors. This may involve adding new element IDs where necessary

As a temporary hold over, we can configure the Ember builds to not strip the selectors for prod builds: https://github.com/mainmatter/ember-test-selectors#configuration

Currently rewriting the selectors to not use the data-test-* attributes. This will work in the short term, but is not readable and will make updating and/or adding tests a bit painful. A better solution will be to add IDs to important UI elements and make sure the UI templates make sense semantically.
We should also look into making the authoring of tests easier. We could look at setting up Page Models in Testcafe, basically creating a page level API for interacting with the UI in tests

https://github.com/eclipse-pass/pass-acceptance-testing/pull/8
https://github.com/eclipse-pass/pass-ui/pull/1191
Decided to keep the data-test-* attributes in prod builds after all. The don't add extra information and only increase bundle size a small amount.
Ran into issues running the acceptance tests in my Windows environment, even though they ran without issue in other environments (Linux, Mac)


Create Struct subclass only once.

Avoid creation of a new class each time method metrics_available is run. Use pre-defined class.
There's performance penalty related to creating classes over and over and it's generally a bad practice.

cc @lucasponce

Thanks for the optimization.
It escaped to my code in some switching from Java into Ruby.