Added option to dartanalyzer to treat lints as errors.

Fixes #25977

/cc @bwilkerson @tonygentilcore

lgtm

Thanks for the review! I don't have write access so I can't merge the change myself.

Argh, just realizing right now that you don't use GitHub PRs. Do you want me to resent this on codereview.chromium.org?

No. I'm fine with a PR as long as it's fairly short (the UI is hard to deal with for large changes).
Have you signed the CLA (as described on https://github.com/dart-lang/sdk/wiki/Contributing)?

Yeah the CLA is signed (I work for Google btw). There's an automated check for it on this page.

So there is. Missed that.


How does proto define this type? map[string][]int64

type mess struct {
	uid int64
	list map[string][]int64
}

The following approach is wrong
message Mess {
  int64 uid = 1;
  map<string, repeated int64> list = 2;
}

The following method works, but it’s not the result I want.
message Mess {
  int64 uid = 1;
  message Repeatedint64 {
    repeated int64 versus_user_id = 1;
  }
  map<string, Repeatedint64> list = 2;
}

https://github.com/protocolbuffers/protobuf/issues/2388
I have checked the relevant information, but go-zero does not support external references. How should I solve this?

我下载了google官网的包，放到环境变量里。然后执行
protoc -I . --go_out=./proto --go-grpc_out=./proto  ./proto/source/test/slot.proto
报错：
The import path must contain at least one period ('.') or forward slash ('/') character.
排查后发现是官方包struct.proto文件内容option go_package = "types";
改为option go_package = "./types";后重新生成，可以正常生成了，
但是打开生成后的文件发现types 引用路径有问题：
import (
	types "/types"
...
)

请问这个问题怎么解决？

Did you file the issue in the wrong place? This is go-zero, not proto. Maybe https://protobuf.dev/ can be helpful to you.


Added information about IE security zones

Q
A

content fix?
yes

New article?
no

Related issues?
n/a

What's in this Pull Request?
Additional information about OAuth implicit flow and Internet Explorer security zones

Hi @waldekmastykarz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Added documentation for scripts

Pls merge this gautam bro :)

Please don't submit AI-generated docs 🥲


KeyError: '_module' error

I get the below error when running the script ::
PS C:\Users\nhr\Desktop\Gmail_Api> python quickstart.py
Traceback (most recent call last):
File "quickstart.py", line 32, in
main()
File "quickstart.py", line 14, in main
creds = store.get()
File "C:\Python27\lib\site-packages\oauth2client\client.py", line 407, in get
return self.locked_get()
File "C:\Python27\lib\site-packages\oauth2client\file.py", line 54, in locked_get
credentials = client.Credentials.new_from_json(content)
File "C:\Python27\lib\site-packages\oauth2client\client.py", line 302, in new_from_json
module_name = data['_module']
KeyError: '_module'
Steps to Reproduce the Problem
I have followed the sames procedure described in https://developers.google.com/gmail/api/quickstart/python

Install Python , PIP and all related modules
Enabled GMAIL API and saved the credentials.json file in project directory
Installed Google Client Library
Copied and Saved the Quickstart.py file in project directory
Run the python command to run quickstart.py

These are the files in my project directory:
09/25/2018  07:45 AM              .
09/25/2018  07:45 AM              ..
09/25/2018  07:46 AM              .idea
09/09/2018  09:03 PM               431 client_secret.json
09/09/2018  08:42 PM               431 credentials.json
09/09/2018  08:33 PM             4,285 index.html
09/09/2018  08:40 PM             3,076 index.js
09/09/2018  08:56 PM             1,001 quickstart.py
09/09/2018  08:47 PM               340 token.json
6 File(s)          9,564 bytes
Specifications

Python version (python --version) : 2.7
OS (Mac/Linux/Windows) : Windows 10

Can you fill out the steps to reproduce the problem?

Do you already have a file in your working directory called token.json?

please answer this, new to this why do we require token.json ?

Hi, My apologies for late response. I have added the steps and other details to the description.

The file token.json is created after you authorize access to your account. This error indicates that the file exists, but isn't in the expected format. What does your token.json file contain? Try renaming it to something else and running the quickstart again.

The file token.json is created after you authorize access to your account. This error indicates that the file exists, but isn't in the expected format. What does your token.json file contain? Try renaming it to something else and running the quickstart again.

Thanks a lot. It worked fine after renaming token.json. now we have a new token.json in its place created by the script.

The file token.json is created after you authorize access to your account. This error indicates that the file exists, but isn't in the expected format. What does your token.json file contain? Try renaming it to something else and running the quickstart again.

Worked well. Thanks a lot. :)

In both of your cases, how did the original (non-working) token.json file get there? Did you create it? Did the folder already have one in there, from a quickstart run in another programming language?

I am not sure, but it looks like the token.json was created by the script
at the first run, which did not work.
On Wed, Sep 26, 2018, 11:49 AM Eric Koleda<EMAIL_ADDRESS>wrote:

In both of your cases, how did the original (non-working) token.json file
get there? Did you create it? Did the folder already have one in there,
from a quickstart run in another programming language?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/gsuitedevs/python-samples/issues/38#issuecomment-424763855,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYGrLewIJPzXHoFbIJ8zvNowlZfBAlVKks5ue6IagaJpZM4WgjSp
.

Hi, how do I create a token.json? Is there an example?

Does anyone have solution for it or not
PS C:\Users\Dhaval\Downloads\uds-master> python3 uds.py push ~\Desktop\ffmpeg.exe
Traceback (most recent call last):
File "uds.py", line 534, in
main()
File "uds.py", line 452, in main
uds = UDS()
File "uds.py", line 46, in init
self.api = GoogleAPI()
File "C:\Users\Dhaval\Downloads\uds-master\api.py", line 19, in init
self.reauth()
File "C:\Users\Dhaval\Downloads\uds-master\api.py", line 25, in reauth
credentials = store.get()
File "C:\Users\Dhaval\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\oauth2client\client.py", line 407, in get
return self.locked_get()
File "C:\Users\Dhaval\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\oauth2client\file.py", line 54, in locked_get
credentials = client.Credentials.new_from_json(content)
File "C:\Users\Dhaval\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\oauth2client\client.py", line 302, in new_from_json
module_name = data['_module']
KeyError: '_module'

So... I just went through this rigamarole as well with GCP. In the end, I was able to generate the credentials.json file and the subsequent token.json.
However, now I am getting a KeyError: '_module' error when calling ezgmail.init(). I have the prerequisite modules and files already. All solutions say to delete the token.json file and rerun the authorization process, but this does not work.
Here is the traceback:
  File "<stdin>", line 1, in <module>
  File "c:\Installs\Python\lib\site-packages\ezgmail\__init__.py", line 471, in init
     creds = store.get()
  File "c:\Installs\Python\lib\site-packages\oauth2client\client.py", line 407, in get
     return self.locked_get()
  File "c:\Installs\Python\lib\site-packages\oauth2client\file.py", line 54, in locked_get
     credentials = client.Credentials.new_from_json(content)
  File "c:\Installs\Python\lib\site-packages\oauth2client\client.py", line 302, in new_from_json
     module_name = data['_module']
 KeyError: '_module'

*** UPDATE ***
I changed a line in the ezgmail/init.py file (line 471) from:
  creds = store.get()

to:
 creds = None

Immediately upon running
 import ezgmail

apparently a new token.json file is written. I then reverted the ezgmail/init.py file (line 471) back to...
 creds = store.get()

... and now I can run ezgmail.init() with no issues!
This represents my limited knowledge of python and I am only just beginning, so please bear with my explanation above.  I aim to try and explain as best as I can how I found a solution so that someone can possibly walk away with an answer as to why a solution works rather than just "it works" and I can move on.

The root cause for this error is that the current token.json (or storage.json) file you're using was written by an app using newer Python auth client libraries (meaning your code wrote the file with minimal content).
The oauth2client library is older and requires such a file (written by itself) include a _module entry in the JSON payload which doesn't exist. Sure, temporarily renaming creds = None works, but in reality, you can just blow away that token file and run your script again (and re-auth) without changing any of its code.
If you're confused by this and want to see examples like an old one using oauth2client and a new one using the replacement client libs where your code has to write the (abridged) token file that's incompatible with oauth2client, see both of the Drive samples I created at https://github.com/wescpy/gsuite-apis-intro/tree/master/python.


Dialog.Error(String) documentation should mention that Error('') does rollback but does not show a blank message box

This page doesn't seem to mention the common pattern (in base and third-party developer code) of calling Error('') to mean 'I want the usual rollback to happen, subsequent code to be skipped, etc. - but I don't need any message window, not even an empty one'.
A common use of that is showing a Confirm() dialog stating that some possibly unexpected thing happened, and asking the user if they want to continue, and if they answer No then doing Error('') to bail out and rollback prior changes, without having to write something like 'The update was aborted to respect the warning' or whatever folk used to do... which is ugly and not done these days.
This should be documented clearly to indicate to new developers that this is a supported/common pattern. Thanks!

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

ID: 6ffada02-b6d2-c76a-cd64-67880036a8b5
Version Independent ID: 99658790-afae-1abd-ab80-e7205a4a1bbb
Content: Dialog.Error(String [, Any,...]) Method - Business Central
Content Source: dev-itpro/developer/methods-auto/dialog/dialog-error-string-joker-method.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

FYI...


Add the code generation of injection

This PR also adds the generation of "injection" for further reconciler.

https://github.com/knative/test-infra/issues/770
This can be manually overrided with "cla: yes"
https://github.com/knative/test-infra/issues/770#issuecomment-494506556

cla: yes

@adrcunha Do you know why the CLAbot is complaining about "author consent"?

@adrcunha Do you know why the CLAbot is complaining about "author consent"?

No. Commits were force pushed 6 times, then the error happened. Then there were 2 more force pushes, and the latest one seems OK. My theory is that one of the 6 force pushes had a wrong user. However, since there's no history to prove or reject my theory, it's just a theory. I'll force the CLA and let's move on.

It seems like @adrcunha theory might be correct, that some of the force commits may have overridden ones with co-authors, which we might not be able to bypass.
@adrcunha to confirm.

It seems like @adrcunha theory might be correct, that some of the force commits may have overridden ones with co-authors, which we might not be able to bypass.
@adrcunha to confirm.

As I said, I can't confirm it because there's no history. But the new label should solve the issue anyways.

/lgtm

@googlebot rescan

/retest

/lgtm

@googlebot I consent.


Export Button

"Embed this visualization" works for the straight embedding, but does not work for a link, and I find I often need some more details. There's a lot we could do, but I suggest we have a button which brings up a dialog with the HTML in the bottom and the following options at the top:

