bug: v0.3.0 prep_for_syncer refs v0.2.1 syncer image

Describe the bug
https://github.com/kubestellar/kubestellar/blob/v0.3.0/scripts/kubectl-kubestellar-prep_for_syncer#L37
Steps To Reproduce
See above
Expected Behavior
The latest release should reference the latest syncer image.
Additional Context
No response

/cc @clubanderson
/cc @yana1205

@yana1205 please PR against release-0.3 only - Thank you

ahh - good point @MikeSpreitzer - yes we do


Snaking voting phases exclusive to team, plus rewrite of state machine

Guess this PR isn't as consolidated to one feature as I thought.
As the title says, this PR adds several things:

A rewrite of the state machine handling to avoid duplicated code.
Voting phases now snake between teams to avoid the coin-flip meta. Previously, all votes from all teams were tallied simultaneously. Since ties were broken at random, one team could end up screwed out of their choices.

Note that this PR does not aim to finalize the interface in any way. Debug tools and displays have intentionally been left in.


Some downloads fail with KeyError: 'url'

Verbose log
Y:\YouTube Channels\NonCompete>youtube-dl --cookies "Y:\cookies.txt" --add-metadata --write-description --write-info-json --write-thumbnail --download-archive "Y:\ytdl_archive.txt" --all-subs --sleep-interval 1 -ciw -o "%(title)s.%(ext)s" -v https://www.youtube.com/channel/UCkZFKKK-0YB0FvwoS8P7nHg
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--cookies', 'Y:\\cookies.txt', '--add-metadata', '--write-description', '--write-info-json', '--write-thumbnail', '--download-archive', 'Y:\\ytdl_archive.txt', '--all-subs', '--sleep-interval', '1', '-ciw', '-o', '%(title)s.%(ext)s', '-v', 'https://www.youtube.com/channel/UCkZFKKK-0YB0FvwoS8P7nHg']
[debug] Loading archive file 'Y:\\ytdl_archive.txt'
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dlc version 2020.10.26
[debug] Python version 3.8.6 (CPython) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg git-2020-04-15-51db0a4, ffprobe 4.3.1-2020-10-01-full_build-www.gyan.dev
[debug] Proxy map: {}
[youtube:channel] UCkZFKKK-0YB0FvwoS8P7nHg: Downloading channel page
[youtube:playlist] UUkZFKKK-0YB0FvwoS8P7nHg: Downloading webpage
[download] Downloading playlist: Uploads from NonCompete
[youtube:playlist] UUkZFKKK-0YB0FvwoS8P7nHg: Downloading page #1
[youtube:playlist] playlist Uploads from NonCompete: Downloading 159 videos
[download] Downloading video 1 of 159
[youtube] T78IcPvnBtQ: Downloading webpage
[debug] Default format spec: bestvideo+bestaudio/best
[info] Video description is already present
[info] Writing video subtitles to: Viva South America! El Breadcast!.live_chat.json
ERROR: 'url'
Traceback (most recent call last):
  File "youtube_dlc\YoutubeDL.py", line 841, in extract_info
  File "youtube_dlc\YoutubeDL.py", line 896, in process_ie_result
  File "youtube_dlc\YoutubeDL.py", line 1691, in process_video_result
  File "youtube_dlc\YoutubeDL.py", line 1893, in process_info
KeyError: 'url'

Description
For some videos, I'm seeing failures with KeyError: 'url' (and this time I've actually confirmed the videos work on the site). This is with the latest release version.

all-subs related.

This seems to be caused by the change that disables sleep interval for subtitles. As such, enabling --sleep-subtitles makes it work again and I can download videos along with the live chat and other subtitles just fine.
When --sleep-subtitles is on it uses the dl function to download, which in turn chooses the correct handler for live chats. Without it, it seems to use some new code that tries to directly download the subtitles instead, which breaks:
At line 1894 in YoutubeDL.py:
                            if self.params.get('sleep_interval_subtitles', False):
                                dl(sub_filename, sub_info) <---------- This works fine
                            else:
                                sub_data = ie._request_webpage(
                                    sub_info['url'], info_dict['id'], note=False).read() <---- This breaks at sub_info['url']
                                with io.open(encodeFilename(sub_filename), 'wb') as subfile:
                                    subfile.write(sub_data)

It seems the correct thing to do would be to use the dl function in both cases, but temporarily disable (or just set to zero) the sleep interval while the function is called. Something like that at least.
I'm not familiar enough with the code to make a patch myself, but hopefully this helps. At least there's workaround for those who need it!

@im-coooding that is actually quite interesting that you mention it. I will revert current effort to deactivate live_chat and pass along some default value of 0 for sleep interval for subtitles.


Hyper quit unexpectedly

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Mac OS X 10.11.5
Hyper.app version: Version 1.0.0 (<IP_ADDRESS>3)
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/himynameisdave/8a95c1f718dffc58aa511ffabdd80af9
Relevent information from devtools N/A
The issue is reproducible in vanilla Hyper.app: Is Vanilla

Issue
Mac was asleep, then on wakeup I was greeted with this Apple standard warning!

Skimming through the contents of the generated report, I can't see anything that seems too out of place... 🤔  I was then able to restart it and get back to work.
I had two windows open, both fullscreen and both subdivided into three "windows" each, making six shells total. Most of these were running some background processes (like docker and grunt) at the time I noticed the issue.

