Stuck at "Waiting for the build to show up in the build list - this may take a few minutes"

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

fastlane has been stuck at "Waiting for the build to show up in the build list - this may take a few minutes (check your email for processing issues if this continues)" step. Tried 3 times, I'm on latest version, and I haven't received an email regarding processing issues.
Command executed

The lane that has the following code:
  log = changelog_from_git_commits(
    commits_count: 5,
    merge_commit_filtering: "exclude_merges"
  )
  log += "\n (last 5 commits)"
  upload_to_testflight(
    beta_app_review_info: {
      # redacted
    },
    team_id: "redacted,
    distribute_external: true,
    notify_external_testers: true,
    groups:["redacted"],
    beta_app_feedback_email: "redacted",
    changelog: log,
    beta_app_description: "redacted."
  )

Complete output when running fastlane, including the stack trace and command used
[...]
[12:04:13]: Successfully uploaded package to App Store Connect. It might take a few minutes until it's visible online.
[12:04:13]: --------------------------------------------------------------------------------------------------
[12:04:13]: Successfully uploaded the new binary to App Store Connect
[12:04:13]: If you want to skip waiting for the processing to be finished, use the `skip_waiting_for_build_processing` option
[12:04:13]: Note that if `skip_waiting_for_build_processing` is used but a `changelog` is supplied, this process will wait for the build to appear on AppStoreConnect, update the changelog and then skip the remaining of the processing steps.
[12:04:13]: Waiting for processing on... app_id: redacted, app_version: 1.0.3, build_version: 97, platform: IOS
[12:04:14]: Read more information on why this build isn't showing up yet - https://github.com/fastlane/fastlane/issues/14997
[12:04:14]: Waiting for the build to show up in the build list - this may take a few minutes (check your email for processing issues if this continues)

Last line repeats forever every 30 secs.
Environment

🚫 fastlane environment 🚫
Stack

Key
Value

OS
12.5

Ruby
3.1.2

Bundler?
false

Git
git version 2.32.1 (Apple Git-133)

Installation Source
/opt/homebrew/Cellar/fastlane/2.209.0/libexec/bin/fastlane

Host
macOS 12.5 (21G72)

Ruby Lib Dir
/opt/homebrew/Cellar/ruby/3.1.2/lib

OpenSSL Version
OpenSSL 1.1.1n  15 Mar 2022

Is contained
false

Is homebrew
true

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
13.4.1

