Flutter:IOS  error: unknown target triple 'unknown-apple-ios9.3.0'

Hello Everyone ,I'm a new flutter .I have some problem that my device are low or something and then I've just changed the architecture from standard to armv7f . when I've run my project with Xcode, the program threw error :
error: unknown target triple 'unknown-apple-ios9.3.0', please use -triple or -arch
Command /Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang failed with exit code 1

Does anyone know How to fix it . Thank you in advance.
My device is iPad mini . ios9.3.6

Hi @izzabellerina
can you please provide your flutter doctor -v ,
your flutter run --verbose
Thank you

Hi @izzabellerina
can you please provide your flutter doctor -v ,
your flutter run --verbose
Thank you
`
=== BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Debug ===
Target 'Pods-Runner' of project 'Pods' was rejected as an implicit dependency for 'Pods_Runner.framework' because its architectures 'armv7 arm64' didn't contain all required architectures 'arm64 armv7f'
=== BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Debug ===
warning: no rule to process file '/Users/armotesomboonkaew/Downloads/flutter_blue-master 2/example/ios/File.swift' of type sourcecode.swift for architecture armv7f
error: unknown target triple 'unknown-apple-ios9.3.0', please use -triple or -arch
Command /Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang failed with exit code 1

Could not build the precompiled application for the device.
Error launching application on @mote500iPadMini1stGen.`

@iapicca Sorry to respond you slow and late.
this is my flutter doctor -v
`[✓] Flutter (Channel beta, v1.14.6, on Mac OS X 10.15.3 19D76, locale en-TH)
• Flutter version 1.14.6 at /Users/armotesomboonkaew/flutter/flutter
• Framework revision fabeb2a16f (6 weeks ago), 2020-01-28 07:56:51 -0800
• Engine revision c4229bfbba
• Dart version 2.8.0 (build 2.8.0-dev.5.0 fc3af737c7)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/armotesomboonkaew/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
• Xcode at /Applications/Xcode-beta.app/Contents/Developer
• Xcode 11.4, Build version 11N123k
• CocoaPods version 1.7.5
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 43.0.1
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] Connected device (1 available)
• @mote500iPadMini1stGen • 7e260d8b3956e35fe20400b14ef03a2ab4376008 • ios • iOS 9.3.6
• No issues found!
`

@iapicca
this is my flutter run -verbose
[  +45 ms] executing: [/Users/armotesomboonkaew/flutter/flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +62 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] fabeb2a16f1d008ab8230f450c49141d35669798
[        ] executing: [/Users/armotesomboonkaew/flutter/flutter/] git describe --match v*.. --first-parent --long --tags
[  +15 ms] Exit code 0 from: git describe --match v*.. --first-parent --long --tags
[        ] v1.14.6-0-gfabeb2a16
[  +16 ms] executing: [/Users/armotesomboonkaew/flutter/flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +18 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/beta
[        ] executing: [/Users/armotesomboonkaew/flutter/flutter/] git ls-remote --get-url origin
[  +18 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +83 ms] executing: [/Users/armotesomboonkaew/flutter/flutter/] git rev-parse --abbrev-ref HEAD
[  +21 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] beta
[  +11 ms] executing: sw_vers -productName
[  +29 ms] Exit code 0 from: sw_vers -productName
[        ] Mac OS X
[        ] executing: sw_vers -productVersion
[  +24 ms] Exit code 0 from: sw_vers -productVersion
[        ] 10.15.3
[        ] executing: sw_vers -buildVersion
[  +29 ms] Exit code 0 from: sw_vers -buildVersion
[        ] 19D76
[  +54 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +6 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +62 ms] executing: /Users/armotesomboonkaew/Library/Android/sdk/platform-tools/adb devices -l
[  +11 ms] Exit code 0 from: /Users/armotesomboonkaew/Library/Android/sdk/platform-tools/adb devices -l
[        ] List of devices attached
[  +31 ms] executing: /Users/armotesomboonkaew/flutter/flutter/bin/cache/artifacts/libimobiledevice/idevice_id -h
[ +521 ms] executing: /usr/bin/xcode-select --print-path
[  +18 ms] Exit code 0 from: /usr/bin/xcode-select --print-path
[        ] /Applications/Xcode-beta.app/Contents/Developer
[   +1 ms] executing: /usr/bin/xcodebuild -version
[ +311 ms] Exit code 0 from: /usr/bin/xcodebuild -version
[        ] Xcode 11.4
Build version 11N123k
[   +6 ms] /usr/bin/xcrun simctl list --json devices
[ +244 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[   +8 ms] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +1 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +36 ms] "flutter run" took 1,433ms.
Target file "lib/main.dart" not found.
#0      FlutterCommand.validateCommand (package:flutter_tools/src/runner/flutter_command.dart:739:9)
#1      RunCommand.validateCommand (package:flutter_tools/src/commands/run.dart:320:19)
#2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:619:11)

#3      FlutterCommand.run. (package:flutter_tools/src/runner/flutter_command.dart:537:33)

#4      AppContext.run. (package:flutter_tools/src/base/context.dart:150:29)
#5      _rootRun (dart:async/zone.dart:1126:13)
#6      _CustomZone.run (dart:async/zone.dart:1023:19)
#7      _runZoned (dart:async/zone.dart:1518:10)
#8      runZoned (dart:async/zone.dart:1465:12)
#9      AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#10     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:527:20)
#11     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
#12     FlutterCommandRunner.runCommand. (package:flutter_tools/src/runner/flutter_command_runner.dart:338:21)

#13     AppContext.run. (package:flutter_tools/src/base/context.dart:150:29)
#14     _rootRun (dart:async/zone.dart:1126:13)
#15     _CustomZone.run (dart:async/zone.dart:1023:19)
#16     _runZoned (dart:async/zone.dart:1518:10)
#17     runZoned (dart:async/zone.dart:1465:12)
#18     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#19     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:288:19)
#20     CommandRunner.run. (package:args/command_runner.dart:112:25)
#21     new Future.sync (dart:async/future.dart:224:31)
#22     CommandRunner.run (package:args/command_runner.dart:112:14)
#23     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:231:18)
#24     run.. (package:flutter_tools/runner.dart:64:22)
#25     _rootRun (dart:async/zone.dart:1126:13)
#26     _CustomZone.run (dart:async/zone.dart:1023:19)
#27     _runZoned (dart:async/zone.dart:1518:10)
#28     runZoned (dart:async/zone.dart:1502:12)
#29     run. (package:flutter_tools/runner.dart:62:18)

#30     AppContext.run. (package:flutter_tools/src/base/context.dart:150:29)
#31     _rootRun (dart:async/zone.dart:1126:13)
#32     _CustomZone.run (dart:async/zone.dart:1023:19)
#33     _runZoned (dart:async/zone.dart:1518:10)
#34     runZoned (dart:async/zone.dart:1465:12)
#35     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#36     runInContext (package:flutter_tools/src/context_runner.dart:64:24)
#37     run (package:flutter_tools/runner.dart:51:10)
#38     main (package:flutter_tools/executable.dart:65:9)
#39     main (file:///Users/armotesomboonkaew/flutter/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#40     _startIsolate. (dart:isolate-patch/isolate_patch.dart:299:32)
#41     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

@iapicca
and finally this is run build whole error.
Running Xcode build...
Xcode build done.                                           10.3s
Failed to build iOS app
Error output from Xcode build:
↳
** BUILD FAILED **
Xcode's output:
↳
=== BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Debug ===
Target 'Pods-Runner' of project 'Pods' was rejected as an implicit dependency for 'Pods_Runner.framework' because its architectures 'armv7 arm64' didn't contain all required architectures 'arm64 armv7f'
=== BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Debug ===
warning: no rule to process file '/Users/armotesomboonkaew/Downloads/flutter_blue-master 2/example/ios/File.swift' of type sourcecode.swift for architecture armv7f
error: unknown target triple 'unknown-apple-ios9.3.0', please use -triple or -arch
Command /Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang failed with exit code 1
Could not build the precompiled application for the device.
Error launching application on @mote500iPadMini1stGen.

@iapicca and my whole expanded dependencies for letting you know. Maybe some of dependencies are  problem instead.
dependencies:   flutter:     sdk: flutter   hex: ^0.1.2   sqflite: ^1.2.0   path_provider: ^1.6.0   intl: ^0.16.0   shared_preferences: ^0.5.6   grouped_buttons: ^1.0.4   flutter_time_picker_spinner: ^1.0.6   sqflite_porter:     git:       url: git://github.com/tekartik/sqflite_more       ref: dart2       path: sqflite_porter   permission: ^0.1.5   flutter_svg: ^0.15.0   easy_localization: ^1.4.0   screenshot: ^0.1.1   mp_chart: ^0.1.2   flutter_share: ^1.0.2+1   time_formatter: ^1.0.0+5   flutter_launcher_icons: ^0.7.4   fragment: ^0.3.1   share_it: ^0.5.1+3   mqtt_client: ^5.5.1   barcode_scan: ^2.0.1   bot_toast: ^2.3.0   connectivity: ^0.4.8+2

@iapicca and the lastly finally error when I run project from XCODE from Runner.xcworkspace file . and This is I got a problem
error: unknown target triple 'unknown-apple-ios9.3.0', please use -triple or -arch
Command /Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang failed with exit code 1

iPad Mini architecture is armv7f.  Huh.

What do you mean by "I've just changed the architecture from standard to armv7f"?

Possibly fixed by https://github.com/flutter/flutter/pull/52021#issuecomment-595368764 in v1.15.18.  Can you try on master or dev channels?

What do you mean by "I've just changed the architecture from standard to armv7f"?

normally , the architecture  is ARCHS_STANDARD  and arm64, and then I cannot run my project to IPad mini. The Xcode threw the error that can compile with  armv7f  and then I changed . After that they threw a new error is Flutter:IOS error: unknown target triple 'unknown-apple-ios9.3.0'
Thank you very much  for your reply.

Gotcha.   Can you see if it's fixed on master or dev?

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so it's always better to open new issues and reference related issues.


Topology view improvements

Consuming the events from inventory and updating the topology graph on events (resource creation only, now). Also adding a platform nodes to the graph.
Here is short demo with curl+ui: https://www.dropbox.com/s/ie8wptu61mjzayz/topology-on-websocket.ogv?dl=0

lgtm - @mtho11 wdyt?

LGTM - Merged #771


Julia Codegen Plugin

Hello, I would like to build a codegen plugin for a Julia client. https://julialang.org/
Given a query/mutation, the plugin should generate the minimum amount of code, not a type for the entire GraphQL schema.
I poked around and it seems like everyone generates the entire graphQL schema to a file. I don't want to do this because my GraphQL API is massive and I'm only using a small part of it.
Are there any plugin examples that do this? I could not find any.
Thanks!

Thank you for creating it @markhalonen !
We are not highly familiar with Julia language, but we are really open for new ideas and concepts. So feel free to create a PR if you wish!


Seems EOF is no longer used

I'm getting errors on SPSS 23 that are resolved if I remove the EOF opcode.
Attached a file that seems to open just fine, but then when trying to analyze it (for example **descriptives > frequencies **) gives:
Incomplete (truncated) SPSS Statistics data file: path

The attached data file contains 1 case, 2 variables. This is the smallest way I could reproduce the error.
sample.sav.zip
Would you accept a PR that removes the EOF opcode functionality?

Dove a little deeper and came up with #23.


Zigbee fan

Description:
Adds a ZigBee fan platform. The implementation is based on ZigBee Cluster Library Specification Revision 6 Draft Version 1.0. It has been tested with https://www.homedepot.com/p/Hampton-Bay-Universal-Wink-Enabled-White-Ceiling-Fan-Premier-Remote-Control-99432/206591100.
It depends on a couple of outstanding PRs in the underlying zigpy library:

https://github.com/zigpy/zigpy/pull/15
~~https://github.com/zigpy/zigpy/pull/16~~ (only necessary as a workaround for my particular fan)

So it can't be merged just yet.
Known issues:

Like all of the current zigbee platforms, it  does not configure reporting, so state changes triggered outside of home assistant do not propagate to home assistant. I think fixing this is out of scope for this PR.

Pull request in home-assistant.github.io with documentation (if applicable): https://github.com/home-assistant/home-assistant.github.io/pull/4626
Example entry for configuration.yaml (if applicable): N/A
Checklist:

[x] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[N/A] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[N/A] New dependencies are only imported inside functions that use them ([example][ex-import]).
[N/A] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Hi @igorbernstein,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @igorbernstein,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Seems like this broke with the release of the entity registry.

Looks great! I see that the linked PRs have been merged and we have also updated the zigpy dependency since then. Is this ok to merge?

(don't mind the WIP bot, it doesn't like that the first commit msg contains the word WIP)

I need to verify that this still works with the latest changes that @rcloran made. I should have time this weekend to rebase

Sorry for the delay, I haven't had a chance to revisit this. I'll try to do it tonight

0.66 is 👍
Thanks! 🎉

@igorbernstein2 can you please post bellows devices for you wink enabled Fan controller? interested to see what is the 'device_type'? Working on a thermostat implementation and apparently those may have "Fan Control" cluster as well.

@Adminiuga  sorry for the delay:
Device:
  NWK: 0xa0f9
  IEEE: 00:22:a3:00:00:16:c5:7d
  Endpoints:
    1: profile=0x104, device_type=14
      Input Clusters:
        Basic (0)
        Identify (3)
        Groups (4)
        Scenes (5)
        On/Off (6)
        Level control (8)
        Fan Control (514)
      Output Clusters:
        Identify (3)
        Ota (25)

How to you go about pairing this fan module?  I'm not having much luck.  thanks.


Internal error: merge job timed out after xxx seconds

Internal error: merge job timed out after xxx seconds error occurred. But other error looks correct.
This error appears v0.63.0~, but v0.62.0 works fine in my project (I can not provide this project because it is private...). I can not know how to reproduce this error.
Can I set merge timeout value to avoid timeout?

We had the same issue, you can set 'merge_timeout' either in your .flowconfig or as a command line flag. That helped for us :-)