Did you have any other screens connected/disconnected when this happened? @himynameisdave

Yes, one external monitor was connected at the time.


Bump Project Version, Esbuild Version For New Release

Overview
Since there have been several updates to this package since the 0.4.0 I figured I could put together a pr to release a new version with several fixes and enhancements.
Here is the diff from this branch to the 0.4.0 release:
https://github.com/phoenixframework/esbuild/compare/v0.4.0...kinson:bump-versions
If it is too early to cut another release for this project let me know, I can either close it or trim it to just include the esbuild update and GitHub CI update!
Changes
This pr:

updates the esbuild binary version from 0.14.29 -> 0.14.41
adds elixir 1.13 (OTP 25) to the test matrix for GitHub CI
bumps the project version to 0.5.0
updates the README and CHANGELOGS with version update details

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

@josevalim thanks for the quick review! I will create a pr into the main phoenix repo with these updates once the new tag/release is created 💜

released!

I created a pr in the main Phoenix repo here https://github.com/phoenixframework/phoenix/pull/4841


Move all external project dependencies to third_party repo

We could use submodules but I find them difficult to work with. This isn't changing how we use externalproject, it is just consolidating all the core uses into third_party.


Console Log Improvements

Resubmitting in favor of https://github.com/NativeScript/android-runtime/pull/933 which appeared empty after reverting the revert commit.

run ci


HHH-17421 Add test for HHH-17421

https://hibernate.atlassian.net/browse/HHH-17421

Thanks for the test. Superseded by https://github.com/hibernate/hibernate-orm/pull/8324


[CELEBORN-1493] Check admin privileges for http admin requests

What changes were proposed in this pull request?
Why are the changes needed?
Does this PR introduce any user-facing change?
How was this patch tested?

waiting for #2599

cc @pan3793 @RexXiong @SteNicholas

Thanks, Merge to main(v0.6.0)


Figure out why jose-jws-test.html doesn't work with coverage.

typeof(window.CryptoKey) is undefined on Chrome 37. Utils.isCryptoKey is returning the incorrect result.

Fixed in https://github.com/square/js-jose/commit/f4b699f289877f417545498763f5b659503b203e


[snapshot] Fix Swift compiler version check for SnapshotHelper.swift

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
Closes #18792
Description
There was a wrong compiler version check for UIImagePNGRepresentation deprecation, which led to a compile error in snapshot
Testing Steps

Checkout this branch.
Go to snapshot and open the Example.xcodeproj
Open SnapshotHelper.swift with a Swift version set at 4.0
Compile it.
:rocket:

@googlebot I consent.


fix: remove check that doesn't allow both wallet and wallet_address to be provided

This check causes errors because after perps is initialized wallet_public_key is set and this is used in the Trader and Product classes to init perps, so they will always throw an error since by the time they are called perps has been initialized and both wallet and wallet_public_key are available.

Closed because the fix for this was merged along with this pr #15


Needlessly running a method where a variable can take it's place

(Describe the goal of this PR. Mention any related Issue numbers)
Running time of a method is greater than accessing a variable, and thus it's place should be taken by a final variable.
Category (place an x in each of the [ ])

[ x] bolt (Bolt for Java)
[ ] bolt-{sub modules} (Bolt for Java - optional modules)
[ ] slack-api-client (Slack API Clients)
[ ] slack-api-model (Slack API Data Models)
[ ] slack-api-*-kotlin-extension (Kotlin Extensions for Slack API Clients)
[ ] slack-app-backend (The primitive layer of Bolt for Java)

Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to the those rules.

 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.


MacOS error dyld[40451]: missing symbol called

I am getting this error when trying to run this on my 13" MacBook Pro 2020 running macOS Monterey (v12.5.1) :
dyld[40451]: missing symbol called
zsh: abort      node test.js

Hey, thanks for reporting. Is it one of those Macs with an ARM processor, right? I don't have one on hand, but I'll try to reproduce the issue anyhow.

Can you tell me how you are trying to run this?
Is "node" available on your environment path?

My mac is actually Intel based.
This is my path, so it doesn't look like I have node there.. I'll have to dig in further. I don't know how to add things to my environment path on mac.
/Library/Frameworks/Python.framework/Versions/3.10/bin:source $HOME/.cargo/env:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin/python3:/usr/local/go/bin:/Library/Apple/usr/bin:/Library/Frameworks/Python.framework/Versions/3.10/bin:source /Users/danraeder/.cargo/env:/Users/danraeder/.cargo/bin


Improving card of each page

Is your feature request related to a problem? Please describe.
The card is too simply i want to add some shadow and make it intresting. can i work on this issue under GSSoC'21

@dinesh9718 can you please assign this issue to me.

@chetan-rns  can you please assign this issue to me.

Thanks 😊


Cursor won't remain set to its assigned style whilst click-and-dragging to move a View

Is this a bug report?
Yes
Have you read the Contributing Guidelines?
Yes
Environment
$ node_modules/.bin/react-native-macos info

Environment:
  OS: macOS High Sierra 10.13.5
  Node: 9.11.1
  Yarn: 1.3.2
  npm: 5.6.0
  Watchman: 4.9.0
  Xcode: Xcode 9.3.1 Build version 9E501
  Android Studio: Not Found