Swift Version
5.6.1

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./fastlane/Fastfile`
(redacted)

fastlane gems

Gem
Version
Update-Status

fastlane
2.209.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

error_highlight
0.3.0

did_you_mean
1.6.1

atomos
0.1.3

rexml
3.2.5

CFPropertyList
3.0.5

claide
1.1.0

colored2
3.1.2

nanaimo
0.3.0

xcodeproj
1.22.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.8.0

terminal-table
1.8.0

plist
3.6.0

public_suffix
4.0.7

addressable
2.8.0

multipart-post
2.0.0

word_wrap
1.0.0

optparse
0.1.1

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

artifactory
3.0.15

babosa
1.0.4

colored
1.2

highline
2.0.3

commander
4.6.0

excon
0.92.4

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-httpclient
1.0.1

faraday-multipart
1.0.4

faraday-net_http
1.0.1

faraday-net_http_persistent
1.2.0

faraday-patron
1.0.0

faraday-rack
1.0.0

faraday-retry
1.0.3

ruby2_keywords
0.0.5

faraday
1.10.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.5

faraday-cookie_jar
0.0.7

faraday_middleware
1.2.0

fastimage
2.2.6

gh_inspector
1.1.3

json
2.6.1

mini_magick
4.11.0

naturally
2.2.1

rubyzip
2.3.2

security
0.1.3

xcpretty-travis-formatter
1.0.1

dotenv
2.8.1

bundler
2.3.11

simctl
1.6.8

jwt
2.4.1

uber
0.1.0

declarative
0.0.20

trailblazer-option
0.1.2

representable
3.2.0

retriable
3.1.2

mini_mime
1.1.2

memoist
0.16.2

multi_json
1.15.0

os
1.1.4

signet
0.17.0

googleauth
1.2.0

httpclient
2.8.3

webrick
1.7.0

google-apis-core
0.7.0

google-apis-playcustomapp_v1
0.10.0

google-apis-androidpublisher_v3
0.25.0

google-cloud-env
1.6.0

google-cloud-errors
1.2.0

google-cloud-core
1.6.0

google-apis-iamcredentials_v1
0.13.0

google-apis-storage_v1
0.17.0

rake
13.0.6

digest-crc
0.6.4

google-cloud-storage
1.38.0

emoji_regex
3.2.3

jmespath
1.6.1

aws-partitions
1.617.0

aws-eventstream
1.2.0

aws-sigv4
1.5.1

aws-sdk-core
3.132.0

aws-sdk-kms
1.58.0

aws-sdk-s3
1.114.0

set
1.0.2

forwardable
1.3.2

logger
1.5.0

pathname
0.2.0

shellwords
0.1.0

cgi
0.3.1

date
3.2.2

timeout
0.2.0

stringio
3.0.1

securerandom
0.1.1

uri
0.11.0

openssl
3.0.0

digest
3.1.0

io-nonblock
0.1.0

ipaddr
1.2.4

io-wait
0.2.1

zlib
2.1.1

resolv
0.2.1

time
0.2.0

open-uri
0.2.0

mutex_m
0.1.1

net-protocol
0.1.2

net-http
0.2.0

ostruct
0.5.2

english
0.7.1

erb
2.2.3

strscan
3.0.1

abbrev
0.1.0

io-console
0.5.11

tempfile
0.1.2

delegate
0.2.0

fileutils
1.6.0

tmpdir
0.1.2

base64
0.1.1

singleton
0.1.1

open3
0.1.1

nkf
0.1.1

prettyprint
0.1.1

pp
0.3.0

find
0.1.1

yaml
0.2.0

psych
4.0.3

generated on: 2022-08-13

Update: It's still the case but Xcode Archive + Upload via GUI works perfectly (build shows up).
Using fastlane, the waiting takes literally forever and no build ever shows up in Xcode connect too.

Update: It's still the case but Xcode Archive + Upload via GUI works perfectly (build shows up).
Using fastlane, the waiting takes literally forever and no build ever shows up in Xcode connect too.

We’re constantly working to improve our features based on feedback like this, so I’ll be sure to share your request to the API product team.

@canpoyrazoglu i'm running into the same issue after Xcode upgrade to 14.2 any solution for this?

Same issue . Xcode 14.2, fastlane version 2.213.0

yep, same for me.
looks like something with Apple's api.
I got email that build available, can install new build via TestFlight, but fastlane still prints "Waiting for the build to show up in the build list".
And when I run bundle exec fastlane pilot builds - new build is not listed here.

Thx roman-stetsenko-reed - passing the app_identifier in pilot solved it for me!

Hi, i am facing this issue, please how am i supposed to the app_identifier through fastlane?

You would add the following line to your Appfile:
app_identifier("com.your.bundleID")

Thank you


api_docs: Fix enum strings in parameters to have quotes.

Fixes the rendering of parameters that are enums to show enum strings with quotation marks, while integers will continue to be rendered without quotation marks.
This allows for an empty string to be passed as an enum value and be rendered as such in the documentation. Null will be rendered without quotation marks, like integer values. See example screenshot below.
This reuses some of the code added in #20409 to render the details of object parameters. At some point it would be good to look at refactoring for duplicated code. Perhaps as a step towards adding some of these details (enum, default, required/optional) to the return values and events.
HTML diff.
Screenshot of example string and integer enum, both with null as an option:

Merged, thanks @laurynmm!


packit: Use C locale for updating %changelog

Having the wrong locale running on your machine shouldn't
result in a changelog with foreign characters.
Example of a bad date here:
https://src.fedoraproject.org/rpms/oci-kvm-hook/pull-request/2#request_diff

@TomasTomecek How do I add a test for this?

how come we never run into this since Franta is using CZ locale :)
Stef, thank you so much for providing the fix. While browsing the code and looking for where this is actually happening (spoiler: it's inside rebase-helper), I'd suggest adding it to our specfile wrapper class where we have a method to update changelog:
https://github.com/packit/packit/blob/6cab7cc68c448ca1dbdbfe1fad880b33515102c3/packit/specfile.py#L62-L70
and I'd also test it that way: create a new test case here, in the file where specfile-related tests live:
https://github.com/packit/packit/blob/6cab7cc68c448ca1dbdbfe1fad880b33515102c3/tests/integration/test_spec.py
@csomh @lachmanfrantisek @jpopelka would you agree?

Just to clarify: this is not about bad characters, this is about producing a non-English date string.
If this is urgent and needs to be done in Packit, then I would recommend overriding get_new_log in packit.specfile.Specfile, to set and reset the locale and call get_new_log between them from rebase-helper. But ultimately, this should be fixed in rebase-helper.

@csomh It seems like Packit supports an old version of rebase-helper (without update_changelog) and a new version. To fix this bug for any user of Packit, and thus consider it actually fixed for good ... it seems that I would need to fix it in both versions of rebase-helper. So it seems like I have a choice to make:

Try to fix this in all versions of rebase-helper
Wait until Packit removes support for older versions of rebase-helper and fix it in rebase-helper
Fix it in Packit wrapper function (see above).

Am I understanding this right? If so, how should I proceed?

@stefwalter sorry, I missed that.
Looking at what versions are released for rebase-helper in Fedora, it seems  that the only place where supporting an older (pre 0.16.2) version of rebase-helper would matter is EPEL 7. But Packit is not built for EPEL 7, and we don't plan to do that, but @TomasTomecek might know something, I'm missing here.
So I say, fixing the locale issue in the latest version of rebase-helper is the way to go here. And we should drop this compatibility wrapper in Packit.

Thanks @csomh I'll follow the approach you described above during the day of learning.

how come we never run into this since Franta is using CZ locale :)

Not anymore..;)
a) I've switched from CZ locale.
b) I use packit from toolbox container.

Submitted pull request to rebase-helper: https://github.com/rebase-helper/rebase-helper/pull/851
However this pull request remains valid, and adds a test that the fix actually works. Packit is a project integrating many things, and it should be useful to have an integration test here that the changelog is valid.

rebase-helper PR has been merged. https://github.com/rebase-helper/rebase-helper/pull/851

it seems zuul doesn't have any additional locale available :/

@csomh What would I need to do to get this moving forward?

@stefwalter Besides rebasing, you could install few locales in the test environment so that this test actually does something. files/tasks/rpm-test-deps.yaml seems the right place for this; I would create a new task there, and install langpacks-{cs,de}.
Then, you could check the change locally by  first building a test image with make build-test-image and then running the test with make check-in-container TEST_TARGET=tests/integration/test_spec.py. Or just push it to this PR and wait for the tests to run in Zuul.


Improve UI of Screen sequence and transition diagram

The transition mapping's styling, color scheme can be improvised to make it more aesthetic.

@dikshika-s , could you please elaborate more details, NTT-tx think should start with simpler tasks, so it depends on what kind of complexity you will need to implement.

Respected Sir
Greetings of the day!
By improvising the model appearance I meant enhancing the box outline, colors, font size styles and other frontend related elements. To implement it one would require UI and frontend development knowledge.

@uehaj2022 CC: @dikshika-s @kushagra20251 @ShivomA @kiku-h @imshresth0810
Update video from @dikshika-s.
Flonnect_2022-11-16_b81cd514-4be0-4e73-8d16-3a1c83a34ac1-trim-0.zip


Update package.json

martinet fails to install on node v0.10.X due to msgpack failing to compile for older versions of node. Since it's unused, consider removing it.

good catch, thanks!


error mp4 file not valid

Describe the bug
i uploaded two .mp4 clips which generated my gopro. the software returns the error because it does not recognize the mp4 file as a file generated by the gopro
To Reproduce
Steps to reproduce the behavior:
select file and merge
Error: mp4 file not valid
Desktop (please complete the following information):

OS: WIndows 10

GoPro camera and files information
(If applicable)

GoPro Model: gopro10
GoPro Files info: 4k 4/3

.
27 nov 2021 12-00.log
[IMPORTANT] Logs (attach log file)

Hello, thank you for providing all the information. I've reviewed the log files that you've attached, I've seen that it doesn't detect part of the metadata, perhaps in the new video files for the GoPro 10 it's not included, I don't have this model, the tests I did to develop ReelSteady Joiner were with a GoPro Hero 6 Black and a GoPro Hero 9, I've compiled a trial version removing the GoPro file check, if with this version you can perform the process, please, let me know to update ReelSteady Joiner.
If there is any way to obtain test files made with GoPro Hero 10 I would greatly appreciate it to be able to carry out tests, you can contact me through my email<EMAIL_ADDRESS>Download ReelSteady Joiner v1.2.3-alpha: https://drive.google.com/file/d/1r1VjKvrMLtdH3BiuKdiNaQaPhRXuEtPW
Regards, and thank you very much.

Hi, I downloaded version v1.2.3 and tried to run the files. the process continues creating the merge folder, but inside it does not create the .mp4 merge file.
I'm sending you the log file
good day
29 nov 2021 08-57.log
y

Hi, I've checked the log file you attached, it's the same problem, it does not detect the part that contains the gyroscope data in the video file, I start to think that the problem may be related to your files and not ReelSteady Joiner. Is there an option to get a clip recorded with your camera? I do not need anything special, only that it is recorded with correct settings for later use in ReelSteady, with a few seconds of video it would be enough only to review the video file and the data.
I've gotten video files from a GoPro Hero 10 and these contained the necessary gyro data in the file, it's strange that your files don't have it.
Anyway, could you specify the following information ?:

How have you extracted the videos from the GoPro?
Could your antivirus be interfering with the process?
Have you previously edited those files with any other program?

Thanks.


Create AMPL 1.6.3 Snapshot

It looks like the docker tagging scheme has been modified, so I'll have to make some changes to the Makefile after this 1.6.3 release

While trying to create the snapshot, I get an error:

I'm not sure if this is a bug or a limitation in pFDA. The only thing I can think is that the snapshot size is too big and maybe there is a limit. I attempted this with the CPU image, as the GPU image is 6x larger
I will need to talk with someone from pFDA to verify the behavior

The snapshots have been uploaded

For some reason the size of the snapshots is larger than anticipated. I've been getting the following errors in pFDA. I emailed Sam & Yulia about how to proceed.

It might have to be that our snapshot just stores the state of the .env file with pre-filled values

I will revert to the original script where we are able to download the image and run it

This has been completed:

I've created this snapshot, which comes with a pre-configured .env file


Update KTikZ for KDE 5

KtikZ development continues upstream, but retaining the older KDE 4-based package obstructs removal of its dependencies.
Attn. @layus

I have a patch working for qtikz with qt5. We can live without KDE integration and temporarily drop ktikz.


Figures 4 & 5: Scalings of Lineage Length and Lineage Number

Merge when you're comfortable with it @namn12

Merge when you're comfortable with it @namn12

it looks blocked from one of the ealirer commits

Merge when you're comfortable with it @namn12

it looks blocked from one of the ealirer commits @shak360

our old friend back again?
Traceback (most recent call last):
File "./genFigures.py", line 14, in
ff = eval(nameOut + '.makeFigure()')
File "", line 1, in
File "/home/jenkins/jenkins/workspace/r-lab_lineage-growth_PR-109-head/lineage/figures/figure5.py", line 13, in makeFigure
numb_of_lineage_h1, accuracy_h1, bern_MAS_h1, bern_2_h1, MASlocBern, locBern2, MASbeta, beta2, betaExp_MAS_h1, betaExp_2_h1 = Lineages_per_Population_Figure()
File "/home/jenkins/jenkins/workspace/r-lab_lineage-growth_PR-109-head/lineage/figures/Fig_Gen.py", line 136, in Lineages_per_Population_Figure
_, _, all_states, tHMMobj, _, _ = Analyze(X1, numStates)
File "/home/jenkins/jenkins/workspace/r-lab_lineage-growth_PR-109-head/lineage/Analyze.py", line 22, in Analyze
fit(tHMMobj, max_iter=200, verbose=True)
File "/home/jenkins/jenkins/workspace/r-lab_lineage-growth_PR-109-head/lineage/BaumWelch.py", line 181, in fit
get_nonleaf_NF_and_betas(tHMMobj, NF, betas)
File "/home/jenkins/jenkins/workspace/r-lab_lineage-growth_PR-109-head/lineage/UpwardRecursion.py", line 177, in get_nonleaf_NF_and_betas
assert np.allclose(betas_row_sum, 1.)
AssertionError

so the test that was failing was test_bw_4, which is a test for a 1 state lineage. all 1 state lineage models were failing because i initilzied E_bern to be 1/numStates. so when numStates was 1 (in this case), E_bern would start at 1. And the pdf of a cell that dies (ie bern =0) is zero in a distribuiton with bern=1, which was causing Betas to be NaN. So now the bern parameter is initialized randomly between (0.01 , 0.99) and the test passed and the AIC for one state seems to be working on the other branch. let me know if this sounds right to you guys @aarmey @shak360 and then can we merge this?

Sounds reasonable to me.

@namn12 merge this and #110 . Good work and good catch on the bug.


Generate correct response header

The response header version is not the same as the API version (but it
can be derived from it). Since RequestHeader provides a convenient way
to build a response header, use this instead.
Signed-off-by: Adrian Preston<EMAIL_ADDRESS>

thanks @prestona


Added Scrollable Dialog Box

Description
Added Dialog box option in main home page, then added the dialog screen in which other dialog boxes can be added.
And in dialog screen added the scrollable dialog box.
Fixes #19
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

[x] Manual testing
[x] Widget testing
[x] Unit testing

Checklist

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

After Change

Hey @sagardev2301 there is some merge conflict, please resolve it first


Add Interface Boundary to Graph

the graph needs to take in a list of query results and a view option object, and that's it. everything else should be a functional transform of that state to a pretty graph


Using tsrc with GitLab CI

Today you have two options.

Deploy a ssh key on all the runners
Patch the manifest to replace<EMAIL_ADDRESS>URLs with<EMAIL_ADDRESS>and use  --no-update-manifest  option when running sync - See #275

Both are not ideal. Let's see if we can find better solutions!

We could also update the documentation. See also #238

We could also update the documentation. See also #238

Well, looks like we have a workardound now: #338. Thanks @Lecrapouille :)


PGN Editor: ply cut leaves the game result intact

This is confusing, as it feels as if it's winning at move 20

fixed in v2.3.121, where if you cut plys from the mainline you also remove the Result tag.


Securimage integrate OWA Exchange

How intagrate Securimage in OWA Exchange 2013?

I personally have no idea if it's possible.  I don't think so though.  You'd need to install PHP within IIS which is fine, and while you could edit the OWA login templates to display the captcha, I seriously doubt you have the ability to edit the backend code that handles the login (though I could be wrong).  My guess is that the stuff that handles that is proprietary code compiled into DLLs and other things you don't have the ability to change.
I'd suggest posting on a Microsoft forum asking if it would be possible to hook into OWA's login code to run your own code.  If it were possible, you'd then likely be better off using an ASP.NET / C# captcha that would integrate a little easier than PHP.
Feel free to post back if you find anything out.


[Question] Is it necessary to incorporate a local caching mechanism into the instance creation process?

Question
Recently, I fixed an issue related to instance reuse; for more details, please refer to https://github.com/apache/hertzbeat/issues/2886. To quickly address the online bug, my solution was somewhat crude. Below is the modified code, which prevents instance reuse problems caused by IP switching.

However, if a user or organization needs to monitor multiple Kafka instances, the current logic becomes ineffective, as it continuously creates new Kafka instances, potentially leading to performance degradation. Similarly, the existing code for RocketMQ repeatedly creates instances without enabling reuse. If an enterprise monitors multiple instances and requires high real-time performance with short monitoring refresh intervals, the repeated creation of instances becomes a significant concern.

Therefore, I am considering whether it is necessary to introduce a local caching mechanism (such as using Caffeine). The key could be a combination of app+ip+host, and the value would be the corresponding instance. Additionally, setting a relatively small cache size to prevent memory overflow and implementing an LRU eviction policy—which aligns well with our monitoring scenarios—would effectively allow the reuse of created instance objects, and we can start by implementing this for the Kafka client first and subsequently enable a gradual rollout.
If the community agrees that this solution is viable, please assign it to me, and I am willing to implement it.

+1 connection local caching mechanism is already use in ssh db and soon. suggest we use ConnectionCommonCache in kafka client like others.
@zhangshenghang how about this?

+1 connection local caching mechanism is already use in ssh db 和 soon. suggest we use ConnectionCommonCache in kafka client like others. @zhangshenghang how about this?

This is a very good idea，@doveLin0818 Can you complete it? If you can, I will assign it to you.
@doveLin0818  You can refer to
https://github.com/apache/hertzbeat/blob/4236c438be50fd941f039be958d22fb6a14415aa/hertzbeat-collector/hertzbeat-collector-basic/src/main/java/org/apache/hertzbeat/collector/collect/ssh/SshCollectImpl.java#L74

+1 connection local caching mechanism is already use in ssh db 和 soon. suggest we use ConnectionCommonCache in kafka client like others. @zhangshenghang how about this?

This is a very good idea，@doveLin0818 Can you complete it? If you can, I will assign it to you.
@doveLin0818 You can refer to
https://github.com/apache/hertzbeat/blob/4236c438be50fd941f039be958d22fb6a14415aa/hertzbeat-collector/hertzbeat-collector-basic/src/main/java/org/apache/hertzbeat/collector/collect/ssh/SshCollectImpl.java#L74

@zhangshenghang hi,please assign to me

BTW, another point. We can find that a new ConnectionCommonCache object is created in each collection class. Is this the best practice? Because each ConnectionCommonCache has a corresponding observation thread, etc., which takes up more resources.
How about use only one ConnectionCommonCache for these connection cache function.

BTW, another point. We can find that a new ConnectionCommonCache object is created in each collection class. Is this the best practice? Because each ConnectionCommonCache has a corresponding observation thread, etc., which takes up more resources. How about use only one ConnectionCommonCache for these connection cache function.

Hi tom , I will try to optimize this.


Add govulncheck.sh script

Codecov Report

Merging #87 (360ba0c) into main (cedc066) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##              main       #87   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           60        60
  Lines         1693      1693
=========================================
  Hits          1693      1693

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Ability to select/copy all the text in the session

Summary of the new feature/enhancement
Sometimes there are situations when you need to select/copy all the text in the session.

ConEmu has such functionality (Menu → Edit → Copy All).

Proposed technical implementation details (optional)

Keybinding for action Select All Text.

May intersect with #715.

I should add this to the #4993 scenario.

@carlos-zamora what is the plan for this feature? Looks like an important feature.
Select by mouse/trackpad works. Adding by keyboard short cut would be easy to use.

https://github.com/microsoft/terminal/issues/5886

For what its worth and to whomever finds this useful, if you zoom out really far with ctrl + scroll wheel highlighting the complete buffer is much less painful even for relatively long histories. Not ideal of course, workable.

Make sure one can choose between plain text and formatted text e.g. with colors. MobaXterm supports saving as RTF, preserving formatting.

We already support RTF and HTML copy, @sba923 :)

Typing my comment, on my phone, I kinda remembered reading about this somewhere... 😝 I promise I will check out the docs first thing in the morning 😜

We already support RTF and HTML copy, @sba923 :)

I know, but you can't choose dynamically.
You have to define this in the settings. So it's either "always formatted" or "always plain text."
Being able to choose between formatted copy or unformatted copy e.g. by using two different key bindings would be nice.
Or maybe the choice should be at paste time, not at copy time. (I know, one day Win+V will allow pasting the plain-text version of whatever's on the clipboard. But this is not yet for corporate systems running non-Insider Windows 10 builds 😉).

Or maybe the choice should be at paste time, not at copy time.

That's completely up to the app reading the clipboard. Can't do anything about that.

Being able to choose between formatted copy or unformatted copy e.g. by using two different key bindings would be nice.

You mean like...
{ "command": { "action": "copy", "copyFormatting": "rtf" }, "keys": "ctrl+shift+r" },
{ "command": { "action": "copy", "copyFormatting": "html" }, "keys": "ctrl+shift+h" },
{ "command": { "action": "copy", "copyFormatting": ["html", "rtf"] }, "keys": "ctrl+win+alt+shift+h" },

? From the docs for the copy action.

Or maybe the choice should be at paste time, not at copy time.

That's completely up to the app reading the clipboard. Can't do anything about that.

I know. It's too bad too many apps don't provide a way to choose between plain-text paste and formatted paste.
I paste tons of stuff into Teams, from various sources like VScode and Windows Terminal, and I often don't get what I'd expect.
My current workaround is to first rewrite the clipboard as plain-text-only with some one-liner, such as this running in a WSL2 tab in Windows Terminal 😉
pwsh.exe -noprofile -c Get-ClipboardText | fold -w80 | clip.exe

(this also tackles a Teams issue with the lack of line-wrapping but that's another issue altogether)

Being able to choose between formatted copy or unformatted copy e.g. by using two different key bindings would be nice.

You mean like...
{ "command": { "action": "copy", "copyFormatting": "rtf" }, "keys": "ctrl+shift+r" },
{ "command": { "action": "copy", "copyFormatting": "html" }, "keys": "ctrl+shift+h" },
{ "command": { "action": "copy", "copyFormatting": ["html", "rtf"] }, "keys": "ctrl+win+alt+shift+h" },

? From the docs for the copy action.

Exactly. [blushing mode on] My bad for not RTFM. [blushing mode off]. I'll try defining a "copy as plain text" keybinding and see whether my muscle memory can be trained to using that.

Leaving the text format aside, what is the status of being able to copy all text to the clipboard? Since we just got the ability to change the language to English, I can now actually read the menu options and in the "Command Palette" window there is nothing that is related to selecting text. Ctrl+A just inserts ^A instead of selecting the text.
This is a major blocker because I might have a program that spits out dozens of screens worth of output and I want to search for the first error in the output (since usually a single error triggers a cascade of error messages, checking the last one is useless).

@gyorokpeter

I might have a program that spits out dozens of screens worth of output and I want to search for the first error in the output

https://docs.microsoft.com/en-us/windows/terminal/search

@gyorokpeter

I might have a program that spits out dozens of screens worth of output and I want to search for the first error in the output

https://docs.microsoft.com/en-us/windows/terminal/search

Unless you want wildcard search, regular expression search, find all, count, match whole word, find and replace, mark line and a million other things you'll never have inside a terminal...
One common scenario is using a command that dumps a huge JSON on the screen and you want to copy it over for further processing and formatting (yes, you can use stdout redirect but sometimes you forget, maybe you want to see the output first before saving it, etc).

I have found a workaround to select all text:

Scroll to the beginning of the console window Ctrl+Shift+Home
Start selecting text by holding left mouse button
While holding left mouse button press Ctrl+Shift+End, it will move the cursor to the bottom of the window and all console's text will be selected
Release left mouse button
Press Ctrl+C to copy the text

FYI ctrl+a works if the tab is powershell but not if its cmd

I probably have a silly workflow, but I just wanted to echo that for me "Select All" (Ctrl+A) and "Copy" (Ctrl+C) console output would be a pretty sweet feature to have.

Dear devs, could you please implement this? As a blind developer I'm not able to use the mouse at all and, as it seems, there is no way of selecting text in Windows Terminal using the keyboard only. Otherwise, a great app, thank you.

Sorry but this is super unhelpful, it selects all the text in the prompt including the beginning
Microsoft Windows [Version 10.0.19044.1706]
(c) Microsoft Corporation. All rights reserved.

How about the ability to just select the current writable buffer??


refactor(errors): Make Error Consistent

Please Review:
@mannyrivera2010 @mleeBoeing @rkenyon1969
Purpose:
Make errors consistent

Need to update the test case:
tests/ozpcenter_api/test_api_listing.py:249


Added new pf classes to app template.

Insights page layout will be changed a bit in a future. This PR prepares PF classes to transition once the changes in Chrome are ready. Has no effect right now.

:shipit:


[replaces #126] Source prepare-env.sh in the start.sh script to override possible leftovers from build

Replaces #126 (I couldn't rebase it...)
Fixes https://github.com/CoffeaTeam/coffea-casa/issues/125

@mapsacosta could you please check if I rebased properly?

I think everything is fine, so I will try to merge it and test on coffea-casa. Thanks a lot @mapsacosta!

Awesome, thanks @oshadura let me know and I'll test on the EAF too :)


$cordovaFile.listDir disappers

Upgrading ngCordova from 0.1.12-alpha to 0.1.14-alpha, listDir function disappears!
In source code this function is commented.
Why? Can you reintroduce it?
Thank you.

up!

+1

+1

+1

=3

+1 = 4
What happened with that?

+1  ... I had to write my own, i didn't know it was commented. =(

+1

also mentioned in #833

+1

+1

+1

+1

+1, why is this gone ? Is there an alternative ?

You can use this as alternative for listDir method:
function listDir(path){
  window.resolveLocalFileSystemURL(path,
    function (fileSystem) {
      var reader = fileSystem.createReader();
      reader.readEntries(
        function (entries) {
          console.log("entries");
          console.log(entries);
        },
        function (err) {
          console.log(err);
        }
      );
    }, function (err) {
      console.log(err);
    }
  );
}
//example: list of www folder in cordova/ionic app.
listDir(cordova.file.applicationDirectory + "www/");

Thanks Vicent!

Wow, thanks Vincent!

Thanks Vincent !

I don't see how you can manage your cacheDir without a listDir() function ... makes no sense.

LOL. This ticket was opened nearly two years ago. Open source FTW.

WTF


Fix spelling inconsistency and typos

Fix spelling inconsistency :
sqlorArgs => sqlOrArgs
Fix typo :
sqlOrAgrs => sqlOrArgs

Codecov Report

Merging #1317 into master will not change coverage.
The diff coverage is 92.59%.

@@           Coverage Diff           @@
##           master    #1317   +/-   ##
=======================================
  Coverage   57.35%   57.35%
=======================================
  Files          44       44
  Lines        7818     7818
=======================================
  Hits         4484     4484
  Misses       2777     2777
  Partials      557      557

Impacted Files
Coverage Δ

session_query.go
61.7% <100%> (ø)
:arrow_up:

engine.go
61.53% <100%> (ø)
:arrow_up:

session_raw.go
57.57% <75%> (ø)
:arrow_up:

xorm.go
68.18% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks


Get-DbaService - Rewritten command to have better error handling and messages

Type of Change

[x] Bug fix (non-breaking change, fixes #7470 )

See issue for details...
As with most big rewrites, diff is not very helpfull - better have a look at the target source code.

@nvarscar can you please take a look and approve? 🙇🏼  I'm really excited about this fix, as I encountered it as well.

@wsmelton too if you're around

One review plus I just tested it and it works! Merging 🥳


For JSX Attributes, map over unions of props for contextual types

This way an individual attribute can fulfill any of them, rather than failing to get a contextual type unless it fulfills all of them.
Fixes #17427

Do you know why getPropertyOfType wasn't working for that? It appears to already have a special case for union types.

@andy-ms Since it's a union and the property only exists on one option of the union, the property gets marked as CheckFlags.Partial, then omitted from the getPropertyOfType result. Acually, looking into it, I can just use getTypeOfSymbol on getUnionOrIntersectionProperty rather than what I've got here, which should also enable some cacheing of cacheable results.

Is there any reason why JSX should differ from object literals here?

@andy-ms Thanks for the prompt - I hadn't realized we were doing something slightly differently for non-jsx-related contextual types. Elsewhere we were using getTypeOfPropertyOfContextualType instead of getTypeOfPropertyOfType, which fully explains the difference in behavior (getTypeOfPropertyOfContextualType does the mapping I had inlined here). Now for all JSX contextual typing, we use getTypeOfPropertyOfContextualType, as we should.


[CI] Jenkins rebuilds docker image for every branch

Problem
The current logic to rebuild the docker image is when a new branch is pushed. This has the following two drawbacks:

If one of the dependencies in requirements.txt is changed during the development of the new branch, the docker image is not rebuilt.
If a new branch does not change anything in the dependencies,  the docker image is rebuilt twice (one for /usr/local and another one for /opt/rocm), which is a wasted of resources (about 40 mins and some disk space).

Proposal
The docker image should be rebuild if and only if some dependencies or the Dockerfile itself are changed. Here is the general idea of the proposal:

Include not only the branch name, but also the commit id (git rev-parse --short HEAD) in the tag of the image. Therefore, the name of each image should has the form: ${MIOPEN_IMAGE_URL}:miopen_ci_${branchname}_${commitID}_[usr|opt]
Every time Jenkins is notified of a change to a branch (either new commits are added or new branch pushed), it tries to find the latest image built for the branch.

If there is one, obtain the commit id from the image name and compare with HEAD to see if there is a change in the dependencies or Dockerfile.

If so, build a new image with HEAD as the commit id in the tag.
If not, use the last built image. Note that the develop branch can only fall into this case since Jenkins should always have images built for the develop branch.

If an image is not found, it means this is a new branch or this branch (cannot be the develop branch) has not changed any dependencies or the Dockerfile before. Then Jenkins compares HEAD with the fork point between this branch and the develop branch to see if there is a change in the dependencies of the Dockerfile.

If so, build a new image with HEAD as the commit id in the tag.
If not changed, use the latest built image for the develop branch. Note that the commit id of this image is the same or later than the fork point. In this way, the new branch will always be tested with the latest dependencies installed in the image.

Benefits
The effect of the proposed mechanism is a great drop of the number of rebuilds. The docker image is rebuilt if and only if some dependency requirements are changes with one exception mentioned in Case 5.1.
Demos
Here are some scenarios to illustrate the proposed image building mechanism.
                      F is the fork point between the two branches.
Assume I first pushed mybranch when P is the HEAD. Then I added two commits and pushed again when Q is the HEAD.
Case 1: no dependency change in mybranch
Since I am not changing anything about the dependencies, there is no need to build a new image for mybranch. And since no images are built for mybranch, both pushes fall into the scenario of 2.ii.b, i.e. neither git diff P..F nor git diff Q..F contains dependency changes. And in this case, the image used for mybranch is the latest built image for the develop branch. Then we have two sub-cases:

the latest image is ${MIOPEN_IMAGE_URL}:miopen_ci_develop_[A|F]_[usr|opt]
In this case, the dependency requirements have not changed since A or F. Therefore, it makes sense for P and Q to used to same image as A or F.
the latest image is ${MIOPEN_IMAGE_URL}:miopen_ci_develop_B_[usr|opt]
In this case, some dependency requirements have changed by another PR, which have been merged into the develop branch before my push of mybranch. It makes sense for mybranch to be tested with the latest dependency requirements to catch bugs as early as possible (at least before mybranch is merged).

Case 2: dependency changes in my first push but no more changes in my second push
When I first push mybranch with head P, Jenkins finds that the difference between F and P contains dependency changes. Therefore, it rebuilds the image with tag miopen_ci_mybranch_P_[usr|opt] as in 2.ii.a.
At this moment, the image tag for the develop branch can point to A, F, or B. In the case of A or F, mybranch has a newer version of the dependency requirements. However, in the case of B, the newer version of the dependency requirements could be on the develop branch. In this case, mybranch is tested with its local change to the dependency requirements. The local change could be tested again when mybranch is merged into the develop branch.
The second push of mybranch with head Q does not have more changes to the dependency requirements compared to P. Therefore, Jenkins will use the previously built image with tag miopen_ci_mybranch_P_[usr|opt] as indicated in 2.i.b.
Case 3: no dependency change in my first push but some are in my second push
The first push (with head P) falls into 2.ii.b, in which case the latest built image for the develop branch is used.
For the second push with head Q, Jenkins still cannot find an image with a tag that include mybranch. However, since the second push includes changes to the dependency requirements, the image is rebuilt with a new tag miopen_ci_mybranch_Q_[usr|opt] as indicated in 2.ii.a. Note that Jenkins compares Q with F to make the decision since P and F are equivalent in terms of dependency requirements. And since mybranch has local changes to the dependency requirements, those on the develop branch are ignored when testing mybranch.
Case 4: dependency changes in my first push and more dependency changes in my second push
The first push falls into 2.ii.a since this is the first time mybranch is pushed and the image is rebuilt with tag miopen_ci_mybranch_P_[usr|opt].
The second push falls into 2.i.a, in which case Jenkins rebuild the image again with a new tag miopen_ci_mybranch_Q_[usr|opt].
Case 5: What happens when mybranch is merged
                     A           F           B               M
branch develop O --- O --- O --- O --- O --- O ------------- O
                                  \                         /
mybranch                           X --- X --- X --- X --- X
                                               P           Q

In the above diagram, M is the merge point. The image is rebuilt for the develop branch in the following cases:

mybranch has changes but no changes after F on the develop branch
This case implies that the last built image for the develop branch has a tag pointing to A or F, but cannot to B. For the develop branch, M contains the dependency requirements changes that is not found in F or A. Therefore, the image is rebuilt with tag miopen_ci_develop_M_[usr|opt] as indicated in 2.i.a.
Note that this image is duplicate to the latest image built for mybranch. The proposed mechanism cannot eliminate such duplicates.
mybranch has changes and some commits (say B) after F on the develop branch have changes
In this case, the merge must have resolved any conflicts of dependency requirements. And Jenkins will trigger a rebuild if and only if mybranch has changes that are not covered in B, i.e. M is different from B in terms of dependency requirements. And the image is rebuilt according to 2.i.a.

When mybranch does not have dependency requirements changes, no rebuild will happen when mybranch is merged into the develop branch since M must be the same as B or F or A in terms of dependency requirements. And this falls into 2.i.b.

My solution to this problem was to provide an override parameter to the pipeline which can be manually set forcing the generation of a new Docker image ? See here for code

@JehandadKhan Thank you for the code. That is indeed a simple workaround to build docker image during the branch development. I'll borrow it to get my PR #1673 done first.

@junliume Please add @pfultz2 to this discussion. Also I recommend labeling with request_for_comments.

@zhanglx13 I highly recommend removing all the mentions of develop from the description. Technically, this branch is equal to all other branches except some details unrelated to this topic (e.g. it's special role in the promotion process etc.)

The idea of reusing docker images from the "parent" branches is good but it's generic implementation seems too complex.
Why docker build caching is not effective anymore? I remember it worked well some time ago.

[Informative] @zhanglx13 Thank you very much for the detailed analysis 👍 !

@atamazov It seems to me that other branches are merging into the develop branch, which is considered to be the only relevant difference. This is only for demonstration purpose and the complex proposal handles all branches the same way.
I am new to docker and I'll take a look at docker build caching. Or if you know someone who is more familiar with the topic, could you ask him/her to take a look?


TakeOff via MAVLink command: Altitude command param vs. vehicle parameter

Sending the MAVLink command MAV_CMD_NAV_TAKEOFF with param7 (altitude set point), it seems not to be clear which altitude should be the target. Although the SP is set to value provided with the command, takeoff takes place considering the global vehicle parameter MIS_TAKEOFF_ALT in the first phase. Then, after some time, the SP provided is considered and the vehicle adjusts its altitude accordingly. This could result in a massive overshoot. Setting MIS_TAKEOFF_ALT to 0.0 solves that issue and the param7 of the command is considered as TakeOff-Altitude.
SITL Example: MIS_TAKEOFF_ALT is set to 10m; Param7 is set to ground altitude + 1:

@jgoppert This seems to be LPE related. EKF2 SITL does not show this effect at all. Any idea?

I thought this is expected behavior.  Mis takeoff alt is the min takeoff alr, so it gets to that within nav accept radius of 1m, then follows mission alt, not sure how the estimators could be different here.

@jgoppert Thanks for clarification. I'll re-check my SITL settings with EKF2.

Ok. That's explaining it, MIS_TAKEOFF_ALT was set to 2.5m compared to 10.0 for LPE SITL. That means, after reaching 1.5m mission altitude was taken into account. Thanks again for the clarification. I'll close this issue.

We should probably change the default for lpe sitl so others are not confused as well.


SpanContext struct initialization fails after samplingState introduced

Requirement - what kind of business use case are you trying to solve?
We're trying to use struct initialization (mostly in tests) to instantiate a SpanContext.
Problem - what in Jaeger blocks you from solving the requirement?
When updating from jaeger-client-go 2.17.0 to 2.21.1, we're encountering nil deferences when calling SpanContext.IsSampled() and other methods which now introspect the samplingState field. When creating a SpanContext via struct initialization, this doesn't have a default value so there are several methods which do not work.
The following code fails with the new version:
jaeger.SpanContext{}.IsSampled()

Stack trace:
panic(0x1366d00, 0x16998f0)
	/usr/local/Cellar/go/1.13.6/libexec/src/runtime/panic.go:679 +0x1b2
go.uber.org/atomic.(*Int32).Load(...)
	/Users/build/go/pkg/mod/go.uber.org/atomic@v1.5.0/atomic.go:41
github.com/uber/jaeger-client-go.(*samplingState).isSampled(...)
	/Users/build/go/pkg/mod/github.com/uber/jaeger-client-go@v2.21.1+incompatible/span_context.go:144
github.com/uber/jaeger-client-go.SpanContext.IsSampled(...)
	/Users/build/go/pkg/mod/github.com/uber/jaeger-client-go@v2.21.1+incompatible/span_context.go:180

Proposal - what do you suggest to solve the problem or improve the existing situation?
Making the samplingState a struct instead of a pointer to a struct so the default value works again, or guarding against any accesses which can dereference a nil struct by returning a useful default value.

SpaContext is not intended to be initialized as empty struct. There is a constructor function that you can use, which performs proper initialization.

Ok I can update our usages to call NewSpanContext instead.
FYI there are a few locations in the jaeger-client-go code that still use struct initializers without instantiating samplingState:

https://github.com/jaegertracing/jaeger-client-go/blob/7cedac84688a3141758443da36a0fecff9b73d45/jaeger_thrift_span_test.go#L296-L297
https://github.com/jaegertracing/jaeger-client-go/blob/7cedac84688a3141758443da36a0fecff9b73d45/span_context.go#L37
https://github.com/jaegertracing/jaeger-client-go/blob/7cedac84688a3141758443da36a0fecff9b73d45/tracer_test.go#L140
https://github.com/jaegertracing/jaeger-client-go/blob/7cedac84688a3141758443da36a0fecff9b73d45/config/options_test.go#L111
https://github.com/jaegertracing/jaeger-client-go/blob/7cedac84688a3141758443da36a0fecff9b73d45/zipkin/propagation.go#L83
https://github.com/jaegertracing/jaeger-client-go/blob/7cedac84688a3141758443da36a0fecff9b73d45/zipkin/propagation.go#L111
https://github.com/jaegertracing/jaeger-client-go/blob/7cedac84688a3141758443da36a0fecff9b73d45/zipkin/propagation.go#L114


.env variable examples in README

Everything else in the README is clear, and I can get the app running on my on computer just fine, but I can't seem to get the login working. I can't figure out what I'm supposed to set these four variables in the .env file to. Nothing happens after I press "Allow" on in the Youtube/Spotify login window.
SPOTIFY_CALLBACK=XXXXX
SPOTIFY_LINK_CALLBACK=XXXXX
YOUTUBE_CALLBACK=XXXXX
YOUTUBE_LINK_CALLBACK=XXXXX

Could the README be updated to provide example values for the environmental variables (for a dev environment)?

@Otju
Will update as soon as I can. Let me know if anything else is unclear and I can answer them and add them to the README :)

@Otju Updated!
Let me know if anything else is unclear and I'd be happy to answer them and update the README :)

Thanks @bundit!
Seems I did have the right values for those callback-urls, and the problem was actually with the database. Had to mess with my node version and update the pg-package to even get the error message to show up. I got the database connection working, but the login doesn't work cause obviously the database doesn't have the correct tables (it's just an empty database).
Could you provide the database schema (tables and their row names/types) or the SQL-commands used in creating them? The schema could be in the README, or as a separate file. The db could also be initialized in the code when in a dev environment (with CREATE TABLE IF NOT EXISTS or something else).

@Otju ah yeah that makes sense too why it wouldn't be working. I'll leave the db schema here and note to myself to add documentation for this in the future.
CREATE TABLE users (
	id UUID PRIMARY KEY,
	username VARCHAR(255),
	email VARCHAR(255) UNIQUE NOT NULL,
	created_at TIMESTAMP
);

CREATE TABLE user_profiles (
	user_id UUID REFERENCES users(id),
	oauth_provider VARCHAR(255) NOT NULL,
	provider_id VARCHAR(255) NOT NULL,
	refresh_token VARCHAR(255),
	images TEXT,
	username VARCHAR(255),
	"profileUrl" TEXT,
	PRIMARY KEY(user_id, oauth_provider)
);

CREATE TABLE user_playlists (
	user_id UUID REFERENCES users(id),
	source VARCHAR(255) NOT NULL,
	external_id VARCHAR(255) NOT NULL,
	title VARCHAR(255),
	"isConnected" BOOLEAN NOT NULL,
	index SMALLINT NOT NULL,
	img TEXT,
	total INT NOT NULL,
	PRIMARY KEY(user_id, source, external_id)
);

CREATE TABLE keys (
	source VARCHAR(255) NOT NULL,
	type VARCHAR(255) NOT NULL,
	key VARCHAR(255) NOT NULL,
	last_updated TIMESTAMP,
	PRIMARY KEY(source, type)
);

In addition, I think you will need to import the uuid module for creating users.
CREATE EXTENSION IF NOT EXISTS 'uuid-ossp';
Let me know if you have any other issues getting it up an running


Selection of 'PM' Time Missing from Datetime Control

This is a bit odd but the date time control is missing 'PM' selection for time (see attached). I simply call it with this code:
<input id="add-schedule-datetime" type="datetime-local" />
Any ideas as to why it's missing? The time isn't 24-Hour so you can't select a 'PM' time no matter what you try.

This is <input type="date"/>?
And the exact same code without xwalk shows both AM and PM?

If I use your code it only shows the date and I need date and time. I tested this on Nexus 7 2014, Nexus 5, and Samsung Galaxy 5 with all the same results missing the 'PM'.

Let me ask it differently :)

How do you show the date/time widget?
If you remove crosswalk and keep everything else the same, do you get the same result?

If I remove crosswalk and use the Default WebView in Telerik it displays as expected, with both 'AM' and 'PM' selections. It's only with the crosswalk plugin that it behaves like this.

Thanks, can you please also enlighten me how the date widget is shown exactly?

Let me know if this is what you want. You basically click the input and the date time widget displays in the app. This is a hybrid app that uses jQuery Mobile on Cordova 5.0.0.

I was wondering if that picker was a plugin or an html input type, so it's the latter, right?
Did you search the upstream repo's issues / google / StackOverflow for something similar already? It must be something with the Chromiun version that xwalk ships.

Yes it's the latter, just an html input type. I have checked StackOverflow but found nothing related; however, I haven't searched specifically for reported issues in the Chromium version though.

A quick Google found me this Chromium bug.
So which crosswalk version / chromium version are you on?

According to that bug report this was resolved in version 46. What version are you using? This bug may have resurfaced again.

Correct, but it may be only related to 45 (not sure). The latest version of this plugin uses 45, but we'll update to 46 in the foreseeable future. Interesting to know if the issue also pops up on 44 (plugin version <IP_ADDRESS>).

Yes this issue also pops up in your plugin versions <IP_ADDRESS> and 1.2.0. Any ideas when your next update might be?

Hey @rmolley, I just updated to the latest Crosswalk bits and the datetime-local widget looks great on my device. Hopefully for you too!

Yes, it works. Thank you!


Close idle dbs

Previously idle dbs, especially sys dbs like _replicator once opened
once for scanning would stay open forever. In a large cluster with many
_replicator shards that can add up to a significant overhead, mostly in terms
of number of active processes.
Add a mechanism to close dbs which have an idle db updater. Before hibernation
was used to limit the memory pressure, however that is often not enough.
Some databases are only read periodically so their updater would time
out. To prevent that from happening keep the last read timestamp in
the couch file process dictionary. Idle check then avoid closing dbs
which have been recently read from.
(Original idea for using timeouts in gen_server replies belongs to
Paul Davis)
COUCHDB-3323
Related to previous PR from before monorepo merge:
https://github.com/apache/couchdb-couch/pull/236

This is a test restarting a 3 node cluster with 130k databases (100k regular and 30k "_replicator" dbs).

On the left is master and on the right is this PR.
There were no requests issued to the cluster during this time. All activity is due to internal replication and replicator.
Left side crashed and restarted after some point, probably due to high memory usage.
Notice memory consumption and process count continue to rise after internal sync is finished due to the replicator having to launch change feeds and checkpoint 30k * 8 = 240k shards. Closing idle shards in this case has helped free enough memory to let replicator finish checkpointing all the shards.

Another test with 130k dbs (100k dbs regular + 30k `_users)  dbs.