@osmestad Thanks :) I'll try it!

For me, I found out the problem was that flow was getting stuck inferring the type of an accumulator in a reduce operation. It took a long time using trial & error to narrow it down, but once I added a type annotation to the accumulator in the reduce callback, the merge operation completed in a reasonable time. Hope that helps.

For us I suspect the types may have had a circular dependency, but I have not tried to reduce a test case of it (simplifying the code some seemed to help the speed, was only happening on the build server for us so painful to isolate). Looking at the Flow logs might also be useful, as that gives a bit more info than the console output :-)

I've the same problem. Is there any profiling option that I can use to narrow area of issue. I've large project and it took 34 seconds before. Now it takes ~ 1500 seconds.

I'm usually not a fan of "me too" posts, but well me too! Today the flow check stopped working for no apparent reason. I've already added type definitions to the recursive methods I've found but that didn't help. If I use flow check-contents it works but takes forever... I'm out of ideas on how I can fix that.

A same error occurred at 0.86 👀

Still causing at 0.87 👀

Still causing at 0.93

Reinstalling packages helped me in my case

One workaround is to add the offending path/file to [untyped] in .flowconfig, particularly if the issue is coming from within node_modules (otherwise, if in your source, you should try to fix the error!).
[untyped]
.*/node_modules/package-with-merge-timeout/filename.js

You can also add merge_timeout=N under [options] to make testing the fix a bit quicker. Set N to a value that is long enough that flow should complete without the the merge job timeout, but short enough you're not waiting forever. YMMV depending on size of your codebase.

There is not enough information to repro, and the issue is likely fixed a while ago.


2 problems, and a fix

I have 2 problems. (One fixed, one to go)
a) The download address in the code, doesn't work.
Solution: Change
CLAMAV_DOWNLOAD_LINK="http://nbtelecom.dl.sourceforge.net/project/clamav/clamav/$CLAMAV_VER/clamav-$CLAMAV_VER.tar.gz"
to
CLAMAV_DOWNLOAD_LINK="https://www.clamav.net/downloads/production/clamav-$CLAMAV_VER.tar.gz"
b) OpenSSL is not found on /usr (installed via brew, no longer included on 10.11)
I got this on the ./configure part (probably)
checking for OpenSSL installation... /usr   configure: error: OpenSSL not found.
Brew give this warning, it may help:

Apple has deprecated use of OpenSSL in favor of its own TLS and crypto libraries
Generally there are no consequences of this for you. If you build your
own software and it requires this formula, you'll need to add to your
build variables:
LDFLAGS:  -L/usr/local/opt/openssl/lib
CPPFLAGS: -I/usr/local/opt/openssl/include

So, after adding a
-with-openssl=/usr/local/opt/openssl to the ./configure command, it worked.
BTW: Yes, it's one dash, idk why it worked, maybe is case insensitive.

Thank you.  I will incorporate your fixes.  Sorry for the long delay.

It now builds and runs on Mojave.  I ended up having the install process build openssl from source, the way it does for clamav and fswatch, rather than introduce a dependency on brew.


Write function body and associated test(s): total_cumulative_deaths

Function body and associated test(s) need to be developed for CanadaCovidMetrics.total_cumulative_deaths.

Done


Pull k8s images before kubeadm init

Coverage increased (+0.04%) to 75.537% when pulling dda6f8f52d67c19f799c6f8c35ac709d83be32e5 on rastislavszabo:master into 9382819bced553b7f75a597cf0fe7a959d5442eb on contiv:master.


multiple background colors in one view

Hi!
I want to have many differently coloured background sections in one View. Is there a way to set color of a View part based on a Key word? I tried to do it in the renderRow() method in ListView, but it didn't override the background color set in TabVieExample _renderScene.
Like this:
https://i.stack.imgur.com/LkQEF.png
Thanks!

Inside your renderRow function you can check for something in rowData and set background color accordingly. You should post the question on StackOverflow to get more detailed answers which other people can also find easily.


Info about upcoming removal of packages in the General registry

As described in https://discourse.julialang.org/t/ann-plans-for-removing-packages-that-do-not-yet-support-1-0-from-the-general-registry/ we are planning on removing packages that do not support 1.0 from the General registry. This package has been detected to not support 1.0 and is thus slated to be removed. The removal of packages from the registry will happen approximately a month after this issue is open.
To transition to the new Pkg system using Project.toml, see https://github.com/JuliaRegistries/Registrator.jl#transitioning-from-require-to-projecttoml.
To then tag a new version of the package, see https://github.com/JuliaRegistries/Registrator.jl#via-the-github-app.
If you believe this package has erroneously been detected as not supporting 1.0 or have any other questions, don't hesitate to discuss it here or in the thread linked at the top of this post.

Thank you @KristofferC for the heads up. This package was mostly an exercise for myself back in 2014 to learn the ropes of package creation by wrapping up a neat little hack back when our ecosystem was less mature, thus I am closing this issue and opting for sun setting it at this point.


WIP: Add ServiceAcountName and kubernetesApiAutomation fields to EdgeConnect CRD

CRD
Description
Please include the following:

A meaningful PR title guidelines
A summary of the change
Which issue is fixed (if there is one)
Relevant motivation and context

How can this be tested?
Please include some guiding steps on how to test this change during a review.

What environment is necessary for the change to be noticeable ?
What needs to be enabled/applied for the change to be noticeable ?

Checklist

[x] Unit tests have been updated/added
[x] PR is labeled accordingly with a single label
[x] I have read and understood the contribution guidelines