Packages: (wanted => installed)
  react: 16.0.0-alpha.12 => 16.0.0-alpha.12
  react-native: Not Found

react-native-macos: 0.18.0-alpha.3

Aim
Basically I want the cursor to imitate the accessible behaviour you'd expect from NSSplitViewController: the cursor style should reflect whether it's hovering over the divider or not.
Steps to Reproduce
Run the code from the Reproducible Demo section.

Move your cursor onto the pale-cloured divider bar
Try clicking-and-dragging, moving it from side-to-side.
Release your cursor.
Move your cursor off the divider.

Expected Behaviour

The cursor should become set to resizeLeftRight style.
The cursor should remain in resizeLeftRight style.
The cursor should remain in resizeLeftRight style provided that it's still overlapping the divider bar.
The cursor should return to arrow style.

Actual Behavior

✅
╳ The cursor returns to arrow style. Even if an adaptation is added to restore resizeLeftRight style upon any call of the whilstDragging function, the cursor will repeatedly fight to return to arrow style. I think this is related to the divider changing position (possibly re-rendering in the process, and affecting the responder behaviour), but I'm not sure.
✅ (maybe by co-incidence)
✅ (maybe by co-incidence)

See video on Twitter.
Reproducible Demo
Use the code snippet from this Snack with react-native-macos.
Note: commit 7c8be91 from July 3rd 2018 to master must first be applied to fix the Cursor module import issue.

A very good case of how tricky it could be to replicate native UX. Instead, it's appealing to rewrite this with the native component, which is not easy too. I have a little time this weekend, trying to fix as many bugs as I can. Will play with this too.


Fix color values

Fix color values by adding # in the beginning of them.

@AhmedElaguab  Thanks! I missed this.

@ghosh You're welcome.


please can someone send me the code just for the checker?

discord is tbzGAMING#0001

Why

The function is called quickChecker, you can copypaste that yourself


Get Request for ng2-admin not working

Question about the project
I'm trying to do a get request to populate the pieCharts given in the repo with my own information. But after the first get the pieCharts are not populated. only after changing pages then going back it populates.

Same. Well, I'm populating the data from a POST request but it doesn't work.
I've subscribed to the observable inside the constructor, I've tried calling _loadpiecharts and _updatepiecharts in the complete part of the subscription as well as/instead of inside ngAfterViewInit, I've tried subscribing to update the contents of 'charts' in ngAfterViewInit, regardless, the piecharts don't display correctly. The numbers are in there, but the percentage isn't calculated and the graphics don't appear.
Am I doing something obviously wrong? The format of the data I'm retrieving within the subscription is identical to that in the unaltered service so it can't be that.

OK so the only way I could get this to work was by replacing ngAfterViewInit with ngAfterViewChecked, and in the 'complete' section of my subscription, setting the this._init flag to false.
incidentally, if you end up with an infinite loop a quick google suggests this is due to any Console.log's you may have in your code, so try taking these out.
Hope this helps.
`constructor(private _pieChartService: PieChartService) {
this._pieChartService.getData()
  .subscribe(
    pieData => {
      this.charts = pieData;
    },
    error => console.log(error),
    () => { this._init = false;
    });

}
ngAfterViewChecked(){
if (!this._init){
this._loadPieCharts();
this._updatePieCharts();
this._init = true;
}
}`


You Need A Budget (Independent Publisher) issue

Api Name - shared_youneedabudgetip
Bug description
I cannot find the connector when I try to create a new flow.
In the connectors overview I see the connector, I have a premium environment (pay as you go through an azure subscription)
I am able to select other premium connectors, but the you need a budget one is simply not showing up.
Is this a security bug? (Y/N) No
What is the severity of this bug?

Which region are you connecting to?

Wow, that was fast!
I'm trying to connect to Switzerland.

@Amjed-Ayoub can you confirm that this connector is available in the Switzerland region?

In the documentation it says the following about availability:
Power Automate	Premium	All Power Automate regions except the following:
-   US Government (GCC)
-   US Government (GCC High)
-   China Cloud operated by 21Vianet
-   US Department of Defense (DoD)
I am trying with a new environment in Europe now.

When I use an environment in Europe, I can see the connector as I would expect.
It seems that it's not deployed to Switzerland and the documentation is outdated.

Adding @jopanchal - is there a known region restriction for Switzerland?

Hello @vansummeren and @troystaylor,
I hope you are doing well.
In our deployment we don't have a specific region for Switzerland, but we include it within Europe deployment.
I can confirm that the connector is deployed in Europe region in Power Automate(Includes Switzerland region), please see below.

And in Switzerland region in Logic Apps as well. Please see below.

Thank you so much.
Best regards,
Amjed Ayoub


ENG-1858 - feat: refer to new Elements consolidate interfaces

Description
Updates ~elements.create~ to createElement and ~storeCreditCard~ to atomicCards.create

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


Auto Restart

After a while, servers accumulate too much RAM ,
In 2 hours I get to use 12GB, 80 mbps connection
Please make autoRestartTime , thanks.