In this case replicator is not involved so the effect is easier to see. Memory consumption is reduced significantly (10GB -> 200 MB).

I know it is out of the scope of this PR but I want to mention the possibility anyway.
We could store extend statistics in the process dictionary. For example we could store following keys:

rx_access_time
tx_access_time
rx_bytes (payload size of last request (last request or counter since we open?))
tx_bytes (payload size of last request (last request or counter since we open?))
rx_requests
tx_requests
In this case we would kill two birds with one stone.

we would account for both reads and writes when we make a decision to close the file.
we can extend couch_debug to return :

recently accessed files
top N files which write the most
top N files which read the most

Would it be a good idea to prevent sysdb (things like _dbs or _nodes) from closing?

I did think about special casing _nodes and _dbs but from what I remember _dbs for example had change feeds hanging of it so it wasn't idle. I'll have to double check but I think those are kept alive anyway.

@iilyak
On the stats idea, we could keep a few stats in there to help debug and monitor issues. Some of the things like total bytes in and out can be found out from the port drivers (efile and inet). But we could keep them in one place in the process dict to avoid digging too deep for the port driver info.
We would also have to split read and write stats since we have a separate updater process. But definitely doable.

Made a few updates. Should be ready for another look.
Also checked that in an idle cluster _nodes and _dbs don't become idle. It's probably because there are change feeds listening to them.