Codecov Report
Attention: Patch coverage is 84.61538% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 67.33%. Comparing base (bbb96f4) to head (d7d6e0c).
Report is 1 commits behind head on main.

Files
Patch %
Lines

.../webhook/validation/edgeconnect/service_account.go
84.61%
1 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2977      +/-   ##
==========================================
+ Coverage   67.31%   67.33%   +0.01%
==========================================
  Files         262      263       +1
  Lines       14302    14315      +13
==========================================
+ Hits         9628     9639      +11
- Misses       3845     3846       +1
- Partials      829      830       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

could you please run:
make doc/api-ref

and commit files.

@amohrd, Can you please confirm that we want to use kubernetesApiAutomation in the CRD?
The corresponding feature in Dynatrace is called "Kubernetes Automation for Workflows" without the "API".

@amohrd, Can you please confirm that we want to use kubernetesApiAutomation in the CRD? The corresponding feature in Dynatrace is called "Kubernetes Automation for Workflows" without the "API".

@agrimmer
kubernetesAPIAutomation: I see it as a quite technical description, as the workflow actions use the API for command execution. kubernetesAutomation: I understand it as a more generic term, and better fits to "Kubernetes Automation for Workflows". Also the whole CRD already include various "API",  therefore I prefer kubernetesAutomation


go1.15 on windows (#12589)

We don't support go1.15 on 1.12.x branch, however this will allow users who chose to build with go1.15 on windows to be successful
backport #12589

🚀


Create Login View

Add the following for this:

Create a login form
Create a view
Make an entry to the application's URL file for login
Make an entry in the project's URL file (for accessing the application's URLs)
Create the base and login template


GraphQL Syntax Errors Aren't Recoverable Post Push

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v17.6.0
Amplify CLI Version
7.6.22
What operating system are you using?
Mac
Amplify Codegen Command
codegen models
Describe the bug
I generated a model schema and pushed to Amplify console. That push failed because of a syntax error. I've fixed that syntax error, regenerated the models but pushing again runs into the syntax issue even if that code doesn't exist anymore in the current version.
Expected behavior

Bad syntax error is caught before push. Bad schema syntax should not be allowed to attempt to compile.

Amplify push should update the current state of the app.

Reproduction steps

Create a Flutter Amplify project
Add graphql
Add a model

type Todo @model {
    id: ID!
    done: Boolean!
    due: AWSTimestamp!
}

amplify push
Modify GraphQL to have bad syntax

type Todo @model {
    id: ID!
    done: Boolean!   :
    due: AWSTimestamp!
}

amplify push --> fails
Modify GraphQL to fix syntax
amplify push
Observe that error still occurs

Attempted remediation strategies:

Destructive update

amplify push --allow-destructive-graphql-schema-updates

Push before pull

amplify pull
(fix syntax again)
amplify push

Manually add another change to 'break' cache
Push with force

amplify push --force

Update api to remove conflict resolution

amplify update api --allow-destructive-graphql-schema-updates
(remove conflict resolution)
(succeeds at update)
amplify push (fails with same error)

GraphQL schema(s)

# Put schemas below this line

type Todo @model {
    id: ID!
    done: Boolean!
    due: AWSTimestamp!
}

Log output

# Put your logs below this line

Syntax Error: Unexpected :

Additional information
I've run into this a few times now and the only reliable way to fix this is to completely teardown amplify and rebuild the entire application. Obviously this cannot work for production apps.
I've also found that S3 is storing the outdated schema in #current-cloud-backend/ (unfortunate that the name uses escaped characters because it breaks the default VSCode's ability to list the file) in a zipped folder. I've attempted to do some surgery and modify the amplify-meta.json directly but expectedly ran into issues where the project could no longer operate at all.

Also, interestingly, it doesn't appear that the changes are even picked up at all. I'm running amplify status -v and it doesn't list the GraphQL changes as part of the update.

I updated the latest patch problem and ran:
amplify codegen models
amplify status -v
amplify build
amplify push --force --allow-destructive-graphql-schema-updates

and result - finding a syntax error from a prior build.

I wonder if this is a better fit for the cli repo directly.


Revert "pinentry: drop gtk2"

This reverts commit edb8b7994c383d3754cbb03b2bdb6016026e19cc.
Description of changes
Revert the removal of the gtk2 flavour in #270266, since people are still using the gtk2 pinentry for reasons.
gtk2 still won't end up in the runtime closure of systems not using it, but something like https://github.com/NixOS/nixpkgs/pull/133542 might be a more principled approach that removes it from the build-time closure too.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

since people are still using the gtk2 pinentry for reasons.

Why would you want to still use it? It is since 3 years deprecated and there is no DE using it anymore.

It's the default in home-manager:

  services.gpg-agent.pinentryFlavor
      Which pinentry interface to use. If not null, it sets pinentry-program in gpg-agent.conf. Beware that pinentry-gnome3 may not work on non-Gnome systems. You can fix it by
      adding the following to your system configuration:

          services.dbus.packages = [ pkgs.gcr ];

      For this reason, the default is gtk2 for now.

      Type: null or one of “curses”, “tty”, “gtk2”, “emacs”, “gnome3”, “qt”

      Default: "gtk2"

      Example: "gnome3"

      Declared by:
          <home-manager/modules/services/gpg-agent.nix>

That's probably worth changing anyway, but in the meantime let's bring back the gtk2 pinentry.

It's the default in home-manager:

I am not very amused that we are held back by an out of tree project that is setting bad defaults and not properly forwarding the overrides.

I think this also affects programs.gnupg.agent.pinentryFlavor, which mentions gtk2 as a valid option. (Actually, also the code itself handles gtk2 as a valid option, only the type check throws this error:
      type = types.nullOr (types.enum pkgs.pinentry.flavors);

(I may look at outdated code, I have nixos-unstable checked out here, but then, the pinentry change is alsready on unstable).

@dschrempf I've opened #277221 to fix that. I hope I didn't mix things up.
The linked home-manager PR got merged, so we can close this, right?

Unless gtk2 itself is removed from nixpkgs I don't see why this shouldn't continue to be available

It is just no longer available as a default, you can still explicitly opt into this.
Also removing gtk2 in one go is near impossible and a pretty huge task, so we need to start somewhere and default options and dead software are a pretty good start.

if it is a bottleneck for builds then a separate derivation could be provided as a middle-ground solution instead.

I don't think we need to go that way. gpg pinentry is pretty small and pretty fast built.

It is just no longer available as a default, you can still explicitly opt into this.

No, #270266 broke this.


Use credentialName to specify credential resource name and support mTLS for external cert management at ingress gateway.

PR 780 updates API for the secret resource name, this PR implements functionalities at pilot and gateway agent side.
This PR also supports mTLS for external cert management at ingress gateway.
Fixes #11397, #9030 and #7976

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: JimmyCYJ
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS
pkg/OWNERS
security/OWNERS
tests/OWNERS
vendor/istio.io/api/OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio-unit-tests

prow/istio-pilot-multicluster-e2e.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio-pilot-multicluster-e2e

prow/e2e_pilotv2_auth_sds.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio_auth_sds_e2e

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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio-unit-tests

prow/istio-pilot-multicluster-e2e.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio-pilot-multicluster-e2e

prow/e2e_pilotv2_auth_sds.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio_auth_sds_e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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

/test istio-unit-tests
/test istio-pilot-e2e-envoyv2-v1alpha3

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio-pilot-multicluster-e2e

prow/e2e_pilotv2_auth_sds.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio_auth_sds_e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-unit-tests.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio-unit-tests

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: JimmyCYJ
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/e2e_pilotv2_auth_sds.sh
52ebaa67da42034faa68f9529107bc3a54ba7190
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
9ff72ec47d0763d4faf9de2ca343784f7d2a6b5c
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
9ff72ec47d0763d4faf9de2ca343784f7d2a6b5c
link
/test istio-integ-k8s-tests

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

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

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
9ff72ec47d0763d4faf9de2ca343784f7d2a6b5c
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
d7eed844fc9ee13f2619d6ced792ff355de8cb8d
link
/test istio-pilot-multicluster-e2e

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
d7eed844fc9ee13f2619d6ced792ff355de8cb8d
link
/test istio-pilot-multicluster-e2e

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

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

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

Test name
Commit
Details
Rerun command

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

prow/istio-pilot-multicluster-e2e.sh
d8c4d1af68b0ae99379a62f8b664fa83a9b8d173
link
/test istio-pilot-multicluster-e2e

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

Please undo the validation change.
Also, are you sure that this is compatible with Istio 1.0 proxies (esp the envoy config changes)?. If not, then it might need a guard for proxy version

Thanks for the review. The validation change is reverted, and I added a check for proxy version before creating Listener config for gateway.
Please take a look.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
d8c4d1af68b0ae99379a62f8b664fa83a9b8d173
link
/test istio-pilot-multicluster-e2e

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

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

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: JimmyCYJ, myidpt
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

Is this PR already adding support for autoamatic certificate reloading through e.g. cert-manager and Let's Encrypt?

Yes. @JimmyCYJ is working on the tutorial on istio.io.

Hi @muhlba91, the tutorial https://github.com/istio/istio.io/pull/3224 is under review. Thanks.

I want to configure a mTLS ingress gateway, but

TLS certs comes from cert-manager as TLS secret
but the root cert for the client side certs comes from my own PKI as Generic secret

I run helm-chart istio-1.1.0-rc.0 with --set gateways.istio-ingressgateway.sds.enabled=true and configure a Gateway with tls.credentialName: istio-ingressgateway-certs like:
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: topsecret
spec:
  selector:
    istio: ingressgateway-istio
  servers:
    - port:
        number: 443
        name: https-istio
        protocol: HTTPS
      tls:
        mode: MUTUAL
        credentialName: istio-ingressgateway-certs

The I configure a certificate with secretName: istio-ingressgateway-certs like:
apiVersion: certmanager.k8s.io/v1alpha1
kind: Certificate
metadata:
  name: topsecret
  namespace: istio-system
spec:
  secretName: istio-ingressgateway-certs

and a separate generic secret with name: istio-ingressgateway-certs-cacert and the key cacert which contains the root-cert for the client side certs from my own pki like:
apiVersion: v1
kind: Secret
metadata:
  name: istio-ingressgateway-certs-cacert
  namespace: istio-system
type: Opaque
data:
  cacert: {...}

But the ingress-sds container throws following error: cannot find secret istio-ingressgateway-certs-cacert for ingress gateway.
When I look at secretfetcher then there is a delete statement for the cacert secret.  https://github.com/istio/istio/blob/0e9cecab053aab744a7c3a731aacb07fd794d5f9/security/pkg/nodeagent/secretfetcher/secretfetcher.go#L260-L261
But I need the possibility to define the root-cert for the client side in a separate generic secret due to cert-manager generates only TLS-secrets.
Any Idea?

Hi @thomschke, only one secret with secretName: istio-ingressgateway-certs is needed by ingress gateway agent. And this secret should contain key, server cert, and root cert for MUTUAL mode. The ingress gateway agent sends key and server cert in response to SDS request with resource name "istio-ingressgateway-certs", and sends root cert in response to SDS request with resource name "istio-ingressgateway-certs-cacert"

Hi @JimmyCYJ, thanks for your prompt reply.
Cert-Manager generates only TLS secrets with key and server cert. So currently I have to manually merge this TLS secret into a generic secret together with my root cert. That's bad.
If secretfetcher will find a TLS secret then it can look for a responsible generic secret named with suffix IngressGatewaySdsCaSuffix and can merge the root-cert (instead of returning nothing):
https://github.com/istio/istio/blob/0e9cecab053aab744a7c3a731aacb07fd794d5f9/security/pkg/nodeagent/secretfetcher/secretfetcher.go#L152
What do you think?

@thomschke Thanks for providing more context. Is it possible to let cert-manager generate secret with key, server cert and root cert? Changing the ingress gateway agent to extract root cert by scanning secret name suffix is a kind of hack. This affects user experience and could mess up user setup. It would make more sense to let cert-manager provide an option to specify root cert.
cc @myidpt

Hi @JimmyCYJ,
Cert-Manager only create TLS Secrets, which are a kind of Secret that only contains a crt and key file and far and widely no help in sight

https://medium.com/@prune998/istio-1-0-2-envoy-cert-manager-lets-encrypt-for-tls-certificate-merge-7a774bff66c2
https://github.com/jetstack/cert-manager/issues/675#issuecomment-411816455

Yes, at first glance my FR looks like a hack. But in my opinion it's the only way to support auto-reloading for mTLS gateways in the moment.
BTW: Istio helm chart bundles cert-manager. Maybe you have more persuasiveness when you talk to them :-)

