Test-Connection only responds to stops in between pings

Steps to reproduce
Test-Connection anyHostWhoWillTimeout
# Press Ctrl + C immediately

Expected behavior
The command is immediately cancelled and the prompt returns.

Actual behavior
The current ping finishes timing out before the command is cancelled.

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0-preview.6
PSEdition                      Core
GitCommitId                    7.0.0-preview.6
OS                             Microsoft Windows 10.0.18362
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Hmm... that was supposed to work. Perhaps I wasn't thorough enough. Thanks for reporting it, I'll check it out this weekend sometime. Fingers crossed it's a quick fix. 😉


Adding more functionality to app.js

I love this project and I want to work on this part. Can I work?

Carry on


Resolve SSL local issuer and header redirect not working for OpenID

There are 2 issues that I am trying to resolve with this PR:

Issue #3 that I reported - the issue was caused by potentially older version of CA Cert or it may be due to environment specific certificate provided in the original repo.
header() was not redirecting in OAuthOpenIDExample.php because

@hlu2 and @diana-derose Please review the PR when you get a chance.

@curiousfreelancer
Thanks for contributing.
Hao

You're welcome and thanks @hlu2 for merging my PR.


AttributeError: can't set attribute

Describe the bug
Im getting some strange error when building the sphinx documentation for a project that I maintain
I tried different versions of the package, removing extensions ...
writing output... [  0%] CHANGELOG
/usr/share/miniconda/lib/python3.9/site-packages/pydata_sphinx_theme/bootstrap_html_translator.py:27: RemovedInSphinx60Warning: generate_targets_for_table() is deprecated
  self.generate_targets_for_table(node)
/home/runner/work/gdsfactory/gdsfactory/docs/CHANGELOG.md:688: WARNING: None:myst reference target not found: CrossSection

Exception occurred:
  File "/usr/share/miniconda/lib/python3.9/site-packages/pydata_sphinx_theme/bootstrap_html_translator.py", line 29, in visit_table
    self._table_row_index = 0
AttributeError: can't set attribute
The full traceback has been saved in /tmp/sphinx-err-bcb2whfa.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!

https://github.com/gdsfactory/gdsfactory/issues/553
How to Reproduce
You can see the error here
https://github.com/gdsfactory/gdsfactory/runs/7584210854?check_suite_focus=true
git clone https://github.com/gdsfactory/gdsfactory.git
cd gdsfactory
pip install -r requirements.txt
pip install -r requirements.txt
cd docs
make html
open build/html/index

OS
Linux
Python version
3.10 from mambaforge
Sphinx version
5.1.1
Sphinx extensions
    "matplotlib.sphinxext.plot_directive",
    "myst_parser",
    "nbsphinx",
    "sphinx.ext.autodoc",
    "sphinx.ext.doctest",
    "sphinx.ext.mathjax",
    "sphinx.ext.napoleon",
    "sphinx.ext.todo",
    "sphinx.ext.viewcode",
    "sphinx_autodoc_typehints",
    "sphinx_click",
    # "sphinx_markdown_tables",
    "sphinx_copybutton",
    "sphinxcontrib.autodoc_pydantic",
    "sphinx.ext.autosummary",
    "sphinx.ext.extlinks",

here is the fix
https://github.com/gdsfactory/gdsfactory/issues/553


Cop idea: Redundant safe navigation for basic types

Describe the solution you'd like

hsh = [['a', '1'], ['b', '2']]
# hsh = nil

# bad
hsh&.to_h { |k, v| [k, v] } || {}
# bad
hsh&.to_h || {}

# good
hsh.to_h { |k,v| [k, v] }
# good
hsh.to_h

arr = ['a', 'b', 'c']
# arr = nil

# bad
arr&.to_a || []

# good
arr.to_a

str = "foo"
# str = nil

# bad
str&.to_s || ''

# good
str.to_s

Rather than creating a new cop, extending the existing Lint/RedundantSafeNavigation should perhaps be considered."

I do not think we need a new cop, Lint/RedundantSafeNavigation can be extended, as it was once extended for constants.


Test against Postgres db in CI workflow

🚀 Feature Request
Currently the back end defaults to a sqlite database to test against. This is really convenient for local development as it doesn't require spinning up an instance of postgres or having docker installed.
I don't have a good reason why we shouldn't just test against a postgres service in all our back end workflows (different python versions etc.). However, if we can avoid a situation where all of a contributors tests pass locally on their workstation then fail against postgres in our CI pipeline, that would be great.

closed with #488


Fix typo