This is running a crud benchmark while having 30k _users (system dbs) loaded.

Probably a better idea to use hashes to identify runs rather than Jenkins build numbers.

@davisp ah good point. 6194 is the build for master and 6200 is the close-idle-pr build

Just realized I didn't explicitly +1 this... so:
+1


Globus refresh tokens

Currently, we can perform file transfers by performing the Globus login flow on each campaign invocation. Next, we want to be able to automatically 'refresh' our tokens for the available refresh expiration period.
Will require:
[] Saving/accessing/authorizing refresh tokens in campaign.py

Working in dev-clean. Close upon merge to main.

Closed.


AWS Authentication w/o body

While following this document https://docs.aws.amazon.com/elasticsearch-service/latest/developerguide/es-managedomains-snapshots.html I was able to use Insomnia for the PUT request under "Registering a Manual Snapshot Repository" (takes a JSON body) but when I use the PUT request under "To manually take a snapshot" that does not have a body, Insomnia gives me the message, "AWS authentication not supported for provided body type".
When I configured Postman with the same request and authentication it worked.

Insomnia Version:  Version 6.5.4 (<IP_ADDRESS>76)
Operating System:  MacOS 10.14.5

Details

AWS IAM v4 Authentication
PUT request
No body

"AWS authentication not supported for provided body type"
Same config in Postman is bueno 👍