@JimmyCYJ you just found the biggest issue with Istio and Cert-Manager !
Your solution is to pre-create the secret istio-ingressgateway-certs as a plain Secret (not a TLS one), put your CA inside it, then push your Certificate resource.
Cert-Manager should add the key/CRT to your secret.
I won't rant again. Both Istio (Galley ?) and Cert-Manager need to do something to ease the creation and management of those certificates. I had a strong No on the Cert-Manager side, so far...
One thing you could do is create a new issue and explain your issue... maybe at some point someone will understand we need a change to support SSL certs from Cert-Manager in Istio...

Hi @prune998 @thomschke, thanks for all the context. I just create a feature request https://github.com/istio/istio/issues/12132, please feel free to add comments there. That would be easier for us to keep tracking of the issue.

@JimmyCYJ Thanks!
BTW: CRD validation has to change -->  If credentialName is specified then privateKey, serverCertificate and caCertificates are no longer required.

@thomschke That validation change is in https://github.com/istio/istio/pull/11991, you don't need to add privateKey, serverCertificate, and caCertificate if you have specified credentialName.

@JimmyCYJ GJ


cmake: Set BOARD_ROOT from environment

To ease building off-tree boards,
if BOARD_ROOT is not set from command line but set in the
environment, take that value.
Otherwise assume ZEPHYR_BASE as before.

Codecov Report

Merging #8550 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8550   +/-   ##
=======================================
  Coverage   63.92%   63.92%
=======================================
  Files         427      427
  Lines       41100    41100
  Branches     6939     6939
=======================================
  Hits        26275    26275
  Misses      11678    11678
  Partials     3147     3147

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 daef3cc...7621572. Read the comment docs.

The precedence between cmake variable and environment variable must be the same for BOARD and BOARD_ROOT.

Ok, I just took the logic from BOARD and reused it directly for BOARD_ROOT, with a change to have the ZEPHYR_BASE default.

This is too much duplicated code just to allow configuring a single variable through an environment variable.
Either this needs to be refactored, e.g. through the use of macros. Or we need to find a better solution.
E.g. a mechanism from https://github.com/zephyrproject-rtos/zephyr/pull/1380 might also be able to "ease building off-tree boards". I assume you want BOARD_ROOT as an environment variable because you already have BOARD in an environment variable and it makes sense for those two variables to be maintained in the same, or perhaps a set of config files.
This same use-case I believe can be covered by the ideas in #1380. So please, either refactor this solution, investigate if #1380 would apply for your use-case, or elaborate on your use-case so that I can find the best solution here.

Thanks @SebastianBoe . Yes it is a lot of code. The original PR was much simpler and did what I needed https://github.com/aescolar/zephyr/commit/7621572b5f619af3d283bfbe2d3003544c695352   ;)
I took a look at that old PR 1380, but I can't see anything in that discussion that would help me. My use case is roughly:

set the BOARD_ROOT and BOARD environment variables,
Run a script which will call other scripts which will eventually invoke cmake (in batch).
unset those environment variables

I would like to avoid creating extra files, and I want to avoid needing to modify those scripts.
I could imagine other people may have other use cases to set the BOARD_ROOT from the environment like they already can with BOARD.

This use-case can be covered by having the script accept CLI arguments that are passed verbatim to cmake.
Until we find a way to refactor BOARD and BOARD_ROOT this will have to do.


Databse error when trying to edit the wiki

Iain, i had to restore the db. If you made any channges from this Sunday(28th) let me know and il recover them for you. Anything before that should be fine. Hoping i did not forget to move any of my changes across..

Iain, i had to restore the db. If you made any channges from this Sunday(28th) let me know and il recover them for you. Anything before that should be fine. Hoping i did not forget to move any of my changes across..


Move config subscriptionInitialPosition from parameter into ConfigurationData

Motivation
Seems subscriptionInitialPosition was not passed down success, and There was some redundant to config subscriptionInitialPosition by both parameter and ConfigurationData. This PR fixes these issues.
Modifications

fix subscriptionInitialPosition pass down, and redundant config issue.
add e-2-e test.

Result
issues fixed.

retest this please


Build pure product, allowed mixed, native mixed and JMV mixed tests i…

…n parallel

It looks like the state of the tree and local Maven repo is not propagated to the subsequent stages by default. Looks like we need to stash/unstash explicitly https://www.jenkins.io/doc/pipeline/steps/workflow-basic-steps/#stash-stash-some-files-to-be-used-later-in-the-build