[check box] embed full visualization as an iframe (what we have now)
[check box] generate HTML link with this title (edit box with "brunel visualization" as the default title)
[check box] generate caption with brunel script (tagging it as 'code' and 'small' works well for this)
[edit box] title ("Brunel Visualization" default, if not defined on page)
[edit box] description (none if not defined on page)

How about another option here to copy the resulting JS?  This would supply the javascript that Brunel generated for pasting into a self-contained web page (no service calls).

yes, that would be good. agree

Added all suggested features in new UI.


tonic-build: support boxing fields

Motivation
This exposes https://github.com/tokio-rs/prost/pull/802 in Tonic, though obviously until that's merged and released this diff does nothing.
This allows the user to request boxing of specific fields. This is useful for enum types that might have variants of very different size.
Solution
Expose the method

The underlying prost-build PR was merged; Any chance we can get this one merged as well? :)

yup sorry for the delay, IRL has kept me busy


[BEAM-7784] Fixup for Guava upgrade

Original PR: https://github.com/apache/beam/pull/8899

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

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run JavaPortabilityApi PreCommit

R: @chamikaramj

LGTM.

Yeah, reorderingbeamjoins #9040 was merged after my commit and testing couldn't catch this since both sets of precommits were run where neither had eithers commit

Thanks for fixing this.

+1, thank you


Error video no play

D/NativeCrypto(28749):  sslNotify, appData=0x9888f340 ret=1
D/NativeCrypto(28749): ssl=0x8b67a840 info_callback where=0x4008 ret=256
D/NativeCrypto(28749): ssl=0x8b67a840 SSL3 alert write:W:CN warning close notify
D/NativeCrypto(28749): ssl=0x8b67a840 info_callback ignored
I/System.out(28749): Close in OkHttp
I/System.out(28749): close [socket][/<IP_ADDRESS>:33900]
E/flutter (28749): [ERROR:flutter/lib/ui/ui_dart_state.cc(186)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: Unexpected runtime error, null, null)

Link test "https://assets.mixkit.co/videos/preview/mixkit-woman-on-a-video-call-in-outdoors-39813-large.mp4"

Can you provide more detailed information?

Can you provide more detailed information?

