Failure notification daily

Hello! I began receiving a daily notification in HA that my one drive local backup failed. However, when I check everything is backed up and uploaded to my cloud storage successfully. This has been in place and working for several months however I believe this issue might be caused by the size of the backup? Is that possible? I extended the DB purge cycle to 60 days and now my backups are about 1.5GB.

Would be helpful to see the logs :)

Yes, of course. Sorry, I should have included this.
ODB Log.txt

Yes so notice it actually tells you what the problem is in the logs. Backup is timing out, makes sense this can now happen if your backup size has increased. See here:
https://github.com/lavinir/hassio-onedrive-backup#hass-api-timeout-minutes

I read right past that line! Sorry to bother you.
-------- Original message --------
From: lavinir @.>
Date: 7/16/23 8:41 AM (GMT-05:00)
To: lavinir/hassio-onedrive-backup @.>
Cc: Jason Vorbeck @.>, Author @.>
Subject: Re: [lavinir/hassio-onedrive-backup] Failure notification daily (Issue #98)
CAUTION: EXTERNAL SENDER. THINK before the link!
ATTENTION: The sender of this email could not be verified and may not match the person in the "From" field. Use extra caution here. SPF Fail
Yes so notice it actually tells you what the problem is in the logs. Backup is timing out, makes sense this can now happen if your backup size has increased. See here:
https://github.com/lavinir/hassio-onedrive-backup#hass-api-timeout-minutes
—
Reply to this email directly, view it on GitHubhttps://github.com/lavinir/hassio-onedrive-backup/issues/98#issuecomment-1637074566, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALISAX7LIWPYG64YS2KQVSDXQPOLFANCNFSM6AAAAAA2LHXWTY.
You are receiving this because you authored the thread.Message ID: @.***>
CAUTION: This email is NOT from anyone in our company. Do not click links or open attachments unless you recognize the sender and know the content is safe.


Eloquent has() doesn't work with embedsOne/Many

I've two relationships that make use of embed
public function memberApplication(){
	return $this->embedsOne("App\Models\Member");
}

public function notes(){
	return $this->embedsMany("App\Models\Note");
}

When I try to check if the relationship exists, like that
Guest::has("notes")->count();
Guest::has("memberApplication")->count();

Both queries returns me this error:
BadMethodCallException with message 'Call to undefined method Jenssegers\Mongodb\Query\Builder::getOwnerKey()'
Same thing occurs when I use whereHas() method, but it only happens to embeds like relationships.

Why was there a "wontfix" label added? Looks like the issue is still relevant. Is there a specific reason it's not going to be fixed?


Refactored calls to sizeof() macro.

In the original code there are a couple of uses of the sizeof() macro with dynamically allocated memory. This is a issue as sizeof will not be evaluated to the correct value.
However, in the code it is possible to calculate the size of the allocated memory, as it is allocated on the same block where sizeof() is used. Thus I used the known value and replaced the sizeof() calls with it.
I also added some templates for error checking, as the enif_alloc() function can fail and will return NULL if it does. This must be treated, otherwise the program will ungracefully crash.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Lecture mode: Different approach to focus ("attention")

Need [low, medium, high]: Medium
Requester: Teacher
(Please refer to Lecture mode issue #2007 for top-level discussion)
As opposed to collaboration mode, where focus can be requested, in lecture mode the focus should be viewed in similar terms as attention in traditional lecturing. By default, session creator (teacher) should automatically have everyone's attention (everyone focusing on his actions / cursor). But when teacher gives the floor to a student (or pair, perhaps), the "attention" should be assignable to a student.
For pair work, this should work in a way that the student that has the "attention" can pass it to another non-observer student, but cannot take it. Only teacher should be able to take the "attention".

When a guest joins the session, they will automatically follow the host. In turn, if the host follows a guest, then all other guests would in turn transitively start following the guest that the host is now following. So some of this experience is already possible today.
That said, what we don't currently enable, is the host being able to force users to be pinned and/or denying read-only users from being able to force a focus. With that in place, the experience you're looking for would be possible. We're actually tracking this issue already, so I'm going to close this as a duplicate.


Support a context when showing error messages

Closes #14.
Implementation is kind of thrown out there, didn't spend too much time making things nice 😄 I'm also very interested in what format you think would work best. Not sure my choice of wording/error message is clear to read. Any ways, this is a first pass to get something working at least, because without this my experience has been that it's really hard to figure out where an error message is coming from. :)
cc @josevalim @msaraiva

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Don't fail if quotes file does not exist.

Currently the bot fails to start if there's no quotes.yml.
Instead,we should either create a skeleton file if none exists or just not create a file until a quote is added.

Anybody running with this one? If not, I'll knock it out for my first crack at things.

What kind of skeleton file are you looking for here? One with actual values like quotes.yml.example or just an empty file (as if we just did touch quotes.yml) perhaps with a comment in it explaining what to do with it?

I introduced this bug, but I've gotten significantly busier lately and have very little time to fiddle with this. Go ahead and take it!

@herbmillerjr Copying quotes.yml.example sounds like a great solution, but what happens if that file doesn't exist? It might be better to just create a blank YAML file to store stuff in.

Mmm... actually I was just making a comparison to the example file, not suggesting copying it (for that very reason). Okay, blank it is.

Fixed by #69


Equivalent to CURL_IPRESOLVE_V4 in Guzzle ?

Hi,
I've had a quick look through the docs and can't see anything there.
Is there an equivalent to curl's CURL_IPRESOLVE_V4 for Guzzle to force it straight to IPv4 for DNS resolution and server communication ?

I guess you may just need to add it as the custom curl option.

Yep, you'd need to use custom cURL options for something this cURL specific.


Use Promises within nodeJsAction proxy code

Trying to dial down the callback hell in the proxy code.
The underlying reason is that with attachments, doInit() is becoming asynchronous.
FYI @rabbah

Will be PG1 577.

@bjustin-ibm thanks for the review, I agree with your suggestion and will make doInit async now.

Re-submitting for your reviewing pleasure. I continued the callback witch-hunt deeper into service.js and runner.js. I removed a bunch of deadcode and simplified some logic in the process.
PG2 303 is running.

Good catch on the invalid reject(e). I updated the tests so that they would have caught it.
I disagree on the other changes, though. Removing the trues and the names of unused params seems like a recipe to run into undefineds later on. If JS had a Unit type I'd use that, but it doesn't, so I used the simplest thing that's not falsy. I added comments to help the future reader, though.

@psuter You are welcome to prefer the existing pattern, but your comment seems to indicate that you may be under the impression that promises must be resolved on some value, which is not true.  In order to make my suggestion completely clear (which you are still welcome to disagree with), I have coded it up in this commit: c9918fb0f396383e74ff34656e1fb6de7ece6e70
The overall idea is that if your function doesn't produce a meaningful result that the caller is interested in, then it is reasonable to indicate success by resolving on nothing. This is a very common pattern with promises and is well documented:
http://www.2ality.com/2014/10/es6-promises-api.html
http://exploringjs.com/es6/ch_promises.html
http://www.datchley.name/es6-promises/

Thanks for the pointers. I'm still unconvinced that it's a good idea to send undefined values around, whatever the accepted standard is. Feel free to remove/edit and squash the changes in if you feel strongly enough about this, I won't object.
In other news, PG1 646 is blue-ish.

@psuter I left my changes out as this is just a matter of readability preference.


SteamOS

by @kmfrk (1370 points, 519 comments)

by @rkalla at 2013-09-23T17:12:50.000Z
This is effectively the software-only equivalent of a console launch.Is anyone else skeptical that they can motivate publishers to spend time/money porting their games to the Linux platform?Valve certainly has a better chance than most at pulling this off (and likely enough user/market data to make this seem like a valid investment) I am still super skeptical that these publishers are going to spend the time porting their AAA releases to this platform.A good chunk of the console games barely make it to PC/Windows as it is, let alone a PC/Linux platform... seems like a tough sell.If the goal is an entertainment OS with streaming and DVR capabilities in addition to the few Linux compatible games on Steam, that's a bit different of a story but not a huge commercial win I don't think (unless they having some amazing partnerships planned with Netflix/Amazon/Vudu/Hulu for streaming that I am not thinking of).If the goal is to make Steam into an entertainment platform (not just games) it is interesting to watch all these platforms converge on this "entertainment delivery pipeline" solution.

by @jsnell at 2013-09-23T17:27:20.000Z
Well, there is some teasing along those lines there:> Watch for announcements in the coming weeks about all the AAA titles coming natively to SteamOS in 2014.But more importantly, control of Steam gives Valve a lot of leverage in this. It's easy to see the reasons for Linux support in indie games. But harder to see why mid-tier developers like Paradox or Eugen started to port their latest games to Linux. It's a good bet that some kind of a Steam-related carrot was involved there.

by @theschwa at 2013-09-23T17:09:03.000Z
"In SteamOS, we have achieved significant performance increases in graphics processing, and were now targeting audio performance and reductions in input latency at the operating system level."I'm really interested to see the trickle down effect that this will have on other Linux flavors. Having a push for standardization from a big name company like Valve should provide a better Linux experience for everyone.

by @xSwag at 2013-09-23T17:08:08.000Z
Oh my god. This provides a genuine chance for year of the linux desktop in 2014, for gamers at least.

by @TazeTSchnitzel at 2013-09-23T17:12:19.000Z
Valve are working on improving Linux debugging tools too, which is great for developers.

by @bryanlarsen at 2013-09-23T18:52:15.000Z
Which works out to $25 million to get the E.T. license, $100 million to make millions of cartridges to bury in the desert, and $200,000 on actual development.

by @esw at 2013-09-23T18:39:06.000Z
Yeah, but you know that we'll only get two announcements and the third will be delayed indefinitely.

by @corresation at 2013-09-23T17:20:35.000Z
http://store.steampowered.com/search/?os=linuxValve has been planning this for a while, and has had pretty good success motivating a lot of smaller publishers to target Linux, even though the base isn't there yet, and of course have moved their own incredibly valuable games to Linux. Major game engines (Unreal 3D, Unity, among others) already support Linux as an equal target.

by @RyanZAG at 2013-09-23T17:34:46.000Z
This is basically a similar approach Android took for phones. It leverages off Linux while replacing user facing pieces to give the user a simplified and consistent environment that they expect.However as with Android, that isn't enough to get it to take off. Android had huge traction problems to start with until they managed to get the big device makers to train users and market the devices. SteamOS is going to have the same rough and long launch period, but the ultimate payoff is very big as the tech is all there and it's just a matter of getting the big game publishers to join up.As with mobiles and Google, if anybody can succeed in the gaming industry, it's Steam. So I definitely wouldn't write this one off, I think it will take off in a few years time if they can get the big players - both software and hardware - to join up.

by @matthew-wegner at 2013-09-23T17:21:16.000Z
Valve has three announcements this week: http://kotaku.com/looks-like-well-hear-about-valves-new-hard...One of the announcements is widely believe to be the long-rumored Steam Box hardware...

by @makmanalp at 2013-09-23T17:46:42.000Z
The interesting thing is that convincing AAA publishers to port merely requires them to think it'd be profitable.20 years ago, if someone asked to raise a quarter of a billion into a video game, they'd be laughed out of the building. Now it happens often. Over the years, increasingly big-budget games got funded and created.So, all they need to do is to replicate a similar process. First you move all the indie games and source engine games to linux to "bootstrap" your market. Maybe make them free or a few bucks. Once you have a paying userbase from that, you show financials to bigger fish in an attempt to draw them in. Lather, rinse, repeat.Even better is the fact that publishers love "free money" that doesn't require them to gamble. A port of an already successful game is much less risky than a new game. And (conditions permitting, like engine support) it costs a very small amount compared to the actual budget. So they can work on smaller margins to still turn a healthy profit.

by @saraid216 at 2013-09-23T17:14:39.000Z
Gaming is one of the last legitimate refuges of Windows dominance. There are a lot of us who wouldn't care about (read: would be willing to switch) their OS if not for PC gaming.


Use maven-bundle-plugin to add OSGi metadata to consume within Eclipse

I've put .java into the bundle name.

Can you elaborate on what this does?

Eclipse is based on OSGi, and this metadata is necessary to be consumed by the GoogleCloudPlatform/gcloud-tools-eclipse project.  The maven-bundle-plugin computes OSGi metadata and writes it into the META-INF/MANIFEST.MF:

provides a unique name and version
documents the packages it imports and exports, and their implementation dependencies
documents the compatible java runtime
OSGi provides strong classpath isolation between bundles.

The jar can be used outside of OSGi — this will have no effect on other consumers.
The MANIFEST.MF will look like the following:
Manifest-Version: 1.0
Bnd-LastModified:<PHONE_NUMBER>686
Build-Jdk: 1.8.0_77
Built-By: bsd
Bundle-ManifestVersion: 2
Bundle-Name: Google Cloud Java management client library
Bundle-RequiredExecutionEnvironment: JavaSE-1.7
Bundle-SymbolicName: com.google.cloud.tools.app.java
Bundle-Version: 0.1.0.SNAPSHOT
Created-By: Apache Maven Bundle Plugin
Export-Package: com.google.cloud.tools.app.api;version="0.1.0",com.googl
 e.cloud.tools.app.api.deploy;version="0.1.0";uses:="com.google.cloud.to
 ols.app.api",com.google.cloud.tools.app.api.devserver;version="0.1.0";u
 ses:="com.google.cloud.tools.app.api",com.google.cloud.tools.app.api.ge
 nconfig;version="0.1.0";uses:="com.google.cloud.tools.app.api",com.goog
 le.cloud.tools.app.api.module;version="0.1.0";uses:="com.google.cloud.t
 ools.app.api"
Import-Package: com.google.cloud.tools.app.api;version="[0.1,1)",com.goo
 gle.cloud.tools.app.api.deploy;version="[0.1,1)",com.google.cloud.tools
 .app.api.devserver;version="[0.1,1)",com.google.cloud.tools.app.api.gen
 config;version="[0.1,1)",com.google.cloud.tools.app.api.module;version=
 "[0.1,1)",com.google.common.base;version="[17.0,18)"
Originally-Created-By: Apache Maven Bundle Plugin
Require-Capability: osgi.ee;filter:="(&(osgi.ee=JavaSE)(version=1.7))"
Tool: Bnd-<IP_ADDRESS>509101326

That's interesting. A couple of question for things I don't understand

Does this affect our bundling strategy (This wont make fat jars, right)?
Can eclipse resolve the dependencies directly for these?
How exactly will we (as IDE agnostic developers) be restricted in our library's use of dependencies that aren't OSGI spec? Can it slow us down as we add new non-osgi compatible dependencies for example (com.bob.strings:best-string-parser-ever:1.2.3)?

This patch only changes the MANIFEST.MF (no fat jars).  Eclipse / OSGi uses the Import-Package and Export-Package entries to wire up bundles to each other; if you look carefully, this bundle now only exports what it imports except for com.google.common.base, which comes from Guava, which also carries OSGi metadata.
New dependencies will be cause new packages to be added to the Import-Package header, and will only affect us; we'll need to find providers of those packages and otherwise wrap them ourselves.  I held off submitting the patch until I saw you'd removed the the app-engine-sdk for partially that reason.
That said, if you think you'll be pulling in all sorts of dependencies, then it may be better to hold off on this patch and we'll just wrap app-tools-lib-for-java and its dependencies ourselves.

I think this is actually okay, just wanna hold off till after Friday, perhaps Monday (May 9th) when we're relatively stable.

Maybe .lib instead of .java?

Makes sense — I originally put .java since the project seemed focused on Java support, but that can be changed of course.

LGTM, pending @loosebazooka's thumbs up

:+1:

@briandealwis Can you rebase? And then we can merge it into master.

Bah, Update branch doesn't rebase.

Rebased @meltsufin

Thanks!

Why/how is this only exporting com.google.cloud.tools.app.api? I think this may have broke with the package rename.

for me the 'api' package is a signal that only classes in that package are
part of the public api,  however it seems like that's not the case since
CloudSdk at least is needed to actually do anything with this library.
Basically if we have an 'api' package then everything else should be under
internal and not for public consumption.
On Wed, Jun 8, 2016 at 2:48 PM, Elliotte Rusty Harold <
<EMAIL_ADDRESS>wrote:

Why/how is this only exporting com.google.cloud.tools.app.api? I think
this may have broke with the package rename.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/app-tools-lib-for-java/pull/40#issuecomment-224690278,
or mute the thread
https://github.com/notifications/unsubscribe/AHf5HR-q9j52bf7edBafB1vQdz0MHoYGks5qJw5vgaJpZM4IYC1K
.

Currently we're using 'api' package to designate interfaces and classes that are not Cloud SDK specific. So, if we ever move away from using gcloud, we won't have to touch this package. The 'cloudsdk' package itself has an 'internal' subpackage that is not for consumption by clients.

I don't think that's meaningful to clients.  In the end the contract is the
set of public classes I need to use to do stuff with this library.  Perhaps
a solution is to just bring all of the .api package's contents up one level
and removing .api as a concept all together.
I do agree that we don't want to expose through our service methods the
fact that they're implementing their actions through the gcloud CLI, and
that we want nice Java-esque APIs to wrap those interactions,  but it's OK
to expose the bits of the public API that require Cloud SDK configuration
as part of the 'public API'.
On Wed, Jun 8, 2016 at 3:10 PM, meltsufin<EMAIL_ADDRESS>wrote:

Currently we're using 'api' package to designate interfaces and classes
that are not Cloud SDK specific. So, if we ever move away from using
gcloud, we won't have to touch this package. The 'cloudsdk' package itself
has an 'internal' subpackage that is not for consumption by clients.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/app-tools-lib-for-java/pull/40#issuecomment-224696441,
or mute the thread
https://github.com/notifications/unsubscribe/AHf5HUJ_tM7gYA0k1-gZ6H6ILuZKI1elks5qJxOYgaJpZM4IYC1K
.

Created an issue to fix this: #110


Getting IDBDatabase Error in Banana Cake Pop After Upgrade .NET and HotChocolate Versions

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Banana Cake Pop
Describe the bug
We currently updated our GraphQL project from .NET 6 to .NET 8 and have started experiencing an issue when starting the application.  At the start of the day, everything was working fine and I worked on multiple incidents, however none of them required changes to GraphQL.
About an hour ago, I created a new branch off of our main code branch, made some GraphQL changes (to return an additional property), and when I started up the application Banana Cake Pop does not load, just the spinner is shown.  Checking the Chrome console I see the following error:

Thinking maybe I screwed something up, I switched back to the main code branch, which was working all morning, and I created several branches off of it which also worked fine, but even when I run the main branch now I still get the same issue.
However, two of my colleagues simply tried to run the main branch code (which was originally working for me) and immediately experienced this issue.  We are all running the same project and in the same environment setup:
.NET Version 8
Visual Studio Professional 17.8.3
HotChocolate.AspNetCore 13.7.0
HotChocolate.AspNetCore.Authorization 13.7.0
Chrome Version 120.0.6099.129
Steps to reproduce
Seems to happen in various way, but for my specific case:

Upgrade GraphQL project from .NET 6 to .NET 8
Upgrade HotChocolate.AspNetCore from 12.6.2 to 13.7.0
Upgrade HotChocolate.AspNetCore.Authorization from 12.14.0 to 13.7.0
Make changes to the GraphQL contracts (e.g., add a new property)
Run application to open Banana Cake Pop

Relevant log output
638.1bd26ed5.chunk.js:1 This app is being served cache-first by a service worker.
197.26a35fa9.js:2 Uncaught DOMException: Failed to execute 'deleteObjectStore' on 'IDBDatabase': The specified object store was not found.
    at Proxy.<anonymous> (https://localhost:7122/graphql/static/js/197.26a35fa9.js:2:342558)
    at migrateSchema (https://localhost:7122/graphql/static/js/main.3a5aa400.js:2:561850)
    at upgrade (https://localhost:7122/graphql/static/js/main.3a5aa400.js:2:566836)
    at IDBOpenDBRequest.<anonymous> (https://localhost:7122/graphql/static/js/431.a3ab1df2.js:2:4803074)
(anonymous) @ 197.26a35fa9.js:2
migrateSchema @ main.3a5aa400.js:2
upgrade @ main.3a5aa400.js:2
(anonymous) @ 431.a3ab1df2.js:2
IndexedDB (async)
o @ 431.a3ab1df2.js:2
(anonymous) @ main.3a5aa400.js:2
await in (anonymous) (async)
(anonymous) @ main.3a5aa400.js:2
ol @ 431.a3ab1df2.js:2
Sc @ 431.a3ab1df2.js:2
(anonymous) @ 431.a3ab1df2.js:2
C @ 431.a3ab1df2.js:2
T @ 431.a3ab1df2.js:2

197.26a35fa9.js:2 Uncaught (in promise) DOMException: Uncaught exception in event handler.
o @ 197.26a35fa9.js:2

graphql/:1 Uncaught (in promise) DOMException: Version change transaction was aborted in upgradeneeded event handler.
Promise.then (async)
(anonymous) @ main.3a5aa400.js:2
ol @ 431.a3ab1df2.js:2
Sc @ 431.a3ab1df2.js:2
(anonymous) @ 431.a3ab1df2.js:2
C @ 431.a3ab1df2.js:2
T @ 431.a3ab1df2.js:2

Additional Context?
Version
13.0.0
DB 19

I was able to find a work-around for this by going into the Developer Tools in Chrome and manually deleting the banana-cake-pop database from under Application --> Storage --> IndexedDB.  Is there any suggestion on what the root cause of this as we only updated to the latest packages, which I would not expect to break the database.


Is there a way to get the selected row or index, besides the text

I was trying to use this to choose from a list of [id:String], but since it only accepts [[String]], I created two lists one of [id] and one of [[String]], I want to know if I can get the selected row (index) in order to get the id from the other list, thanks.
Note: I have repeating strings so IndexOf will not help

Please can you give the solution ?


separate DB, CLI from contender-core

contender-core should only require as dependencies the core requirements for running generators and spammers; alloy, tokio, and "core" libs like eyre & serde.
We're importing a bunch of implementation-specific dependencies into a single crate (rusqlite & r2d2 for the sqlite DB implementation (used for the CLI), clap for parsing CLI args, toml for the testfile tx generator) that don't belong in contender-core. There should still be a db module in contender but only for trait definitions. Everything else should be split into a new crate, where it can still make use of the contender-core crate, but manage its dependencies separately.

#31


Auto scroll button intersects with community message for certain screen arrangements

You can see this even more so in device mode.

Resolving this since button disappears in offending window sizes now.


[📚] Do we need SSL certs for Messaging on IOS? The docs are not clear.

The docs simply say to create a cert, but there are multiple types of cert.
When setting up my Push Notifications, I see this "Configure" box which is not mentioned in your docs:

Bringing me here:

And here:

I'm guessing we don't need this, because we are using APN Auth Key, but the docs are unclear here, and I don't want to waste 4 or 5 hrs after finding out these don't work in production...

Also, is there a reason the docs here: https://pub.dev/packages/firebase_messaging are so diferent than the steps here: https://firebase.flutter.dev/docs/messaging/ios-integration ?
I'm confused which I should follow, the ones on pub.dev has several additional steps, the one on flutter.dev is presumably more recent?

@esDotDev  You should check out flutter firebase docs . This explains it well!! You can check the documentationon how to use most of the firebase plugins in flutter.

I linked to those docs. My confusion is that the docs on pub.dev are different than the docs on:
https://firebase.flutter.dev/docs/messaging/ios-integration
Should we consider the instructions on pub.dev deprecated?
Additionally, the docs do not mention whether we need to deal with the "Configure" btn on Push Entitlement. If the answer is no, this would be a good thing to mention.

The README/pub.dev docs are due an upgrade when we get around to them. Follow the documentation website docs for now.
Also in regards to your initial question, yes you'll need to configure the certificate for whatever environment you're using.

Another thing that is not mentioned is that you have to upload those certificates to firebase as well apart from the keys

Really? Thanks!
I really can't stress enough how important it is that these docs are complete and exhaustive. Please keep in mind the 3 day approval cycle, and the extremely high testing/deploy time can be if errors are detected in production here. This can literally eat up days or even weeks of dev-time because you miss some tiny setting somewhere.
Please keep in mind the audience is often windows/android devs who know nothing about any of this stuff and just need a straight line from A > B in bullet form.

@esDotDev yeah agree I spent 4 days trying to figure this out on IOS, another thing to keep in mind is to ask for notifications in IOS when initializing FCM in flutter. This isn't mentioned anywhere in the docs and is frustrating when you spend 2 days wondering what you did wrong.
  final FirebaseMessaging _firebaseMessaging = FirebaseMessaging();

  @override
  void initState() {
    super.initState();
    _firebaseMessaging.configure(
      onMessage: (Map<String, dynamic> message) async {
        print("onMessage: $message");
      },
      onLaunch: (Map<String, dynamic> message) async {
        print("onLaunch: $message");
      },
      onResume: (Map<String, dynamic> message) async {
        print("onResume: $message");
      },
    );

   _firebaseMessaging.requestNotificationPermissions(
        const IosNotificationSettings(
            sound: true, badge: true, alert: true, provisional: false));

    _firebaseMessaging.onIosSettingsRegistered
        .listen((IosNotificationSettings settings) {
      print("Settings registered: $settings");
    });

    _firebaseMessaging.getToken().then((String token) {
      print("token: $token");
    });
  }

Here is the code.
In the end I didn't have to use the authenitcation key I generated on the Apple developer platform. Instead I generated 2 certificates both for Development and Production and used them instead of the key in firebase console. I can recieve push notifications this way on IOS but the image is not being displayed. If you figure out a way to get the image to display let me know.
One more thing to keep in mind while processing additional data in notification via the data field is that it is received differently in IOS and Android.
On Android, the message contains an additional field data containing the data. On iOS, the data is directly appended to the message and the additional data-field is omitted.
To receive the data on both platforms:
    var data = message['data'] ?? message;
    String expectedAttribute = data['expectedAttribute'];

The docs will be getting a full overhaul when we get around to working on messaging.

Not to be too much of a squeaky wheel but this sure would've been nice to know when the announcement was made last mth about these new extensions. I fully switched over, thinking they were good to go, and now realize the team is in a "They'll be done when they're done" head space, which puts my project in pretty bad spot.
Thanks @Ashar2shahid for all the tips, that is immensely helpful

Closing in favour of trying the latest messaging documentation which has been significantly updated along with major version updates: https://firebase.flutter.dev/docs/messaging/overview If you still have a problem please raise a new GitHub issue with up to date information and code snippets if possible. Thanks.


upgrading to offscreen 2.1.0

Need to bump this as another thing in BSI needs a new feature in 2.1.0. @ryantmer, @apalaniuk is it cool if I do a release of my-courses after this change is in and update BSI?

LGTM :+1:


TLSContext cannot find secrets without Legacy Mode from another namespace

Describe the bug
Without legacy mode enabled on 1.13, Ambassador complains that it cannot find the Kubernetes secrets containing the TLS certificates defined in TLSContext that refers to secrets in a different namespace.
For example:
apiVersion: getambassador.io/v2
kind: TLSContext
metadata:
  annotations:
    meta.helm.sh/release-name: app-ui-staging
    meta.helm.sh/release-namespace: app-staging
  labels:
    app.kubernetes.io/instance: app-ui-staging
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: app-ui
    helm.sh/chart: app-ui-0.6.0
  name: app-ui-staging-0
  namespace: app-staging
spec:
  alpn_protocols: h2, http/1.1
  ambassador_id:
  - external
  hosts:
  - example.ai
  min_tls_version: v1.2
  secret: example-ai-wildcard.core

2021-04-21 03:10:36 diagd 1.13.0 [P20TAEW] INFO: starting Scout checker and timer logger
2021-04-21 03:10:36 diagd 1.13.0 [P20TAEW] INFO: starting event watcher
2021-04-21 03:10:40 diagd 1.13.0 [P20TAEW] ERROR: Secret example-ai-wildcard.core unknown
2021-04-21 03:10:40 diagd 1.13.0 [P20TAEW] INFO: app-api-staging-0.app-staging.1: <RichStatus BAD error='TLSContext app-api-staging-0 found no certificate in secret example-ai-wildcard in namespace core, ignoring...' hostname='ambassador-external-6c4f79dbc8-8qrnz' version='1.13.0'>
2021-04-21 03:10:40 diagd 1.13.0 [P20TAEW] ERROR: Secret example-ai-wildcard.core unknown
2021-04-21 03:10:40 diagd 1.13.0 [P20TAEW] INFO: app-ui-staging-0.app-staging.1: <RichStatus BAD error='TLSContext app-ui-staging-0 found no certificate in secret example-ai-wildcard in namespace core, ignoring...' hostname='ambassador-external-6c4f79dbc8-8qrnz' version='1.13.0'>
2021-04-21 03:10:40 diagd 1.13.0 [P20TAEW] ERROR: Secret example-ai-wildcard.core unknown
2021-04-21 03:10:40 diagd 1.13.0 [P20TAEW] INFO: app-ui-staging-redir-example.app-staging.1: <RichStatus BAD error='TLSContext app-ui-staging-redir-example found no certificate in secret example-ai-wildcard in namespace core, ignoring...' hostname='ambassador-external-6c4f79dbc8-8qrnz' version='1.13.0'>

If legacy mode is enabled, everything works fine.
When I exec into the container, I can see that Ambassador only finds the TLS secrets that are used in TLS Origination, but not for TLS termination:
/ambassador/snapshots/core/secrets-decoded $ ls -al
total 12
drwxr-xr-x    3 ambassad root          4096 Apr 23 01:38 .
drwxr-xr-x    3 ambassad root          4096 Apr 23 01:38 ..
drwxr-xr-x    2 ambassad root          4096 Apr 23 01:38 ambassador-consul-connector-internal

The secrets are also missing in the snapshot file.
I can confirm that the RBAC for the pod is correct. If I use kubectl from within the container, it can get the secrets.
Versions (please complete the following information):

Ambassador: [e.g. 0.32.1] 1.13.x
Kubernetes environment [e.g. Minikube, bare metal, Google Kubernetes Engine] GKE
Version [e.g. 1.8.1] 1.18

@esmet Please let me know if I've missed any details.

I've fixed this by upgrading the CRD.
I installed Ambassador via the Helm Chart and Helm does not manage the lifecycle of CRDs.
Relevant issue: https://github.com/datawire/ambassador/issues/3417

Sorry for the delay here! I note that secret_namespacing isn't set in your TLSContext -- do you by chance have tls_secret_namespacing set in your ambassador Module? One of those must be set as of... geeez... 1.7 or something? for cross-namespace secrets to work this way.

@kflynn No. I don't think these were in the documentation so I did not enable them.
In any case, I managed to fix this with the following findings:

I used a different Consul Connector TLS secret for TLS origination that was created by a different ambassadaor_id. Using the "right one" fixed TLS orgination for Consul Connect
I found out that you cannot add .namespace when the secret is in the same namespace as the TLSContext. Removing it fixed the issue.

Everything else works.

OK -- I'm going to go ahead and close this, then. Reopen if need be, thanks!


Create CODE_OF_CONDUCT.md

this file is missing so i added this

@Timothyxxx   this good for maintaining a community

Thanks for your CODE_OF_CONDUCT. We've merged it.


Update many optimization

Optimization on update_many code, done by @oleduc.
(Was previously committed to master, then changed with universal-instance-type)

Approved

Approved (but still waiting for changes in this PR)


Fixed array/object mismatch bug in PaginatedList

The last commit to PaginatedList throws errors because $this->request can sometimes be an array and not an HTTPRequest object.  This fix checks if $this->request is an instance of HTTPRequest before calling getURL().
Also, the & separator parameter passed to HTTP::setGetVar() has been removed, as that method already uses an encoded & to build the query string.  This would result in pagination links with some encoded ampersands, and some unencoded ampersands.
The PaginatedListTest class was also updated to correctly parse the encoded links generated by PaginatedList.

request should ONLY be a HTTPRequest. When is it assigned a non-request?

I've answered my own question... the constructor.

@tractorcow it's actually specified as ArrayAccess in the source, and indeed the constructor default is an array. Perhaps this is legacy from when PaginatedList was passed $_GET?

Requested changes temporarily; We need to look at the cause of the poor template behaviour before we merge this, as I have a feeling this is highly regression prone.

Actually it may be 4.0.1 before we patch this.

Ok, so this is the code I'm using to test.
https://gist.github.com/tractorcow/4b2f624b37c4eb515a39c5e11142ef41
I'm getting double-escaped &s in my html output.
I think the default needs to be & not &. Can you update to fix?

                        Prev

                                1

                                ...

                                6

                                7

                            8

                                9

                        Next

Yep so it seems I had already a workaround in my standard Pagination include for this, by using .RAW for the links. I think then @tractorcow we need instead to change HTTP::setGetVar() to have only & separators and not &.  All escaping should be handled by casting in the template layer.

Oops, forgot about HTTPTest, just made the changes and pushed.

@tractorcow I'll leave you to merge this as I'm not sure what the 4.0 branch is for now (ie: 4.0.1 or 4.0.0) and not sure whether you want this is 4.0.0 or 4.0.1

It's for 4.0.0 for now; Yes please hold off until we tag stable (tomorrow).


Make security_group description column not nullable

Creating a security group without a description and running apply renders QueryFailedError: APPLY error: ["SecurityGroup cloud create error: The request must contain the parameter GroupDescription"]. However, the security group entity marks it as NULLABLE here. This will involve regenerating the migrations following the steps here.

@ajejoseph22 this could be a good issue after you are done with the AWS policy/object diff.


Incorporate Monitor Query Beta-2 UX study feedback into August (Beta-3)

Tracking issue to update the August Release to incorporate the TBD Beta-2 UX Study Feedback
/cc @scottaddie

[x] responses must be a list in batch response , a type is unnecessary.
[x] avoid magic strings for aggregation and provide an enum
[x] aggragation param must be plural (it's a list)
[x] #20011
[x] Consider a json handler for the response.

Documentation

[x] the response heirarchy in metrics must be there on the readme
[x] Authenticate the client heading must be create the client
[x] In the samples, remove time generated from queries - confusing along with start_time
[x] Add a section on duration, start and end time, additional_workspaces
[x] Try using pandas on metrics for time series in samples

Action items from 7/23 meeting:

[x] Add an "Advanced scenarios" section to the README. Within that section, provide a link to the multi-workspace logs query code sample at https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/monitor/azure-monitor-query/samples/sample_log_query_multiple_workspaces.py.
[x] Add a logs query response object hierarchy tree to the README. - this will be done along with flattenning
[x] Add a metrics query response object hierarchy tree to the README. https://github.com/Azure/azure-sdk-for-python/pull/19903
[x] Rename the _log_query_client.py file to _logs_query_client.py.
[x] #20015

[x] Add instructions to the README for locating the resource ID. https://github.com/Azure/azure-sdk-for-python/pull/19929

Closing - created separate issues for which there is no action item currently


Where's last result and indexed result?

in python kernel we can refer to them via _ for last result and _3 for Out[3].

Not yet implemented, but good idea.  I don't see any reason not to use the same variable names in this kernel.  Of course it would conflict with any usage of a variable named _ in the user's code, but presumably the same issue exists in python, and I don't think _ is used for anything special in Perl 6.  ( $_ is used of course but _ without a sigil isn't, as far as I know).  We also need to support allowing Out[3] to refer to Out[3].


Hugo cannot identity correct translation string when multiple variants of a language code is used

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.78.2 linux/amd64 BuildDate: 2020-11-13T14:14:29Z

Does this issue reproduce with the latest release?
Yes.
Steps to Reproduce
The problem can be reproduced by following the steps below:

Copy the current multilingual example to local directory: https://github.com/gohugoio/hugo/tree/b8758de19ec75b4565075314f9578270a092bc6f/examples/multilingual

cd into the local directory and create zh files by executing the commands:

cp content/home.en.md content/home.zh.md
cp i18n/en.toml i18n/zh.toml
cp i18n/en.toml i18n/zh-cn.toml

Modify i18n/zh.toml, set other= Correct title

Modify i18n/zh-cn.toml, set other= Wrong title

Add the following content into config.toml

[languages.zh]
weight = 1
title = "Test ZH language"
[[languages.zh.menu.main]]
url = "/zh/home"
name = "ZH Home"
weight = 0

Execute hugo server.
When hugo server is executed, a warning message is displayed, indicating something is wrong:

WARN 2020/11/24 10:10:59 Failed to get translated string for language "zh" and ID "head_title": %!s(<nil>)

Open browser and go to url HUGO_URL/zh/home. We should expect that the page title is Correct title. However, Wrong title is displayed as the page title.

Comments
This is probably related to #7838.

I believe I am also experiencing this issue, and it still exists in Hugo 0.86.0.

I think you can fix the problem by remove either zh.toml or zh-cn.toml
What's the difference between this language file? I thought there same?

@smailzhu  That doesn't "fix" the problem. These files could certainly be different. Using another example to illustrate the point: maybe the site owner wants to have en.toml, and en-au.toml, in which the first file contains the default English translations while the second file contains translations for a specific English-language dialect in Australia.

About this question I think it may have some problem with i18n in hugo
issue #3564 also mention some similar issue with nil pointer, does this problem caused by go-i18n?

@shaform I cannot reproduce the warning with en.toml and en-au.toml
it seems i18n somehow has problem with zh.toml and zh-cn.toml . It works without warning with zh-hant.toml + zh.toml, zh-hk.toml + zh.toml, zh-tw.toml + zh.toml, zh-hant.toml + zh-hk.toml + zh-tw.toml + zh.toml, or  zh-hant.toml + zh-hk.toml + zh-tw.toml + zh-hans.toml

it seems i18n somehow has problem with zh.toml and zh-cn.toml .

That's interesting. I wonder what's so special about zh-cn? It seems https://github.com/nicksnyder/go-i18n/issues/72 (which causes #3564) has already been fixed though.

I have seen this issue with pt + pt-BR as well.

I've done some investigation and it seems that the issue is caused by the language matcher, which would match pt with pt-BR and zh with zh-cn. Here is a sample code that illustrate the problem:
tag_pt_BR := language.Make("pt-BR")
tag_pt := language.Make("pt")
tags := []language.Tag{tag_pt_BR, tag_pt}
matcher := language.NewMatcher(tags)
m, _, _ := matcher.Match(tag_pt)
fmt.Println(tag_pt, "is matched with", m)

It would output:

pt is matched with pt-BR

@shaform It seems like switching the order of the tags presented to language.NewMatcher(tags) may fix this. (But I haven't looked into if it breaks anything else.)
i.e. Switching:
- tags := []language.Tag{tag_pt_BR, tag_pt}
+ tags := []language.Tag{tag_pt, tag_pt_BR}

A simple test:
tag_pt_BR := language.Make("pt-BR")
tag_pt := language.Make("pt")
tags := []language.Tag{tag_pt, tag_pt_BR}
matcher := language.NewMatcher(tags)
m, _, _ := matcher.Match(tag_pt)
fmt.Println(tag_pt, "is matched with", m)
m2, _, _ := matcher.Match(tag_pt_BR)
fmt.Println(tag_pt_BR, "is matched with", m2)

prints the expected:
pt is matched with pt
pt-BR is matched with pt-BR

I don't understand exactly what's happening here, but using an underscore instead of a hyphen in the i18n file name gives you the desired behavior.
defaultContentLanguage = 'en'

[languages.en]
weight = 1

[languages.zh]
weight = 2

[languages.zh-cn]    # HYPHEN
weight = 3

content/
├── _index.md
├── _index.zh-cn.md  # HYPHEN
└── _index.zh.md

i18n/
├── en.toml
├── zh_cn.toml       # UNDERSCORE
└── zh.toml

Test site, see README:
git clone --single-branch -b hugo-github-issue-7982 https://github.com/jmooring/hugo-testing hugo-github-issue-7982
cd hugo-github-issue-7982
hugo server

This was resolved in v0.112.0.
Minimal example:
git clone --single-branch -b hugo-github-issue-7982 https://github.com/jmooring/hugo-testing hugo-github-issue-7982
cd hugo-github-issue-7982
hugo

There's something strange going on here.
I've taken your example, added a similar i18n/en.toml and changed the config.toml to this
baseURL = 'https://example.org/'
languageCode = 'en-US'
title = 'Hugo GitHub Issue #7982'

defaultContentLanguage = 'en'
defaultContentLanguageInSubdir = true

[languages.en]
contentDir = 'content/en'
languageCode = 'en'
languageName = 'English'
weight = 1

[languages.zh]
contentDir = 'content/zh'
languageCode = 'zh-CN'
languageName = 'Mandarin'
weight = 2

...and the error reappeared
WARN  Failed to get translated string for language "zh" and ID "foo": %!s(<nil>)

If I change defaultContentLanguage back to zh, the error is gone.

I've update the test conditions per your remarks and can reproduce the problem.
git clone --single-branch -b hugo-github-issue-7982 https://github.com/jmooring/hugo-testing hugo-github-issue-7982
cd hugo-github-issue-7982
hugo

Expected result in console:
WARN  "foo in en"
WARN  "foo in zh"

Actual result in console:
WARN  "foo in en"
WARN  Failed to get translated string for language "zh" and ID "foo": %!s(<nil>)
WARN  "foo in zh-CN"

I suspect this is an upstream problem, but it will take some effort to dig into.

https://github.com/gohugoio/hugo/blob/9dd687027f2b37bdb94d51fed403066b9f7b9a45/langs/i18n/i18n.go#L118-L120
We're throwing the warning even when the error is nil. That block should be wrapped with if err != nil.
As for why it's choosing the wrong i18n file, I am suspicious of how the files are ordered. I'm guessing the logic is something like, "Yeah, zh-CN looks good, let's use that. No need to look any further."


The ' key (Q in QWERTY) does not work in chromium

When the service is turned off or I just run dvorak in the terminal, everything works, but when the service is running the upper left key (Q in QWERTY) stops working in chromium on any layouts. At the same time, everything is fine in any other applications. It does not work both when the service is started automatically at startup and when it is started manually. I am using the latest version of Manjaro KDE and Chromium(96.0.4664.110)

I only do a remapping of the Q if isUmlaut is true. Did you set the -u parameter? If you did you need to the dvorak intl. layout with dead keys, otherwise behavior is undefined.

I now also removed this in the service file, I comited -u by accident. Thx for reporting


Self-closing curly braces {{ do not appear inside a tag

VSCode Version: 1.20.0-ins
OS Version: win7x64

In .html file, you type a tag

, and type {{ inside of it ... and what you expect is that curly braces will auto-close. But they don't.
They want to auto-close only in situations when there is a space after your caret:

duplicate of https://github.com/Microsoft/vscode/issues/42956


Handling CLOB Datatype/

Is there any Example that can guide on how to extract / display / manipulate the contents of the Clob returned from the the Database. Currently when getString of such datatype only returns oracle.xdb.XMLType@XxXXxX

The issues are for bugs and new features. Please, submit questions  on StackOverflow and tag  with activejdbc and javalite

@goldxfisch have you looked at: http://javalite.io/clob_support_and_caching ?

If you are not getting your content, maybe your column type id not VARCHAR?


Handle all Podman states

Issues: #123

Definition of Done
The PR shall be merged only if all items mentioned in CONTRIBUTING.md have been followed. In case an item is not applicable as described, please provide a short explanation in the description.

[x] All steps in CONTRIBUTING.md have been handled

I have also asked @krucod3 to review this PR, because this change may have effect to #48.


Failure to load built-in stylesheet resources

Hi,
Trying to compile lean from scratch on Windows and starting with Tutorial 1.
Expected Behavior
Backtesting-desktop should open.
Actual Behavior
Launcher fails with:
[18668] ###!!! ABORT: LoadSheetSync failed with error 80040111 loading built-in stylesheet 'resource://gre-resources/counterstyles.css': file c:/builds/moz2_slave/try-w32-0000000000000000000000/build/src/layout/style/nsLayoutStylesheetCache.cpp, line 758
[18668] ###!!! ABORT: LoadSheetSync failed with error 80040111 loading built-in stylesheet 'resource://gre-resources/counterstyles.css': file c:/builds/moz2_slave/try-w32-0000000000000000000000/build/src/layout/style/nsLayoutStylesheetCache.cpp, line 758

Any idea?

Backtesting desktop is no longer supported


[SPARK-10873] Support column sort and search for History Server.

[SPARK-10873] Support column sort and search for History Server using jQuery DataTable and REST API. Before this commit, the history server was generated hard-coded html and can not support search, also, the sorting was disabled if there is any application that has more than one attempt. Supporting search and sort (over all applications rather than the 20 entries in the current page) in any case will greatly improve user experience.

Create the historypage-template.html for displaying application information in datables.
historypage.js uses jQuery to access the data from /api/v1/applications REST API, and use DataTable to display each application's information. For application that has more than one attempt, the RowsGroup is used to merge such entries while at the same time supporting sort and search.
"duration" and "lastUpdated" rest API are added to application's "attempts".
External javascirpt and css files for datatables, RowsGroup and jquery plugins are added with licenses clarified.

Snapshots for how it looks like now:
History page view:

Search:

Sort by started time:

Can one of the admins verify this patch?

ok to test

Haven't looked at the code, but the zebra striping is messed up when you have multiple attempts.

the striping is the same as the current history server (1.5) when you have multiple attempts.  The only difference I see if the striping on the first 2 columns is disabled on the current version.   I'm fine with changing it if its not to much hassle.

Test build #48961 has started for PR 10648 at commit 808558f.

Test build #48961 has finished for PR 10648 at commit 808558f.

This patch fails RAT tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48961/
Test FAILed.

Test build #48966 has started for PR 10648 at commit f1f8feb.

Test build #48966 has finished for PR 10648 at commit f1f8feb.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48966/
Test FAILed.

Merged build finished. Test FAILed.

Test build #48968 has started for PR 10648 at commit 2abe73c.

Test build #48968 has finished for PR 10648 at commit 2abe73c.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48968/
Test FAILed.

Hi @vanzin , I added two new fields (duration Long, lastUpdated Date) into ApplicationAttemptInfo in api.scala.
The jenkins compile is complaining about mima binary check failures. Did you have a clue for this? Thank you!
[info] Compiling 16 Scala sources and 2 Java sources to<EMAIL_ADDRESS>[info] spark-core: found 2 potential binary incompatibilities (filtered 621)
[error]  * synthetic method <init>$default$5()Boolean in object org.apache.spark.status.api.v1.ApplicationAttemptInfo does not have a correspondent in new version
[error]    filter with: ProblemFilters.exclude[MissingMethodProblem]("org.apache.spark.status.api.v1.ApplicationAttemptInfo.<init>$default$5")
[error]  * method this(scala.Option,java.util.Date,java.util.Date,java.lang.String,Boolean)Unit in class org.apache.spark.status.api.v1.ApplicationAttemptInfo does not have a correspondent in new version
[error]    filter with: ProblemFilters.exclude[MissingMethodProblem]("org.apache.spark.status.api.v1.ApplicationAttemptInfo.this")
...
...
[error] (core/*:mimaReportBinaryIssues) spark-core: Binary compatibility check failed!
[error] Total time: 111 s, completed Jan 7, 2016 1:13:24 PM
[error] running /home/jenkins/workspace/SparkPullRequestBuilder@3/dev/mima ; received return code 1

Since the constructor is private anyway, it's fine to just do what the error message says and add the exclusions to MimaExcludes.scala.

Test build #48976 has started for PR 10648 at commit 4322851.

Hi @vanzin , thanks for the info.
BTW, I have fixed the issue you mentioned about messed zebra striping. Now the zebra striping is exactly as what we have originally (the first two columns display pure white).

Test build #48976 has finished for PR 10648 at commit 4322851.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48976/
Test FAILed.

Test build #49025 has started for PR 10648 at commit d7cc895.

Test build #49025 has finished for PR 10648 at commit d7cc895.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49025/
Test PASSed.

@vanzin @srowen @JoshRosen @steveloughran @andrewor14 @rxin   I want to get input on what people think of using the datatables? If everyone is ok with it we can review in more detail.  I think starting out with history page and then perhaps changing all the UI to use datatables to its consistent across the board.

I don't have a problem with it if the license are ok; I just don't have the cycles right now to review the code. :-(

Screenshot looks amazing, thanks @zhuoliu!
I think @JoshRosen has some opinions on using a database for the history server.

I have some opinions too, as with YARN timeline service integration, it's essentially hooked up to a database, both for publishing and retrieval. It might be interesting to consider a dual mode, where things are published to HDFS/s3/WASB during a run (availability will match that of the analytics work itself), and for retrieval, after any initial load & parse, persist in a DB. You could do that with the current FS records,  extracting useful summary data "completed" "incompatible-version" in the process.

The use of a database seems like it could be separate from this pr.  This pr does change it to use the rest api but the main change here is the use of datatables to allow for sorting, searching, etc.
I haven't heard any issue with using that so I'll go ahead with a more thorough review.  If I missed someone concern with that please let me know ASAP.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49810/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, test this please

Test build #49878 has started for PR 10648 at commit ad6ce01.

Test build #49878 has finished for PR 10648 at commit ad6ce01.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49878/
Test PASSed.

Test build #49899 has started for PR 10648 at commit b62d77b.

Test build #49899 has finished for PR 10648 at commit b62d77b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49899/
Test FAILed.

Test build #50109 has started for PR 10648 at commit 638a903.

Test build #50109 has finished for PR 10648 at commit 638a903.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50109/
Test PASSed.

Test build #50155 has started for PR 10648 at commit d7e679e.

Test build #50155 has finished for PR 10648 at commit d7e679e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50155/
Test PASSed.

The paging doesn't seem to be working can you take a look.
Then could you also line up the table indentation with the other text for "Event log directory"

Test build #50196 has started for PR 10648 at commit 1d3343f.

Test build #50196 has finished for PR 10648 at commit 1d3343f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50196/
Test PASSed.

Hi @tgravescs , finally fixed the paging stuff in RowsGrouping. :)

Test build #50248 has started for PR 10648 at commit 430d283.

Test build #50248 has finished for PR 10648 at commit 430d283.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50248/
Test PASSed.

Merged build finished. Test PASSed.

@tgravescs @zhuoliu are you guys interested in more UI work? I have some ideas that I never found time / people to work on ... I think they will make the UI a lot more useful.

@rxin  I would be interested in hearing them if they make debugging easier and usability better.  We'll have to prioritize it with the rest of the work we have.  We would like to change most of the pages to use the datatables to keep things consistent so if its things we can do at the same time it might be easier.

mostly looks good a few more minor things.

can you highlight the page you are on
file a jira to have attemptid always show up in rest api so spark shell reports attempt id
when you click an application id with multiple attempts, open the latest (ie 3 if 3 attempts).

JIRA file here: https://issues.apache.org/jira/browse/SPARK-13064

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50299/
Test FAILed.

Jenkins, test this please

Test build #50306 has started for PR 10648 at commit 0c55712.

Test build #50306 has finished for PR 10648 at commit 0c55712.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50306/
Test FAILed.

Jenkins, test this please

+1 for the latest changes, the unit tests failures are unrelated.  Going to give this a little while but if they are hung again commit this anyway since the latest changes made don't affect unit tests.


Question about Info Addon: What If I use Typescript instead of prop-types?

currently the Info Addon shows blank in the prop types section (which annoyingly can't be disabled...) but I use Typescript so this is kinda moot. I think Typescript has a bigger market share than prop-types now so it would be interesting to have an option to show typescript types instead of prop-types. This is probably above my typescript skill level right now but would appreciate a discussion on this (or a pointer in case this has been discussed before).
thanks!

Looks related to #1519

ah yes. happy to close my issue and just watch that.


Make async-std optional

closes #244

This looks good. Is it semver-major?

This looks good. Is it semver-major?

I don't believe so.

futures-lite now has a prelude, updated the imports to use it.


This adds a default implementation to the KafkaProducer interface to …

…support the use of headers
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Rerun tests please

@hijker FYI, we do not add new features to older branches except under extremely rare circumstances. Can you re-target this PR at the trunk branch (rebasing if necessary to resolve any merge conflicts)?
Also, we should probably add some tests that utilize the new partitioner method introduced in this PR (though it's fine to wait until the KIP is approved to do that since discussion/voting may lead to a shift in the design; up to you).


Upgrade to the new "No CAPTCHA" reCaptcha

Google introduced a new reCaptcha
https://www.google.com/recaptcha/intro/index.html
It would be great if this plugin supported it. Are there any plans to support it?

Hello,
We haven't yet worked on this ticket since it's been superseded by other tasks for commercial reasons.
I wouldn't be able to give an ETA, but we certainly wouldn't be able to get round to working on this until mid-February.
Sorry :/

:+1: This will be a nice improvement, thanks :)


memory test compilation fixes

The memory test wasn't compiling off of master for me. These changes helped me.

Build status:

Linux
Mac
Windows

These were already covered by #1174, which has just now made its way to master


Fix use feature to detect any features in list

This makes features can be described in a list syntax.

smart idea.


tag-self-close should ignore menuitem when it has a closing tag

At the moment if I set tag-self-close to true then it will error on this code:
<menuitem>Do Thing</menuitem>
and it should not do this if there is a closing tag, as there is in this case.
The error is this:

void element should close itself

It would be nice if there was an exclude option for this rule too, like there appear to be for other rules.

I think you've misunderstood something about the html spec here (and this is a very confusing part of html, so that's not too surprising).
menuitem is a void element, meaning (1) it should not contain any content and (2) the closing tag may (and should, generally) be omitted. So your example should perhaps be <menuitem label="Do Thing">. The htmllint option tag-self-close is used to require all void elements to be self-closing (like <br/>, for example). This was the preferred style in html4. However, it is not typically used in html5, and menuitem is only defined in html5. So it seems that you should just not be using tag-self-close.
A rule to exclude certain elements seems strange because html has only exactly list of void elements, and they behave the same. It would be very unusual to require code to style some of them differently than others.

Ah ok, I'll switch to using ul/li then.


Update website templates to set User Registration to None by default

Summary
Resolves #4893

@valadas thanks for linking that.  Honestly, that's what took me so long - finding that enum!  😆


Update scala-library to 2.13.11

About this PR
📦 Updates org.scala-lang:scala-library from 2.13.8 to 2.13.11
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-lang", artifactId = "scala-library" }
}]

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

Superseded by #162.


Running on Digital Ocean

I need to learn how to use this! I have a very cool idea that I believe this will help me with!
I now just need to understand some basics. How do I get this on the web?
I have a DigitalOcean Droplet with Gesture on it. I edit the github_username & blog_url to what I need.
I point my DNS to the DO droplet and nothing. Can you please help me with this lol

Sure man! You can configure NGINX to reverse proxy the serving port from gisture or place the generated static files directly from NGINX as well. And for DNS, first setup fhe nameservers and then add the A/CNAME records for fhe domain from the DigitalOcean network tab.

If you are trying to access gisture directly from the serve feature, it won't be accessible publicly since it runs on <IP_ADDRESS> and not <IP_ADDRESS>.

Wow u caught me red-handed lol

You can configure NGINX to reverse proxy the serving port

This is what I will need help with

feel free to DM me on Twitter.

I reached out under the same username

Thanks for helping solve the issue!


Wsl shell always shows an error message on the daily inform.

/usr/lib/ubuntu-release-upgrader/release-upgrade-motd: 31: cannot create /var/lib/ubuntu-release-upgrader/release-upgrade-available: Permission denied
This is what the message shows on wsl once a day. Although it seems to be no harm, I still wonder how can I avoid this.
I have tried the chmod command to open the permission of release-upgrade-available, but it doesn't work. Since the release-upgrade-motd belongs to root, there shouldn't be any permission denied problem.

this

this

Thanks. However I ran into trouble when trying some seemingly incomplete fix in the link.
Someone on that page posted a 'harmful' fix of an official package because he posted this fix in an inappropriate way - offering a non-official ppa contain a special version designed by himself. This package destroyed my apt and I have no idea about how to deal with this situation. Since my working environment in wsl is easy to rebuild, I reinstalled the wsl on my pc.
Surprisingly, this issue - the error message - didn't pop up since then. So in a nut shell, Reinstalling wsl may fix this problem, although it's not a good idea.

I ran into the same issue. I resolved it by removing /var/lib/ubuntu-release-upgrader/release-upgrade-available. This causes /usr/lib/ubuntu-release-upgrader/release-upgrade-motd to skip to the second condition, but since I don't login as root it never tries to write the file.

@spaulaus  maybe this is a solution, but I don't like it. You say, it never tries to write the file unless you login as root? However, whenever I login, no matter if I am root, it always tries to write the file.
I have to admit that I made the conclusion too soon in the last comment. I said that reinstalling wsl will help, but it didn't. The message pop up later again. Luckily, I managed to 'fix' it by opening the access of /usr/lib/ubuntu-release-upgrader (not the file, but the folder) using the command
$sudo chmod 664 /usr/lib/ubuntu-release-upgrader

and the message haven't shown since then. It's strange for that release-upgrade-motd should have the root authority but it cannot write the file.

@sffred : I agree that this isn't an optimal solution since it prevents the file from being generated.
Here's the contents of /usr/lib/ubuntu-release-upgrader/release-upgrade-motd on my WSL install.
stamp=/var/lib/ubuntu-release-upgrader/release-upgrade-available
if [ -f "$stamp" ]; then
        # Stamp exists, see if it's expired
        now=$(date +%s)
        lastrun=$(stat -c %Y "$stamp") 2>/dev/null || lastrun=0
        expiration=$(expr $lastrun + 86400)
        if [ $now -ge $expiration ]; then
                # Older than 1 day old, so update in the background
                /usr/lib/ubuntu-release-upgrader/check-new-release -q > "$stamp" &
        elif [ -s "$stamp" ]; then
                # Less than 1 day old, and non-empty, so display now
                cat "$stamp"
                echo
        fi
elif [ "$(id -u)" = 0 ]; then
        # No cache at all, so update in the background
        /usr/lib/ubuntu-release-upgrader/check-new-release -q > "$stamp" &
fi

As you can see, the first if checks if the file exists. If the file does not exist, the elif checks if the executing user is root. My solution prevents the upgrade check from happening at all, by ensuring that both of these conditions are false since the file doesn't exist and I don't login as root.
By default /usr/lib/ubuntu-release-upgrader has the following permissions:
drwxr-xr-x  2 root root 4.0K Jun 13 07:21 ubuntu-release-upgrader

Setting 664 on this directory will give drw-rw-r--, which will prevent any script from actually navigating into the folder at all. This gives the same effect as my solution since the script cannot confirm that the file exists, and the script is not being run as root. See the image below:

@spaulaus : I made a mistake because I am not familiar enough with chmod. Actually I set it to 664 first and then found it was wrong and set it to 775 instead. Sorry for that and I have updated the comment.
I looked at my  /usr/lib/ubuntu-release-upgrader/release-upgrade-motd and it's just the same. Yeah, I agree with you that deleting the file will prevent it from being generated again, but when I deleted the file the message is still there. (Actually this is the first thing that I tried.) It's really odd, but I'd ignore this issue and go on my work. Anyway, I have solved it now, at least to some extent.

@sffred : No worries! It's interesting that the script is executing as a user in the root group but is not the root user.
One more question: Did you get this message after upgrading to a new version of Ubuntu (e.g. from 19.10 -> 20.04) using do-release-upgrade?

After a little more research, I found the following solution, which prevents the release check all together. This is probably the safest solution.
sudo chmod -x /etc/update-motd.d/91-release-upgrade

@sffred : No worries! I've confirmed that the script is running as the user logging in.
One more question: Did you get this message after upgrading to a new version of Ubuntu (e.g. from 19.10 -> 20.04) using do-release-upgrade?

hhh that's a sad story. I run do-release-upgrade several month ago and it totally broke my disto. I really experienced a hard time because I tried to fix it manually and it took me several hours. And I chose to reinstall wsl at last. After that I still get the message so I came here for help.

After a little more research, I found the following solution, which prevents the release check all together. This is probably the safest solution.
sudo chmod -x /etc/update-motd.d/91-release-upgrade

Really? But I found that /etc/update-motd.d/91-release-upgrade is already a executable script
$ l /etc/update-motd.d/91-release-upgrade -l
-rwxr-xr-x 1 root root 299 Dec 19  2019 /etc/update-motd.d/91-release-upgrade*

That's correct. /etc/update-motd.d/91-release-upgrade is executable. -x removes the execute bit from the script and prevents it from executing.
-rw-r--r-- 1 root root  299 Jun 18 08:29 91-release-upgrade

Reproducing again in 22.o4 WSL

Can confirm :
Welcome to Ubuntu 22.04 LTS (GNU/Linux <IP_ADDRESS>-microsoft-standard-WSL2 x86_64)

 * Documentation:  https://help.ubuntu.com
 * Management:     https://landscape.canonical.com
 * Support:        https://ubuntu.com/advantage
/etc/update-motd.d/50-landscape-sysinfo: 17: cannot create /var/lib/landscape/landscape-sysinfo.cache: Permission denied

Is there a downside to fixing it this way?
sudo chmod a+w /var/lib/landscape/landscape-sysinfo.cache

@QNimbus Hello, have you found a solution to your problem? I'm also having the same error message on my terminal. I'm new to Linux and just started out with Ubuntu 22.04 LTS on WSL2. I kinda do not understand what the problem is or how to fix it?

sudo apt remove python3-twisted
sudo apt install python3-pip
pip3 install twisted

sudo apt remove python3-twisted
sudo apt install python3-pip
pip3 install twisted

Does Ubuntu 22 uses python3-twisted for something? Otherwise one could just remove it (first line above).

sudo apt remove python3-twisted sudo apt install python3-pip pip3 install twisted

I think it's better to:
sudo apt remove landscape-common
sudo apt autoremove

@msveshnikov

Reproducing again in 22.04 WSL

@QNimbus

cannot create /var/lib/landscape/landscape-sysinfo.cache: Permission denied

This is a new error in 22.04, not related to the previously fixed issue that this thread refers to.  Apparently they left the Landscape feature from Ubuntu Server in the 22.04 WSL distro when it probably should have been removed.
Also tracking this at this Ask Ubuntu question, but it probably needs a Launchpad bug report if there isn't one already (haven't checked yet, but will).

/usr/lib/ubuntu-release-upgrader/release-upgrade-motd: 31: cannot create /var/lib/ubuntu-release-upgrader/release-upgrade-available: Permission denied
please has anyone found a solution to this prompt message? please help I'm new to linux


PSADT deployment error

As far as I can tell, AdobeAcrobatReaderDC is the only PSADT example in this repo. I'm getting an 0x80070002 error when I try to deploy it. If I build the AdobeAcrobatReaderDCMUI package using the standard tooling, it deploys without issue.
Also, just a suggestion, but at 420MB, perhaps it would be better to use something smaller as an example, say Brave.
https://github.com/brave/brave-browser/releases/latest
Thanks!

I've expected the same issue and I'm trying currently the workaround to specify as install command:
%SystemRoot%\sysnative\WindowsPowerShell\v1.0\powershell.exe -executionpolicy bypass -command .\Deploy-Application.ps1
instead of
Deploy-Application.exe -DeploymentType "Install" -DeployMode "Silent"
maybe we need to write it as **.**Deploy-Application.exe -DeploymentType "Install" -DeployMode "Silent"

I've expected the same issue and I'm trying currently the workaround to specify as install command:
%SystemRoot%\sysnative\WindowsPowerShell\v1.0\powershell.exe -executionpolicy bypass -command .\Deploy-Application.ps1 instead of Deploy-Application.exe -DeploymentType "Install" -DeployMode "Silent"
maybe we need to write it as **.**Deploy-Application.exe -DeploymentType "Install" -DeployMode "Silent"

mh okay, it's in all variantes "The system cannot find the file specified. (0x80070002)" but the output of the source looks not so bad:


Support Dynamic Blocks: Read Evaluation Graphs and Block Visibility Parameters

Is your feature request related to a problem? Please describe.
We currently implement a converter from DWG to SVG. The converter is aimed to provide SVG fragments (templates) defined in dynamic blocks. An application shall compose these fragments/templates to a complex drawing. This requires that not only block records can be read and converted into SVG groups. Dynamic block are subgrouped as defined in a BLOCKVISIBILITYPARAMETER object. The BLOCKVISIBILITYPARAMETER is associated with a block record. It essentially defines subsets of the entities of the block record and allows to set one of them optionally visible. These subsets shall be converted into SVG groups of the SVG group representing the block record.
Analysis
I am not yet an AutoCAD expert. But I understood the following that can be seen in the DXF export of my DWG file:
A block record that is a dynamic block is associated with a BLOCKVISIBILITYPARAMETER object as follows:

The XDictionary associated with a block record that is a dynamic block has an entry named ACAD_ENHANCEDBLOCK.
This points to an EVALUATION_GRAPH object.
One of the graph nodes points to the BLOCKVISIBILITYPARAMETER object.

The details of the function of the evaluation graph seem not to be important to solve my problem. I also do not need the objects associated with the other nodes of the graph.
Desired Solution

Implement a reader for BLOCKVISIBILITYPARAMETER.
Implement a reader for the EVALUATION_GRAPH.

The converter then can read the ACAD_ENHANCEDBLOCK dictionary entry from the block-record object, find the graph node with
the BLOCKVISIBILITYPARAMETER, and read the subgrouping to create SVG groups.
The Problems

I do not have any documentation about EVALUATION_GRAPH and BLOCKVISIBILITYPARAMETER data fields, neither a description of the DWG structure nor DXF group codes.
I can guess from the DXF export what the fields of an EVALUATION_GRAPH and its graph nodes should be including the field type. I succeeded to read a EVALUATION_GRAPH, without understanding the meaning of all fields. The result is sufficient to route to the BLOCKVISIBILITYPARAMETER object.
Reading the BLOCKVISIBILITYPARAMETER is difficult. The sequence of data in the DXF seems to be different from the DWG data.

I succeeded to read the subgroup data of the BLOCKVISIBILITYPARAMETER after I deternmined the offset of an expected field manually. After that I could use the subgroup data in the converter. I could create SVG subgroups containing the expected entities. But most of the data found in DXF cannot be found.

I could solve the problems reading BLOCKVISIBILITYPARAMETER data fields, by analysing the data from the stream and guessing a sequence of field types. All DWG files I have so far could be read successfully. A systematic investigation creating various dynamic blocks will hopefully harden the aproach.

I solved it as far we need it for our DWG-to-SVG converter.
It is, of course, only a small part of the whole story. Unfortunatley I did not find yet any documentation about the fields of the objects involved.
We will crete a PR, perhaps it helps to proceed.

ACadSharp has bee forked: see https://github.com/mme1950/ACadSharp/issues/3


Populate with Array doesn't work anymore

I don't know if its me and i just do it the wrong way or if this is a bug but populate with an array of fields to populate doesn't work.
The params i try to use look like this:
const params = {
populate: {
contentSections: {
populate: ['firstSection', 'secondSection', 'anotherSection']
},
}
If i try to fetch Data from the Backend i get the following error: "Invalid populate field. Expected a relation, got string"
I also tried it with "contentSections" with the same error.
I used the same syntax with an earlier version of strapi so i know this worked before.
Would be nice to know if i do something wrong or if this is a bug either in strapi or in strapi-module.

@l0ggik try
const params  =  {
  populate: [
     'contentSections.firstSection',
     'contentSections.secondSection',
     'contentSections.anotherSection'
  ]
}

for me this way works as expected.

For me it does not work on the auth.populate strapi config


We need to store the file_size as something other that an integer

We had this in the code pre Sufia 7.
solr_doc[Solrizer.solr_name('file_size', STORED_SYMBOL)] = object.content.size.to_i
I think that was to allow for sizes over the solr integer field.

Here is the error message we see:
/rsolr/client.rb:288:in `adapt_response': RSolr::Error::Http - 400 Bad Request (RSolr::Error::Http)
Error: 'ERROR: [doc=3t945q772] Error adding field 'file_size_is'='3091666335' msg=For input string: "3091666335"','code'=>400}}

I am still seeing this error when importing file on QA.  Either the code did not get out to qa, I deployed yesterday mid morning or this is not entirely fixed.

Looks like I didn't set FileSet to use the custom FileSetIndexer.

@cam156 what size file do we need to use to test this?

@awead one of the files that is failing is<PHONE_NUMBER> (see the comment above).  I'm guessing than is larger than needed, but a good number to work with.  I'm guessing the solr documentation would give you the real limit.

@cam156 that's a little over 3 GB. What's the present size limit on file uploads?

@awead I believe the cloud files are 7G

Fixed, maybe, but there might be another issue with the work size display. https://scholarsphere-qa.dlt.psu.edu/concern/generic_works/k35694774

Check this file since the above file is no longer in the system: https://scholarsphere-qa.dlt.psu.edu/concern/parent/k6d56zx87n/file_sets/4b8515n08m


[WIP] chore(flat-components): change wepack to vite

This pull request is still process on,here is the discussion https://github.com/netless-io/flat/issues/1143

Whether to consider storybook-builder-vite

Whether to consider storybook-builder-vite

Thanks, let me test if it works.

Whether to consider storybook-builder-vite

please see https://github.com/netless-io/flat/issues/1143#issuecomment-981253929，to get more detail

I'll close the push request and wait for me to finish before making it .


Lost connection to Mount Pod after OOM

I have set 'mountPropagation: HostToContainer' in the work Pod.
After oom occured in mount Pod, the mount Pod restarted immediately(using Dynamic provisioning). But when I run the command 'ls', it returns nothing.
I suppose the connection between work Pod and mount Pod will be bind and I can still see the files from the mount Pod after the mount Pod restarted.
How can I fix this?

Hi @lixiaofengfeng , can you provide the response of df -h in work pod?

Filesystem                      Size  Used Avail     Use%  Mounted on
JuiceFS:lsp-drive-bce  1.0P  9.9T   1015T   1%       /home/user/data

It seems ok that mountpoint is not disconnected

But the command "ls /home/user/data" returns nothing. But there are many files in this directory.
Btw, after I restarted the work Pod, the mount Pod is restarted the same time, and the command "ls /home/user/data" returns the files that existed.

I suppose, after the 'mount bind', I will have the same access to the mount point on '/home/user/data'.

I just created a work pod and a mount pod, and restarted the container 'jfs-mount' within the mount pod by 'kill -9'. In the work pod container, the command 'ls /home/user/data' returns the right answer.
The situation that the command 'ls /home/user/data' returns nothing only occured when the mount pod restarted because of oom so far.

@lixiaofengfeng hi, I did not reproduce it. Can you try again the OOM situation ?

I wondor why mount different PVC to /home/user and /home/user/data. They may affect each other. I guess that when mount pod oom, /home/user/data is the result of subpath of /home/user directory.

Long history. I found that the results are different when the mount pod container is kill by 'kill -9' or oom. If I restart the container by command 'kill -9', command 'ls /home/user/data' would return the real data, but it returns nothing when oom occurred. BTW, it would recover through restarting work pod when oom occurred. What would juicefs-csi-driver do when oom occurred or restarted container manually？


Nagios reports "NRPE: Unable to read output" for netns check

focal-ussuri, charms revision 20.10
check_status_file.py script cannot be executed because /usr/bin/python is not available on the machine. It was available on bionic but it does not exist on focal.
ubuntu@network-1:/etc/nagios/nrpe.d$ cat check_netns.cfg
check netns
The following header was added automatically by juju
Modifying it will affect nagios monitoring and alerting
servicegroups: juju
command[check_netns]=/usr/local/lib/nagios/plugins/check_status_file.py -f /var/lib/nagios/netns-check.txt
ubuntu@network-1:/etc/nagios/nrpe.d$ sudo /usr/local/lib/nagios/plugins/check_status_file.py -f /var/lib/nagios/netns-check.txt
sudo: unable to execute /usr/local/lib/nagios/plugins/check_status_file.py: No such file or directory
ubuntu@network-1:/etc/nagios/nrpe.d$ head -1 /usr/local/lib/nagios/plugins/check_status_file.py
#!/usr/bin/python
ubuntu@network-1:/etc/nagios/nrpe.d$ /usr/bin/python
-bash: /usr/bin/python: No such file or directory

(by phausman)
Subscribing ~field-high as this issue occurs on a customer deployment.
The workaround could be to create a symlink to /usr/bin/python2 on the affected nodes.

(by guoqiao)
A MP made with simple shebang change:
/usr/bin/python -> /usr/bin/env python3
https://code.launchpad.net/~guoqiao/charm-nrpe/+git/nrpe-charm/+merge/397298

(by guoqiao)
I have verified python3 is installed by default on trusty, so we don't need to worry about python3 installation.
Code already merged into master as ce7cc4ecee33aeffb6ed9025c693db9c18ff1771


Upgrade to upstream 1.25.0 and sync Helm chart for missing features

Towards: https://github.com/giantswarm/giantswarm/issues/24865

Bumped azuredisk-csi to upstream version 1.25.0
Increased qps limits for csi-provisioner and csi-attacher to match upstream
Increased WorkerThreads for csi-provisioner to match upstream
Increased csi-attacher timeout from 600s to 1200s to match upstream
Add support for http/s proxy settings to azuredisk
as per KEP 2067 add toleration for node-role.kubernetes.io/control-plane label and update nodeSelector
add rbac to get nodes for azuredisk-node to match upstream
Make some hardcoded fields in the template a value so that we can set them differently in Vintage and in CAPZ

runOnMaster nodeSelector label
AZURE_CREDENTIAL_FILE location

Summary of Changes from upstream project
1.22

chart: As per KEP 2067, node-role.kubernetes.io/master taint is replaced with node-role.kubernetes.io/control-plane
Bugfix for default iops on UltraSSD disks

1.23

support PremiumV2_LRS disk type
support location in storage class
chart: support http/s proxy settings
increase csi-attaccher timeout

1.24

chart: increase client QPS settings for csi-attacher , csi-provisioner

1.25

support advanced perf profile on linux nodes
chart: add no_proxy settings

ToDo:

[x] Check node-role.kubernetes.io/control-plane on a Workload Cluster azure vintage that the label is set on master node

Vintage MC cluster do have the label but WC do not. i will make this a value in the values.yaml and default to old value

[x] Check node-role.kubernetes.io/control-plane on glippy that label is set on master node
[x]  Check location and contents of AZURE_CREDENTIAL_FILE in vintage vs Capz

vintage: /etc/kubernetes/config/azure.yaml
capz: /etc/kubernetes/azure.json
[x] Support having different values for AZURE_CREDENTIAL_FILE in vintage vs capz

Tests:

[x] Vintage WC

[x] Check Rendered Yaml helm get manifest and compare to current version
[x] Create Volume
[x] Delete volume

[ ] CAPZ

[ ] Create Volume
[ ] Delete Volume

I expected that something is not quite right with this PR since it was pushed to the test catalog as "add azuredisk-csi-driver-app-1.21.0-gs4-67d2032765d14fb6dd904c777c4221c1e7487ccb.tgz" rather than 1.25.0 ... need to figure out how to get the right version

FYI i am having an issue on capz i need to resolve, and probalby retest on vintage as well
https://kubernetes.slack.com/archives/CEX9HENG7/p1669828285278899
so i will probably have to apply some changes and re-ask for reviews eventually , sorry i rushed it :) :pray:

Issue was resolved by setting disable-avset-nodes=false , which it has been set to false upstream by default since 1.18.0 https://github.com/kubernetes-sigs/azuredisk-csi-driver/commit/9a1b96c1b4973bc329a9dd4f30816ba9925ee789
I will try to understand what does it acually do then we should decide what to do with default setting for legacy ( which is set to true )  @whites11

Upstream description of the flag
that's an optimization if it's set as true, it would skip avset vm list check directly, it's ok to set as false by default, which may cause a little more single vm list operation on VMSS clusters, but that's ok

I will do a second test on vintage , i already did one with the flag as it is now and it worked fine at least on a brand new vintage cluster i created on purpose wich runs both master and workers as VMSS

Asking for re-review due to some code changes and because this is my first time pushing code to an app :)
@giantswarm/team-phoenix i did get a review from @whites11 before but he is off now so i just added the whole team back as a reviewer since this does impact you to some extent.
I did all i think was needed to keep compatibility between vintage and capz for the helm chart.
I also tested the upgraded chart on a Vintage WC but not a MC
From what i understand even when released this won't apply to any vintage cluster until is added to a new release for WC or updated in Terraform for MC ?
Thanks


[installer] Update the installer

What does this PR do?

Add experiment unit file
Setup and start the updater experiment

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge -c

cancelled the merge as the e2e test didn't pass, and it's not blocking yet the pipeline from merge, so you have to check it manually before merging for now
=== FAIL: tests/updater TestCentOSARM/TestInstallerUnitLoaded (0.10s)
    suite.go:249: No change in provisioners, skipping environment update
    remotehost.go:82:
        	Error Trace:	/go/src/github.com/DataDog/datadog-agent/test/new-e2e/pkg/components/remotehost.go:82
        	            				/go/src/github.com/DataDog/datadog-agent/test/new-e2e/tests/updater/linux_test.go:107
        	Error:      	Received unexpected error:
        	            	ls: cannot access /usr/lib/systemd/system/datadog-installer.service: No such file or directory
        	            	: Process exited with status 2
        	Test:       	TestCentOSARM/TestInstallerUnitL

@raphaelgavache I added a skip for the test on rpm based distros

/merge


[Bug]: priority scheduling doesn't work according to token_per_s. The token_per_s of requests with higher priorities is not higher than that of requests without priority settings.

Your current environment

The output of `python collect_env.py`
Your output of `python collect_env.py` here

Model Input Dumps
No response
🐛 Describe the bug
Does the vllm priority scheduling work? I've set the "max-num-seqs" to 32. At first, I sent 30 requests with a priority of 0 and 1 request with a priority of -3 to access it together. Later, I  sent 30 requests with a priority of 0 and 1 request with a priority of 0 to access it together. The "token_per_s" of the request with a priority of -3 is the same as that of the requests with a priority of 0 later. What's the problem?
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

same issue


Allow admins to add unique links and remove existing ones on the live site

Rather than submitting new commits for every new shortened link, admins should be able to sign-in at h4i.app and add/remove forwards.

IMO this requires too much work to be worth it, we'd need a full stack app with auth and a database - I don't think the benefit outweighs the development time, and its not a very interesting technical problem either in my eyes (I wouldn't want to spend time writing it out as I don't find it compelling).

Yeah, not sure why you were assigned for this. I'll work on it this summer, or have the infra team work on it. Will also be rewriting the backend completely in express and hooking into the memberDB tool's api to figure out which members are actually registered in our org (which i think is a pretty cool use for it).

Honestly I still don't think it's that worth it as we do have our own membership list through GitHub - I only see value in having this be someone's first full stack app as it sits in the weird spot of extremely simple business logic that still requires a full stack which is both too simple and too much work. Up to you all ofc on final call though

Appreciate the note, but making commits for every change is not the way I want this infra to work. I also don't think authentication and the backend are a lot of work, as it's nothing new from what we've done with memberDB tool, and think this would be a good project for the summer with its small scope.

If there are features I would want to see from a more complex version it would be link visibility - which shortcuts do we want put on the main page vs. which we don't (like an "unlisted" link)

Additionally maybe analytics of some sort


Added PodDisruptionBudget

Description
Changes proposed in this pull request:

Added PodDisruptionBudget.

Related issue(s)

https://github.com/kyma-project/nats-manager/issues/92

/test pull-nats-manager-unit-test

/test pull-nats-manager-unit-test

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

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

/test pull-nats-manager-build

/cla-recheck


AR button is not showed in Safari on iPad Pro on iOS 13

Description
AR button is not showed in Safari on iPad Pro on iOS 13. According to Apple's list of supported devices here down in the page this device has support for AR on Safari.
Live Demo
Browser Affected

[ ] Chrome
[ ] Edge
[ ] Firefox
[ ] Helios
[ ] IE
[X] Safari

OS

[ ] Android
[X] iOS
[ ] Linux
[ ] MacOS
[ ] Windows

Versions

model-viewer: v0.6.0
three.js: r105

@fselcukcan thanks for the report. Could you share a link to a demo that exhibits this behavior for you, or else paste the <model-viewer> HTML you are using so we can try to repro ourselves?

The ar-button is not shown anymore because the test :
const IS_IOS = /iPad|iPhone|iPod/.test( navigator.userAgent ) && !self.MSStream;
fails on iPads with iOS 13/iPadOS 13 Beta.
see:
https://stackoverflow.com/questions/57599945/how-to-detect-ios-13-on-javascript
https://stackoverflow.com/questions/57765958/how-to-detect-ipad-and-ipad-os-version-in-ios-13-and-up

iPadOS 13 will be released to the public on September 30th, so it would be amazing if this fix could make it into the next release to avoid model-viewer's AR feature breaking on every iPad.

@scanblue-sr ahh, thanks for the added details.
@freinbichler acknowledged, we'll try to get this in to v0.7.0 which should go out before the end of the month.

FYI while I do think we'll have v0.7.0 before the end of the month, we will also be back-porting this fix and releasing it as v0.6.1 for the benefit of folks who won't be able to update to v0.7.0 right away.

Re-opening, as it appears that the logic in our fix was wrong. A new fix should go out later today.

FYI v0.6.2 has been released including the updated fix for this issue.

@scanblue-sr @cdata  Instead of testing the user agent I have an idea of a feture detection as this so it works even if the mentioned iPad user agent string looks like a desktop browser. In my experience it had nothing like iPad thus failing the regexp test above. It had Macintosh in it instead.
SO I have wrote the following function:
function doesSupportAnchorRelAR() {
  return document.createElement("a").relList.supports("ar");
}

after research and finding that: https://webkit.org/blog/8421/viewing-augmented-reality-assets-in-safari-for-ios/

@fselcukcan as I mentioned in another conversation , we already perform this test. Unfortunately, this test will be true for both Chrome and Safari on iOS, but in some versions of Chrome the behavior is different (this is why we have the quick-look-browsers attribute, to allow folks to opt-out of Chrome specifically).
That said, it's possible that you're right that we could skip the user agent detection at this point. We just need to make sure that we don't have any false positives where browsers do support rel="ar" but have different behavior from iOS Safari.

I see now all the things about Content-Type header requirement. But I do not see practicality of opting out of it in Chrome on iOS. Since there is no ARCore as there is in Android, Chrome can only make use of Quick Look.

@fselcukcan the reason why is because Chrome on iOS is able to launch Quick Look. Quick Look is a separate app, not a part of the web, and not part of Safari (the same goes for Scene Viewer on Android). The problem is that on Chrome on iOS, there is an extra requirement that is very hard for some folks (setting the Content-Type header). This is why we have an opt-in approach, because otherwise we would display the AR button and instead of launching Quick Look the file would just download in many cases.

I see now. Thank you for detailed explanation again.


Add image search support

URLs

Yandex https://yandex.com/images/search?text=test and /video/
Yahoo https://images.search.yahoo.com/search/images?p=test https://video.search.yahoo.com/search/video?p=test
Google https://www.google.com/search?q=test&tbm=isch https://www.google.com/search?q=test&tbm=vid
DDG note both params are needed https://duckduckgo.com/?q=test&iax=images&ia=images https://duckduckgo.com/?q=test&ia=videos&iax=videos
Bing https://www.bing.com/images/search?q=test https://www.bing.com/videos/search?q=test