@llowinge this is the idea. All tests are now linked from product/*/group-* modules. Tests are assigned to them in a round robin fashion with target group count 10 (configurable). Let's see how well it's going to work here.

Native mode is TBD


make CollectionUtilities internal

You should not extend types you dont own
also CollectionUtilities.IsNullOrEmpty  is not a safe general purpose method. if the IEnumerable is a forward only approach, then this method will consume the first value effectively creating a bug.

Fixes #1722.

@jmprieur @SimonCropp @Dreamescaper This is a breaking change.

yes it is a breaking change


[firebase_auth]: Flutter Windows cannot sign in with provider

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Auth
Which platforms are affected?
Windows
Description
Unable to sign in with provider for Flutter Windows. I tried to use MicrosoftAuthProvider to sign in with FirebaseAuth but I am getting this error: [firebase_auth/unknown-error] Operation is not supported on non-mobile systems.
However this works on Flutter android and ios.
I am using firebase_auth version 5.2.0.
Reproducing the issue
MicrosoftAuthProvider microsoftAuthProvider = MicrosoftAuthProvider();
microsoftAuthProvider.setCustomParameters({
        "tenant": <tenant-id>,
      });
UserCredential user =
          await FirebaseAuth.instance.signInWithProvider(microsoftAuthProvider);

Firebase Core version
3.4.0
Flutter Version
3.22.3
Relevant Log Output
[firebase_auth/unknown-error] Operation is not supported on non-mobile systems.

Flutter dependencies

Expand Flutter dependencies snippet

dependencies:
- cloud_firestore 5.3.0 [cloud_firestore_platform_interface cloud_firestore_web collection firebase_core firebase_core_platform_interface flutter meta]
- cupertino_icons 1.0.8
- date_time_picker 2.1.0 [flutter intl]
- dropdown_plus 0.0.9 [flutter]
- dropdown_search 5.0.6 [flutter]
- expandable 5.0.1 [flutter]
- file_picker 8.0.7 [flutter flutter_web_plugins flutter_plugin_android_lifecycle plugin_platform_interface ffi path win32 cross_file web]
- firebase_auth 5.2.0 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 3.4.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_storage 12.2.0 [firebase_core firebase_core_platform_interface firebase_storage_platform_interface firebase_storage_web flutter]
- flutter 0.0.0 [characters collection material_color_utilities meta vector_math sky_engine]
- flutter_launcher_icons 0.13.1 [args checked_yaml cli_util image json_annotation path yaml]
- flutter_signature_pad 3.0.1 [flutter]
- flutter_toggle_tab 1.4.1 [flutter]
- get 4.6.6 [flutter]
- hive 2.2.3 [meta crypto]
- hive_flutter 1.1.0 [flutter hive path_provider path]
- image_picker 1.1.2 [flutter image_picker_android image_picker_for_web image_picker_ios image_picker_linux image_picker_macos image_picker_platform_interface image_picker_windows]
- intl 0.17.0 [clock path]
- mailer 6.1.2 [async logging intl mime path meta]
- path_provider 2.1.4 [flutter path_provider_android path_provider_foundation path_provider_linux path_provider_platform_interface path_provider_windows]
- pdfrx 1.0.83 [collection crypto ffi flutter http path path_provider rxdart synchronized url_launcher vector_math web]
- sliding_up_panel 2.0.0+1 [flutter]
- table_calendar 3.0.8 [flutter intl simple_gesture_detector]
- timezone 0.9.4 [path]
- url_launcher 6.3.0 [flutter url_launcher_android url_launcher_ios url_launcher_linux url_launcher_macos url_launcher_platform_interface url_launcher_web url_launcher_windows]

Additional context and comments
No response

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hello @jingfeng-cheah, as specified by the error, this is not supported by the C++ SDK. You can request this feature on the CPP repository: https://github.com/firebase/firebase-cpp-sdk


1.23: SD-4095 finalize change notes for C/C++

@Semmle/cpp - I've made some small changes to the text in the first commit. The second commit just fixes the sort order in the first table.
I revised the text describing Configuration::hasPartialFlow so that it matched the bullet point in the C# change notes (see #2430). Happy to make changes in either/both PRs but it makes sense for the description to be shared text.

My apologies - I've got my self confused. Will amend that last commit after my next meeting 😞

I think that the change notes file should now contain the correct wording.


I am getting a message like this: "An error occurred. Please try again later.(Playback ID: W.....cC)"

I am passing video id but still getting this error on playing video using this library. while using that video id video is playing in official youtube player. So need to remove this library from my project

The same happens with me :/

I noticed it happens with some of videos only. 90% of videos are pretty good

@m-akopov can you please confirm that it happens only when playing programmatically? It seems very strange to me.
My understanding of this issue is that YouTube does not allow certain videos to be played on an embedded player.

@PierfrancescoSoffritti
Yes I confirm. From XML is fine. I tried everything, even set ReadyListener after 2000 milliseconds :D and then onReady cueVideo, or loadVideo. But without success.
Now I have  and embedded video in iFrame with HTML and the same problematic videos(and all others) work fine.

Can you send me a few ids of videos that have this problem?

Sure, here are videos:
https://youtu.be/Dzbb0OWnl2A
https://youtu.be/X1Jqv1bESS8

I was having the same issue as @m-akopov but I think I found a way to solve it. Here's how I'm setting the videoID programmatically:
    private fun setupVideo(vidId: String) {
        val youtubePlayerView: YouTubePlayerView = binding.resourceVideo
        lifecycle.addObserver(youtubePlayerView)
        youtubePlayerView.enterFullScreen()
        youtubePlayerView.addYouTubePlayerListener(object : AbstractYouTubePlayerListener() {
            override fun onReady(youTubePlayer: YouTubePlayer) {
                youTubePlayer.loadVideo(vidId, 0f)
                youTubePlayer.play()
            }
        })
    }

On the XML side I removed the autoPlay and videoId properties.

In my case, this error was when I tried to use youtube links instead of video Id.

i solve this issue just removing autoplay and viodeo id from xml file


Makefile fixes and building for darwin/arm64

Fixups in Makefile:

set plugin version based on the newest git tag starting with v
set OS and ARCH based on the system on which the code is built
set top-level name for local installation of the plugin to kentik in order to make usage of local installation of the plugin easier

Goreleaser:

enable building darwin/arm64 version of the plugin

I've addressed the review comment. Please, take another peek.


Unexpectedly found nil while unwrapping an optional value

I've found that on the swift-2.0 branch, the PKHUDAssets class causes the fatal error on line 20 when returning an image from the bundle. I'm not sure why, I copied the entire PKHUD folder into my project so I have Images.xcassets and everything. I've checked and the bundle for the PKHUDAssests class is being returned correctly (I think). But for some reason the UIImage is returned as nil..

I fixed the issue. The Info.plist had a blank "Principal Class" Value. I set it to PKHUDAssets and now everything works fine


elmr config logout fails

If I go to http://localhost/auth/elmr/config then press logout (http://localhost/auth/elmr/session?mode=logout) I receive the following error in the browser:
HTTP Status 500 – Internal Server Error

Type Exception Report

Description The server encountered an unexpected condition that prevented it from fulfilling the request.

Exception

java.lang.NullPointerException
	edu.illinois.techservices.elmr.servlets.SessionServlet.sessionDestroyed(SessionServlet.java:139)
	edu.illinois.techservices.elmr.servlets.SessionServlet.service(SessionServlet.java:41)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:741)

Note The full stack trace of the root cause is available in the server logs.
Apache Tomcat/9.0.11

See referenced commit above. This is ready to be pulled and tested again.

Logout now works as expected in the browser, but one of the tests that I wrote before this change is now failing. Specifically, if I preform a curl against elmr logout without a cookie I receive a 302, before I received a 500. Not sure what you think the correct behavior should be in this case. Maybe a 400 or a 401? Though I can also see the point of just redirecting to shib to make sure the shib cookies are killed off.
$ curl -I  http://<IP_ADDRESS>/auth/elmr/session?mode=logout
HTTP/1.1 302 302
Server: nginx/1.14.0
Date: Wed, 31 Oct 2018 20:51:01 GMT
Connection: keep-alive
Location: http://<IP_ADDRESS>/auth/Shibboleth.sso/Logout

The 302 status is what should have been happening all along. The 500 status with this test was a manifestation of the error that 93a5baa addresses; it was the unexpected result. If you change your test to assert the 302 status we can close this issue.

Done! Thanks.

Done. Thank!


[CUBRID] Display Partition Tables

Description
Currently, Partition Table is display as normal Table.
Partition table should not be shown in the navigation list and should be included in the normal Table instead.
Solution
Make a folder Partitions under table and display all partitions which is related to each table.

Thank you for request


[DOCFIX] Cherry-pick: Correct documentation on HDFS and secure HDFS (#7521)

Correct documentation on HDFS and secure HDFS

Futher improvement

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20505/
Test PASSed.


Fix Issue #126, implemented get_email

Can one of the admins verify this patch?

[test]

Also, there is a test for get_username method. You can use something similar and simple as well.

In the case of Pagure, we are not using any library, but calling the API on our own. Look at the other methods, how they are implemented in ogr/services/github.py. The API documentation of pagure can be seen e.g. at https://src.fedoraproject.org/api/0/

Just a small correction, here should be probably ogr/services/pagure.py instead of  ogr/services/github.py.

@lachmanfrantisek @marusinm Thank you! I will get it fixed today.

I cannot make check locally on either Windows or Mac OS machine; I think it has to be a Fedora distro. Sorry lads, tried my best :(

I cannot make check locally on either Windows or Mac OS machine; I think it has to be a Fedora distro.

It should not be dependant on the Fedora. I've described the testing in the review which can solve the problem.
In the case of pre-commit, you can find better log output here.
It's a group of static-analysis tools/checks and it finds some problems in your code..;-)

Two things can be fixed by my suggestions from review.
The missing problem is ogr/services/pagure.py:840:5: E303 too many blank lines (2)

Sorry for the late response, I just saw this, I will be working on it! Thank you for being patient with me!!

Sorry for the late response, I just saw this, I will be working on it! Thank you for being patient with me!!

No problem, feel free to ask if you need some help..;-)

@yzhang2907 thank you for your contribution! You commits were included in the follow-up PR from @marusinm which has been merged recently.


Fix main menu current item underline

Fix the main menu to show the correct active item
Add ".idea" for .gitignore file
Replacing closing tag with self closing

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.andris.svarcs seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Fix the main menu to show the correct active item
Add ".idea" for .gitignore file
Replacing closing tag with self closing

Thanks this looks great! Merging today.

Also, I would like to take an opportunity and mention that this is my very first contribution to Open Source. :)


Integrate freq_measure library, general cleanup

Full changelog:

Integrate the frequency measuring library (FreqMeasure)
Update lib-common
Fix several small bugs; ensure the project compiles
Create a permanent ./build directory, using git tricks

@vogeldylan @shimismith please review


Authenticator Icon

Is your feature request related to a bug/problem? If so, please link to the relevant issue or describe it
No
Describe the solution you'd like
Is there a way to change the default icon the authenticator displays in apps such as Authy?
Describe alternatives you've considered
Additional context
https://discord.com/channels/416652224505184276/606541782607724560/919755677486960661

Not possible as far as I could tell


ROX-10097: Configure dependabot to watch the docs/Dockerfile

Description
Dependabot configuration to watch the docs/Dockerfile.
Checklist

[ ] Investigated and inspected CI test results
[ ] ~Unit test and regression tests added~
[ ] ~Evaluated and added CHANGELOG entry if required~
[ ] ~Determined and documented upgrade steps~
[ ] ~Documented user facing changes (create PR based on stackrox/openshift-docs and merge into rhacs-docs)~

Testing Performed
Will see the GitHub reaction.

Does it make sens when we use latest image?
https://github.com/stackrox/stackrox/blob/5544c7d5ce8593cdf9495c36811f96ef5a92b200/docs/Dockerfile#L13

Does it make sens when we use latest image?

I dunno. I'd prefer having 120 as a part of the tag, and I wonder, no, I rather doubt that dependabot would bump nginx-120 to nginx-121.

If we use official nginx image then it will work as expected


Users can export citations to reuse them

Most of this epic story has already been completed for the Aggie Experts pilot in 2022. The one thing that is missing is that when the user exports citations, their dates of epublication, official publication or any other date are combined. That results in an import problem for MIV.
After discussing the issue with faculty and the MIV team, the date listed in the citation should be the official publication date.
The RIS file allows reuse of the citation data in Zotero.
BibTex is a citation format that was also requested at one of the faculty meetings at the College of Engineering.

[x] #39
[ ] #50

Previously conference papers were not exported with publication dates, which made them rejected in MIV.


Add sweeps to runs in public API

Also

Add unit tests
Add manual test scripts
Warn about WANDB_* environment variables when running unit tests
Stop silently clearing WANDB_* environment variables (in tests)
Refactor graphql mocking to make it easier (for me) to understand
Add docstrings

WB-568

Test failures look unrelated.

Test failures were in fact related but it was hard to see because of flakey unit tests.
I had changed a test fixture that silently unsets any WANDB_ environment variables before running any unit test. Sometimes it's handy to set things like WANDB_DEBUG=True to help debug things. This must have exposed some environment variable leakage from other unit tests.
I added another fixture to save and restore environment variables before / after each unit test. Everything's passing now.


update docstrings for df batch4

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[x] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #
Closes #

Checklist

[ ] Code follows style guide (ruff)
[ ] Tests added
[x] Documented the changes

If you have comments or can explain your changes, please do so below.

thanks
there's still some things, but i'll push a commit to this branch, don't worry about it, feel free to start a different task

closing then, as this has been taken care of in other prs - thanks for your pr! going strong on the left join one 💪


Is it still considered "active" if I set the minimum and maximum upload speeds to 0?

Hello. New user here.
Very recently had my dedicated seeding drive fail and I don't think I can replace it any time soon. Found this amazing tool and desperate enough to try it out but I'm not looking to spoof my ratio. I'm a bit too paranoid about that for now. Lol. But I do want to continue increasing my seedtime for the torrents I have been seeding until I can buy a new drive.
I currently have a torrent loaded from a public tracker to test it and my settings are set to 0 minimum and maximum upload speed. But in the dashboard, it shows Torrents in queue: 0 which makes me think it's not considered as currently active?
Again, since I just found out about this a couple of hours ago, I'm still a bit paranoid. So not personally ready to test it on an actual private tracker torrent yet. I hope it's okay to ask this here. Thank you.
I'll add photos of my dashboard and settings below:

Hello, cramberries,
First, your setup is perfectly fine. Having a zeroed max speed is fine to
use.
The "queued torrents" will start to increase if you had more torrent than
the "simultaneous seed" config. Queued torrents are basically in queue,
inactive.
When you'll start using it on real tracker please adapt the "torrent
client" config to latch you real client. You don't need to match the exact
same version but at least match the client (qbittorrent, utorrent, ...)
Le lun. 7 oct. 2024, 22:22, cramberries @.***> a écrit :

Hello. New user here.
Very recently had my dedicated seeding drive fail and I don't think I can
replace it any time soon. Found this amazing tool and desperate enough to
try it out but I'm not looking to spoof my ratio. I'm a bit too paranoid
about that for now. Lol. But I do want to continue increasing my seedtime
for the torrents I have been seeding until I can buy a new drive.
I currently have a torrent loaded from a public tracker to test it and my
settings are set to 0 minimum and maximum upload speed. But in the
dashboard, it shows Torrents in queue: 0 which makes me think it's not
considered as currently active?
Again, since I just found out about this a couple of hours ago, I'm still
a bit paranoid. So not personally ready to test it on an actual private
tracker torrent yet. I hope it's okay to ask this here. Thank you.
I'll add photos of my dashboard and settings below:
image.png (view on web)
https://github.com/user-attachments/assets/9a04e8df-476b-4053-9899-b67fcfe2227d
image.png (view on web)
https://github.com/user-attachments/assets/b294f0eb-cdbd-45ab-b751-d775684d5e17
—
Reply to this email directly, view it on GitHub
https://github.com/anthonyraymond/joal-desktop/issues/113, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZH5YKS3GGAECIVESEA5L3Z2LUP3AVCNFSM6AAAAABPQWXWWCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGU3TCMZZGM4TSNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@anthonyraymond
Thank you so much for the quick response and I apologise for the delay of mine. After reading your reply, I decided to test it on a general tracker I'm in but it only updates every 30 minutes so I decided to wait until then before responding. I can confirm it is working great for me and it's counting up the seed time without increasing my upload.
Thank you so much for creating this tool.  Lol. Thank you again!

@anthonyraymond
I hope you don't mind me coming back in and just asking this question in this thread. But would it be possible to set a specific port? I cannot see anything in the settings for that.
I have a particular one that I use for a specific private tracker and it hasn't changed in years so that might be noticeable if moderators are checking that sort of thing. Thank you again!

@anthonyraymond
Thank you. I was looking at the main JOAL repo and saw there's info for specifying the port so I thought this would have one as well. But I couldn't ID a file that looks like a config file in the directory so I asked.
Sorry for bothering you about it but I'm looking forward to the next iteration. Thank you!


Fix effect drifting

Description
The effects were drifting slightly, barely noticeably except for very long runs.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:
If something is unclear, please submit the PR anyways and ask about what you thought was unclear.

[x] This branch is based on develop
[x] This PR is targeted to merge into develop (not master)
[x] I have added an entry under [next] in CHANGELOG.md
[x] I have formatted my code with flutter format
[x] The continuous integration (CI) is passing

LGTM
We should start to add some GIFs and Images to make it clear what we are changing, sometimes is I find it hard to visualize the change only looking on the code, what do you think?

Sounds like a very good idea, how are you creating the gifs you put on your PRs?

LGTM
We should start to add some GIFs and Images to make it clear what we are changing, sometimes is I find it hard to visualize the change only looking on the code, what do you think?

Sounds like a very good idea, how are you creating the gifs you put on your PRs?

To record the android screen I use adb screen record command, and to record iOS screen, I just run on the simulator, and record the screen from the notebook. Then I this site to convert the video to gif, since github only support gif.

LGTM, I'm not sure I understand the drift but the code looks fine.

When an effect hits the edges, either 0.0 or 1.0 progress, it will have tiny bit left of the delta time as overflow and that overflow needs to be caried over to what is happening next.


Basic application structure for Flask app  (#5)

Basic application structure for Flask app

Please merge and rebase


[Feature Request]: StateBackedIterable should support lazy observing of element size by extending ElementByteSizeObservableIterable

What would you like to happen?
As it does not, Iterables.registerByteObserver triggers element iteration and prefetching/reading. Otherwise byte observing would occur lazily during iteration.
Issue Priority
Priority: 2 (default / most feature requests should be filed as P2)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

Renaming to reflect the end goal.  I was looking into doing this at the Prefetchables level but it was difficult given overridden methods having colliding types for ElementByteSizeObservableIterable and PrefectIterable.
Planning instead of just fixing up StateBackedIterable and possibly exposing a generic way to wrap an iterable to make it cheap to byte size observe.


Remove unsupported paginator

Caught by downstream CLI tests: https://github.com/aws/aws-cli/blob/develop/tests/functional/logs/test_get_log_events.py
We will probably will have to stop paginating on repeated next tokens.

Codecov Report

:exclamation: No coverage uploaded for pull request head (log-paginator@566f1f8).

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 a9b5115...566f1f8. Read the comment docs.


Error with nesterov?

Fitting a simple mlp with learning_rule = nesterov returns an error consistently (with sgd there is no error). Not sure if this is enough information or not.
/home/anaconda/lib/python2.7/site-packages/IPython/kernel/main.py:10: SettingWithCopyWarning: A value is trying to be set on a copy of a slice from a DataFrame
/home/anaconda/lib/python2.7/site-packages/IPython/kernel/main.py:11: SettingWithCopyWarning: A value is trying to be set on a copy of a slice from a DataFrame

RuntimeError                              Traceback (most recent call last)
 in ()
35     nn = Classifier(layers=[Layer("Rectifier", units=100),Layer("Softmax", units =4)], learning_rate=0.01, n_iter=10, learning_rule = 'nesterov')
36
---> 37     nn.fit(train_X, train_y)
38
39     p=nn.predict(test_X)
/home/anaconda/lib/python2.7/site-packages/scikit-neuralnetwork/sknn/mlp.py in fit(self, X, y)
910         yp = self.label_binarizer.transform(y)
911         # Now train based on a problem transformed into regression.
--> 912         return super(Classifier, self)._fit(X, yp, test=y)
913
914     def partial_fit(self, X, y, classes=None):
/home/anaconda/lib/python2.7/site-packages/scikit-neuralnetwork/sknn/mlp.py in _fit(self, X, y, test)
788         for i in itertools.count(1):
789             start = time.time()
--> 790             self.trainer.train(dataset=self.ds)
791
792             self.mlp.monitor.report_epoch()
/home/src/package/pylearn2/training_algorithms/sgd.py in train(self, dataset)
468             value = param.get_value(borrow=True)
469             if not isfinite(value):
--> 470                 raise RuntimeError("NaN in " + param.name)
471
472     def continue_learning(self, model):
RuntimeError: NaN in hidden0_W

Try dropping your learning rate down, by 10x or more. NaN happens with any learning rule if the gradient descent diverges.  If this is the problem I will add a better error message to explain this...

That did it, dropping the learning rate to 0.001 worked. Thanks! (one additional note is that the docs list a parameter called 'dropout' but it appears to really be 'dropout_rate').

If you use PIP, it's still dropout from version 0.1. dropout_rate is in the latest from source and not yet released.  I'll check the versioning of the documentation!  Thanks.

About to merge a fix for this that explains the solution... Renamed the issue so it's easier to search/find.  Thanks for reporting it!


Clean up manifest apply output

This cleans up a bunch of output that should only go to the log, not be visible to the user during normal operation.

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

Test name
Commit
Details
Rerun command

e2e_operator
a9c167c673e295168bd2482c87f56c4035e834ac
link
/test e2e_operator

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

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

Test name
Commit
Details
Rerun command

e2e_operator
9955eab6e6f61ff2dc6508f4fad9457a9bf67e59
link
/test e2e_operator

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

In response to a cherrypick label: #457 failed to apply on top of branch "release-1.4":
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pkg/kubectlcmd/client.go
M	pkg/manifest/installer.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/manifest/installer.go
Auto-merging pkg/kubectlcmd/client.go
CONFLICT (content): Merge conflict in pkg/kubectlcmd/client.go
Patch failed at 0001 Clean up manifest apply output


make 0-rtt configurable

In case folks want to do it for non-idempotent requests, etc.

Can you clarify a bit more about what the config knob should guard? Allowing non-safe requests to be sent over 0-RTT?

This was fixed in #20242 and #20167


Ionic Keyboard - keyboard.hide() not working on iOS device.

Bug Report
Ionic Info
Ionic:

	ionic (Ionic CLI)          : 4.1.2
	Ionic Framework            : @ionic/angular 4.0.0-beta.15
	@angular-devkit/core       : 0.7.5
	@angular-devkit/schematics : 0.7.5
	@angular/cli               : 6.1.0
	@ionic/ng-toolkit          : 1.0.7
	@ionic/schematics-angular  : 1.0.5

Cordova:

  cordova (Cordova CLI) : 8.0.0
  Cordova Platforms     : ios 4.5.5
  Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 2.2.3, (and 10 other plugins)

System:

  ios-deploy : 1.9.2
  NodeJS     : v8.9.3 (/usr/local/bin/node)
  npm        : 6.4.1
  OS         : macOS
  Xcode      : Xcode 10.0 Build version 10A255

Describe the Bug
On a text input field, the keyboard does not hide when user hits enter key on device.
Related Code
	- Template .html

	<ion-input
		formControlName="name"
		autocapitalize="off"
		autocomplete="off"
		type="text"
		(keyup.enter)="hideKeyboard()">
       </ion-input>

	- Component .ts

	hideKeyboard() {
           this.keyboard.hide();
        }

Expected Behavior
The keyboard should close/hide when user hits enter key.

Mh, I think this is hard to implement because keyboard is a native cordova plugin, so it cannot be integrated (from my point of view) into Ionic directly. So i think you have to make it manually (write a service for example).

It can, I'm using the above, too.
@samachola did you include the keyboard this way:
import { Keyboard } from '@ionic-native/keyboard/ngx';
Another thought, perhaps you want to apply a slight setTimeout() delay to your this.keyboard.hide();  call - could be a timing issue?

Ionic native plugins cannot be added to the core / angular package of ionic, so I close it here because such a feature you have to include by yourself like @olivermuc described


[big refactor]mmlu is needed

Where can i find mmlu benchmark in big refactor branch?
I have implement some kind of rwkv world benchmark adapted from huggingface model,
https://github.com/xiaol/rwkv-hf-lm-evaluation-harness
not so sure how to proceed to improve the benchmark result based on the evaluation code.

Hi! I believe you're looking for this


targeting android 11: storage

https://developer.android.com/about/versions/11/privacy/storage
https://developer.android.com/training/data-storage/manage-all-files

Done.


Add a user manual to the website

Adding a user manual will help the users understand the flow of the website and working of the website
I would like to work on this issue can you please assign me @swapnilsparsh @sakshikhachane

Adding a user manual will help the users understand the flow of the website and working of the website
I would like to work on this issue can you please assign me @swapnilsparsh @sakshikhachane

Can you explain what type of user manual or any website which is having a user manual ?

We can have it like this in our home page
or we can even create a separate like this  https://summerofcode.withgoogle.com/how-it-works/
here we can create a detailed explanation https://prod-cdn.preprod.co-vin.in/assets/pdf/user-guide-citizen-registration-ver-1.2.pdf  like this which is available in cowin website on clicking know how to book our appoinment

in our case we can give detailed explanation of the working of emergency messages and calls how we can create contacts step by step

@coding-geek21
Ok, work on it.

@coding-geek21 Any updates ?

Yes half completed I will make PR tomorrow


feat(CategoryTheory/Sites): constructors for Mayer-Vietoris squares

In this PR, we give constructors for Mayer-Vietoris squares, which allows the construction of these squares in the case of topological spaces.

[x] depends on: #14959
[x] depends on: #14957
[x] depends on: #14951
[x] depends on: #14890
[x] depends on: #14935
[x] depends on: #14936

Thanks for the review!
bors merge


Spirit name still being random even after you name dictionary of spirits

Describe the bug
Spirit name will remain random even after you name the dictionary of spirits in a anvil wich from my understanding should't be the case
To Reproduce
Steps to reproduce the behavior:

name dictionary of spirits in an avill (example:jeff)
combine the named dictionary of spirits in a crafting with any unbound binding book (example:book of binding foliot)
you will see that the crafting output to will say that the outputed bound book is spose to have a spirt name that is the same as the name of the dictionary of spirits (example:jeff)
craft the bound book
look at the spirit name it will still have been randomised

Expected behavior
the bound books spirit name is the same as the name of the dictionary of spirits (example:jeff)
System (please complete the following information):

Occultism Version: 1.136.1
OS: Windows
Minecraft Version: 1.20.1
list of mods:
architecture 9.2.14
cupboard 2.7
curios 5.9.1
embeddium 0.3.30
geckolib 4.4.7
gpumemleakfix 1.8
jei <IP_ADDRESS>
kubejs 2001.6.5-build.14
memorysettings 5.5
modonomicon 1.74.1
occultism 1.136.1
probejs 6.0.1
rhino 2001.2.3-build.6
saturn 0.1.3
SmartBrainLib 1.15
smoothboot(reloaded) 0.0.4

Additional context
needed for a modpack

An initial test does not reproduce this (the resulting book has the same name as the renamed dictionary), I will test further to see if I can repro it.
@oncer56 is this in SP or on a server?

I did a quick test in ATM 10 and could not reproduce the issue there either, does it still happen for you on latest occultism?

its SP.
after testing it, it works fine on the 1.21.1 version.
also tested it with just occultism and its dependecies on the 1.136.2 version of occultism and the the name is still random even after i name the  dictionary of spirits so it does not look like comptibilty issue

Mea culpa - I overlooked that the issue was on 1.20.1
Will test there!

thank you for the fix


Getting stuck

When running Aider I just get stuck here in the terminal. I have installed everything according to the instructions. Am I doing something wrong perhaps?
(venv) jw project % aider
Model: gpt-4
Git repo: .git
Repo-map: universal-ctags using 1024 tokens

I'm going to close this issue, but feel free to re-open if you're able to provide additional information for me to help debug.


Issue with v16

I've just tried to use v16 on a new install (node_exporter 0.18.1) - all I get is the top 'filter bar' but no graphs etc.
Everything seems to work as expected with v15.
Let me know if there's any extra debugging info I can provide...

Yeah it's borked on v16. I think they changed ids again in prometheus.


Segmentation Fault when using a different planner in the pick & place demo

Description
When I modify the pick and [place demo by adding a new planner (nvidia cumotion planner), I hit a segmentation fault when the task that uses the cumotion planner is executed. This is not any error in the cumotion planner which returns a plan successfully but my move it task mtc node is not being able to read the message and causing an Address not mapped error.
Your environment

ROS Distro: Humble
OS Version: e.g. Ubuntu 22.04
Source or Binary build? Binary
If binary, which release version?
If source, which branch? Humble branch binary
Which RMW (Fast DDS or Cyclone DDS)? Fast DDS.

Steps to reproduce
Add a new planner in the code:
auto cumotion_sampling_planner = std::make_shared<solvers::PipelinePlanner>(node, "isaac_ros_cumotion");
	cumotion_sampling_planner->setProperty("goal_joint_tolerance", 1e-5);
...
/****************************************************
	 *                                                  *
	 *               Move to Pick                       *
	 *                                                  *
	 ***************************************************/
	{  // Move-to pre-grasp
		auto stage = std::make_unique<stages::Connect>(
		    "move to pick", stages::Connect::GroupPlannerVector{ { params.arm_group_name, cumotion_sampling_planner } });
		stage->setTimeout(5.0);
		stage->properties().configureInitFrom(Stage::PARENT);
		// initial_state_ptr = stage.get();
		t.add(std::move(stage));
	}