Hello
There is a memory leak in the stats read this to fix it.
You could use something like this to make it auto Restart
Start server with (this is important so the server autorestarts):
#!/bin/bash while ! node MultiOgar do   sleep 1   echo "Restarting program..." done
Save this as whateveryouwant.sh
do
chmod +x whateveryouwant.sh
start the script aka MultiOgar
./whateveryouwant.sh
or do it in a screen
make the autorestart bash:
`#!/usr/bin/perl
use strict;
use warnings;
use Proc::ProcessTable;
my $table = Proc::ProcessTable->new;
for my $process (@{$table->table}) {
# skip anything other than Passenger application processes
#next unless $process->fname eq 'ruby' and $process->cmndline =~ /\bRails\b/;

# skip any using less than 1 GiB
next if $process->rss < 4_103_741_824;

# document the slaughter
(my $cmd = $process->cmndline) =~ s/\s+\z//;
print "Killing process: pid=", $process->pid, " uid=", $process->uid, " rss=", $process->rss, " fname=", $process->fname, " cmndline=", $cmd, "\n$

# try first to terminate process politely
kill 15, $process->pid;

# wait a little, then kill ruthlessly if it's still around
sleep 5;
kill 9, $process->pid;

}
Save this as whateveryouwantbutnotthesame.sh dochmod +x whateveryouwantbutnotthesame.sh`
be sure to have perl installed and the Proc::ProcessTable module
and this is really bad if you have other things running on your server  cuz it just kills everything over 4gb even root stuff (if you want to skip root stuff add next if $process->uid == 0 or $process->gid == 0;
change this line accordingly
next if $process->rss < 4_103_741_824;
this would kill the server at some weird 4 GB.
do
crontab -e
and add a line to run the autokill script every 20 min or so
*/20 * * * * /home/multiogar/whateveryouwantbutnotthesame.sh
exit and save.
Not really tested it like this but i'm using a similar method to keep our servers alive.
sauce
Have fun testing this
Greeb
-Agarlist

.sh So this method doesn't work with windows?

@NatsuTheGreat i think you could with batch files can't give you the code tho, dont even have windows 💯

adding bots can also make memory leak
just adding 70 bots leaving it for 2 hours will make the server laggy as hell

windows please

@GREEB Killing all active processes above 4GB RAM usage on a machine is a very dangerous practice.
There is a little gem called PM2, a widely known process manager for Node.js. We can use it's auto-restart functionality either programmatically or through a config file. I'll show how to do it using a config file, because it will be easier for people to follow.

Install PM2 globally by running npm install pm2 -global
Create a new file under the /src folder named process.json with the following content:

{
  "apps" : [{
    "name": "MultiOgar",
    "script": "index.js",
    "instances": 1,
    "exec_mode": "fork",
    "autorestart": true,
    "max_memory_restart": "200M",
    "env_production": {"NODE_ENV": "production"},
    "interpreter_args": "--nouse-idle-notification --max-old-space-size=4096"
  }]
}

Launch MultiOgar with pm2 start process.json --env production

That's all! PM2 will now auto-restart MultiOgar upon crashes and the same is true for when it reaches over 200MB RAM usage. By passing --env production we ensure that Node.js runs in production mode. The flag --nouse-idle-notification turns off idle garbage collection which makes the GC constantly run and --max-old-space-size increases the heap memory limit of the instance (remove flag if not enough memory).

@acydwarp Thanks for the post Acyd. Yeah dangerous practices is what i do 👎 .

@acydwarp It's giving me minimatch problems. It says I need to upgrade to 3.2.0 higher. Also, when I start the server, is just automatically kills it. I don't even get to read the problem.

@acydwarp only can run 1 process with pm2..

Increase instances

What else does PM2 do but auto restart?

I am using node v6.3.0 ;/

@acydwarp  I cant understand

@GREEB There is a serious problem with agarlist. You have to shoot mass to move, it happened randomly. I lost all all 200k mass because of this :/ (instant merge)

there is really some memory leak, after 4 days of running, my servers crashed with out of memory :)
Added some logging to look where memory leak exactly

Have you found out the memory leak source?

Underlying Node modules can also be a source of memory leaks. The ws module for example has a bunch of memory leaks as pointed out in this PR: https://github.com/websockets/ws/pull/734. I would also take the advice from this PR and cleanup MultiOgar's self = this by using arrow functions instead.

var self = this lags the server?

Arrow function seem cool. Just why does var self = this cause problems? Does it cause any issues with uws?

The PR points out that the use of self = this leads to memory leaks when used in closures. With arrow functions, this remains sticky to it's original context so it eliminates the need for self = this.
GC is a complex process and even valid code can cause memory leaks. One can pass the --expose-gc flag to Node and control garbage collection manually to avoid your app from cashing, but GC is generally not something you want to trigger as it blocks the app from executing while it's doing it's job. Best thing to do is to trace down the leaks.

@NatsuTheGreat: no

I see now. But does it cause issues with uws? That project is on ws.

I don't know. There is some a little memory leak with ws. I just get a dump from the test server with a lot of players (bubble-wars.tk:443).
Will investigate where memory leak happens exactly.

@Barbosik Alright. Good luck.
@acydwarp Do you know if var self = this; causes issues with uws? I am using uws. I do see quite an amount of var self = this; I used var self = this.gameServer; but that shouldn't do anything because I didn't have closures.

It's unrelated to uws, uws is c++ code with a Node.js interface which allows it to run more efficient than ws. I noticed a 5% drop in CPU consumption when I switched from ws to uws, using 200 bots to benchmark.
But, CPU consumption is actually fine with MultiOgar, it's the memory allocation that's mostly an issue and causes most of the lag.