Any chance of this actually getting fixed?

Still no progress? Postman, I've come to you...


Sytem failure when registrater for a new account

Hi, I recently got invited to register an account at zen.coderdojo.com where I filled out my details. After pressing submit I received a server error.
{"statusCode":500,"error":"Internal Server Error","message":"An internal server error occurred"}

As the error is a server-side error, there's not much more that I can include in this bug report. No further information was provided.

@DragonBe Could you send on the exact details you entered to<EMAIL_ADDRESS>please? Also, did a second attempt fix the issue?

Wait for more information, closing until we receive it.


Partial loading of skf

If the skf is very large, loading and decompressing the whole thing is slow. It might be possible just to define and get a header struct, which would be useful for ska nk.

Could try #[serde(skip_deserializing)] on variants fields

Tried skip_deserialising trait, and making a smaller struct which omits the large fields. Both take just as long to read in. It's possible this is due to either the CBOR format used (I think more likely) or the compression, but either way this would require a breaking change to the .skf format to work. I've got a couple of thoughts about that, but I will track them in #17 instead.


TSC Meeting (01 Feb 2022): New pack content overrides, AWS, v3.7.0 progress

February 2022 @StackStorm/tsc 1 hour planned meeting: Tuesday, 01 Feb 2022, 09:30 AM US Pacific.
See #33 for more info about how to join.
(TLDR; Zoom link: https://us02web.zoom.us/j/81082101702?pwd=N1V4TWdYRVQ4SXBsaFh1TFYvVDA0UT09)
TSC Meeting Host: @armab
Meeting Agenda
DEMO: New Pack content loading by @amanda11  (15min)

https://github.com/StackStorm/st2/pull/5506
Amanda will present a demo showcasing the new feature from the upcoming v3.7.0 that adds configuration in /opt/stackstorm/overrides/ to control the pack content loading: a whitelist/blacklist for sensors, rules, etc.

AWS p1 (bizdev): Change of account ownership request by @punkrokk (BlueCycle)
Context: 1.5+ years ago @punkrokk created a dedicated AWS account and requested on behalf of StackStorm project FOSS credits in AWS Open Source program. This way the original StackStorm AWS account (child org) moved away from Extreme AWS (parent org) to the new umbrella of created AWS FOSS account (parent org). This helped the project to keep its infra and builds operating as before.

@punkrokk (representing BlueCycle, a StackStorm partner) is looking for someone else to take the AWS account ownership, including the billing and business mgmt.
StackStorm Partners are invited to the meeting, as the project is seeking a potential new AWS account holder or AWS parent org.

AWS p2 (technical): Future plans
Context: AWS FOSS credits will expire mid 2022 with no possibility for extending, per AWS support. StackStorm release automation and e2e testing rely on AWS API where st2cicd requests on-demand VMs to automatically test the stackstorm end-to-end installation in a fresh environment. This happens for every build x 4 OS flavors we support (EL7/8, U18/20).

Discuss the future plans about AWS infra

Short-term: Further reduce the AWS usage
Long-term: Move away from AWS (?)
Consequences for the st2cicd, packaging, future releases, OS support, deployments (?)
etc

StackStorm v3.7.0 release progress

Release Manager: @cognifloyd
Release Assistant: still missing a volunteer. Who from @StackStorm/maintainers?
Target month: March 2022
Project & Roadmap: https://github.com/orgs/StackStorm/projects/23

New feature: Disable/Enable st2 content on load: https://github.com/StackStorm/st2/pull/5506 by @amanda11
#88 plan:

RockyLinux will be a drop-in replacement for EOL'ed CentOS8
update the build process, packaging, CI/CD

Problem: py3.6 was EOL in Dec 2021: still used in EL7, EL8, U18

Participants
Attendees:

@StackStorm/maintainers: @armab @amanda11 @m4dcoder @winem @cognifloyd @guzzijones + Partners: @punkrokk (BlueCycle), @mickmcgrath13 (Bitovi), @bishopbm1 (Encore)

Meeting Minutes
DEMO: New Pack content loading blacklist/whitelist /opt/stackstorm/overrides/ feature by @amanda11

PR & Discussion: https://github.com/StackStorm/st2/pull/5506
@amanda11's notes from today's demo and TSC feedback:

https://github.com/StackStorm/st2/pull/5506#issuecomment-1027166605

@punkrokk @bishopbm1 and others are invited to document their ideas in more detail in the PR above ^^
@m4dcoder would be welcomed for review
More :eyes: are needed from the @StackStorm/maintainers

AWS
FOSS AWS Credits for parent AWS org will expire on March 31, 2022.

@bishopbm1 - Encore is relying heavily on StackStorm in its core automation and will examine what they can do to help with the infrastructure: Internal Encore cloud platform, AWS parent org (if possible). Bradley will need the usage stats, details, budgets, to follow-up with the approval, etc, but very positive about the outcome in supporting the project in general.
@bishopbm1, @punkrokk, @armab will work together on details and follow-up, keeping the TSC informed.
@guzzijones will raise the question of sponsoring the st2 infra as well from his work org. @mickmcgrath13 as well?
@punkrokk to explore the possibility to use the AWS spot instances in e2e testing (as promised in the previous meetings: July '21, Aug '21)
Short-Term:

Ideally: transfer the AWS ownership from @punkrokk BlueCycle to @bishopbm1 Encore

Mid-Term:

Before AWS FOSS Credits expiration (March 31, 2022)
Optimise the AWS costs further:

docs.stackstorm.com -> to be migrated from S3 to GH static pages (https://github.com/StackStorm/st2docs/issues/1096) - needs a volunteer
api.stackstorm.com -> to be migrated from S3 to GH static pages (https://github.com/StackStorm/st2apidocs/issues/16) - needs a volunteer
rebuild and right-size the st2cid, remove LDAP, internal monitoring servers, etc
IDEA middle-ground:

run st2cicd release/control instance in Encore infra, still using the AWS APIs for requesting the short-lived VMs for e2e testing on-demand
Can save 80%+ of the AWS monthly costs
Less work is needed for st2cicd workflows refactoring, still using AWS API

Long-Term:

Is it possible to get rid of AWS (almost) completely? (To be discussed in the next TSC Meeting)
Refactor and migrate the st2cicd release automation and e2e logic from the black AWS-hosted box with limited access

Utilize the GH Actions
Public transparent CI/CD/Release automation could be observed and verified by anyone

better security, fewer supply-chain risks

Helpful small e2e step example: @winem is working on running the st2-self-check script in GH Actions
More Ideas - to be discussed in the next TSC Meeting

Thanks, everyone for joining!
If you have any adjustments, additions, or notes about the meeting minutes, - please to post more details in the comments below.

New information from the Slack Discussion that AWS FOSS credits expiry date is 31 July 2022, not 31 March 2022.


Fixed fisher install command

The fisher add has changed to fisher install

Ahh I forgot to update the change here. Thanks for sending in a PR :)


Multi-level subdomain requests fail

Works
curl http://my.multi.subdomain.com

Does not work (Could not resolve host: my.multi.subdomain.com)
<?php
$client = new GuzzleHttp\Client();
$res = $client->request('GET', 'http://my.multi.subdomain.com');

As with most issues, this was the fault of my code.


Add content on custom tracing port

Add content on custom tracing port.

CC @srujann
The doc link is broken for now until the changes get pushed to mater.


@sort:installs @category:languages no longer shows extensions

VSCode Version: Code - Insiders 1.14.0-insider (63def6eb989d62f23d3711ace0f795ce9f23551d, 2017-06-28T05:07:46.148Z)
OS Version: Darwin x64 15.6.0
Extensions: none

Steps to Reproduce:

Run command "Preferences: Language Extensions"
-> The extension viewlet opens, but no longer shows any results.

I'm using this command for the 'more' link on the 'Tools and languages' button on the Welcome page. (You might not see this button if you don't fall within the split Quick Links experiment.)

Assigning to June for discussoin

The @category: syntax was never supported. I guess we were just lucky that the Marketplace ignored the @ before it, making it category:, which is the correct syntax.
Now that we must drop the catregory: syntax, we have some parsing code which converts that to query filters.
The fix here is to simply remove the @.


Chnage download btn link

Now the download button points to the docs (I've deployed a test build to vercel: https://blendos-co.vercel.app)
(I also spent most of my time figuring out how to deploy your site to vercel lmao)

A bit messy, but I couldn't find #distros anywhere


refactor: use string intern

Current benchmark in comment section is a fake one, I will add real tests later


Mark and remove inactive users from a group

As mentioned in #853, it would be nice to remove inactive users from a group:

due to inactivity: notify and mark as inactive after 1 month (grey out in member list), notify again after 2 months, remove from group after 3 months

Questions were raised by @djahnie

What counts as activity? (read or write access, imho read is enough)
What is the process and what are the timeframes?

I think activity should be counted per group and read/write access should both be counted as activity. It will therefore be possible to get removed from a group while being active in another one. It will also be possible to stay active by just visiting the group from time to time.

I agree on your definition of activity and that it has to be counted per group.
What are the first steps to implementing something like this? I guess a new field in the database that has the binary choice of being set to 'active' or 'inactive' is needed, as well as some code, that counts the time since the last activity of a user. This is more of a backend task, am I right?
(In case this is not obvious, I'm trying to find more starter tasks...^^)

This also ties a bit into my ponderings on how to record activity in general (so it's clearer which people/groups are actively using it) - my ponderings led me to think in terms of 3 levels of activity:

registered user (signed up, but do nothing)
online user (they log in, click around)
active user (they do pickups!)

Maybe it's also useful to distinguish between users that are active online (writing messages, editing things, etc) and passive ones (just click around a bit), but maybe not...
Historic data is better stored in influxdb I think and exported periodically, but the application also would/might/should have some historic data too, where it makes sense for the logic of the application (I think the application should not read from influxdb).
Perhaps a storage format for inside the application would be an activity table and record into time buckets per user+group, e.g. recordActivity(user_id, group_id, time_bucket, activity_type) where time_bucket might be a day , and activity_type might be online|offline. the data could be sent to influxdb db at the same time, then the application can delete from the activity table when it makes sense for the application logic (e.g. you only actually need to know the last activity).
So, actually, rethinking as I write, if we need to only know the last activity then could store that directly in the user/group join table (last_online_activity_at, last_offline_activity_at), and write to influxdb as before.....
... or even just repurpose the history table, and check when their last activity was (although that does not record things like logging in, but could (and just filter it from the normal view), but could end up with too much data in there that we don't really need.
The activity table could have counters for each time bucket row too if we wanted to keep track of how much activity too, not just whether there was some.
Seems enough thoughts for now.

@tiltec and I have been looking at this in a bit more detail this morning. We decided to implement the simplest useful answer to the problem which was to add a single "last_seen" field to the GroupMembership model. In terms of the user, the field is intended to represent the last time a user had used Karrot (via a browser or the app) to actively view data associated with a group.
Our initial implementation was to use the current_group field in AuthUser. This worked to some extent, however it was only updated when we use the 'Change group' menu option on a browser. (We also noticed that it is possible to open two different groups in two different browser tabs, which may lead to some issues but didn't seem to break anything.)
We discussed a few options:

Add a hook in the backend which checks if read_access permission has been requested on a group and then update last_seen at that point. There are two problems with this. First, updating the DB when only a request has been made is probably not a good idea. Second, a future frontend implementation might cache several groups' data which would make it look like a user had done something in a group when they hadn't.
Change responsibility to the frontend to call a function in the backend to report that a user had done something in a group.  We would implement a post-function in the backend for the frontend to report that a specific user had accessed a group.

Option 2 probably resolves the potential problems from option 1, but might have some new issues?

The approach still seems good to me, I'd say it's worth a try.

Option 2 is now working and has a mark_user_active endpoint.
We now envisage the following tasks:

[ ] Implement calling the mark_user_active endpoint into the frontend

[ ] Add a boolean field "active" to GroupMembership (default=active) which is set to True whenever the endpoint is called

[ ] Write a script which looks for inactive users and sets the active field to False after a time period, and removes inactive users after a second time period

[ ] Write an email template for users who have been flagged as inactive to warn them they will be removed from the group / tell them they have now been removed

[ ] Implement greying out of inactive users in the frontend

The current backend implementation marks a user as inactive_at = <date> after 30 days of not visiting the group and sends out an email to the user.
The group API has a field active, which is False when the user is inactive.
#957 marks inactive users in the member list.
Future improvements:

only show active members in the group member count (GroupGallery)
remove long-time inactive users from the group https://github.com/yunity/karrot-backend/pull/528

We decided to wait for feedback for the current implementation to find out if the automatic removal is actually needed right now.


Curate nebula-graph topic

Please confirm this pull request meets the following requirements:

[X] I followed the contributing guidelines: https://github.com/github/explore/blob/main/CONTRIBUTING.md.
[x] I am not the sole author or employee of a company who created the topic or collection I'm changing.

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[X] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
[ ] Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

[X] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[X] My folder contains a *.png image (if applicable) and index.md
[X] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

NebulaGraph, hosted on GitHub, is a distributed, fast open-source graph database. There are over 36 tagged repositories currently under the nebula-graph topic.

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing topic or collection, or at least not a single one
[ ] My suggested edits are not about curating a new topic or collection, or at least not a single one
[ ] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Hi @kenyonj , could you help me review this PR again? Thanks for your time!


New examples

Adding c examples for message federates and combination federates. The goal is to have them be able to interact with the C++ equivalent.  I haven't run full tests yet.

Should an entry be added to the cmake file for copying the run[3].sh files to the build directory? Some of the examples have the scripts copied, others don’t.

Someday I would like to make sure those scripts are actually working.  I think in this case I just copied from another directory,  I guess we probably should delete them since I don't think they will work at present.  And they are unix specific.  We should probably invest some time in the examples repo and make sure the scripts work and everything runs smoothly when that is operational.  What is your opinion should the *.sh be deleted or just left and we will deal with them later?

I think dealing with paths and the directory to execute the run.sh scripts from is similar to the problem of running the examples as part of the test on Travis, except spread out over a large number of nearly identical files - lots of things can break from a small name or location change. Having a cleaner solution in an examples repository would be better than having a bunch of outdated scripts lying around in this one.
Most of the run.sh scripts don’t seem to work on LC without changing the -name and -target arguments to —name and —target.
I just got an idea for running the examples we keep in this repo on Travis without dealing with hard-coded paths.


Bug Report: WebUI through browser is not working

Describe the bug
I cannot view the status of IPFS from my browser
To Reproduce
Steps to reproduce the behavior:

Install via brew cask install ipfs
Install the ipfs-companion Firefox extension
Open "IPFS Desktop.app" and view the successful peering from within the app
Select "Open WebUI" from the ipfs-companion menu and get an error that ipfs daemon needs to be run.

Expected behavior
I expect the WebUI to be accessible from any interface that can load localhost:5001/webui
Screenshots
View from IPFS Desktop:

View from localhost:8080:

Desktop (please complete the following information):

OS: macOS 10.14.4
Browser: Firefox
Version: go-ipfs v0.4.19

Hi @jasikpark you should be able to visit the webui via http://localhost:5001/webui
http://localhost:8080 isn't expected to work out of the box as the API has CORS restrictions in place that only allow access from a site loaded from <IP_ADDRESS>:5001.
If the api allowed access from content loaded via :8080 then any site that you visited via IPFS would be able to access your api, not just webui.

You very much should be able to use the "Open Web UI" button from within companion tho, and it should be able to talk to the api provided by IPFS Desktop... that is odd.

@jasikpark can you share the full url that you get redirected to when you hit "Open Web UI" in companion?

Full URL:
http://<IP_ADDRESS>:8080/ipfs/QmfQkD8pBSBCBxWEwFSu4XaDVSWK6bjnNuaWZjMyQbyDub/
I also typed http://<IP_ADDRESS>:5001/webui/ into the bar and got redirected to http://<IP_ADDRESS>:5001/ipfs/QmXc9raDM1M5G5fpBnVyQ71vR4gbnskwnB9iMEzBuLgvoZ/
it also failed to connect to the API

I've tried uninstalling and reinstalling go-ipfs and IPFS Desktop multiples times, in addition to nuking ~/.ipfs in concert with reinstallation. For some reason, the api only works when accessed through IPFS Desktop, rather than a web browser

Does ipfs-companion work?

ipfs-companion works fine - though when i click the "Show WebU" button within ipfs-companion the page it loads fails to access the API.
but ipfs-companion is redirecting and loading and pinning perfectly.

@jasikpark I've wiped my local set up and tried to recreate this, but it seems to be working. Can you share the API and Adresses sections of your config, to help diagnose this? Have you ever installed ipfs before via some other mechanism?

I think it might be that I have both go-ipfs and IPFS Desktop installed, and maybe that's a problem. Though I tried both of them installed by themselves and that did not fix the problem once it appeared.

@jasikpark assuming this is fixed in go-ipfs 0.6.0 / ipfs-desktop 0.12.2 – please upgrade and reopen if you still experience this problem.


Explore refactoring local NodeManagement DetailedDisovery to use SPINE model

Instead of creating the DetailedDiscvovery data structures in device, entity, features and operations using Information(), use the SPINE model data structure instead and therefor remove custom code for handling this.

Won't do. This would make the code much more complicated.


Feature reques: pgn output with comments

Is it possible to output a new pgn that marks the undecided positions then adds data from the analysis as a comment?

I'll add that to the wish list.  Not sure how soon it will happen, though, I have other higher-priority enhancements that I'm working on when I get a chance.


Warning: Unexpected character input

https://github.com/apereo/phpCAS/blob/ff165bf94c62b4cb76c5eebd3ea3d157c24e527c/source/CAS.php#L1877
Warning: Unexpected character in input: '\' (ASCII=92) state=1 in /web/staging/CAS/CAS-1.3.8/CAS.php on line 1877
I must be the only poor schmuck required to run this on PHP 5.2. Is \CAS_Client required? or would CAS_Client suffice?
I know it's just a warning, but that warning is interfering with the redirect to our login page.
If I remove the \ everything appears fine, but I honestly can't say I know what all the ramifications are.
If I grep -FRi "(\\CAS" the repo this is the only instance I can find (excluding ./docs)

The slash was introduced with php namespacing in php 5.3
https://www.php.net/manual/de/language.namespaces.importing.php
Removing it should make no difference for you.

Closing since:
https://github.com/apereo/phpCAS/blob/ff165bf94c62b4cb76c5eebd3ea3d157c24e527c/composer.json#L13

Trying reply by email.
Super. Thanks for the help.

From: Joachim Fritschi<EMAIL_ADDRESS>Sent: Sunday, September 1, 2019 4:48:56 AM
To: apereo/phpCAS<EMAIL_ADDRESS>Cc: Macdonald, Jay<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [apereo/phpCAS] Warning: Unexpected character input (#314)
The slash was introduced with php namespacing in php 5.3
https://www.php.net/manual/de/language.namespaces.importing.php
Removing it should make no difference for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/apereo/phpCAS/issues/314?email_source=notifications&email_token=AAOJSTZRNGIZF3G2Q52CKFTQHOMZRA5CNFSM4IRO42HKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5T7SWY#issuecomment-526907739, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAOJSTZMPXO7GDK6BYEL74LQHOMZRANCNFSM4IRO42HA.


why not share pid namespace

From cri-o.io says
Pods are a kubernetes concept consisting of one or more containers sharing the same IPC, NET and PID namespaces and living in the same cgroup.

but in the code, seems don't share pid namespace for now, is there some special reason?
https://github.com/kubernetes-incubator/cri-o/blob/master/server/container_create.go#L916

We probably should change the description.  Kubernetes currently does not share the PID Namespaces,  Well I guess it is optional.

We probably should change the description.  Kubernetes currently does not share the PID Namespaces,  Well I guess it is optional.

Kubernetes now support pid namespace from 1.7 :) (ref: https://github.com/kubernetes/kubernetes/pull/51634), and we have enabled this feature in our 1.8 environment with docker 1.13 so far so good :), wondering why cri-o don't support pid namespace, we are also investigating to use cri-o(since it have some features we need, like pid-max, ignore image volume and more light weight than docker for kubernetes)
If there's no other reason than old kubernetes didn't support share pid namespaces, I'd like to open a pr  working on it.

Kubernetes now support pid namespace from 1.7 :) (ref: https://github.com/kubernetes/kubernetes/pull/51634), and we have enabled this feature in our 1.8 environment with docker 1.13 so far so good :), wondering why cri-o don't support pid namespace, we are also investigating to use cri-o(since it have some features we need, like pid-max, ignore image volume and more light weight than docker for kubernetes)
If there's no other reason than old kubernetes didn't support share pid namespaces, I'd like to open a pr working on it.

We will certainly add it, I believe they were duplicating the Docker-1.12 behaviour that kubernetes currently uses.  @runcom how hard would this be to fix?

Yes open a PR.
I have been pushing back on upstream kubernetes about making this a requirements. since I think having multiple containers in the same POD with different Security is an interesting use case.  Think about a container with an init container that loads a kernel module and then the primary container can be locked down and just use the module.
systemd in a container also does not work well with the PID namespace shared with the Pause container.  Although I think CRI-O can get rid of the need for the Pause container, which would be awesome.

@weiwei04 if you do make a patch to share the PID namespace can you add it as an option to the daemon to allow people to optionally run it shared.


Next

Closing, as we don't have access to Travis for now. Moving my WIP to https://github.com/VojtechVitek/qmd.

In general LGTM


Gradle & las2peer v1.1.1

This PR removes ant from the template project and replaces it by gradle.
Besides that, the template project uses las2peer v1.1.1 and JUnit v4.13.2 now.

Codecov Report

Merging #25 (0ffcf7b) into master (8cb0583) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##              master       #25   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity         3         3
===========================================
  Files              1         1
  Lines              7         7
===========================================
  Hits               7         7

Impacted Files
Coverage Δ
Complexity Δ

...peer/services/templateService/TemplateService.java
100.00% <ø> (ø)
3.00 <0.00> (?)

Continue to review full report at Codecov.

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


loading CSS - cannot assume node_modules path

edge case:
In our Cypress example recipes repo https://github.com/cypress-io/cypress-example-recipes each separate application is in a subfolder and all dependencies are installed at the root level. Thus we only need to clone the repo, run npm install from the root and then inside each examples/... folder we can start the app and run Cypress tests. Example package json from branch that uses this plugin is testing-redux-store and pull request https://github.com/cypress-io/cypress-example-recipes/pull/211
The application is in examples/blogs__testing-redux-store and its package.json is just
{
  "name": "blogs__testing-redux-store",
  "version": "1.0.0",
  "description": "Testing Redux store using Cypress",
  "private": true,
  "scripts": {
    "start": "../../node_modules/.bin/react-scripts start",
    "cypress:run": "../../node_modules/.bin/cypress run",
    "cypress:open": "../../node_modules/.bin/cypress open"
  },
  "browserslist": [
    ">0.2%",
    "not dead",
    "not ie <= 11",
    "not op_mini all"
  ]
}

(or at least it should be this)
"cypress-plugin-snapshots": "1.0.6" is installed in the root folder.
But if I run tests I get an exception
Error:    CypressError: Timed out retrying: cy.readFile("node_modules/diff2html/dist/diff2html.css") failed because the file does not exist at the following path:

/Users/gleb/git/cypress-example-recipes/examples/blogs__testing-redux-store/node_modules/diff2html/dist/diff2html.css

Because this error occurred during a 'before all' hook we are skipping all of the remaining tests.

I have not looked at the dependency loading code, but I know it can be in an unexpected place, see how I load CSS for cypress-dark here https://github.com/bahmutov/cypress-dark/blob/0460aa71a23080780605936257896ed3f0777cb9/src/utils.js#L7 which is ugly.
I was thinking a better way would be to somehow to ask the plugin where it is installed - because it has access to the true __dirname without browserify wrap. Then the front end code can correctly do its part, or even ask the plugin via cy.task for the path to load.

for now I work around this by installing cypress-plugin-snapshots in examples/blogs__testing-redux-store folder, but I really would prefer only the root folder install.

Yeah paths are hard in Cypress. Because if you want to use cy.readFile you need to access it's path relative to Cypress directory.
I like your suggestion to do it via the plugin in a task. Will investigate it.
Thanks for the feedback by the way. Much appreciated.

@bahmutov I am now loading the files via a task. This should solve the issues where paths are not 100% reliable.
I am not 100% convinced by this solution, it seems a bit slow. Perhaps I could cache the results in memory somehow so it doesn't have to execute the task every time. But I am not sure how this would behave since the tests run in isolation.
Can you verify if it also works in your setup?


[Bug Report]: TypeError: Cannot read properties of undefined (reading 'createElement') when enable builtins.presetEnv

System Info
System:
OS: macOS 12.5.1
CPU: (10) arm64 Apple M1 Pro
Memory: 212.36 MB / 32.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.19.1 - /usr/local/bin/node
npm: 8.19.3 - /usr/local/bin/npm
Browsers:
Chrome: 111.0.5563.110
Safari: 15.6.1
Details

Reproduce link
https://github.com/web-infra-dev/modern.js/tree/test/rspack-3-28
Reproduce Steps

pnpm install
cd tests/integration/webpack-builder
DEBUG=builder pnpm dev:rspack
open the browser

Everything works fine when disable presetEnv:

tests/integration/webpack-builder/scripts/shared.ts

colsed at #2503


Auto-submit pub packages autoroll PRs

Part of https://github.com/flutter/flutter/issues/106371
I've been manually reviewing and approving these PRs for about a month (https://github.com/flutter/flutter/pulls?q=is%3Apr+author%3Afluttergithubbot+"Roll+pub+packages"+is%3Aclosed+) and nothing has gone wrong. The PRs that were closed without merging required manual intervention that was resolved in a separate PR.
This PR has the bot add the autosubmit label when creating the PR. I will make a follow-up cocoon change to have the cocoon bot auto approve. Any autoroll that breaks pre-submits will still require manual intervention.

@goderbauer FYI next time you manually roll pub packages I should get auto-added as a reviewer via codeowners, so you shouldn't have to remember to cc me.


optimize lro/rpc-legacy test case

Cadl Ranch Contribution Checklist:

[ ] I have written a scenario spec
[ ] I have meaningful @scenario names. Someone can look at the list of scenarios and understand what I'm covering.
[ ] I have written a mock API
[ ] I have used @scenarioDocs for extra scenario description and to tell people how to pass my mock api check.

@chunyu3 @pshao25 @archerzz Please help review


Fix/jquery

Apologies for the mess... only the last commit is relevant. Do with what you please!

@emcniece had some trouble with a merge conflict so I cherry-picked this over to the branch. commit still has your name on it for the cred!


Profile View bands

https://forum.dynamobim.com/t/set-profileview-band-profile-1-or-profile-2-to-new-profile/55677/4?u=mzjensen

Don't have time for this.


Add generic bounds assist

pub fn read_perf_data<P>(path: P) -> Result<(), Box<dyn Error>> {
    let mut file = File::open(path)?;
}

The compiler knows that path does not satisfy the bounds for File::open. Having an assist to add the needed bounds to P would be great.

This is hard to do right now, because rust-analyzer's own analysis does not record failed trait bounds yet. (The error message you're seeing comes from rustc.)


Communication SMS Readme issues

Section Link:

Suggestion:
Add Source code| Package (Pypi)| API reference documentation | Product documentation
As shown in the figure below about app configuration:

2.
Section Link:

Reason:
Name 'SmsClient' is not defined.
Suggestion:
Add:
from azure.communication.sms import SmsClient
3.
Section link:

Reason:
Name 'PhoneNumber' and 'SendSmsOptions' is not defined.
Suggestion:
Add:
from azure.communication.sms import SendSmsOptions, PhoneNumber
4.
Section link:

Suggestion:
Add Hyperlink for "[azure_core]":
[azure_core]: https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/core/azure-core/README.md
@jongio for notification.

@v-xuto I'm on call this week and I'm working on these issues you submitted. This is a PR that addresses your issues :)
https://github.com/Azure/azure-sdk-for-python/pull/16825


tap on another textinput hide keyboard so need to tap again on textinput for keyboard open and type

tap on another textinput hide keyboard so need to tap again on textinput for keyboard open and type

having this same issue

http://facebook.github.io/react-native/docs/scrollview.html#keyboardshouldpersisttaps


(AUTOMATIC) opensource update

Makes the address normalizer's normalize function check all languages.

I've also modified the source so that the next automatic roll doesn't delete these changes again.


Add Debian 12 as molecule scenario and update role's meta information

Cannot get Debian 12 working. Will try again some other time.


Package for pypi/pip

It would be very nice, if you could provide pyflame via pypi.
Installing via pip is easy.
Are there reasons why there is no wheel up to  now?

Hi devs, please leave a comment to this feature request.  Thank you.

I don't care any more. I use py-spy now.


frontend_tests: Make better use of assert methods

assert.foo() methods give more specific error messages than assert().

Nice, merged, thanks @andersk!


Using Gridster with React

Hi,
Our App is REACT based, can you please let me know if Gridster is compatible with REACT, and it will be appreciated if you can post a sample

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain biomassiv.es has exceeded the max emails per hour (119/100 (119%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext3.iad.github.net ([<IP_ADDRESS>]:50759 helo=github-smtp2b-ext-cp1-prd.iad.github.net)
by chi-server32.websitehostserver.net with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.87)
(envelope-from<EMAIL_ADDRESS>id 1cWmiX-0032eG-EM
for<EMAIL_ADDRESS>Thu, 26 Jan 2017 10:16:09 -0600
Date: Wed, 25 Jan 2017 07:39:42 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1485358782;
bh=WJ+8Ijmnd3VF+eEZjEwBxw7yea2rlYjJg15NMOG1DJw=;
h=From:Reply-To:To:Cc:Subject:List-ID:List-Archive:List-Post:
List-Unsubscribe:From;
b=U5OYcAMU7B9YTf4NyUnl5wBEle581CssKhOkYdX4VbpDTZOTpU3s4hD9nHrju8jz/
N/FowRxq0IaLEh/NSMUZWnW9PBpHG6Xt5A7dLwd75ejEkk3m7uaIf/qd4MAmzwZayc
vpHK/FUEjgfzX+E+xp7cEJswzbBBylfOqeM9nRvU=
From: rajani kanth ponugoti<EMAIL_ADDRESS>Reply-To: "ducksboard/gridster.js"<EMAIL_ADDRESS>To: "ducksboard/gridster.js"<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Subject: [ducksboard/gridster.js] Using Gridster with React (#588)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5888c6be371d9_76e33fe1a86fb13818777f";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: rajniponugoti
X-GitHub-Recipient: biomassives
X-GitHub-Reason: subscribed
List-ID: ducksboard/gridster.js <gridster.js.ducksboard.github.com>
List-Archive: https://github.com/ducksboard/gridster.js
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AELU4mOuJO4XWYJ538X67YtZ2d1XoI3-ks5rV2y-gaJpZM4Ltobc
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>----==_mimepart_5888c6be371d9_76e33fe1a86fb13818777f
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Hi,
Our App is REACT based, can you please let me know if Gridster is compatible with REACT, and it will be appreciated if you can post a sample
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ducksboard/gridster.js/issues/588
----==_mimepart_5888c6be371d9_76e33fe1a86fb13818777f
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Hi,
Our App is REACT based, can you please let me know if Gridster is compatible with REACT, and it will be appreciated if you can post a sample
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5888c6be371d9_76e33fe1a86fb13818777f--

Hi,
Our App is REACT based, can you please let me know if Gridster is compatible with REACT, and it will be appreciated if you can post a sample


ARVIR bug fixes

Handles:

https://artsyproduct.atlassian.net/browse/EV-88
https://artsyproduct.atlassian.net/browse/EV-89
https://artsyproduct.atlassian.net/browse/EV-87
https://artsyproduct.atlassian.net/browse/EV-90

LGTM. I know Erik is assigned, but happy to merge if you want @orta