Expected behaviour
The plan shou;ld be executed and shown in rviz
Actual behaviour
ctory
[pick_place_demo-1] [INFO] [1719169477.331862724] [moveit_task_constructor_demo]: Sending goal
[pick_place_demo-1] [INFO] [1719169477.341134866] [moveit_task_constructor_demo]: Goal accepted by server, waiting for result
[pick_place_demo-1] Stack trace (most recent call last) in thread 680136:
[pick_place_demo-1] #22   Object "", at 0xffffffffffffffff, in
[pick_place_demo-1] #21   Object "/usr/lib/x86_64-linux-gnu/libc.so.6", at 0x7fce84306a3f, in
[pick_place_demo-1] #20   Object "/usr/lib/x86_64-linux-gnu/libc.so.6", at 0x7fce84274ac2, in
[pick_place_demo-1] #19   Object "/usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.30", at 0x7fce84506252, in
[pick_place_demo-1] #18   Object "/pick_place_demo", at 0x55a59a754a30, in std::thread::_State_impl<std::thread::_Invoker<std::tuple<main::{lambda()#1}> > >::_M_run()
[pick_place_demo-1] #17   Object "/opt/ros/humble/lib/librclcpp.so", at 0x7fce8477ec3e, in rclcpp::spin(std::shared_ptr<rclcpp::Node>)
[pick_place_demo-1] #16   Object "/opt/ros/humble/lib/librclcpp.so", at 0x7fce8477eb44, in rclcpp::spin(std::shared_ptr<rclcpp::node_interfaces::NodeBaseInterface>)
[pick_place_demo-1] #15   Object "/opt/ros/humble/lib/librclcpp.so", at 0x7fce8477e920, in rclcpp::executors::SingleThreadedExecutor::spin()
[pick_place_demo-1] #14   Object "/opt/ros/humble/lib/librclcpp.so", at 0x7fce8477741c, in rclcpp::Executor::get_next_executable(rclcpp::AnyExecutable&, std::chrono::duration<long, std::ratio<1l,<PHONE_NUMBER>l> >)
[pick_place_demo-1] #13   Object "/opt/ros/humble/lib/librclcpp.so", at 0x7fce8477734e, in rclcpp::Executor::get_next_ready_executable_from_map(rclcpp::AnyExecutable&, std::map<std::weak_ptr<rclcpp::CallbackGroup>, std::weak_ptr<rclcpp::node_interfaces::NodeBaseInterface>, std::owner_less<std::weak_ptr<rclcpp::CallbackGroup> >, std::allocator<std::pair<std::weak_ptr<rclcpp::CallbackGroup> const, std::weak_ptr<rclcpp::node_interfaces::NodeBaseInterface> > > > const&)
[pick_place_demo-1] #12   Object "/opt/ros/humble/lib/librclcpp_action.so", at 0x7fce83af051b, in rclcpp_action::ClientBase::take_data()
[pick_place_demo-1] #11   Object "/opt/ros/humble/lib/librcl_action.so", at 0x7fce83ad45bc, in rcl_action_take_result_response
[pick_place_demo-1] #10   Object "/opt/ros/humble/lib/librcl.so", at 0x7fce83ddea25, in rcl_take_response
[pick_place_demo-1] #9    Object "/opt/ros/humble/lib/librcl.so", at 0x7fce83dde76c, in rcl_take_response_with_info
[pick_place_demo-1] #8    Object "/opt/ros/humble/lib/librmw_fastrtps_shared_cpp.so", at 0x7fce81527637, in rmw_fastrtps_shared_cpp::__rmw_take_response(char const*, rmw_client_s const*, rmw_service_info_s*, void*, bool*)
[pick_place_demo-1] #7    Object "/opt/ros/humble/lib/librmw_fastrtps_cpp.so", at 0x7fce81582bb8, in
[pick_place_demo-1] #6    Object "/ros_ws/install/moveit_msgs/lib/libmoveit_msgs__rosidl_typesupport_fastrtps_cpp.so", at 0x7fce7c35bc2c, in moveit_msgs::action::typesupport_fastrtps_cpp::_MoveGroup_GetResult_Response__cdr_deserialize(eprosima::fastcdr::Cdr&, void*)
[pick_place_demo-1] #5    Object "ros_ws/install/moveit_msgs/lib/libmoveit_msgs__rosidl_typesupport_fastrtps_cpp.so", at 0x7fce7c35ba0b, in moveit_msgs::action::typesupport_fastrtps_cpp::cdr_deserialize(eprosima::fastcdr::Cdr&, moveit_msgs::action::MoveGroup_GetResult_Response_<std::allocator<void> >&)
[pick_place_demo-1] #4    Object "/ros_ws/install/moveit_msgs/lib/libmoveit_msgs__rosidl_typesupport_fastrtps_cpp.so", at 0x7fce7c35a598, in moveit_msgs::action::typesupport_fastrtps_cpp::cdr_deserialize(eprosima::fastcdr::Cdr&, moveit_msgs::action::MoveGroup_Result_<std::allocator<void> >&)
[pick_place_demo-1] #3    Object "os_ws/install/moveit_msgs/lib/libmoveit_msgs__rosidl_typesupport_fastrtps_cpp.so", at 0x7fce7c334aef, in moveit_msgs::msg::typesupport_fastrtps_cpp::cdr_deserialize(eprosima::fastcdr::Cdr&, moveit_msgs::msg::MoveItErrorCodes_<std::allocator<void> >&)
[pick_place_demo-1] #2    Object "/install/moveit_msgs/lib/libmoveit_msgs__rosidl_typesupport_fastrtps_cpp.so", at 0x7fce7c30e6fa, in eprosima::fastcdr::Cdr::operator>>(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&)
[pick_place_demo-1] #1    Object "ros_ws/install/moveit_msgs/lib/libmoveit_msgs__rosidl_typesupport_fastrtps_cpp.so", at 0x7fce7c30b54b, in eprosima::fastcdr::Cdr::deserialize(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&)
[pick_place_demo-1] #0    Object "/usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.30", at 0x7fce84576569, in std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::operator=(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&&)
[pick_place_demo-1] Segmentation fault (Address not mapped to object [(nil)])
[ERROR] [pick_place_demo-1]: process has died [pid 680123, exit code -11, cmd '/pick_place_demo --ros-args --params-file _pick_and_place/config/panda_config.yaml --params-file /tmp/launch_params_c3kkiavu --params-file /tmp/launch_params_kranhjdb --params-file /tmp/launch_params_8w289rl6 --params-file /tmp/launch_params_9bwjxh9l --params-file /tmp/launch_params_jj6sut3w'].