I try to rotate this link in the video player and it doesn't work.
BetterPlayer.network(
          "https://assets.mixkit.co/videos/preview/mixkit-woman-on-a-video-call-in-outdoors-39813-large.mp4",
          betterPlayerConfiguration: BetterPlayerConfiguration(
            aspectRatio: 16 / 9,
          ),

Can you provide more detailed information?

I try to rotate this link in the video player and it doesn't work.
BetterPlayer.network(
          "https://assets.mixkit.co/videos/preview/mixkit-woman-on-a-video-call-in-outdoors-39813-large.mp4",
          betterPlayerConfiguration: BetterPlayerConfiguration(
            aspectRatio: 16 / 9,
          ),

Sorry!

This library is not [BetterPlayer].(https://github.com/jhomlala/betterplayer)


Add condition for showing border on team Member on first month date

Changes

Don't show border if expected hours and actual hours are 0.

Self reviewed


Fixes "OperationFailure: Error 24: Too many open files"

Instead of requiring that users modify their .bash_profile (or corresponding file), this change manually sets the open file limit before running mongod. This solves the OperationsFailure which occurs sometimes while running tests.

Implicitly setting OS configuration doesn't seem prudent here; users should expect that inv mongoserver only runs a mongo process.
Not to mention that this will break on non-*nix systems.
I think I'd prefer documenting the fix rather than actually setting ulimit in the task.

Closing for now.

Can we get on the same page re: 'documenting the fix' ? What does that look like?

I've no strong preference in how this is done, as long as we don't continue to spend developer time looking for a solution.

This should be documented here: https://cosdev.readthedocs.org/en/latest/osf/common_problems.html

Can we get on the same page re: 'documenting the fix' ? What does that look like?

As a point of comparison, here is what we say about ulimit in the OSF repo README.md:
https://github.com/CenterForOpenScience/osf.io#additional-configuration-for-mac-os-x


Adds a new Storable interface for things stored in Handles

Storable represents both Entities and References, since both can be stored in Handles. Adds some type safety to these Handle methods (previously things were working on References mostly by accident).
NB: this PR is branched off https://github.com/PolymerLabs/arcs/pull/2869 -- will submit that first.

(Submitted this without submitting PR #2869)


fix crash when selecting some presets

follow up to 5a3a2d876e3a685cfa3391b4f1aeb3c0b4b6e765
to reproduce:

Go to https://ideditor.netlify.app/#map=20.50/-36.93539/174.71695
Click any vertex
Search for the "ASL" preset
click the preset
editor crashes

Great catch. :+1:


boardctl: Added BOARDIOC_PM_CHANGESTATE to BOARDIOC_PM_CONTROL

Summary
Added boardctl BOARDIOC_PM_CHANGESTATE used in BOARDIOC_PM_CONTROL to change the pm state (pm_chargestate()).
Impact
minimal, added a new boardctl to excisting interface.
Testing
Tested on RDDRONE-BMS772, it worked.

Sounds reasonable. I only wonder about the safety of this call, considering the following:

Assumptions: It is assumed that interrupts are disabled when this function is called. This function is probably called from the IDLE loop… the lowest priority task in the system. Changing driver power management states may result in renewed system activity and, as a result, can suspend the IDLE thread before it completes the entire state change unless interrupts are disabled throughout the state change.

from https://nuttx.apache.org/docs/latest/components/power.html

BTW, maybe it is worth adding checkstate() as well, which would completely expose PM system via boardctl and allow for full userspace state handling. Not sure if it is possible since maybe both calls need to happen within same critical section to work. Also, can this really work outside of IDLE loop?

@xiaoxiang781216 I had asked a question since I was not entirely convinced about the safety of the change. It should have not been merged yet.

Hi @v01d, since you are closer to it, should you just hit the revert button?

@v01d @davids5 The enter_critical_section() function is located in the pm_changestate() function.

@v01d @davids5 The enter_critical_section() function is located in the pm_changestate() function.

Ok, so I guess that it is safe to be called anywhere then.
I can't really see why it should need to be called from IDLE loop either.
The only issue I can imagine is an interruption between checkstate and change state .

Hi @v01d, since you are closer to it, should you just hit the revert button?

In this case I don't think it is necessary to revert, as it is not critical.

@cisvanmierlo If you could provide a companion PR including checkstate support it would be appreciated.

@v01d @davids5 The enter_critical_section() function is located in the pm_changestate() function.

Ok, so I guess that it is safe to be called anywhere then.
I can't really see why it should need to be called from IDLE loop either.
The only issue I can imagine is an interruption between checkstate and change state .

Looking at, for example, nrf52_idle.c, we can see a critical section protecting the two calls to changestate as well as any PM calls. Would it be safe to use this from userspace without such a protection? I imagine it could result in some drivers changing state and not others (after first call, if some refused the change) and then an interruption is done before all drivers are reverted to previous state (using the second call). For clarification, I mean this code:
      flags = enter_critical_section();

      /* Perform board-specific, state-dependent logic here */

      _info("newstate= %d oldstate=%d\n", newstate, oldstate);

      /* Then force the global state change */

      ret = pm_changestate(PM_IDLE_DOMAIN, newstate);
      if (ret < 0)
        {
          /* The new state change failed, revert to the preceding state */

          pm_changestate(PM_IDLE_DOMAIN, oldstate);
        }
      else
        {
          /* Save the new state */

          oldstate = newstate;
        }

...

@v01d I was planning on adding that function as well.
For your other question, pm_chargestate() does already has a check, see https://github.com/apache/incubator-nuttx/blob/cdd111a1faec9b40b707797e00c4afae4956fb3f/drivers/power/pm_changestate.c#L238-L247.

Mmm, then I'm not following why the two calls on changestate on most IDLE loops...

@xiaoxiang781216 I had asked a question since I was not entirely convinced about the safety of the change. It should have not been merged yet.

Oh, I was thought that you don't reject BOARDIOC_PM_CHANGESTATE, but want to expose checkstate additionally. And could be done in another PR.


BUG:macos无法截屏

macos 13.5 (22G74) 运行示例项目无法截屏
环境：macos 13.5
pnpm:8.6.11
nodejs：v18.17.1
点击截屏后没有出现示例中的蒙层，无法选取屏幕上的任何东西

[DESKTOP] [15034:1118/170832.697779:ERROR:system_services.cc(34)] SetApplicationIsDaemon: Error Domain=NSOSStatusErrorDomain Code=-50 "paramErr: error in user parameter list" (-50)
[DESKTOP] Error occurred in handler for 'ss:get-shot-screen-img': TypeError: Cannot read properties of undefined (reading 'thumbnail')

没有mac设备，暂时无法解决

Got the same error. I run the project locally (there is no macOS compatible asset in the release)

[DESKTOP] [15034:1118/170832.697779:ERROR:system_services.cc(34)] SetApplicationIsDaemon: Error Domain=NSOSStatusErrorDomain Code=-50 "paramErr: error in user parameter list" (-50) [DESKTOP] Error occurred in handler for 'ss:get-shot-screen-img': TypeError: Cannot read properties of undefined (reading 'thumbnail')

我看见 electron 是有这个方法的
interface DesktopCapturerSource {

    // Docs: https://electronjs.org/docs/api/structures/desktop-capturer-source

    /**
     * An icon image of the application that owns the window or null if the source has
     * a type screen. The size of the icon is not known in advance and depends on what
     * the application provides.
     */
    appIcon: NativeImage;
    /**
     * A unique identifier that will correspond to the `id` of the matching Display
     * returned by the Screen API. On some platforms, this is equivalent to the `XX`
     * portion of the `id` field above and on others it will differ. It will be an
     * empty string if not available.
     */
    display_id: string;
    /**
     * The identifier of a window or screen that can be used as a `chromeMediaSourceId`
     * constraint when calling `navigator.getUserMedia`. The format of the identifier
     * will be `window:XX:YY` or `screen:ZZ:0`. XX is the windowID/handle. YY is 1 for
     * the current process, and 0 for all others. ZZ is a sequential number that
     * represents the screen, and it does not equal to the index in the source's name.
     */
    id: string;
    /**
     * A screen source will be named either `Entire Screen` or `Screen <index>`, while
     * the name of a window source will match the window title.
     */
    name: string;
    /**
     * A thumbnail image. **Note:** There is no guarantee that the size of the
     * thumbnail is the same as the `thumbnailSize` specified in the `options` passed
     * to `desktopCapturer.getSources`. The actual size depends on the scale of the
     * screen or window.
     */
    thumbnail: NativeImage;
  }

很抱歉刚看到，这次更新后直接运行报错，(node:31713) electron: Failed to load URL: http://localhost:9191/index.html with error: ERR_CONNECTION_REFUSED，macos13.5


Automated Package Upgrade

:tada: This PR is included in version 25.6.81 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bug: macos.x86.medium.gen2 resource not accepted when doing an import

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
When uploading our current existing valid circleCI config, the VCE throws an error that the macos.x86.medium.gen2 resource is not a valid enum case.

Minimum reproduction code
https://gist.github.com/mathaeus/2ff2b1d2782977e639ad9746cabc914c
Steps to reproduce
Just import an existing config with that executor
Expected behavior
Valid circleCI executors are also accepted by the VCE.
CircleCI Config SDK version
Whatever the online editor gave me:
# This configuration has been automatically generated by the CircleCI Config SDK.
# For more information, see https://github.com/CircleCI-Public/circleci-config-sdk-ts
# SDK Version: 0.9.0-alpha.16
# VCE Version: v0.10.5
# Modeled with the CircleCI visual config editor.
# For more information, see https://github.com/CircleCI-Public/visual-config-editor

Node.js version
No response
In which operating systems have you tested?

[x] macOS
[ ] Windows
[ ] Linux

Other
No response

Looks like upstream work to address this via https://github.com/CircleCI-Public/circleci-config-sdk-ts/pull/145 ; just helping tie them together.


Remove 'Hidden' attribute from serviceAccount PipelineOption (#24642)

Cherry-picks https://github.com/apache/beam/pull/24642 into the 2.44.0 release branch.
Issue is https://github.com/apache/beam/issues/24649.

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.

Run Java PreCommit

Run Java PreCommit

Run Java_Examples_Dataflow PreCommit


Modify: Compile warning when modified attribute is input

A case like this:
output O:          // line 1
   a = a +1,       // line 2
   b=a +1;         // line 3

needs a warning, because a is modified in line 2--when used in line 3, is it the only value or the new value that's used?

seems to be merged


Enhance MCAD Logs and fix naming consistency

Issue link
Resolves #644
Motivation
In the scenario where we want to create an AW in two different namespaces but with the same name, it is important to know which AppWrapper the logs are referring to.
Previously:

There are a number of instances where the logs only provide the name of the AppWrapper or Job, instead of following the most used pattern of namespace/name as done i.e., here.

There are a couple of places where the logs display name/name, such as here.

A few places have the name/namespace pattern, which can be inconsistent when looking/searching through the logs.

Few instances where i.e., qj.Name is provided in the Info log but not used.

What changes have been made

The logs now consistently refer to the AppWrappers or Jobs in the form of namespace/name.
Typo fixes.

Verification steps

With MCAD running, create an AppWrapper, and check MCAD logs for <namespace/name> of your AW.

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests

/lgtm

/approve


Authentication passwords lost on update

Issue Type: Bug
I'm connected to several postgresql data and when the azure studio is updated all authentication passwords on saved connections are lost and have to reenter all passwords.
Using postgresql extension from Microsoft
Azure Data Studio version: azuredatastudio 1.14.1 (f9b968c1ae514a0d1210788629e1c69125857279, 2019-12-24T02:37:56.865Z)
OS version: Darwin x64 19.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz (4 x 2300)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
8.00GB (0.08GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

azuredatastudio-postgresql
Mic
0.2.2

import
Mic
0.12.0

azdata-sanddance
msr
1.3.9

I'm encountering the same issue with a MSSQL connection.
Version: 1.15.0 (user setup)
Commit: e57eb999fc5f1e000477da8fe0b8679ff0bfd57b
Date: 2020-02-12T20:48:43.389Z
VS Code: 1.42.0
Electron: 6.1.6
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.15063

Create new connection, specify connection type Microsoft SQL Server and Authentication type SQL login. Enter connection information including password and check Remember password checkbox. Click Connect. Connection successfully established.
Open settings.json in Preferences | Settings and the password for the connection just created is empty:

Close and re-open ADS, edit settings.json in Preferences | Settings to specify password. Save settings.json and close and re-open ADS again.
Connect to connection for which password was just added in the settings.json. Connection successfully established.
5, Disconnect. Open settings.json in Preferences Settings and password is empty again.

Created new connection, checked Save Password, connected fine first time, exited, checked settings.json, empty password, added password manually in settings.json, saved settings.json,

I am having the exact same issue as Juan Pablo.
Each and every time Azure Data Studio is updated (had one this morning) all passwords on saved connections are lost. I need to reenter all passwords and gathering those is in my case a real pita!
This is at least for all the PostgreSQL connections. I'm unable to test for MS SQL Server though because those all use Windows Authentication.
One thing I noticed is that after the update of ADS the PostgreSQL extension (azuredatastudio-postgresql) gets reinstalled. Perhaps this has something to do with the issue.
Azure Data Studio version: azuredatastudio 1.16.1 (16801c522df68e63218b39ccb5b4a7431f9bf667, 2020-03-23T22:08:06.892Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.89GB (3.83GB free)

Process Argv

Screen Reader
no

VM
33%

Extensions (1)

Extension
Author (truncated)
Version

azuredatastudio-postgresql
Mic
0.2.5

In addition "updating" the connection with the password just adds a new instance of the saved connection instead of updating the existing one.

I have a similar problem but without to be needed touching the settings! From times to times, the password for connection is lost and I have to retype it.

After my comment about a year ago there seemed to be progress because after updating ADS a couple of times it did not lose any passwords again. I'm pretty much following the release timeline. However since the last update all my passwords are gone again. This is really a major PITA. I even find myself using pgAdmin 4 more frequent which is awful for only querying data.
Azure Data Studio version: azuredatastudio 1.26.1 (796abbdf5fdd341e3528e30a6b777c93e115eb02, 2021-02-24T23:42:55.085Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.89GB (5.07GB free)

Process Argv

Screen Reader
no

VM
31%

Extensions (1)

Extension
Author (truncated)
Version

azuredatastudio-postgresql
Mic
0.2.7

Hate to say "me too", but the last comments on this are over a year ago and this issue is still happening.  I am using the new MySQL extension, but as far as I can tell, it happens for all connections.  See below for more info.
Issue Type: Bug
Save Password checkbox saves password only temporary until next ADS update.
Steps to reproduce:

Create a Server connection and check save password checkbox, then connect and save the connection.
Notice that every time you go to open a connection it works fine.
A new update for Azure Data Studio arrives and gives you a notification to restart to install.  (side note - preferred would be to install on exit NOT on startup of ADS)
Click on restart and wait for install to finish and ADS to come back up.
Check connections and notice that ALL password boxes have been wiped while save password box is still checked.

Azure Data Studio version: azuredatastudio 1.41.2 (0f0960d0e774e1ce903b79378addff96117c27a6, 2023-02-08T20:16:49.071Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No
Preview Features: Disabled

Extension
Author (truncated)
Version

azuredatastudio-mysql
Mic
0.2.6

dacpac
Mic
1.12.0

schema-compare
Mic
1.18.0

sql-database-projects
Mic
0.21.0

azdata-sanddance
msr
4.0.1


Save file to internal storage

Is it possible to download file to internal storage?
I want to download file for permanently internal usage without sharing with other apps.

figure out. DocumentDir is internal storage absolute dir for android.

Can you guide me, in detail plz
how to download in internal storage for get

let dirs = RNFetchBlob.fs.dirs
RNFetchBlob
.config({
  path : dirs.DocumentDir + '/path-to-file.anything'
})

@prasad456


The Facet initializer doesn't allow to pass parameters to Facet

In the current implementation it's impossible to pass parameters to Facet.
For example, the following code:
@term = "category_ids"

search =  Tire.search do |search|
  search.facet 'category' do |facet|
      facet.terms @term, :size => 100
  end
end

generates the following query:
{"facets":{"category":{"terms":{"field":null,"size":100,"all_terms":false}}}}

( note the "field":null )
The reason seems to be the Facet initializer - it's last line performs only instance_eval and not considering the block arity.
It should be exactly like the last line of the Query initializer, where passing parameters is possible.
Fixed it in this fork.

Sorry, fixed it in this fork:
https://github.com/itayco/tire/blob/master/lib/tire/search/facet.rb

Hi, thanks, cleaned up the commit message, added a simple unit test and pushed in master.


How can one use Kubecost to tune requests for Argo Workflows?

Our users often run workflows with large pods, the need lots of CPU/GPU/memory.
I'd like to show users how to optimise these workflows. I'm not an expert on tuning requests, I'm more like a user myself. So as a user I want to know:

How do I know which workflows are costly?
How should I tune the workflow?
How can I monitor cost change over time?

Would anyone be interested in help me out with this please?

@alexec i'd absolutely love to see Kubecost support this and also provide content!
have you seen the integration with spinnaker? https://blog.kubecost.com/blog/spinnaker-kubecost-integration/
i've talked with half a dozen join users that also expressed interest in something similar to this. let me know if'd want to chat about it or if there is more you can share!

Great, I just grabbed time. Sorry for my delay... I recently had a kid, and have been... busy!

@alexec saw this was meeting was canceled on the argo side. know if that was intentional?

agro v2.2 now has a more specific annotation to track resources:
  annotations:
    argocd.argoproj.io/tracking-id: my-app:apps/Deployment:default/nginx-deployment

@alexec draft outline available here! https://docs.google.com/document/d/1JmvkyJe0iJ8TFGX3fT5kP_S6jB_CJ6u_yn9H0YH03So/edit

This issue is being closed because it may not be relevant to the OpenCost project and appears stale. If you feel this was closed in error, please open a new OpenCost issue with updated details or if it is still relevant for Kubecost please open an issue with Kubecost Support.


Opening the diff to a commit results in an error

Description
When attempting to view the diff of a commit in neogit when highlighting the commit and pressing enter, instead of a view popping up showing the diff, an error is shown.
Neovim version
NVIM v0.10.0
Build type: Release
LuaJIT 2.1.1716656478
Run "nvim -V1 -v" for more info
Operating system and version
EndeavourOS
Steps to reproduce

Init a git repo
Create an empty file and add text to it
Commit the file
nvim -nu minimal.lua
Open neogit and select the commit

Expected behavior
A popup should appear, displaying the diff of the commit
Actual behavior
Neovim gives an error, being this:
E5108: Error executing lua: ...test-repo/.min/plugins/neogit/lua/neogit/lib/git/log.lua:53: Failed to parse line: ^[[1mdiff --git a/tmp.txt b/tmp.txt^[[m
stack traceback:
        [C]: in function 'error'
        ...test-repo/.min/plugins/neogit/lua/neogit/lib/git/log.lua:53: in function 'parse'
        ...n/plugins/neogit/lua/neogit/buffers/commit_view/init.lua:51: in function 'new'
        ...min/plugins/neogit/lua/neogit/buffers/status/actions.lua:1100: in function 'cb'
        .../test-repo/.min/plugins/neogit/lua/neogit/lib/buffer.lua:580: in function <.../test-repo/.min/plugins/neogit/lua/neogit/lib/buffer.lua:579>

Minimal config
-- NOTE: See the end of this file if you are reporting an issue, etc. Ignore all the "scary" functions up top, those are
-- used for setup and other operations.
local M = {}

local base_root_path = vim.fn.fnamemodify(debug.getinfo(1, "S").source:sub(2), ":p:h") .. "/.min"
function M.root(path)
  return base_root_path .. "/" .. (path or "")
end

function M.load_plugin(plugin_name, plugin_url)
  local package_root = M.root("plugins/")
  local install_destination = package_root .. plugin_name
  vim.opt.runtimepath:append(install_destination)

  if not vim.loop.fs_stat(package_root) then
    vim.fn.mkdir(package_root, "p")
  end

  if not vim.loop.fs_stat(install_destination) then
    print(string.format("> Downloading plugin '%s' to '%s'", plugin_name, install_destination))
    vim.fn.system({
      "git",
      "clone",
      "--depth=1",
      plugin_url,
      install_destination,
    })
    if vim.v.shell_error > 0 then
      error(string.format("> Failed to clone plugin: '%s' in '%s'!", plugin_name, install_destination),
        vim.log.levels.ERROR)
    end
  end
end

---@alias PluginName string The plugin name, will be used as part of the git clone destination
---@alias PluginUrl string The git url at which a plugin is located, can be a path. See https://git-scm.com/book/en/v2/Git-on-the-Server-The-Protocols for details
---@alias MinPlugins table<PluginName, PluginUrl>

---Do the initial setup. Downloads plugins, ensures the minimal init does not pollute the filesystem by keeping
---everything self contained to the CWD of the minimal init file. Run prior to running tests, reproducing issues, etc.
---@param plugins? table<PluginName, PluginUrl>
function M.setup(plugins)
  vim.opt.packpath = {}                      -- Empty the package path so we use only the plugins specified
  vim.opt.runtimepath:append(M.root(".min")) -- Ensure the runtime detects the root min dir

  -- Install required plugins
  if plugins ~= nil then
    for plugin_name, plugin_url in pairs(plugins) do
      M.load_plugin(plugin_name, plugin_url)
    end
  end

  vim.env.XDG_CONFIG_HOME = M.root("xdg/config")
  vim.env.XDG_DATA_HOME = M.root("xdg/data")
  vim.env.XDG_STATE_HOME = M.root("xdg/state")
  vim.env.XDG_CACHE_HOME = M.root("xdg/cache")

  -- NOTE: Cleanup the xdg cache on exit so new runs of the minimal init doesn't share any previous state, e.g. shada
  vim.api.nvim_create_autocmd("VimLeave", {
    callback = function()
      vim.fn.system({
        "rm",
        "-r",
        "-f",
        M.root("xdg")
      })
    end
  })
end

-- NOTE: If you have additional plugins you need to install to reproduce your issue, include them in the plugins
-- table within the setup call below.
M.setup({
  plenary = "https://github.com/nvim-lua/plenary.nvim.git",
  telescope = "https://github.com/nvim-telescope/telescope.nvim",
  diffview = "https://github.com/sindrets/diffview.nvim",
  neogit = "https://github.com/NeogitOrg/neogit"
})
-- WARN: Do all plugin setup, test runs, reproductions, etc. AFTER calling setup with a list of plugins!
-- Basically, do all that stuff AFTER this line.
require("neogit").setup({}) -- For instance, setup Neogit

Thanks for letting me know :)


OADP-2048: Add replicationsource cron expression support

Related OADP Operator PR: https://github.com/openshift/oadp-operator/pull/1080

/cherry-pick oadp-1.2


"[Diapers Calculator]  Different sizes of containers on the diapers calculator page "

Environment: Windows 11 Home Version 22H2 Build	22621.2506
Chrome Версія 118.0.5993.118 (Official build) (64-bit)
Reproducible: always,
Build found:
Preconditions
https://zero-waste-staging.onrender.com/uk is opened in browser
Steps to reproduce

Click on 'Let`s go' button

Actual result
Diapers Calculator page is opened with containers having different size
Expected result
Сontainers of the same size are located on 'Diapers Calculate` page
User story and test case links
Labels to be added
"Bug", Priority ("pri:  medium"), Severity ("severity: low"), Type ("UI)

Fixed


80-test_cmp_http.t: some generalizations and minor improvements of diagnostic output

This allows the use of specific environment variables, which is needed for our tests with CMP servers other than the internal mock server.
Also tweak the sleep() calls unfortunately needed with one of the external test servers
and enhance the wording and information contained in the status output of the script.

[x] tests are added or updated

The failed external oqsprovider test is unrelated.

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

Thanks for the initial review by @t-j-h. Both comments are handled.
Ping for further reviewing and approvals. The changes are not a big deal.

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This pull request is ready to merge


net/wireguard - Fix additional regression pertaining to opnsense#3571

Issue appears to be in another area.


Plex not sending data

Not sure if I should be sending this to Plex instead, but I'm trying to cover my bases.
I'm sure I've set everything up as I should. Manually checking a specific video works.
However, I'm not getting multiple "POST" log messages as some others have, actually no messages at all. PSD is adding it's own webhook to Plex and is running.
If I POST the example payload from Plex's webpage I get a TypeError, but I figure that is because the payload is malformed or something. I can at least send something and PSD reacts.
Is there a log somewhere in Plex where I would see if Plex is trying something? I tried debug and console, but got nothing useful from them (why even have two places for a log...)
Running plexmediaserver of version <IP_ADDRESS>00-248a2daf0 and latest PSD from pip3, they're both running on the same machine with Debian.

Wow, Plex does not send webhooks unless Push Notifications is enabled also under Settings > General.
Even then, the first time I enabled it it took half hour to actually send a webhook.
I did get an error once that wasn't Plexs fault:
ERROR:plex_sub_downloader.plex_sub_downloader:Exception on /webhook [POST]
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/dist-packages/flask/app.py", line 2190, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.9/dist-packages/flask/app.py", line 1486, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.9/dist-packages/flask/app.py", line 1484, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.9/dist-packages/flask/app.py", line 1469, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
  File "/usr/local/lib/python3.9/dist-packages/plex_sub_downloader/plex_sub_downloader.py", line 28, in respond
    psd.handleWebhookEvent(event)
  File "/usr/local/lib/python3.9/dist-packages/plex_sub_downloader/PlexSubDownloader.py", line 53, in handleWebhookEvent
    self.handleLibraryNewEvent(event)
  File "/usr/local/lib/python3.9/dist-packages/plex_sub_downloader/PlexSubDownloader.py", line 68, in handleLibraryNewEvent
    self.handleDownloadingVideoSubtitles(video)
  File "/usr/local/lib/python3.9/dist-packages/plex_sub_downloader/PlexSubDownloader.py", line 85, in handleDownloadingVideoSubtitles
    subtitles = self.downloadSubtitlesForVideos(missingVideos)
  File "/usr/local/lib/python3.9/dist-packages/plex_sub_downloader/PlexSubDownloader.py", line 186, in downloadSubtitlesForVideos
    subtitles = self.sub.search_videos(videos)
  File "/usr/local/lib/python3.9/dist-packages/plex_sub_downloader/subliminalHelper.py", line 52, in search_videos
    subtitles = subliminal.download_best_subtitles(subVideos, languages=self.languages, providers=self.providers, provider_configs=self.provider_configs)
  File "/usr/local/lib/python3.9/dist-packages/subliminal/core.py", line 651, in download_best_subtitles
    subtitles = pool.download_best_subtitles(pool.list_subtitles(video, languages - video.subtitle_languages),
  File "/usr/local/lib/python3.9/dist-packages/subliminal/core.py", line 139, in list_subtitles
    provider_subtitles = self.list_subtitles_provider(name, video, languages)
  File "/usr/local/lib/python3.9/dist-packages/subliminal/core.py", line 102, in list_subtitles_provider
    if not provider_manager[provider].plugin.check(video):
  File "/usr/local/lib/python3.9/dist-packages/stevedore/extension.py", line 340, in __getitem__
    return self._extensions_by_name[name]
KeyError: 'addic7ed'

And my config.json was:
{
    "plex_base_url": "http://ftp.int.durd.net:32400",
    "plex_auth_token": "<token>",
    "languages": [
        "eng",
        "swe"
    ],
    "webhook_host": "<IP_ADDRESS>",
    "webhook_port": 5000,
    "subtitle_destination": "with_media",
    "subtitle_providers": [
        "opensubtitles",
        "tvsubtitles",
        "addic7ed"
    ],
    "subtitle_provider_configs": {
        "opensubtitles": {
            "username": "<user>",
            "password": "<pass>"
        }
    }
}

Once I removed addic7ed it worked again.

Hm thanks for the note about needing to have push notifications enabled, I'll add that to the readme.
And as for why addic7ed isn't working, it looks like it's partially an issue within Subliminal.
According to this issue, addic7ed added a captcha to prevent things like Subliminal from searching. But the specific reason why we're getting that KeyError: 'addic7ed' is because there's a typo in the subliminal/providers/addic7ed.py file.
For the life of me, I don't know why they don't just completely remove addic7ed as a provider 🤷


Request to host package

This issue was originally filed by<EMAIL_ADDRESS>
https://github.com/prujohn/Buckshot
Latest merge to trunk has a lot of enhancements and bug fixes.  If there is enough time before launch, a refresh to the pub repo would be very much appreciated. :)

This issue has been moved to dart-lang/pub#185.


no matches for kind "Broker" in version "eventing.knative.dev/v1 error

I am following the instructions here:
https://github.com/knative-sandbox/eventing-rabbitmq/tree/master/broker
Everything ok until I try:
kubectl apply -f - << EOF
  apiVersion: eventing.knative.dev/v1
  kind: Broker
  metadata:
    name: default
    annotations:
      eventing.knative.dev/broker.class: RabbitMQBroker
  spec:
    config:
      apiVersion: v1
      kind: Secret
      name: rokn-rabbitmq-broker-secret
EOF

I get this error:
no matches for kind "Broker" in version "eventing.knative.dev/v1"

Am I missing something?
Thank you

Did you install the Knative eventing?
As in, you might not have the CRDs that you get from eventing in there?
What do you get when you do this:
vaikas-a01:wabbit vaikas$ kubectl get crds | grep brokers
brokers.eventing.knative.dev                         2020-06-26T19:48:17Z

Sorry, my fault.
I had some problem to follow the example.  It wasn't totally clear to me.

Oh no worries, maybe need to make the pre-requisites bigger and flashier :)


Fix BASH shell for untagged images (orphans)

The code as it sits doesn't do what it should (at least it didn't for me), here is a fix for the line that will make it work for the BASH example above:
az acr repository show-manifests --name $REGISTRY --repository $REPOSITORY --query "[?tags==null].digest" -o table
| cut -f 1 -d ' ' | xargs -I% az acr repository delete --name $REGISTRY --image $REPOSITORY@% --yes
the  cut -f 1 -d ' '  pipe insertion will take the sha256:9be28e093ed318f2c91281c67af6e5ed760aaf3f1f944efede04e91b79a12635  2019-01-02T22:56:59.5068157Z identifier and strip off the date and last part so the xargs works correctly.
Note I also use table output in the command.

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

ID: cb1b3438-58e2-cd26-3b5e-1a1717fe182f
Version Independent ID: ad41d0b4-39b1-7e24-836e-ff12d5016623
Content: Delete image resources in Azure Container Registry
Content Source: articles/container-registry/container-registry-delete.md
Service: container-registry
GitHub Login: @dlepow
Microsoft Alias: danlep

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@mtmazurik - Marty, thanks for providing helpful feedback on the script. I found that your updated query statement --query "[?!(tags[?'*'])].digest" correctly selected untagged images in my test repo. However, when I formatted the output using --output tsv (as the command was originally written) I found I didn't need to apply your cut command. In my case, I found that the query output contained only the digest:
sha256:1a6fd470b9dddd0849be79e99529a88371dff60c60aab424c077007f6999b4812

I didn't need to strip additional text, and the xargs command worked fine as written.
I'll plan to update the article with your updated query statement. If you have other input, please let us know.
Thanks!
#in-progress

Thanks, Dan for update, and yeah … I didn’t go back to tsv after playing, so glad you worked that out and I figured there was just some changes to the internal structure of the JSON that necessitated the change to the query,
Glad to help out (minorly) that’s cool,
Thanks!
marty
From: Dan Lepow<EMAIL_ADDRESS>Sent: Friday, January 4, 2019 3:14 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Marty Mazurik<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Fix BASH shell for untagged images (orphans) (#21537)
@mtmazurikhttps://github.com/mtmazurik - Marty, thanks for providing helpful feedback on the script. I found that your updated query statement --query "[?!(tags[?'*'])].digest" correctly selected untagged images in my test repo. However, when I formatted the output using --output tsv (as the command was originally written) I found I didn't need to apply your cut command. In my case, I found that the query output contained only the digest:
sha256:1a6fd470b9dddd0849be79e99529a88371dff60c60aab424c077007f6999b4812
I didn't need to strip additional text, and the xargs command worked fine as written.
I'll plan to update the article with your updated query statement. If you have other input, please let us know.
Thanks!
#in-progress
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/21537#issuecomment-451596965, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADjn-1NAQw-XuosgbU3IsRCHPqNEWMXlks5u_-A5gaJpZM4ZnW7w.


How to make Checkbox (JS version) from original mdc?

Hi. I don't see in demo this version (on the buttom) - https://material-components-web.appspot.com/checkbox.html
Also wonder, is this lib compatible with SSR, Nuxt?

@jerakode
Hi,

regarding the checkbox, you have the demo @ input-controls , although the ripple effect is not working yet and I have not added the indeterminate style yet.

regarding SSR and Nuxt, I'm not an expert. In theory everything should work but the mdc-v-ripple directive. componnents should be ok as far as I understand, but I may have missed something from the doc. Did you try it ?

<mdc-checkbox  label="A Checkbox" v-model="isChecked"></mdc-checkbox>

var vm = new Vue({
  data: {
    isChecked: true
  }
})

I have added indeterminate prop and the ripple
https://github.com/stasson/vue-mdc-adapter/tree/master/src/components/checkbox
input-controls

I don't know why, but Nuxt even doen't see this package
This dependency was not found:

To install it, you can run: npm install --save vue-mdc-adapter

Hey. There's a little bug in https://github.com/stasson/vue-mdc-adapter/blob/master/package.json#L15 - https://github.com/nuxt/nuxt.js/issues/917#issuecomment-309280051
AND THANKS for awesome package! It's working!

Also wonder, will Radio will also be upradeted with ripple feature?

Unfortunately, got errors SSR-related. As always. SSR is nightmare.

Sorry again, I fixed it for SSR. If curious:
<template>

      <mdc-checkbox v-if="isChecked" label="A Checkbox"></mdc-checkbox>

            <mdc-radio v-if="isRadio" label="Radio1" name="radios" value="radio1" v-model="picked" checked></mdc-radio>
            <mdc-radio v-if="isRadio" label="Radio2" name="radios" value="radio2" v-model="picked"></mdc-radio>

{{ picked }}

</template>

No SSR errors! Beautiful!
@stasson this package is awesome.
How can I contribute?

@jerakode

Thanks to nuxt/nuxt.js#917 (comment)

Nice finding and thanks for letting me know! I have open #27 for this bug.

Also wonder, will Radio will also be upgraded with ripple feature?

Yes, I plan to support same javascript features and ripples as MDC vanilla components. Radio was not my priority yet ( I have lot of work to do with finalizing api's and bundling ...) But if you need it I'll try to do it within the next couple of weeks.

How can I contribute?

I'll welcome any help. I you find bugs or feature improvements don't hesitate to open issues or submit pull request. The way that I work is as follow: I usually review and decide if I merge directly or if it needs some rework. I have started a wiki](https://github.com/stasson/vue-mdc-adapter/wiki) to put roadmap and contribution infos.
Right now I'm still in development phase so the APIs ( props events and slots ) may change significantly. Also I'm not fully satisfied with the bundling and demo site so this will probably change as well. I'm explaining that so that you don't get upset when I'm doing breaking changes. Once we reach beta I'll freeze the APIs and it will be easier for everyone.
Also, as you know how to use nuxt and ssr, may be you could help there. I have started to read the nuxt doc and it seems you can generate static sites with nuxt. May be we could use nuxt to generate the demo and doc site? would you agree to help with that ?

@stasson sure, I can help with that. I just tried to generate static site with Nuxt and vue-mdc-adapter - worked just fine, http://unwieldy-stream.surge.sh/
Had one webpack caution
WARNING in asset size limit: The following asset(s) exceed the recommended size limit (300 kB).
This can impact web performance.
Assets:
  nuxt.bundle.1f5672929109b4eef126.js (387 kB)

WARNING in entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (300 kB). This can impact web performance.
Entrypoints:
  app (657 kB)
      manifest.77715a711fb590abe21a.js
      vendor.bundle.0406979ce3c26b14ca89.js
      nuxt.bundle.1f5672929109b4eef126.js

But I think it can be solved in future.
As for Radio - I am not hurry you up, do what you consider priority!

@stasson we can easily copy MDC https://material-components-web.appspot.com/card.html
And it will be even better due to Nuxt SPA UX.
Will be happy to help.


confusion and inconsistency with config (mode+target) & cli flags

Is your feature request related to a problem? Please describe.
I, fist thanks for your awesome job
I would like to expose a comprehension problem I had with nuxt with the add of the target property
also according to the documention  build have a --spa flag but there is no ssr flag, it's inconsistent
the target only take effect on universal mode ( when I tested I in 2.13) because in spa mode even if you set the target to server nuxt will build spa with static target (that make sense)
but why add a new property to something which only take effect on 1 mode ? it should be more easy to add a new mode "static" ?
Describe the solution you'd like
a good solution could be deprecated mode and target and use the cli, nuxt [action] [mode] or nuxt [mode] [action] depending on what makes the most sense to you and what is the most future-proof.
in parallel we could add a prefixe to dist / cache folders dist/ -> [mode]-dist/, .[mode]-nuxt/ or in the other way .nuxt/ -> .nuxt-[mode]/ one more time depending on what makes the most sense to you and what is the most future-proof.
I think the 2th proposition could help you in internal to serve the good build and it should also help a cache issue when we change for an another mode (delete dist / .nuxt is boring)
thanks for reading :)

This is a helpful point to make, and we're working to improve things. For example: https://nuxt.com/docs/guide/concepts/rendering/. If you have any particular suggestions, we'd welcome ideas or PRs. ❤


cjdns: added option addExtraHosts

Motivation for this change
This commit restores the automatic addition of cjdns ipv6 address -> hostname mapping
as used to work in nixos-16.09 and before.  It is now optional and turned off by default
because some people complained, that evaluating nixpkgs compiled cjdns
(needed for publicKey to ipv6 conversion).
Things done

[X] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

if it works it's cool with me

To preserve history, I think it'd be better to revert 60ded3f3632d221ca3f82a52392e155517880456 and then add the conditional as a separate commit.

Closed in deference to #21592


Pod labels for Kiali Operator Helm Chart

https://github.com/kiali/kiali/issues/5832

Did you test this? Because it doesn't even build.
$ make build-helm-charts
Will use this helm executable: /home/jmazzite/source/kiali/helm-charts/_output/helm-install/helm
Building Helm Chart for Kiali operator
==> Linting /home/jmazzite/source/kiali/helm-charts/_output/charts/kiali-operator
[ERROR] templates/: template: kiali-operator/templates/deployment.yaml:22:20: executing "kiali-operator/templates/deployment.yaml" at <.Values.deployment.podLabels>: nil pointer evaluating interface {}.podLabels

Error: 1 chart(s) linted, 1 chart(s) failed
make: *** [Makefile:75: .build-helm-chart-operator] Error 1

Did you test this? Because it doesn't even build.
$ make build-helm-charts
Will use this helm executable: /home/jmazzite/source/kiali/helm-charts/_output/helm-install/helm
Building Helm Chart for Kiali operator
==> Linting /home/jmazzite/source/kiali/helm-charts/_output/charts/kiali-operator
[ERROR] templates/: template: kiali-operator/templates/deployment.yaml:22:20: executing "kiali-operator/templates/deployment.yaml" at <.Values.deployment.podLabels>: nil pointer evaluating interface {}.podLabels

Error: 1 chart(s) linted, 1 chart(s) failed
make: *** [Makefile:75: .build-helm-chart-operator] Error 1

make build-helm-charts now runs successfully on my machine! Fixed in 7e8767b84ffdafb9ba1c422a3fe86b194328235c. Maybe after this PR I can open a PR to fix the different conventions between server and operator ( values.deployment.pod_labels and values.podLabels, value.deployment.pod_annotations vs values.podAnnotations etc.)?

How I tested:

make build-helm-charts
helm template --show-only templates/deployment.yaml --set podLabels.testLabel=testValue kiali-operator _output/charts/kiali-operator-1.64.0-SNAPSHOT.tgz
Confirm the testLabel: testValue label is in the pod template:

spec:
...
  template:
    metadata:
      name: kiali-operator
      namespace: default
      labels:
...
        testLabel: testValue


Add support for subnet nat gateway association

One of the common configuration to nat gateway is to subnet nat gateway association. see below:
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/subnet_nat_gateway_association

+1 :)

+1


Skip a givne number of elements

This is similar but different from #5.
There are some use case for skipping a given number of elements.
Once again, Elias Fano makes it possible by counting zeroes and ones in the unary encoded sequence.
One use case was met in tantivy when encoding positions. Given a term, term positions for all of the documents are concatenated and encoded as an increasing sequence.
Accessing the positions in the n-th document is made possible by suming all of the term frequency in
document 1 to (n-1)th and skipping the same amount of values in the elias fano encoded positions.

Thanks for this. Correct me if I'm wrong, but is what you're asking here similar to the functionality of something like the following:
By skipping the same number of EF encoded values are you not performing a visit on the current internal position plus the number that you would like to skip? Something like ef.visit(ef.position() + num_to_skip).
If that's not the case, I might simply be misinterpreting what you're after. Let me know if that is it or not. :)

Oh yes you are right. This is the same spec.
The implementation must be very slow : you could use a popcount.
    for _ in 0..skip {
        pos = get_next_set(&self.b, (pos + 1) as usize);
    }

Fantastic, I'll have a look into maximizing performance and providing a few more helpful methods for common use cases. I will take the performance optimizations into a separate issue. Thanks!


[Wave6] Push updates to the policy settings when made in OldDot

Problem
When admin changes isPreventSelfApprovalEnabled or autoReportingOffset or the submit and close/ submit& approve settings in oldDot, we do not send pusher updates to the users to let them know about this change.
This is important so we can keep the optimistic actions correct for example the optimistic next steps too
cc @rezkiy37 for visibility
Solution
When these settings are changed in oldDot, make sure to queue updates to all policy members
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01311fa61bb7e55b98
Upwork Job ID:<PHONE_NUMBER>755016704
Last Price Increase: 2024-01-19

PR is up, need to clean it up

PR is in review

Merged and deployed


Defining language.decimal on dataTable.defaults is breaking built in type detection

Like the title says, when I define the default value for the language.decimal the built in type detection wont recognize my int table as a 'num'
Debugging the dataTable code, I found that the code returns 'num,' as the type and this somehow brokes the sort.
return _isNumber( d, decimal ) ? 'num'+decimal : null;
When I define type: 'num' on the columnDef when creating the DataTable all goes right.
Same when I remove the default decimal value, all goes right.
Edit: http://jsfiddle.net/d2k5bbt9/

Continuing... I found that it works if I define the language in the DataTable initialization/declaration.
It's related that how the component initialize itself: it checks if the language has been informed but doesn't check for the default configuration of language:
		if ( oInit.oLanguage )
		{
			_fnLanguageCompat( oInit.oLanguage );
		}


Login to phpvms forum

Hi Nabel,
now that the forum is open, I can not make login  this error appears: "Username or password incorrect."  but the password and mail  is correct.
Thank you for all Nabel and excuse my english ;-)

Can you try resetting your password?

ok now works thanks Nabel   ;-)


Add dataset column？

Hello, the Awesome project is nice! I suggest you can add the corresponding dataset benchmarks for each paper.

Hello, the Awesome project is nice! I suggest you can add the corresponding dataset benchmarks for each paper.

Thank you for your constructive suggestion. I will continue to maintain this project at my convenience.


Invalid SQL generated with multiple OrderBy calls

I have something like this:
    protected IQuery<ContentItem, TIndex> PublishedItemQuery<TIndex>()
        where TIndex : class, YesSql.Indexes.IIndex =>
        Session.Query<ContentItem, ContentItemIndex>()
                   .Where(q => q.Published)
                   .OrderByDescending(q => q.CreatedUtc)
               .With<TIndex>();

When I use an additional ordering...
await PublishedItemQuery<FBOIndex>()
    .OrderBy(q => q.Name)
.Skip(offset)
    .Take(limit)
    .ListAsync();

I get a syntax exception using SQLite:
00:01:13.518|Default||6e3ea404-46efc8b3d974e25f.||IStore|Debug|14|SELECT DISTINCT [Document].*,[ContentItemIndex].[CreatedUtc],[FBOIndex].[Name] FROM [Document] INNER JOIN [ContentItemIndex] ON [ContentItemIndex].[DocumentId] = [Document].[Id]  INNER JOIN [FBOIndex] ON [FBOIndex].[DocumentId] = [Document].[Id]  WHERE ([ContentItemIndex].[Published] = @p0) ORDER BY [ContentItemIndex].[CreatedUtc] DESC[FBOIndex].[Name]|

Specifically, there's a comma missing between the fields.
ORDER BY [ContentItemIndex].[CreatedUtc] DESC[FBOIndex].[Name]

I may be doing something disallowed here, but even so, I don't think that's how it should manifest.
Since I'm calling OrderBy() rather than ThenBy(), I actually expected the ordering to be replaced with the second one rather than to end up with ordering by both fields.

Looks like a valid issue


Tall mushrooms can generate on large mushrooms

As per the title, in a world generated with seed<PHONE_NUMBER>160316681 (the same one where I discovered #20) in a Mushroom Fields biome at 44, ~, 5844, I found a handful of large mushrooms that had generated with the new tall mushrooms growing out of them. I figured that this is unintended, so here I am to report it. This can occur with brown and red tall mushrooms, and I saw them on brown and red large mushrooms and green large glowshrooms.

Four instances of this can be seen in this screenshot.

This is intended.

This is intended.


faup.tcl depends on itcl-3.4 causing dependency conflicts on Gentoo

On current Gentoo, itcl-3.4 depends on tcl<8.6, but the only available version of tclx, 8.4.1, depends on tcl>8.6.
https://github.com/flightaware/piaware/blob/2da8bd97686d3c072ed60ad5643062528f99f92e/programs/piaware/faup.tcl#L3
Can this requirement be increased or unbounded to also accept itcl>4 which depends on tcl>8.6?

It can be relaxed, but this does point to a packaging problem in Gentoo.
Itcl3 and itcl4 are effectively different implementations, itcl3 can run on
8.6 just fine and can coexist with itcl4 (not in the same interpreter, but
certainly on the same system); Gentoo's packaging should change to allow
this, much as Debian's does

Good to know, I'll check that out and report it to them. Thanks.

Same in Alpine (my experience), and ArchLinux too. https://aur.archlinux.org/packages/piaware-git/#comment-690453. Would be nice if it could be 'worked around' on Piaware side. Unless trouble is possible.
https://core.tcl-lang.org/itcl/dir?ci=tip claims:

This version [4.1.2] is the next major release to follow
Itcl 3.4.  This version claims to be script level compatible with Itcl 3.4.

This was already fixed on the dev branch (see the commit above)

Overlooked it. 🤦♂ Thanks.

FWIW, the original reason for requiring itcl 3 was that there's a performance regression going from itcl3 to itcl4. This doesn't really matter for piaware but it's significant on the server side so we avoid using itcl4 in other tcl code.


🐛 Bug Report: User entity not found error

📜 Description
When I open my profile in backstage instance where i can see 2 options
User Entity: [user:default/username]
Ownership Entities: [user:default/username]
after clicking on one I am getting this user entity not found error
I am using GitHub Oauth app for login authentication
👍 Expected behavior
why i am getting this error after clicking on my username
i thought it will show my identity
👎 Actual Behavior with Screenshots

👟 Reproduction steps
go to profile
📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Hey! 👋
Are you doing this?
https://backstage.io/docs/integrations/github/org
By the looks of it, you have authentication set up, but don't have an associated user in backstage created (which have to come from somewhere).
👍

Hi! 👋
how can we create user in backstage ?
I am somewhat new to this whole thing
this is my code from catalog.ts
export default async function createPlugin(
env: PluginEnvironment,
): Promise {
const builder = await CatalogBuilder.create(env);
builder.addEntityProvider(
GithubEntityProvider.fromConfig(env.config, {
logger: env.logger,
// optional: alternatively, use scheduler with schedule defined in app-config.yaml
schedule: env.scheduler.createScheduledTaskRunner({
frequency: { minutes: 30 },
timeout: { minutes: 3 },
}),
// optional: alternatively, use schedule
scheduler: env.scheduler,
}),
);
builder.addProcessor(new ScaffolderEntitiesProcessor());
builder.addProcessor(new EntityAnnotationPreprocessor());
const { processingEngine, router } = await builder.build();
await processingEngine.start();
const unprocessed = new UnprocessedEntitiesModule(
await env.database.getClient(),
router,
);
unprocessed.registerRoutes();
return router;
}
this is my auth.ts
github: providers.github.create({
signIn: {
resolver: async (info, ctx) => {
const username = info.result.fullProfile.username;
if (!username) {
// This should never happen...
throw new Error('No username found for the user.');
}
const userRef = user:default/${username};
        return ctx.issueToken({
          claims: {
            sub: userRef,
            ent: [userRef],
          },

@Pvv7707 is this a brand new project or have you done some form of upgrade, wondering if this has started to break recently with a change or if there's something else happening here.

Yes its a new project i needed these user identities to check the entity feedback plugin to work because without the user identities we cant see the feedback responses
below code is my catalog.ts
after using this GithubOrgEntityProvider instead of GithubEntityProvider i am not able to see the components loading in catalog
is there anything i am missing ?
export default async function createPlugin(
  env: PluginEnvironment,
): Promise<Router> {
  const builder = await CatalogBuilder.create(env);
    builder.addEntityProvider(
        GithubOrgEntityProvider.fromConfig(env.config, {
            // this is the ID you've configured in your config.
            // your config example shows you set one up called "development"
            id: 'development',
            // the host and org you want to ingest; you likely don't want to load this org,
            orgUrl: 'url',
            logger: env.logger,
            schedule: env.scheduler.createScheduledTaskRunner({
                frequency: { minutes: 30 },
                timeout: { minutes: 3 },
            }),
        }),
    );

Yes its a new project i needed these user identities to check the entity feedback plugin to work because without the user identities we cant see the feedback responses
below code is my catalog.ts after using this GithubOrgEntityProvider instead of GithubEntityProvider i am not able to see the components loading in catalog is there anything i am missing ?
export default async function createPlugin(
  env: PluginEnvironment,
): Promise<Router> {
  const builder = await CatalogBuilder.create(env);
    builder.addEntityProvider(
        GithubOrgEntityProvider.fromConfig(env.config, {
            // this is the ID you've configured in your config.
            // your config example shows you set one up called "development"
            id: 'development',
            // the host and org you want to ingest; you likely don't want to load this org,
            orgUrl: 'url',
            logger: env.logger,
            schedule: env.scheduler.createScheduledTaskRunner({
                frequency: { minutes: 30 },
                timeout: { minutes: 3 },
            }),
        }),
    );

Hey!
Sorry, I think I miss-read your initial issue, with the GithubEntityProvider it looks like (although you have omitted it in your screenshot I assume) you do get a user entity which is user:default/your-username but I am not sure how you got that user there in the first place 😅 did you import it with a .yaml?
The GithubOrgEntityProvider will only import users and groups automatically AFAIK. However, I think you should still be able to use both providers they are not mutually exclusive.
The GithubEntityProvider will automatically pull in catalog-info.yaml files I believe, or at least it says here:

The GitHub integration has a discovery provider for discovering catalog entities within a GitHub organization. The provider will crawl the GitHub organization and register entities matching the configured path.

Please correct me if I am wrong @benjdlambert
Hope this helps 👍

Hi @Parsifal-M
i was able to add the user identity (myself) inside backstage manually through ts file and providing the location in app config yaml.
But i need to add all my org groups and users individually, how is that possible?
can you please provide some insight?
thanks in advance :)

Hi @Pvv7707, the GitHub Org Provider is what you are looking for an it's documented here: https://backstage.io/docs/integrations/github/org

Thanks @awanlin for quick response
i have org provider added in my catalog.ts but i was not able to see the user identities when i click on profile and then on my identity as i mentioned in my first comment i am getting this error
https://user-images.githubusercontent.com/142504329/261859270-b0a61dc8-dfc7-489b-a223-c031544e91b6.png

Right, the GitHub Org provider will bring in all the users and group that you have access to from GitHub. If you go to http://localhost:3000/catalog?filters[kind]=group&filters[user]=all or http://localhost:3000/catalog?filters[kind]=user&filters[user]=all you should be able to see the Groups or Users that have been imported.
Once that's in place then you can update your auth.ts to do the mapping which is the initial issue you logged.

The dos on how to setup the auth.ts are here: https://backstage.io/docs/auth/identity-resolver
In you case you might want to use this: providers.github.resolvers.usernameMatchingUserEntityName()
The code for how that works is here: https://github.com/backstage/backstage/blob/263e3846d8e0d2270800f7ddc224da902912f53e/plugins/auth-backend/src/providers/github/provider.ts#L135

Thanks again
I can see the groups and all in this http://localhost:3000/catalog?filters[kind]=group&filters[user]=all
I also need  modifying the user for some conditional to check if the user exists in the DB. if they do not, you would create them there before issuing a token.
is that condition possible something like this
if (!userExists) {
  createuser(userRef); // or username, whichever the plugin needs to work
}

That's actually not a recommended pattern and will cause a bad user experience when they log in.
I'd recommend figuring our why Groups get ingested by the GitHub Org Provider and not Users. You should be able to pull in Users this way, it's how many Backstage Adopters do this
@Parsifal-M, any tips on this? I haven't used this myself

Hey @Pvv7707
Can we close this one out? Unless you need more help on this topic? 👍

Thanks for all the help provided


cargo 1.50.0 can not cope with filedescriptor-0.8.0

dooku-~ mcr 10001 %cargo install filedescriptor
Updating crates.io index
Downloaded filedescriptor v0.8.0
error: failed to parse manifest at /home/mcr/.cargo/registry/src/github.com-1ecc6299db9ec823/filedescriptor-0.8.0/Cargo.toml
I expected to get installed. (I came across this after an cargo update on another project)
Steps

install cargo 1.50.0
try to install this version of filedescriptor (which I think is the latest)

Possible Solution(s)
Upgrade to cargo 1.53.0
Notes
The comment at the top of that file says:
# If you believe there's an error in this file please file an
# issue against the rust-lang/cargo repository. If you're
# editing this file be aware that the upstream Cargo.toml
# will likely look very different (and much more reasonable)

I think that the goal of the rewriting is to keep people from being forced to upgrade.

It can be helpful to post the full message:
error: failed to download `filedescriptor v0.8.0`

Caused by:
  unable to get packages from source

Caused by:
  failed to parse manifest at `/Users/eric/.cargo/registry/src/github.com-1ecc6299db9ec823/filedescriptor-0.8.0/Cargo.toml`

Caused by:
  feature `resolver` is required

  this Cargo does not support nightly features, but if you
  switch to nightly channel you can add
  `cargo-features = ["resolver"]` to enable this feature

As you may see, 1.50 does not support this feature. You'll need at least 1.51.   When a project uses new features or syntax, they may require a newer version of Rust to use.
Also, from what I can see, filedescriptor is not a binary crate, so cargo install won't work.

I'm going to close, as I believe the error message indicates why it is unable to use that package.  I think that is about the best the error message can do, as it does not know when a particular feature gets stabilized.   Of course, if you have any suggestions on how it could be clearer, feel free to suggest them.

It can be helpful to post the full message:

Yes, but how does telling me to right nightly cargo, sync with the VERY VERY clear message in the generated Cargo.toml, that tells me that if it doesn't work, then it's a bug that should be reported.
It seems to me that the ENTIRE PURPOSE of rewriting the Cargo toml is to make it backwards compatible.
The message
this Cargo does not support nightly features, but if you
switch to nightly channel you can add

so, this has nothing to do with a nightly feature, but in fact a feature deployed.  Perhaps when the Cargo toml is updated, then it should contain a minimum Cargo version that it can be used with?

Ah, I think I understand.  I think the error message could be improved when it comes from a registry dependency, since the user does not have control over those.
The preamble in the rewritten Cargo.toml also probably isn't too helpful.
There is an unstable enhancement to add specifying the rust version, but it has not been stabilized, yet.


kernel SMB client does not automatically update IP address of the cifs mount after DNS update.

Describe the bug
kernel SMB client does not automatically update IP address of the cifs mount after DNS update.
This is a known issue in some early versions of Linux kernel and should have been fixed.  However i can still reproduce this issue on latest AzureLinux running 5.15+ kernel.
To Reproduce
Steps to reproduce the behavior:

Use an AKS agentnode running AzureLinux

root [ /home/azureuser ]# cat /etc/os-release
NAME="Common Base Linux Mariner"
VERSION="2.0.20240403"
ID=mariner
VERSION_ID="2.0"
PRETTY_NAME="CBL-Mariner/Linux"
ANSI_COLOR="1;34"
HOME_URL="https://aka.ms/cbl-mariner"
BUG_REPORT_URL="https://aka.ms/cbl-mariner"
SUPPORT_URL="https://aka.ms/cbl-mariner"

root [ /home/azureuser ]# uname -r
<IP_ADDRESS>-2.cm2

root [ /home/azureuser ]# rpm -qa | grep keyutils
keyutils-1.6.3-1.cm2.x86_64

Mount Azure fileshare

sudo mount -t cifs //f19b3e40c80e54919ae4890.file.core.windows.net/pvc-9717cacf-4c54-4341-b336-f5398bd41b38 /mnt/pvc-9717cacf-4c54-4341-b336-f5398bd41b38 -o credentials=/etc/smbcredentials/f19b3e40c80e54919ae4890.cred,dir_mode=0777,file_mode=0777,serverino,nosharesock,actimeo=30

After the Azure fileshare is mounted, take note of the IP address used by the mount.  IP address used by the mount should match the IP address of the fileshare endpoint

On Azure storage account, initiate geographic failover which updates the fileshare DNS and point it to the IP address of the secondary site.

See error
Fileshare mount still uses the old IP address of the fileshare

trying to access the fileshare now get "Host is down" error
root [ /home/azureuser ]# ls /mnt/pvc-9717cacf-4c54-4341-b336-f5398bd41b38
ls: cannot access '/mnt/pvc-9717cacf-4c54-4341-b336-f5398bd41b38': Host is down

kernel error logged in syslog
2024-05-14T13:56:27.490067+00:00 aks-azurelinux-13697370-vmss000006 kernel: CIFS: Status code returned 0xc000006d STATUS_LOGON_FAILURE
2024-05-14T13:56:27.490233+00:00 aks-azurelinux-13697370-vmss000006 kernel: CIFS: VFS: \\f19b3e40c80e54919ae4890.file.core.windows.net Send error in SessSetup = -13

Expected behavior
SMB client should detect the DNS change and automatically update mount point to use the new IP address.

@christopherco @microsoft/cbl-mariner-kernel

The article linked at the top is nonsense.
It references kernel fixes from three years ago.
Then, it claims the permanent solution is in the latest AzureLinux 2.0 with kernel version 5.15.159. 🤨
There is a known SMB client regression that is present in 5.15.153. https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=v5.15.158&id=929ba00702cfba77fbb3fbc88d9a0f6e2af2f6b6
Could this regression explain the behavior seen here? If so, it's fixed in 5.15.158.

The article linked at the top is nonsense. It references kernel fixes from three years ago. Then, says we need AzureLinux 2.0 with kernel version 5.15.159 to pick them up. 🤨
There is a known SMB client regression that is present in 5.15.153. https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=v5.15.158&id=929ba00702cfba77fbb3fbc88d9a0f6e2af2f6b6
Could this regression explain the behavior seen here? If so, it's fixed in 5.15.158.

this is fixed in <IP_ADDRESS>-1.cm2 which was release a week ago or so.    thanks for the link to the regression bug too. later i realized that's the case too.


Refine Requirement: Research cyclic consumer behavior to report to USPS and its customers

Requirement needs to be analyzed for the state it is in in the current application (it may appear fully complete), which should then drive questions to the customer about what they truly want. Document findings as a comment here. Questions can be asked to the team or put into the "Customer Questions" document in the General/Files area.
https://github.com/umgc/fall2022/issues/34 will help with this if completed, alternatively you can run the app on your own.

I will help with this one.


respaced do's and don'ts

centered Dos and Don'ts title
removed box shadow
changed background to white-ish
issue #119
@austene
@GianFiyah
@travezripley

These issues are now resolved and your changes have been merged. Thank you for your contribution to SDJS.

Thanks for helping out, we have incorporated your changes into another pull request.


Swift 3 compatibility

Cocoapods of this library not updated to Swift 3

Check this issue : https://github.com/ariok/BWWalkthrough/issues/63

@albertml I'm working on the pod update now.

I'm encountering a lot of issues. CocoaPods is currently not compatible with Xcode 8. I've installed the rc2... let's try that way. Every time I need to update this repo for CocoaPods is a pain...

I'm still getting error while trying to push the pod to trunk. Currently I'm getting "The master repo is not a git repo.". I cannot complete and close this issue... I put it on hold.

Hi @ariok ,
Is there any progress about swift 3? Please let us know how it goes.
Thanks

BWWalkthrough has been updated to fully support Swift 3 with CocoaPods.


Run test on CI of Document's Broken Link

Signed-off-by: Chang Yong Lik<EMAIL_ADDRESS>Description
Added Github Workflow to detect broken links
Resolves https://github.com/linkedin/feathr/issues/393
How was this patch tested?
Manually on Github Actions branch
Output
https://github.com/ahlag/feathr/runs/7115496370?check_suite_focus=true
Log
Run pytest --check-links --check-links-cache docs
============================= test session starts ==============================
platform linux -- Python 3.8.13, pytest-7.1.2, pluggy-1.0.0
rootdir: /home/runner/work/feathr/feathr
plugins: check-links-0.7.1
collected 168 items
docs/README.md ........F...............                                  [ 14%]
docs/quickstart_databricks.md ....                                       [ 16%]
docs/quickstart_synapse.md ............F.                                [ 23%]
docs/concepts/feathr-concepts-for-beginners.md .....                     [ 26%]
docs/concepts/feature-definition.md .......                              [ 29%]
docs/concepts/feature-generation.md ........                             [ 33%]
docs/concepts/feature-join.md ...                                        [ 35%]
docs/concepts/point-in-time-join.md .                                    [ 36%]
docs/design_docs/feature_registry_abstraction.md ....                    [ 38%]
docs/dev_guide/cloud_integration_testing.md F.                           [ 39%]
docs/dev_guide/cloud_resource_provision.md F...F..F...                   [ 45%]
docs/dev_guide/doc_guide.md ..                                           [ 47%]
docs/dev_guide/feathr_overall_release_guide.md .....                     [ 50%]
docs/dev_guide/publish_to_maven.md ....                                  [ 52%]
docs/dev_guide/python_dev_guide.md ....                                  [ 54%]
docs/dev_guide/python_package_release.md F..                             [ 56%]
docs/dev_guide/scala_dev_guide.md ......                                 [ 59%]
docs/dev_guide/update_python_docs.md ....F                               [ 62%]
docs/how-to-guides/azure-deployment.md ................                  [ 70%]
docs/how-to-guides/build-and-push-feathr-registry-docker-image.md .F     [ 72%]
docs/how-to-guides/deploy-feathr-api-as-webapp.md .......                [ 76%]
docs/how-to-guides/expression-language.md ..                             [ 77%]
docs/how-to-guides/feathr-configuration-and-env.md ..........            [ 81%]
docs/how-to-guides/feathr-job-configuration.md ...                       [ 82%]
docs/how-to-guides/local-feature-testing.md .                            [ 83%]
docs/how-to-guides/troubleshoot-feature-definition.md .                  [ 83%]
docs/samples/customer360/Customer360.ipynb ........                      [ 88%]
docs/samples/databricks/databricks_quickstart_nyc_taxi_driver.ipynb .... [ 91%]
..
=================================== FAILURES ===================================
_ /home/runner/work/feathr/feathr/docs/README.md: https://linkedin.github.io/feathr/how-to-guides/streaming_source_ingestion.html _
https://linkedin.github.io/feathr/how-to-guides/streaming_source_ingestion.html: 404: Not Found
_ /home/runner/work/feathr/feathr/docs/quickstart_synapse.md: ../how-to-guides/azure-deployment.md _
../how-to-guides/azure-deployment.md: No such file: /home/runner/work/feathr/feathr/docs/../how-to-guides/azure-deployment.md
_ /home/runner/work/feathr/feathr/docs/dev_guide/cloud_integration_testing.md: ../.github/workflows/scala.yml _
../.github/workflows/scala.yml: No such file: /home/runner/work/feathr/feathr/docs/dev_guide/../.github/workflows/scala.yml
_ /home/runner/work/feathr/feathr/docs/dev_guide/cloud_resource_provision.md: ../how-to-guides/deploy.json _
../how-to-guides/deploy.json: No such file: /home/runner/work/feathr/feathr/docs/dev_guide/../how-to-guides/deploy.json
_ /home/runner/work/feathr/feathr/docs/dev_guide/cloud_resource_provision.md: ../how-to-guides/deploy.json _
../how-to-guides/deploy.json: No such file: /home/runner/work/feathr/feathr/docs/dev_guide/../how-to-guides/deploy.json
_ /home/runner/work/feathr/feathr/docs/dev_guide/cloud_resource_provision.md: ../how-to-guides/deployFeathr.ps1 _
../how-to-guides/deployFeathr.ps1: No such file: /home/runner/work/feathr/feathr/docs/dev_guide/../how-to-guides/deployFeathr.ps1
_ /home/runner/work/feathr/feathr/docs/dev_guide/python_package_release.md: https://pypi.org/manage/project/feathr/ _
https://pypi.org/manage/project/feathr/: 404: Not Found
_ /home/runner/work/feathr/feathr/docs/dev_guide/update_python_docs.md: https://xxx.readthedocs.io/en/latest/feathr.html _
https://xxx.readthedocs.io/en/latest/feathr.html: 404: Not Found
_ /home/runner/work/feathr/feathr/docs/how-to-guides/build-and-push-feathr-registry-docker-image.md: https://localhost:3000 _
https://localhost:3000: HTTPSConnectionPool(host='localhost', port=3000): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fcb44ad0790>: Failed to establish a new connection: [Errno 111] Connection refused'))
=========================== short test summary info ============================
FAILED docs/README.md::/home/runner/work/feathr/feathr/docs/README.md
FAILED docs/quickstart_synapse.md::/home/runner/work/feathr/feathr/docs/quickstart_synapse.md
FAILED docs/dev_guide/cloud_integration_testing.md::/home/runner/work/feathr/feathr/docs/dev_guide/cloud_integration_testing.md
FAILED docs/dev_guide/cloud_resource_provision.md::/home/runner/work/feathr/feathr/docs/dev_guide/cloud_resource_provision.md
FAILED docs/dev_guide/cloud_resource_provision.md::/home/runner/work/feathr/feathr/docs/dev_guide/cloud_resource_provision.md
FAILED docs/dev_guide/cloud_resource_provision.md::/home/runner/work/feathr/feathr/docs/dev_guide/cloud_resource_provision.md
FAILED docs/dev_guide/python_package_release.md::/home/runner/work/feathr/feathr/docs/dev_guide/python_package_release.md
FAILED docs/dev_guide/update_python_docs.md::/home/runner/work/feathr/feathr/docs/dev_guide/update_python_docs.md
FAILED docs/how-to-guides/build-and-push-feathr-registry-docker-image.md::/home/runner/work/feathr/feathr/docs/how-to-guides/build-and-push-feathr-registry-docker-image.md
======================== 9 failed, 159 passed in [28](https://github.com/ahlag/feathr/runs/7115496370?check_suite_focus=true#step:5:29).74s ========================
Error: Process completed with exit code 1.

Does this PR introduce any user-facing changes?
None

Thanks @ahlag for the contribution! Appreciate it!


"Stratos" is an implementation detail

Acceptance Criteria

[ ] WHEN I look at dashboard.fr.cloud.gov, THEN I don't see words or images that say "Stratos"

Implementation sketch
Customization pointers

Duplicate of #906


to_utf8(c5) values_->size() >= byteSize

Description
https://app.circleci.com/pipelines/github/facebookincubator/velox/36847/workflows/29f3c2c4-66d3-4c25-9039-9c92c4b3c918/jobs/245537
Error Reproduction
No response
Relevant logs
No response

Looked at this the input vector is changed during the eval!

repo:
checkout
jf get D50751503
and run
fdb buck2 test @mode/dev-sand //velox/expression/tests/facebook:fb_velox_expression_fuzzer_test6 -- --run-slow-tests
Actual issue:
resize is called with out checking that values buffer is unique. same for dict case potentially we need to check indicies buffer
https://www.internalfb.com/code/fbsource/[5f93b984033c]/fbcode/velox/expression/EvalCtx.cpp?lines=384

https://www.internalfb.com/diff/D50751503


DBZ-1152 Record operation order for changes with the same LSNs

https://issues.jboss.org/browse/DBZ-1152

@reworked that last commit of mine as suggested.


deploy: add support for deploying to a Bucket subfolder

Fixes #5930.
This needs better testing, which I will add separately in #5936.

I think I forgot to apply the BucketPath to deletes. It'd probably be best to submit the end-to-end test PR first and then add an end-to-end test for BucketPath here.

I will look at these Monday, but could you have a look at the commit message guidelines of this repo, esp. the part about starting the commit message title with a capital letter.


Dynamic dimensions in json signature now -9223372036854775808 instead of -1

After the latest llvm update, the json signature for dynamic dimensions in models is now -9223372036854775808 instead of -1.
For example compiling yolov3-12.onnx where the first input normally has shape [-1,3,-1,-1] and the second input is [-1,2]. Compiling with:
--O3 --EmitLib yolov3-12.onnx --onnx-op-stats TXT

works fine. But afterward the json signature now looks like
(session.input_signature())
 [    { "type" : "f32" , "dims" : [-9223372036854775808 , 3 , -9223372036854775808 , -9223372036854775808] , "name" : "input_1" } ,    { "type" : "f32" , "dims" : [-9223372036854775808 , 2] , "name" : "image_shape" }]

(session.output_signature())
[   { "type" : "f32" , "dims" : [1 , -9223372036854775808 , 4] , "name" : "yolonms_layer_1/ExpandDims_1:0" } ,    { "type" : "f32" , "dims" : [1 , 80 , -9223372036854775808] , "name" : "yolonms_layer_1/ExpandDims_3:0" } ,    { "type" : "i32" , "dims" : [-9223372036854775808 , 3] , "name" : "yolonms_layer_1/concat_2:0" }]

where it used to have -1 for the dynamic dimensions.
I'm guessing the -9223372036854775808 is coming from MLIR's ShapedType::kDynamic. However I'm guessing we don't want to present that literal through onnx-mlir externalized interfaces (ie the json signature output of this issue and value to use for --shapeInformation for https://github.com/onnx/onnx-mlir/issues/1995).
Note: if we change the external interfaces from -1 to something new, that should be a version bump in the next onnx-mlir release because it might force downstream users to make changes. Luckily the next onnx-mlir is already going to be 0.4.0 rather than 0.3.3 so if we did want to change it, now's as good a time as any.

I suggest that we tweak the reader to accept -1. It is still the recognized value when printing MLIR files along the compilation, so there is no reason to be different here.

People interfacing values should remain easy to use... thus continuing -1 is the way to go.

People interfacing values should remain easy to use... thus continuing -1 is the way to go.

Agreed. PR #1998 is to handle -1 at onnx-mlir's interface.


Looking for information on debugging a node C++ addon using nan and node-gyp

I'm having problems finding documentation on how to debug a node C++ addon using nan and node-gyp.
There are articles to debug JavaScript and C++ applications, but I cannot seem to find an example of how to configure vscode to use node-gyp instead of g++ plus how to test calling between JavaScript and C++.
As a starting point, is it currently possible to debug node C++ addons with vscode and if so, is there an example project or documentation to reference?
I've been using vscode for JavaScript and loving it...now I'm hoping to do the same for node addons!

Any update on this?

+1

This issue was moved to Microsoft/vscode-cpptools#992

This issue was moved to Microsoft/vscode-cpptools#993