yes, sometimes it has random peaks in update time, up to 80-100 ms. I have no idea what is the reason for such peaks, probably it hapens when somone multisplitting. But I spectated big players and it happens not always when someone make multisplit. I think it's protocol lag, need to catch it

at a glance memory leak occurs inside ws code. It allocates a most large part of memory.
I didn't found other memory leaks in MultiOgar dump.
Needs to wait some time to get snapshot when memory leak will be noticeable.

The random peak in update time is most likely caused by the garbage collector. The GC stores a list of objects and uses an algorithm to decide when to traverse the list and clean up the objects. You can avoid this from happening too often by passing the flag --nouse-idle-notification which disables V8's idle garbage collector. Also you can pass the flag --max-old-space to Node and increase it's value, but you'll need a bunch of available RAM for this (it's set at 1.4GB for 64-bit systems). It's best to avoid GC as much as possible because it hinders performance in real-time applications.

@acydwarp: The random peak in update time is most likely caused by the garbage collector.
no, it may works with very smooth for very long time even when there are a lot of players on the server and they using multisplit. But sometimes it works with peaks and they appears when someone make multisplit. But not always...

@acydwarp I have implemented arrow functions and got of self = this I'll be watching to see how memory goes.

"arrow function" is a lambda expression. It's the same as function. Just a new syntax.
I think there is no difference for compiller and both variants will be compiled to the same code.

Yes and that new syntax got rid of self = this; and closures.

@NatsuTheGreat: got rid of closures
actually "arrow function" is nothing else than closure :smile:
"=>" is standard syntax for closure definition, see C# for example.
It's just alternative syntax to define closure.

found the location of random lag peaks. It's client update, so it's still protocol bottle neck. It can take up to 60 ms, while updateMoveEngine takes max 9 ms

How do you find out how much MS a function takes up? I want to know because I refactored the collisions and I want to see if it got faster/slower.

with process.hrtime and some hand-made logic which allows me to catch the lag moment and measure execution time of code which is interested for me. Actually, collistions works pretty fast. It takes max 10-16 ms on server with 50+ players.
The problem with client update. Client update sometimes take up to 60-80 ms. It leads to lag peaks.

I think memory leak related with disconnected players. It happens when someone using DDoS attack with minions. Disconnection takes a lot of time, so it eats memory

@Barbosik I think that could be true. I have had a server with over 10,000 disconnects and quite a lot of players disconnected. It affected server performance a lot based on a server that just started with about the same amount of players.

https://github.com/F0RIS/Ogar-servers-manager


VMwareGuestInfoService is not implementing static network configuration

VMwareGuestInfoService is not implementing the static network configuration at the moment (I haven't tried this to verify it, but looking at the source, VMwareGuestInfoService is not implementing get_network_details nor get_network_details_v2)
Before I started working on a PR, I wanted to know if there was a specific reason this wasn't included? cc @rgl Thanks!

cc @akutz

At the time I didn't known about it. Funnily enough, today I was looking for it! :-)
@janitha, do you want to work on it? I can do it otherwise.

@rgl I haven't started on this yet and a bit tied up with other work, you can take this on, much appreciated!

@ader1990 Can this PR be merged please?

Feature will now be provided by #107


The link does not open through some email clients

👟 Reproduction steps
When I click on the link through the windows mail application, it sends me to an invalid link because the mail client considers the dot at the end of the link. However when I open the same email on the Gmail site directly it works.

Hey!
[USERNAME] shared some files with you. View or dowload the files with this link: https://exemple.com/share/UtNxQ6M.
Shared securely with Pingvin Share 🐧

I suggest just removing the dot at the end of the link, this will fix the issue on all affected email clients.
👍 Expected behavior
The link should work on all email clients
👎 Actual Behavior
This does not work on all clients because of the dot
🌐 Browser
Mail App Windows

Should be fixed 9b0c08d0cdeeeef217ccba57f593fea9d8858371 :)


Use ResourceGroup in kpt as inventory object

Replace ConfigMap with ResourceGroup CRD as inventory object in kpt
Update prune algorithm to use one inventory object instead of multiple objects
Refactor Inventory interface and functionality into common pkg/inventory package.

Updated prune algorithm to use one inventory object:
https://github.com/GoogleContainerTools/kpt/issues/910

Moved inventory object to its own package:
https://github.com/kubernetes-sigs/cli-utils/pull/189

Creates InventoryClient to encapsulate communication about Inventory object:
https://github.com/kubernetes-sigs/cli-utils/pull/190

Remaining work. Implement ResourceGroup wrapper that implements Inventory interface:
https://github.com/GoogleContainerTools/kpt/issues/912

Part of effort: https://github.com/kubernetes-sigs/cli-utils/pull/227

https://github.com/GoogleContainerTools/kpt/pull/1071

https://github.com/GoogleContainerTools/kpt/pull/1068

https://github.com/GoogleContainerTools/kpt/pull/1057

https://github.com/GoogleContainerTools/kpt/pull/970

https://github.com/GoogleContainerTools/kpt/pull/1068

https://github.com/GoogleContainerTools/kpt/pull/1138

https://github.com/GoogleContainerTools/kpt/pull/1132

https://github.com/GoogleContainerTools/kpt/pull/1128

https://github.com/GoogleContainerTools/kpt/pull/1121