Backtrace or Console output
Use gist.github.com to copy-paste the console output or segfault backtrace using gdb.

I tried cyclone dds as well by the way same result.


singleLabel slot can not read nested property

For this code option slot can read the nested property which is en_name, but the singleLabel slot throws an undefined error.

It works for me, I don't think the nested object is the problem from the library.
Try using another syntax like this, to ensure the option.machinery is not null/undefined before accessing en_name, you can add v-if="option.machinery"
<template slot="singleLabel" slot-scope="props">

{{ option.machinery.en_name }}

</template>

Agree with @lehoaibaokg. If you need more help, I recommend creating a JSFiddle with a full reproducible example so we can take a closer look


Exit menus when hitting Esc (from search field or when key navigating)

Okay, this is ready to be merged.

Tagged v0.1.11


change names and links for Babel plugins fromt 6to5

These are only the plugins that were originall at github.com/6to5

:+1:


Release 1.2: Seele

Release name: [That part 2 Seele and Veliona theme you wish were longer]
Codename: Seele (Honkai Impact 3rd)

LGTM


fix(addon-editor): add deps for starter-kit

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[ ] Documentation content changes

What is the current behavior?
Closes #
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

`npm ci` can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync.
Please update your lock file with `npm install` before continuing.


implement quasar framework