It has a typo.. I fixed that typo (get's should be gets)

I am sorry to report to you that we are dropping support for travis and are therefore not interested in cosmetic changes to this script.

Okay...

Veto, Travis makes it possible to test on OS X. We should expend it instead of drop it.
I would also like to have cross-compiling (Linux->Windows) tests back. Why was it dropped? When I'm not wrong I read something of "too old compiler"? However, Travis is supporting GCC5.

@Croydon it was been dropped because of #23 which will allow us to compile distro-specific packages without hassle. Furthermore we use cloudbees.com at the moment, which also offers OSX dropplets.

Ditto. Thanks for the contribution @PowerKiller. There's no need to reject improvements to something that's not yet been phased out! (Also it's probably my grammar error).
As for Travis: Travis is being dropped because it made huge problems. Compiler bugs specifically. Everything we can do with travis we can do with our own CI too.

lmao, remove the wontfix label. :-1:


Don't use a module for infrastructure setup

I initially believed that this repository would host modules that could be used by disparate projects to set up Kubernetes clusters.  It's become more and more clear that the function of this repository is to actually do the cluster setup, removing the need for creating complex Terraform deployments around this base infrastructure.  We can then push off the less complex task of placing additional deployments, etc. onto simpler sidecar repos that use a module that we provide to tap into the resources of an existing cluster to do those application deployments.
The result of this change of approach is that the infrastructure module is not useful on its own, and we don't want to leave it in place and have to answer potential questions about it later.  This issue requests that this unneeded module be rolled directly into the hardware setup stage.

This was superceded (well, duplicated by) #21.  That issue has comments, this doesn't, so close this one.


cleanup: remove inline namespace usage from tests

Tests, like any other user of the library, should not care about versioning in the single release they link against.  Nor do they need to version their own symbols.
So, remove the inline namespaces from tests.  This then removes the need to include google/cloud/version.h, eliminating warnings that we failed to do so.
One slight complication is test-only friends, which should now be forward-declared outside the inline namespace.

This change is

What script did you use to generate the PR?

What script did you use to generate the PR?

I found the tests that used the inline namespace with something like ...
git grep -l GOOGLE_CLOUD_CPP_INLINE_NAMESPACE_ | grep '_test\.cc$'

And then a sed -i script over those files to delete those macros and any google/cloud/version.h include.


Overloading methods arity and order matters

Overloading a method with arity higher than successor definition causes compiler to fail with wrong number of arguments ... ,  fix is to re-order the overloads with lower arity definition first.
module Foo
  def self.foo(a, b, c, d, e = false)
    puts a, b, c, d, e
  end

  def self.foo(a, b, c, d = false)
    puts a, b, c, d
  end
end

Foo.foo(1,2,3,4,true)

above snippet will fail with
error in line 12
Error: wrong number of arguments for 'Foo.foo' (given 5, expected 3..4)

Overloads are:
 - Foo.foo(a, b, c, d = false)

playground link: https://carc.in/#/r/e9as

Just as extra information, it outputs the following with the preview_overload_orde flag:
$ crystal run -Dpreview_overload_order test.cr
1
2
3
4
true

This is definitely a duplicate of #10231

Duplicate of #10231


FAQ topic or individual URL not shown in address bar

Where to find the issue
https://www.coronawarn.app/en/faq/results/
Describe the issue
If an FAQ topic is selected in the menu, the browser shows the topic but does not change the URL displayed in the address bar.
The same for a specific FAQ article.

This follows on from a related issue https://github.com/corona-warn-app/cwa-website/issues/2668.

Steps to reproduce
Open https://www.coronawarn.app/en/faq/results/ in a desktop browser using the full screen width
Note that the browser address bar shows the URL https://www.coronawarn.app/en/faq/results/
Click on any topic in the left-hand menu
Note that the topic is shown on the right-hand side
Note also that the contents of the browser address bar have not changed
Expected behaviour
The contents of the address bar should reflect the FAQ contents currently being shown.
If, for instance, the user clicks on "Tests" in the menu list of https://www.coronawarn.app/en/faq/results/, then the browser address bar should show https://www.coronawarn.app/en/faq/results/#test corresponding to the FAQ contents being displayed.
If the user then clicks on "When do I receive my PCR test result in the app?" to display the contents of this FAQ, then the browser address bar should show https://www.coronawarn.app/en/faq/results/#result_time
If the user then clicks on "Expand/Collapse all" and then clicks on "Link to this answer" from the next FAQ, then the browser address bar should show https://www.coronawarn.app/en/faq/results/#QRcodes.

Closely related:
If you access the FAQ page using a direct anchor link to e.g. https://www.coronawarn.app/en/faq/results/#result_time & then click on "Link to this answer" on any other FAQ entry, the URL is not updated.
I will open an issue for this soon.

@MikeMcC399
You're probably right, but while you were writing I created #2736. Sorry, I will still leave it open to keep everything clean.

This issue has been resolved by PR #2786 from @JoseRosso


Links and search results for Lagoon CLI documentation return HTTP 404 Not Found

Describe the bug
Search for Lagoon CLI documentation and many results in the first page return a 404 Not Found.
Links in other people's documentation also return a 404 Not Found.
To Reproduce
Steps to reproduce the behavior:

Go to Google, Bing or "some other search engine"
Type keywords "Lagoon CLI"
Click on numerous links in the results, or second-tier links in pages.
See an error page at https://amazeeio.github.io/lagoon-cli/

Expected behavior
A redirect takes me to the new documentation site at https://uselagoon.github.io/lagoon-cli/
Additional context
A method for redirecting github pages after a repository move is described in this Gist
https://gist.github.com/domenic/1f286d415559b56d725bee51a62c24a7
Hopefully it helps.

I've just remembered why we didn't do this...
I'm not sure of the impact it would have on other Github pages-hosted sites in the amazeeio namespace (eg https://amazeeio.github.io/charts/)
My guess is that it would steamroller them, and assume the amazeeio.github.io domain for the userspace pages, not the organisation-space ones.
I'll have a dig through our documentation and try to make sure all the lagoon-cli links are properly update.
No idea what we can do about google search though.  It's so annoying that GitHub doesn't cater for this!


[SPARK-33358][SQL] Return code when command process failed

What changes were proposed in this pull request?
Exit Spark SQL CLI processing loop if one of the commands (sub sql statement) process failed
Why are the changes needed?
This is a bug fix.
Does this PR introduce any user-facing change?
No
How was this patch tested?
env
spark version: 3.0.1
os: centos 7

/tmp/tmp.sql
select * from nonexistent_table;
select 2;

submit command:
export HADOOP_USER_NAME=my-hadoop-user
bin/spark-sql  \
--master yarn \
--deploy-mode client \
--queue my.queue.name \
--conf spark.driver.host=$(hostname -i) \
--conf spark.app.name=spark-test  \
--name "spark-test" \
-f /tmp/tmp.sql

execution log:
# bin/spark-sql  \
> --master yarn \
> --deploy-mode client \
> --queue my.queue.name \
> --conf spark.driver.host=$(hostname -i) \
> --conf spark.app.name=spark-test  \
> --name "spark-test" \
> -f /tmp/tmp.sql
20/11/06 00:06:19 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.spark.client.rpc.server.address.use.ip does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.spark.client.submit.timeout.interval does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.enforce.bucketing does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.server2.enable.impersonation does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.run.timeout.seconds does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.support.sql11.reserved.keywords does not exist
20/11/06 00:06:20 WARN DomainSocketFactory: The short-circuit local reads feature cannot be used because libhadoop cannot be loaded.
20/11/06 00:06:20 WARN SparkConf: Note that spark.local.dir will be overridden by the value set by the cluster manager (via SPARK_LOCAL_DIRS in mesos/standalone/kubernetes and LOCAL_DIRS in YARN).
20/11/06 00:06:22 WARN Client: Neither spark.yarn.jars nor spark.yarn.archive is set, falling back to uploading libraries under SPARK_HOME.

Error in query: Table or view not found: nonexistent_table; line 1 pos 14;
'Project [*]
+- 'UnresolvedRelation [nonexistent_table]

2
2
Time taken: 4.437 seconds, Fetched 1 row(s)

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@artiship how was this patch tested?

@HyukjinKwon I reproduced this issue and then verified the fix manually from my dev env, I would like to add unit test to cover this.

@artiship can you elaborate the reproducible steps and how you tested manually in your env?

@HyukjinKwon
env
spark version: 3.0.1
os: centos 7

/tmp/tmp.sql
select * from nonexistent_table;
select 2;

submit command:
export HADOOP_USER_NAME=my-hadoop-user
bin/spark-sql  \
--master yarn \
--deploy-mode client \
--queue my.queue.name \
--conf spark.driver.host=$(hostname -i) \
--conf spark.app.name=spark-test  \
--name "spark-test" \
-f /tmp/tmp.sql

execution log:
# bin/spark-sql  \
> --master yarn \
> --deploy-mode client \
> --queue my.queue.name \
> --conf spark.driver.host=$(hostname -i) \
> --conf spark.app.name=spark-test  \
> --name "spark-test" \
> -f /tmp/tmp.sql
20/11/06 00:06:19 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.spark.client.rpc.server.address.use.ip does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.spark.client.submit.timeout.interval does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.enforce.bucketing does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.server2.enable.impersonation does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.run.timeout.seconds does not exist
20/11/06 00:06:20 WARN HiveConf: HiveConf of name hive.support.sql11.reserved.keywords does not exist
20/11/06 00:06:20 WARN DomainSocketFactory: The short-circuit local reads feature cannot be used because libhadoop cannot be loaded.
20/11/06 00:06:20 WARN SparkConf: Note that spark.local.dir will be overridden by the value set by the cluster manager (via SPARK_LOCAL_DIRS in mesos/standalone/kubernetes and LOCAL_DIRS in YARN).
20/11/06 00:06:22 WARN Client: Neither spark.yarn.jars nor spark.yarn.archive is set, falling back to uploading libraries under SPARK_HOME.
20/11/06 00:06:46 WARN ShellBasedUnixGroupsMapping: got exception trying to get groups for user qttdw: id: qttdw: no such user

Error in query: Table or view not found: nonexistent_table; line 1 pos 14;
'Project [*]
+- 'UnresolvedRelation [nonexistent_table]

2
2
Time taken: 4.437 seconds, Fetched 1 row(s)

ok to test

Test build #130676 has started for PR 30263 at commit 96c753c.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35287/

Test build #130676 has finished for PR 30263 at commit 96c753c.

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

Merged build finished. Test PASSed.

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

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35287/

Merged build finished. Test PASSed.

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

Test build #130687 has started for PR 30263 at commit 3d71aca.

Looks like something went wrong during rebase. You can either resolve it here or just open another PR.

Test build #130687 has finished for PR 30263 at commit 3d71aca.

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

Merged build finished. Test PASSed.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35298/

add a new test case

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35298/

Merged build finished. Test PASSed.

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

Test build #130710 has started for PR 30263 at commit db231c3.

Looks like something went wrong during rebase. You can either resolve it here or just open another PR.

Sorry for the incorrect rebase. It was resolved and now fix and test case have been merged into one commit.

Test build #130710 has finished for PR 30263 at commit db231c3.

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

Merged build finished. Test PASSed.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35320/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35320/

Merged build finished. Test PASSed.

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

Hi, @artiship . Does this happen only YARN? It seems to work correctly in the local.
spark-3.0.1-bin-hadoop3.2:$ cat 1.sql
select * from n;
select 2;

spark-3.0.1-bin-hadoop3.2:$ bin/spark-sql -f 1.sql
Error in query: Table or view not found: n; line 1 pos 14;
'Project [*]
+- 'UnresolvedRelation [n]

2
Time taken: 2.244 seconds, Fetched 1 row(s)

@dongjoon-hyun
I've verified it again using a distribution dowloaded from the apache spark website:
https://www.apache.org/dyn/closer.lua/spark/spark-3.0.1/spark-3.0.1-bin-hadoop2.7.tgz
This bug still can reproduce in local mode. The spark-sql should break after the first statement failed.
➜  spark-3.0.1-bin-hadoop2.7 cat 1.sql
select * from n;
select 2+2;
➜  spark-3.0.1-bin-hadoop2.7 bin/spark-sql -f 1.sql
20/11/12 11:35:28 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
20/11/12 11:35:38 WARN HiveConf: HiveConf of name hive.stats.jdbc.timeout does not exist
20/11/12 11:35:38 WARN HiveConf: HiveConf of name hive.stats.retries.wait does not exist
20/11/12 11:35:46 WARN ObjectStore: Version information not found in metastore. hive.metastore.schema.verification is not enabled so recording the schema version 2.3.0
20/11/12 11:35:46 WARN ObjectStore: setMetaStoreSchemaVersion called but recording version is disabled: version = 2.3.0, comment = Set by MetaStore<EMAIL_ADDRESS>Error in query: Table or view not found: n; line 1 pos 14;
'Project [*]
+- 'UnresolvedRelation [n]

4
Time taken: 4.39 seconds, Fetched 1 row(s)

Ah, it seems that there is misunderstanding due to the PR description, @artiship .
In the PR description, you mentioned that the output 2 is repeated twice. Is that true?

Thank you so much for your contribution, @artiship .
This landed at master branch for Apache Spark 3.1.0 and branch-3.0 for Apache Spark 3.0.2.
SPARK-33358 will be assigned to you. What is your JIRA id?

I found your id from the commit log.
SPARK-33358 is assigned to you. Thanks.

@dongjoon-hyun  Thanks for your carefully review. The output 2 repeated twice is a testing result I got from my production environment. It seems that it might have a configuration makes it print both command and result.


Conditional Access Trends Workbook (NEW)

Required items, please complete
Change(s):

Created a new workbook to display Conditional Access Trends over time.  Credit for the workbook goes to Michael Crane, I am just doing the pull request.

Reason for Change(s):

Needed to see what changes have occurred to Conditional Access Policies.

Version Updated:

N/A

Testing Completed:

Yes, tested in Commercial and GCC

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Hi @kenrward,
Thanks for your contribution. Can you please fix the below things to proceed further.

Please remove the subscription details from the workbook file to fix the validation failure.
Please Add logo file inside the Images/Logo folder
While testing the workbook, getting the below error. Can you please fix it.

Let me know if you need any further assistance.
Thanks

I am trying to reuse the same logo as the Identity and Access Repo, Microsoft_logo.svg.  Is that not allowed?

I am trying to reuse the same logo as the Identity and Access Repo, Microsoft_logo.svg. Is that not allowed?

Hi @kenrward,
Thanks for fixing the validation failure and other suggested items. We want to maintain a consistency in the folder structure, as part of that we generally use this structure. So, requesting you to follow the same.
Thanks

I am confused.  The structure you sent me was for a Solution.  This is not a Content Hub solution, this is a standalone workbook. I am using a Logo that is placed here like all the other standalone workbooks. What am I missing?

Hi @kenrward your approach is correct, we are looking into this, will update you shortly. Thanks!

Hi @kenrward we are working on this, will update you shortly

Hi @kenrward,
Sorry I thought you have added part of solution. you don't need to make any changes for that. I will review and approve.
Thanks

Hi @kenrward,
Still getting this issue. Can you please fix or suggest if Im missing anything here.

Thanks

That is odd. I thought we fixed that. Let me get some others to help me
test locally.
On Fri, Feb 17, 2023 at 1:47 AM Meena Kumari Chatla <
@.***> wrote:

Hi @kenrward https://github.com/kenrward,
Still getting this issue. Can you please fix or suggest if Im missing
anything here.
[image: image]
https://user-images.githubusercontent.com/108806639/219569849-748abbe3-36af-4e42-ab0b-323434fad017.png
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/Azure-Sentinel/pull/7228#issuecomment-1434170963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFWUYKDKRYEGVLZ5DL6K6TWX4NGLANCNFSM6AAAAAAUOLX27Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Ken

Hi @kenrward,
Hope you are doing well. Please let me know once you are done with the testing.
Thanks

Are you sure you are using the latest file?  I initially had this same error on an earlier version but I just tested again and I am not seeing this.  Also, what workspace are you using when you get this?

Hi @kenrward,
Yes, I have used the latest file from this PR. Let me retest once again and get back to you.
Thanks

Hi @kenrward,
Sorry for the delay, I didn't get a chance to recheck this. I will check and provide update in sometime.
Thanks

Hi @kenrward,
Just verified still getting the below error.

What I supposed to select for UserPrincipalName
Thanks

Let's start over, I will cancel this pull request and do a new one.  Not
sure how you are getting the old code but this error was fixed in a new
version.
On Fri, Mar 3, 2023 at 1:36 AM Meena Kumari Chatla @.***>
wrote:

Hi @kenrward https://github.com/kenrward,
Just verified still getting the below error.
[image: image]
https://user-images.githubusercontent.com/108806639/222648718-555e9b4b-d514-44db-b099-1f38801610e8.png
[image: image]
https://user-images.githubusercontent.com/108806639/222648666-d17e974f-07c6-473c-8a15-4b91a04ccc26.png
What I supposed to select for UserPrincipalName
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/Azure-Sentinel/pull/7228#issuecomment-1453049636,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFWUYNT4L7WDAW4WSPIM73W2GGN7ANCNFSM6AAAAAAUOLX27Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Ken


Use publish action to send markdown files to prose.sh with problem

Hi, Eric. Good jobs on lists.sh and prose.sh.
I am using the publish.yml to send markdown files to prose.sh, like it was pointed on prose.sh help page. (https://prose.sh/help#continuous-deployment).
Could you help me, the files wasn't uploaded.
Lasts lines of log:
debug1: Authentication succeeded (publickey).
Authenticated to prose.sh ([<IP_ADDRESS>]:22).
debug1: channel 0: new [client-session]
debug1: Requesting<EMAIL_ADDRESS>debug1: Entering interactive session.
debug1: pledge: network
debug1: Sending command: scp -v -t /
*.txt: No such file or directory

Is the problem sending markdown file and the actions accepts only txt files?
Thanks.

Yes that's right.  There's an option to change the default.
https://github.com/picosh/scp-publish-action/blob/main/action.yml#L11
You want to set src: *.md in your case.


Import warning

Hello there, working on DTU QUAITOM project, would you look at something for us?
I don't think it causes problem with computation, but it is annoying. Every time I import this module I get the following:
<frozen importlib._bootstrap>:241: UserWarning: enum class termtype not importable from thinmaxflow.src._maxflow. You are probably using a cpdef enum declared in a .pxd file that does not have a .py  or .pyx file.
I am working with python 3.11.2 on Linux machine.

I'm publishing a fix for thinkmaxflow.


Feat/target net48

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


[#24515] Ensure that Go pipelines on Dataflow are not allowed to opt out of runner v2.

This is towards #24515
Note that I removed the worker jar experiment as it will not do anything. Also the data disks are only applicable when using windmill appliance which is not applicable when using streaming engine.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Static check:
runners/dataflow/dataflow.go:75:2: var workerJar is unused (U1000)


Windows disk space checks do not take into account labels

Hi All, I'd like to add the ability to the windows disk check not to check only the mount point (which is the drive letter) but the label of the drive as well.
Drive letters are randomly assigned by default espacially when usb sticks need to be present (like dongles). In that case ignoring only drive letters is not convenient since it is not consistent .
There are two ways we could go about this

Adding one more line under https://github.com/sensu-plugins/sensu-plugins-windows/blob/master/bin/check-windows-disk.rb#L76 to check the same attributes for label this time or
Adding another flag for label.

Any preferences?
MM

Also, come to think of it, a better solution would be to sub (mnt) with (mnt,label) in L76 since Driveletter(mnt) and Label(label) will never be the same, as you can't add as a label for a disk, a driveletter. In that case, the ignoremnt flag would search for both.
MM

I'd happily take a backwards compatible PR that addresses this.

closed per: https://github.com/sensu-plugins/sensu-plugins-windows/issues/20


terraform: add h1 hashes for multiple os_arch in terraform.lock.hcl file

Problem
Dependabot updates terraform.lock.hcl file with h1 hash for new provider version, but only for default platform it runs on. Example PR: https://github.com/mwieczorek/dependabot-test/pull/1
In above example I run locally command
terraform providers lock \
  -platform=linux_arm64 \
  -platform=linux_amd64 \
  -platform=darwin_amd64 \
  -platform=windows_amd64

which produced 4 h1 hashes in file. When depedabot run, it added only one h1 hash (linux_amd64).
If I merge that PR and then clone the repository locally I may run into one of the issues:

(cached providers: plugin_cache_dir set in .terraformrc file) - during local terraform init I'll get an error like Error while importing hashicorp/vault v2.21.0 from the shared cache directory: the provider cache at .terraform/providers has a copy of registry.terraform.io/hashicorp/vault 2.21.0 that doesn't match
(no cached providers) -  localterraform init will update terraform.lock.hcl file with new h1

Proposed solution
Add most common os_arch 'platforms' to 'terraform providers lock' command. F.e. linux_amd64, darwin_amd64, windows_amd64
I understand some providers may not have all of those os_arch combinations, so dependabot should gracefully ignore if one of them is not available.

I wonder if this could be worked around temporarily with a Github Action workflow which adds to the dependabot PR 🤔

Right now, all of dependabot's PRs for Terraform are useless to us because they only include the hashes for linux_amd64, and not any of the other architectures. I'd love to see this either turned into an option (e.g. architectures = [linux_amd64, darwin_amd64]) or default to "all platforms" by default.

It would be valuable to have to option to extend the platform providers locking.
https://github.com/dependabot/dependabot-core/blob/f6acb90326f54ad7b19d6045ad9d8f9960248855/terraform/lib/dependabot/terraform/file_updater.rb#L115
Or we simply add all needed platforms in the hcl.lock through the flags -platform=linux_amd64 -platform=linux_arm64 -platform=darwin_amd64 -platform=windows_amd64.
Parsing the hcl.lock and deciding which to take might be to much.

Wasn't this effectively fixed by?

#4905


Consolidates calicoctl install and configuration content

Description

Consolidates highly redundant and disparate calicoctl install and configuration content
Uses variables in paths to ensure that they retrieve the appropriate version of calicoctl
Adds redirects for deleted pages

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

@caseydavenport Fixes made, could you take another look?

Link errors are spurious, chicken egg

Thanks @doublek ! 💯


Fixes to addition and division

Hey there man!
Can you fix only one issue?
Thanks!

Thanks mannnn


Bump kind version test latest k8s on pr

Update kind to 0.7.0
Run build against latest k8s release on each PR

Not that simple, not sure why this change causes the builds to run out of disk space but for now going to close off.


How to listen to other port than 80/443

Hi,
First I want to thank the developers for this wonderful project.
Is it possible to make a redirect of domain listen in other port then 80/443?
I mean in case that we need to redirect
domain.com:7007 to domain2.com (and secure the port 7007)
This is useful for example for Internet radio stations that used to broadcast on port other than port 80 and want to maintain the structure of their stream URL.

Hi if you want to receive traffic on another port e.g. 7007 you can portforward that port (either in docker or in your firewall)
for in docker

here you would need the first 80 or the first 443 to 7007 depending if the connection is https
e.g
7007:80
then in NPM

place here domain2.com and use port 443
Also please change the tag to Question, this is not a bug
Kind regards

Thanks @lieven121
BTW, I'm not sure that I can change the label from bug to question.
Could you explain how?

I think I'm facing a similar problem as @eranwo
I want to reverse proxy multiple services on default port 443, but i additionally want to host a matrix federation server that requires me to listen to external port 8448. So for some proxy hosts, i.e. portainer.mydomain.tld i need to listen to 80 and 443, but for matrix.mydomain.tld:8448 i want to listen to 8448 as an ssl port. I am not sure how to achieve this. Is it possible to use the existing proxy host configuration but adding a listen directive in the custom configuration field?

I think I'm facing a similar problem as @eranwo
I want to reverse proxy multiple services on default port 443, but i additionally want to host a matrix federation server that requires me to listen to external port 8448. So for some proxy hosts, i.e. portainer.mydomain.tld i need to listen to 80 and 443, but for matrix.mydomain.tld:8448 i want to listen to 8448 as an ssl port. I am not sure how to achieve this. Is it possible to use the existing proxy host configuration but adding a listen directive in the custom configuration field?

you can create a forward host with the port included in the domain name (matrix.mydomain.tld:8448) in your case and add this to the custom nginx config tab: listen 8448 ssl default_server;
I'm surprised this isn't done automatically

you can try using my workaround to use custom ports for host without default:
https://github.com/NginxProxyManager/nginx-proxy-manager/issues/2969

I want to keep it open) It would be great to be able to set custom external ports for proxy manager


Add e2e test to validate hitless reload of Envoy Gateway

Description:
Add a test case in our e2e framework https://github.com/envoyproxy/gateway/tree/main/test/e2e where

a client continuously sends traffic to a backend
Envoy Gateway is restarted multiple times
client successfully sends traffic using existing connection

@dboslee long live connections might be a separate sub issue related to keep alive timeouts (disabled by default)

long lived connections might be a separate sub issue related to keep alive timeouts (currently disabled by default)

To clarify in my case the connections were hitting the default 600 second drain timeout (which can be configured in envoy via --drain-time-s). But this only happens when a given filter chain gets drained. So 10 minutes after restarting the controller all the connections get reset.

@dboslee
I would like to understand more about the new e2c test requirement.
1)What would be short name and description for this new test?
name: gatewayrestartdraining?
description: expect client successfully send traffic using existing connection even after Envoy gateway/controller restart?

We expect the connection is rest after 10 minutes, and then the client should fail to send the request?

@arkodg
Do we have any sample test code that I can mimic? I see two existing tests, ratelimit and accesslog. Which one would be a better baseline code to copy for this specific requirement?
Thanks!

Hi @Ronnie-personal thanks for the looking into this issue, assigning this you for now

a good test to add would be a Reload test where we reload Envoy Gateway multiple times but the the /config_dump output from Envoy Proxy https://www.envoyproxy.io/docs/envoy/latest/operations/admin#operations-admin-interface-config-dump remains unchanged thereby proving that after every restart Envoy Gateway generates identical config
a good example is the ratelimit test https://github.com/envoyproxy/gateway/blob/main/test/e2e/tests/ratelimit.go

@arkodg
Would you help me on one question?
When we say reload Envoy gateway, we mean delete the gateway resource and recreate it?
For example, delete gateway named  'same-namespace' in namespace 'gateway-conformance-infra' and recreate it?
Thanks!

@arkodg
I have some ideas on /config_dump, would you mind checking if I'm on the right track?
Here are the steps in the reload.go test code,

Obtain Kubernetes client from cSuite
Use the client to get Envoy Proxy service by name 'proxy-config' and namespace 'envoy-gateway-system'
(e.g --selector=gateway.envoyproxy.io/owning-gateway-namespace=envoy-gateway-system,gateway.envoyproxy.io/owning-gateway-name=proxy-config)
Get admin url (host/ip and port) from the service
Send http get request to the url/config_dump, save the response.
Send http get request again and compare the config dump with previous response.

Thanks!

hey @Ronnie-personal that looks right ! updating some steps
Obtain Kubernetes client from cSuite

Use the client to get Envoy Proxy service by name 'proxy-config' and namespace 'envoy-gateway-system'
(e.g --selector=gateway.envoyproxy.io/owning-gateway-namespace=envoy-gateway-system,gateway.envoyproxy.io/owning-gateway-name=proxy-config)
port forward to envoy proxy pod, since the admin host is <IP_ADDRESS>
Send http get request to the url/config_dump, save the response.
restart Envoy Gateway using kubernetes client
Send http get request again and compare the config dump with previous response.

@arkodg Would you help me on one question? When we say reload Envoy gateway, we mean delete the gateway resource and recreate it? For example, delete gateway named 'same-namespace' in namespace 'gateway-conformance-infra' and recreate it?
Thanks!

@arkodg Thanks for the prompt reply!
Could you confirm my understanding of 'restart Envoy Gateway'? This is for for the step #4.
Thanks!

Could you confirm my understanding of 'restart Envoy Gateway'? This is for for the step #4.

This means restarting the envoy gateway controller. Which can be done by deleting the controller pods or updating the envoy gateway controller deployment in such a way that it triggers a rollout of new pods for you.

Also update on the issue which initial sparked this conversation, while I initially thought the issue was caused by route order changing this was just a trigger for the actual root cause. Which is that the TCP listener name is set to name of the first route processed by the controller so when routes get processed in a different order or the route the TCP listener is named after is deleted the TCP listener name changes and the listener is drained.

Also update on the issue which sparked this conversation, while I initially thought the issue was caused by route order changing this was just a trigger for the actual root cause. Which is that the TCP listener name is set to name of the first route processed by the controller so when routes get processed in a different order or the route the TCP listener is named after is deleted the TCP listener name changes and the listener is drained.
This happens here
https://github.com/envoyproxy/gateway/blob/708287e53af5146873c91c9a4cda2a9ed8126f03/internal/xds/translator/translator.go#L230
where ir.TCPListener.Name is actually the name of a specific TCPRoute or TLSRoute.

Do we consider it as a bug? What would be the options to fix this root issue?

thanks for highlighting this @dboslee !
as @Ronnie-personal pointed out, we do incorporate the route name in the tcp listener (refer to https://github.com/envoyproxy/gateway/pull/696) since the tcp listener directly maps to a cluster / destination https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/filters/network/tcp_proxy/v3/tcp_proxy.proto#envoy-v3-api-msg-extensions-filters-network-tcp-proxy-v3-tcpproxy .
So as you've shared, if the routes are processed in a different order, the order for the tcp listeners will also change causing the hot restart https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/operations/hot_restart .

Sorting pre or post in the gateway-api layer might be a good enough solution here

Another thing to ensure mainly for performance reasons is to prevent retranslation all together in gateway-api by ensuring the the watchable message for resources  can skip updating to subscriber (gateway-api here) published by the producer (provider) if the contents of the resource list hasn't changed, even if the order of the resource list has.

this is the right approach @Ronnie-personal, you can skip comparison of some fields using cmp.Diff which we use in this project, here's an example.
from the above image, it looks like content is same, some config fields order has changed (might be due to usage of a map which lacks order), unsure if cmp.Diff handles this w/o any extra work or might require additional options like cmpopts.SortMaps to be passed to it
its fine to compare the entire config_dump

@arkodg
I have completed initial coding work, I'm able to test the major logic as a standalone go code locally.
I'm not sure how to test this new e2e test under the entire envoy gateway code stack.
Here is the 'reload' test code https://github.com/envoyproxy/gateway/compare/main...Ronnie-personal:gateway:e2ereload1503
Please feel free to give a quick review.
Thanks,
Ronnie

hey @Ronnie-personal thanks for continue to work on this, can you raise this as a PR so others in the community can also review, tia !


@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/volk-feedstock#11.


Clean code

remove unused import

the merge was undone because of errors in gpt-4-turbo, so you can close the PR


Breakpoint and stackframe class name cleanup

Hi @misolori
Sorry for not providing this feedback earlier since I was in vacation mode when we did the breakpoint css refactoring. Here are some issues I am seeing:

There was a className which I did not fully understand and which I removed codicon-debug-breakpoint-stackframe-dot
There is no class name that repesents just the top stack frame without the breakpoint. There is codicon-debug-breakpoint-stackframe but there shuold be just codicon-debug-stackframe

I plan to:

Rename codicon-debug-breakpoint-stackframe to codicon-debug-stackframe
So if there is just the class codicon-debug-stackframe we decorate just the stacframe, if there is also codicon-debug-breakpoint we decorate both combined
Bring back the inline breakpoitn and stackFrame decoration

There are 3 places where we decorate breakpoints: breakpoint view, glyph margin and inline in the editor. Due to those things might be overly complicated and I will try to clean this up.
Feedback very welcome on how to furhter simplify this.

@misolori I have started a bit with the cleanup. You can see the attached commits for reference. I hope this simplifes the structure and if you have questions do let me know and I can explain more.

@misolori  One of the missing things is that the call stack decoration is not rendered inline - there is just an empty area. I am not sure how you did the rendering in the glyph margin, and what is exactly missing when it is an inline decoration. To repro have the following sample:
debugger
for (var i = 0; i < 100; i++) {
    console.log('helllo theher')
}

Once you step over the for has multiple statmens so there should be an inline decoration in the editor.
I will continue on this tomorrow, but just in case you have ideas on how to fix let me know.

Inline stack decorations are now in:

Thanks a lot!
I have also pushed some cleanup.
There are stil quite some hacks, however I feel this is in a better spot than before, thus closing.


Fix invisible ceiling when using range aiding

This PR adds a check for range aiding to determine if we are relying_on_rangefinder. This fixes the bug we are experiencing where we are limited to the range finders max height above ground when using it in aiding mode. If range aiding is not enabled, and thus the range finder is intended to be the sole source of height observation, then this check will evaluate true and the controller will still properly limit the height above ground.
This PR also includes the removal of _control_status.flags.in_air  check for enabling range aid. There is already checks throughout the ecl that look for being !in_air and if the measurements are bad they are set to ground_clearance. So pretty much this now allows range aiding to be on while on the ground.
Let me know if you have any questions or concerns, thanks!
Jake

Nice fix! Looks good to me.

If enabling range aiding when on ground, we need to make sure if the vehicle is tilted excessively that the switch to baro happens immediately and we don't wait for a timeout. The decision to not  perform range aiding when on ground was due to problems with vehicle being carried and tilted. I worry that changing that behaviour creates unnecessary risk given that baro height is more reliable than range height prior to motor start.

@priseborough Removed.

@priseborough


Use closure as macro's argument can result in "Circular reference" error

Description
I try to use closure as macro's argument, but Xcode shows me a "Circular reference" error, take https://github.com/DougGregor/swift-macro-examples as a example:
@attached(member)
public macro DateStrategy(decode: () -> Date) = #externalMacro(module: "MacroExamplesPlugin", type: "CodableKey")

struct CustomCodable: Codable { // Circular reference
	@DateStrategy(decode: { Date() })
	var date_custom: Date = Date(timeIntervalSince1970: 23343)
}

Environment
I am using swift-DEVELOPMENT-SNAPSHOT-2023-03-13-a-osx to test

Can anyone tag this bug and send it to the correct handler? Thanks!


x/pkgsite: package removal request for [github.com/Abu-Zakaria/termui-kit]

What is the path of the package that you would like to have removed?
My package is at – github.com/Abu-Zakaria/termui-kit
I have already deleted the package from github.

Are you the owner of this package?
Yes

What is the reason that you could not retract this package instead?
I have already deleted the package project files from my local computer and GitHub. The reason I deleted the repo is that I wanted to rename the package name, so I created a new repo with a new name for the package and deleted the old package.

Package removed.