https://github.com/GoogleContainerTools/kpt/pull/1071

https://github.com/kubernetes-sigs/cli-utils/pull/244

https://github.com/kubernetes-sigs/cli-utils/pull/243

https://github.com/kubernetes-sigs/cli-utils/pull/242

https://github.com/GoogleContainerTools/kpt/pull/1177

https://github.com/GoogleContainerTools/kpt/pull/1117

https://github.com/GoogleContainerTools/kpt/pull/1184

Released to alpha in v0.38.0--closing.


Port elixir proxyauth tests from js to elixir

Overview

This PR ports the proxyauth tests from js to elixir. This test was disabled in the js test suite.
The proxy_authentication_handler should be enabled at startup as the changes on chttpd.authentication_handlers config property is not hot-reloaded by CouchDB. An option has ben added to the dev/run script in order to specify a config file to be placed at etc/local.d folder.
Makefile for elixir target now includes a custom config file that enables all the auth handlers for testing.
Testing recommendations

 make elixir tests=test/elixir/test/proxyauth_test.exs

for checking new dev/run option:
 dev/run -n 1 --locald-config test/elixir/test/config/test-config.ini

should place the test-config.ini file at dev/lib/node1/etc/local.d folder
Related Issues or Pull Requests
This PR allows to #2648 to include integration tests for the new JWT auth handler

Checklist

[X] Code is written and works correctly
[ ] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

this should not have merged with so many lines of whitespace only change

and with a merge from master too 🤦♂


Revert "SWDEV-365820 - Refactor build path"

This reverts commit 3f2f7252aac5b9524162027b0237c6c6695f5347.
Reason: it breaks -save-temps in OpenCL
See bug report: https://github.com/ROCm/ROCm/issues/2940

Any reviewer's comments?

@saleelk hey guys are you serious? your change broke your project, the breakage affects your users, it's a bug, it's been months. Why aren't you reverting the breakage, and take the time to think about the right way to do it without keeping things broken. What are you waiting for? no review yet for the revert -- out of reviewers maybe?

Hi @preda , I didnt see this comments as my emails got filtered for "OpenCL" keyword. I have just done a fix in staging, shoudl get reflected here soon! Appreciate your patience

we drove this internally as the solution for this should have come from the compiler. but looks like that hasnt happened,

@saleek why wait for another project (LLVM?) to fix your breakage? Enough time has passed in the broken state, and that other project didn't fix it as you hoped. The simple solution is to revert your breakage, especially that the revert is just fine. Even the reason invoked for the initial change that broke it, i.e. compilation perf, is probably wrong -- please do time the compilation after the revert, and see whether there is any performance penalty (clearly, in my observation, there isn't). So there is absolutely no reason to keep this change, just revert it already!

@preda : Isn't it already reverted via 23c21d5181b70583066fedcc085eed103ae67715? Or are you still seeing the issue?

@mangupta @saleelk
The change https://github.com/ROCm/clr/commit/23c21d5181b70583066fedcc085eed103ae67715 looks good, thank you. I was not aware of it as the issue that's addressed https://github.com/ROCm/ROCm/issues/2940 was not linked in the commit message.

Closing this PR because fully addressed by https://github.com/ROCm/clr/commit/23c21d5181b70583066fedcc085eed103ae67715


Create CI builds

[x] Windows
[x] WASM
[ ] Linux
[ ] Android

Notes on Android SDK Location / scripting:

https://www.appveyor.com/docs/windows-images-software/
https://android.googlesource.com/platform/external/conscrypt/+/3f7489ffe806d7fb8653d94b7a702ffb0088cad2/appveyor.yml
https://docs.microsoft.com/en-us/xamarin/android/troubleshooting/questions/android-sdk-location?tabs=windows
https://stackoverflow.com/questions/25176594/android-sdk-location


regression: Xargo v0.3.9 can't compile compiler-builtins

STR
$ cargo new --lib foo && cd $_

$ echo '#![no_std]' > src/lib.rs

$ cat >Xargo.toml <<EOF
[dependencies.core]
stage = 0

[dependencies.compiler_builtins]
stage = 1
EOF