[ ] remove daisyui
[ ] redesign components

Planning poker :black_joker: results:

Sum: 73
Average: 9.125


RNA api schema

New RNA API schemas.

pushed a change removing references to graph version of the API

removing sequence annotation as it is now a separate topic branch

with the protobuf conversion forthcoming, closing this in favor of #625


PST Timezone is not acceptable

Hi team,
I have very simple configuration for syslog interception with location value. I need to change it to PST - however the value is not acceptable. (not: EST works fine)
Version : otelcol-contrib version 0.114.0
receivers:
syslog:
udp:
listen_address: "<IP_ADDRESS>:55552"
protocol: rfc3164
location: PST
Error:
Error: failed to build pipelines: failed to create "syslog" receiver for data type "logs": failed to resolve syslog config: failed to load location PST: unknown time zone PST
2024/11/19 20:13:53 collector server run finished with error: failed to build pipelines: failed to create "syslog" receiver for data type "logs": failed to resolve syslog config: failed to load location pst: unknown time zone PST
Any ideas?

Nope, try all the options listed - PT/PDT/PST
I cannot get to this workspace ... what is it name?

I cannot get to this workspace ... what is it name?

It's the CNCF OpenTelemetry Slack. You can request an invitation.

Also, based on the example given in the docs, maybe you could try America/Los_Angeles.

That worked!!!!! Very strange - UTC and EST supported, while PST not :)
Thanks anyway

Glad it worked. I'll close this issue now.


Open source Bodyguard: Add unit test using RestQA 🦏

Hi there 👋,
I came across your small project and i though i could contribute to it by adding a way to simply write unit test.
So basically you just need to run the command npm test to have the immediate test result.
However if you want a full Report of the test, including addition information such as api spec, postman collection, performance test, etc... You can run the command npm run happy
Here an example of the report:

However for some reason the code coverage doesn't sound working on your project. I already put a comment on a RestQA issue for the team to look at it.
See Ya!

Live stream available: https://www.twitch.tv/videos/1684201529

I would be happy to get your feedback, or getting some help on RestQA. 😁


ntr: protein mislocalized to nucleus during mitotic interphase

descendent of protein mislocalized to nucleus during vegetative growth
standard def
PMID:10485849

protein mislocalized to nucleus during mitotic interphase FYPO:0005000

edit file: e3d1b713aa95d645445e05c97850f44e29b33b5c
release: 9d851f2dcd57ad8363ea4df4506fc965481b8033


OTG Test - TE-4.2: Leader Failover

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.677%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1314

Relevant Lines:
1808

💛  - Coveralls


Fix edge case where secret names will contain dots (for DNS-named secrets)

@jpds
What this PR does / why we need it:
Our SSL certificate kubernetes secrets are named after their DNS names.  This PR will just change any dots that appear in secret names into dashes, which will allow these secrets to be mounted in the containers.  It's probably an edge case, but it doesn't seem harmful to include.
Which issue this PR fixes
No raised issue.  I can raise an issue if that helps.
Special notes for your reviewer:
It fixes a minor issue, again, for my names regarding DNS-named secrets.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] Chart Version bumped

/ok-to-test

/lgtm


Fix issue in Issues with Role Based Adaptive Authentication example doc

Purpose

Fix issue in Issues with Role-Based Adaptive Authentication example doc.

@IsuraD Do we have to worry about the order here?

@darshanasbg , Since there is no authentication already configured for sample-auth resource, we can add this to the end.
Correct PR : https://github.com/wso2/docs-is/pull/77