$ xargo build
   Compiling core v0.0.0 (file://$SYSROOT/lib/rustlib/src/rust/src/libcore)
    Finished release [optimized] target(s) in 29.77 secs
error: no matching version `= 0.0.0` found for package `compiler_builtins` (required by `sysroot`)
location searched: file://$SYSROOT/lib/rustlib/src/rust/src/libcompiler_builtins
versions found: 0.1.0
consider running `cargo update` to update a path dependency's locked version
error: `"cargo" "build" "--release" "--manifest-path" "/tmp/xargo.XRHsTJQdwa4b/Cargo.toml" "--target" "x86_64-unknown-linux-gnu" "-p" "compiler_builtins"` failed with exit code: Some(101)
note: run with `RUST_BACKTRACE=1` for a backtrace

Meta
$ rustc -V
rustc 1.22.0-nightly (368122087 2017-09-06)

$ xargo -V
xargo 0.3.9
cargo 0.23.0-nightly (34c0674a2 2017-09-01)

Diagnosis
As the error message says the libcompiler_builtins crate has version 0.1.0 in its Cargo.toml but this doesn't match the version in Cargo.lock: 0.0.0.
Why does this happen? When bootstrapping the compiler the Cargo.toml in src/libcompiler_builtins is not used; instead the Cargo.toml in src/rustc/compiler_builtins_shim is used. Because of this the version that appears in Cargo.lock is 0.0.0 -- this matches the version in src/rustc/compiler_builtins_shim/Cargo.toml. However, when Xargo compiles the Rust source it assumes that the source of dependencies for which no path or git information is specified in Xargo.toml will be located at $XARGO_RUST_SRC/lib$name/Cargo.toml where $name is the name of the dependency. Thus Xargo will use the version in src/libcompiler_builtins/Cargo.toml: 0.1.0. Hence the unmet version requirement.
What should we do?
Clearly the Xargo assumption of where the source for the compiler_builtins crate is located is not always right. We can ...
(a) continue using this assumption and patch rust-lang-nursery/compiler-builtins so that the version in its Cargo.toml matches the version in rust-lang/rust. Then we update the submodule in rust-lang/rust. Problem fixed (until someone changes either version ... as there's nothing forcing them to match)
(b) add special syntax to Xargo.toml to let the user specify the exact path of dependencies that reside in the rust-src component. Maybe: $XARGO_RUST_SRC/libcore, $XARGO_RUST_SRC/rustc/libcompiler_builtins, etc. To avoid breaking current users we use $XARGO_RUST_SRC/lib$name as the default if both path and git information is missing in Xargo.toml.
(c) do something else?
Both (a) and (b) seem rather prone to breakage. Maybe (b) more than (a) since the layout of the rust-src crates is by no means stable.
Thoughts?
cc @RalfJung

I think the version numbers should be the same really, so I feel like (a) should be done nevertheless.
But having (b) may be useful anyway? Not sure. The layout of rust-src not being stable is not exactly an argument for "it will always be $XARGO_RUST_SRC/lib$name/Cargo.toml"...

Hey, I am getting the same error. As a complete Rust newbie, I have no idea how to fix this manually. Any pointers?

rustc -V
rustc 1.22.0-nightly (0701b37d9 2017-09-18)
xargo -V
xargo 0.3.9 (7644989 2017-09-06)
cargo 0.23.0-nightly (8118b02ac 2017-09-14)

@pkronstrom downgrade your rust version:
$ rustup toolchain install nightly-2017-08-01
$ rustup default nightly-2017-08-01
$ rustup component add rust-src
$ xargo build

I first tried 2017-09-01 and it was still broken. 2017-08-01 seems to be working; likely there are more recent versions than that which will also work, if you need them.

downgrade your rust version:

Or install an older Xargo version. v0.3.8 works fine with a recent nightly. (cargo install xargo --vers 0.3.8 -f)

(a) continue using this assumption and patch rust-lang-nursery/compiler-builtins so that the version in its Cargo.toml matches the version in rust-lang/rust. Then we update the submodule in rust-lang/rust. Problem fixed (until someone changes either version ... as there's nothing forcing them to match)

I tried fixing the version number of the shim in https://github.com/rust-lang/rust/pull/44742. Unfortunately, this doesn't work; there's code in bootstrap assuming the version numbers of all sorts of things to be 0.0.0.

@RalfJung What about doing it the other way around? That is changing the version of the compiler-builtins crate in the rust-lang-nursery/compiler-builtins repo to 0.0.0. Do you think that will work?
People keep asking about this issue and we are going to have a big influx of cortex-m users tomorrow and I'd rather not have them run into this. So, I have decide to yank the v0.3.9 release until this issue is fixed in master.
Xargo v0.3.8 can build std with jemalloc enabled with a recent nightly, in any case, so the .lock is not currently needed (I'm not entirely sure why it was needed in the first place -- you don't usually ship a .lock file with libraries and the std facade is comprised of libraries only -- did some dependency break semver?)

I don't have a strong opinion.
The reason I implemented the lock file stuff is that semver doesn't help in the presence of deny(warnings), which libstd uses. So, there was a library update deprecating stuff, and then libstd triggered deprecation warnings with the new version, and things failed to build with xargo. That situations has been resolved (libstd no longer uses the deprecated symbols), but it could happen again any time.

Nowadays compile-builtins is pulled in via crates.io, so this should not be a problem any more.


Feature/56

Updated the PR to use the ocm-controller certificate creation. Also, removed the receiver and adding the project description to the repository which is not required.
I also fixed using the local Tiltfile that encodes applying the right generated certificate.
The test is now failing with some missing files which I assume you'll add @shivenduverma-sap.
=== Failed
=== FAIL: e2e TestMpasE2e (0.00s)
    main_test.go:41: running mpas happy path tests
    main_test.go:53: failed to read config file: %w open testdata/podinfo/backend/config.yaml: no such file or directory

After adding back the deleted files, the tests seem to work expect for a couple other small problems.
FAIL: TestMpasE2e/Create_Project/1.12_check_files_are_created_in_project_repo (0.05s)

This failed because some deploy.yaml file was also missing. Probably some deleted things that need to be updated.

I didnt ask for changes? I already made the changes and now I have to adapt again?

I only added the local Tiltfile changes to help you out in adopting the local ocm-controller certificate generation. :)

@shivenduverma-sap adding release for the e2e framework https://github.com/open-component-model/ocm-e2e-framework/pull/35

ocm-e2e-framework v0.6.0 successfully released.

Needs:
      - uses<EMAIL_ADDRESS>        with:
          kustomize: '5.0.1'
          tilt: '0.32.2'

https://github.com/open-component-model/MPAS/blob/feature/56/e2e/suite_test.go#L46
Somehow Tilt is running before installing flux

@Skarlso Any clue on how to fix this?

Hmm, interesting. I will look at it later on. I'm off for now.

I'll look into this. It's super weird and doesn't make a lot of sense. :D

Okay, so for a first, the project controller was installing flux: https://github.com/open-component-model/mpas-project-controller/pull/31

Handling connection for 5000
--- PASS: TestMpasE2e/Setup_OCM_component_for_testing (0.72s)
=== RUN   TestMpasE2e/Configure_Management_Repository
=== RUN   TestMpasE2e/Configure_Management_Repository/management_namespace_mpas-system_exists
    env.go:422: checking if namespace with name: mpas-system exists
--- PASS: TestMpasE2e/Configure_Management_Repository/management_namespace_mpas-system_exists (5.00s)
--- PASS: TestMpasE2e/Configure_Management_Repository (10.02s)
=== RUN   TestMpasE2e/Create_Project
Handling connection for 3000

Things are working locally with go work. Not sure what the problem is on CI 🤔

AARGH.    And since it's not using client-sdk it didn't care that client-sdk has breaking changes all over the place from 26-27.

We can choose to ignore it, or we adopt the new changes in all controllers.

I'm going to adopt it rather than choosing to lag behind. It's gonna take a little while to fix up all the controllers but they are small changes.

I have another idea... We are going to stop using go work in the end-to-end suit. Technically because of tilt it's no longer required. So it should be okay without it actually.

FINALLY!
Handling connection for 5000
--- PASS: TestMpasE2e/Setup_OCM_component_for_testing (0.45s)
=== RUN   TestMpasE2e/Configure_Management_Repository
=== RUN   TestMpasE2e/Configure_Management_Repository/management_namespace_mpas-system_exists
    env.go:422: checking if namespace with name: mpas-system exists
--- PASS: TestMpasE2e/Configure_Management_Repository/management_namespace_mpas-system_exists (5.00s)
--- PASS: TestMpasE2e/Configure_Management_Repository (10.01s)
=== RUN   TestMpasE2e/Create_Project

Looks like either the codeowners formatting is not working or gitea's file fetch strips formatting. 🤔 Going to investigate.

Yep, formatting is off. I'm gonna fix that in git-controller.

ah it was only for gitea 🤦  I left out the break point.

Fixed the formatting on Gitea https://github.com/open-component-model/git-controller/pull/55

https://github.com/open-component-model/MPAS/blob/feature/56/.github/workflows/e2e.yaml#L99
Needs to be changed to make e2e before merge

And the branch to main on the MPAS checkout. :)

You can remove the ref for MPAS, it uses the current branch by default.

Ahhhh f*ck. That's going to be a problem
*** Test killed with quit: ran too long (11m0s).

Question is what killed it. Github, or go test. :)

ah it was go test. nice. I can just increase that.


Set default airline theme

I have the colorscheme nightfly and it automatically sets the theme for airline. How do I disable themes and use the default?
I have tried:
let g:airline_theme= 'default'
let g:airline_theme= 'none'
let g:airline_theme= ''
let g:airline_theme= ' '
let g:airline_theme= 'false'
I only have airline base plugin (not colorschemes) and my nightfly colorscheme plugin.

https://github.com/vim-airline/vim-airline/blob/841a3551965e729e9f1ccd1aca758bdc84af2abf/doc/airline.txt#L159-L161


Annotations on pdfs

Hello!
We are seeing a weird issue with annotations on some pdfs. The appearance of some annotations are rotated at 90 degrees as shown in the screenshot and some are upside down. We observed that this doesn't happen to all the pdfs. Any ideas or explanations are appreciated!

Configuration:

Google Chrome and its version: 85.0.4183.121
windows 10
PDF.js version:v2.5.207

We observed that this doesn't happen to all the pdfs.

This might be a duplicate of #7631; however as-is this issue isn't actionable/valid since there's no test-case provided!
Generally speaking: When opening an issue, please provide all of the information requested in https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md


Problems with MediaElement

If hot reload is not on and Volume is set to a value other than 1 such as 100 it crashes.
Also I have a Samsung phone and after deploying to it the video plays with no volume.
[Enter feedback here]

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

ID: d60d04b7-bb5f-24c6-4281-923117d3eb4a
Version Independent ID: d60d04b7-bb5f-24c6-4281-923117d3eb4a
Content: MediaElement - .NET MAUI Community Toolkit - .NET Community Toolkit
Content Source: docs/maui/views/MediaElement.md
Product: dotnet-communitytoolkit
GitHub Login: @jfversluis
Microsoft Alias: joverslu

This is an issue with the code rather than the documentation as as mentioned in the discussion on the code repository an issue can be raised there


Document LoadBalancer Service safeguard

Public documentation part of https://github.com/elastisys/compliantkubernetes-apps/pull/1924
⚠ IMPORTANT ⚠: This is a public repository. Make sure to not disclose:

[x] personal data beyond what is necessary for interacting with this Pull Request;
[x] business confidential information, such as customer names.

Just merged https://github.com/elastisys/compliantkubernetes-apps/pull/1924 - should this be merged now or when that makes it into a release?