Add a TasksTimeout optional field to pipelinerun

This PR is a partial implementation of TEP #326.
As suggested by @jerop  TasksTimeout is being added to pipelinerun and give a way to users to define how much time are dag tasks permitted to run vs finally tasks.
The idea here is to demo the behavior, and see if the remaining part of the TEP shall be addressed.

Changes
TasksTimeout is used to timeout all dag tasks, finally tasks excluded. The pipelinerun Timeout field is kept and used for finally tasks timeout.
Validates a TasksTimeout if:
- TasksTimeout > 0
- Timeout is specified and TasksTimeout <= Timeout
- Timeout not specified and TasksTimeout <= default Timeout
When TasksTimeout is not defined, the behavior is unchanged.
Have been tested with:
apiVersion: tekton.dev/v1beta1
kind: PipelineRun
metadata:
  name: hello-world-pipeline-run-with-timeout
spec:
  timeout: "0h0m60s"
  pipelineSpec:
    tasks:
    - name: task1
      timeout: "0h0m30s"
      taskSpec:
        steps:
          - name: hello
            image: ubuntu
            script: |
              echo "Hello World!"
              sleep 10
    finally:
    - name: task2
      params:
        - name: echoStatus
          value: "$(tasks.task1.status)"
      taskSpec:
        params:
          - name: echoStatus
        steps:
          - name: verify-status
            image: ubuntu
            script: |
              if [ $(params.echoStatus) == "Succeeded" ]
              then
                echo " Hello World echoed successfully"
              fi

---
apiVersion: tekton.dev/v1beta1
kind: PipelineRun
metadata:
  name: hello-world-pipeline-run-with-taskstimeout
spec:
  tasksTimeout: "0h0m30s"
  pipelineSpec:
    tasks:
    - name: task1
      taskSpec:
        steps:
          - name: hello
            image: ubuntu
            script: |
              echo "Hello World!"
              sleep 30
    finally:
    - name: task2
      params:
        - name: echoStatus
          value: "$(tasks.task1.status)"
      taskSpec:
        params:
          - name: echoStatus
        steps:
          - name: verify-status
            image: ubuntu
            script: |
              if [ $(params.echoStatus) == "Succeeded" ]
              then
                echo " Hello World echoed successfully"
              fi

/kind feature

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[ ] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes

API changes

Added field TasksTimeout to PipelineRun spec

Changes in behavior

When supplied, a TasksTimeout is used as timeout for dag tasks. Finally tasks use Timeout or default timeout.

/ok-to-test

I'm going to add a hold on this just so to ensure we get the TEP merged before this goes in.
/hold

@souleb it would also be great to add an example YAML as part of this PR which shows the new field's usage too.  examples/v1beta1/pipelineruns/no-ci/pipeline-timeout.yaml is another similar example which serves that purpose.

One more thing that would be useful (sorry for the message spam!) would be to include some integration tests as well.  These are quite a bit of work but it's good to have a fully-constructed pipeline run that tests timeout behaviour as part of our e2e test suite.  See test/timeout_test.go for an example that you could base your own test on.

One more thing that would be useful (sorry for the message spam!) would be to include some integration tests as well. These are quite a bit of work but it's good to have a fully-constructed pipeline run that tests timeout behaviour as part of our e2e test suite. See test/timeout_test.go for an example that you could base your own test on.

Sure, will do that. Just I wanted to wait for the TEP to be approved beforehand.

/hold cancel

/hold while waiting for #3881

@sbwsg @jerop @pritidesai this is ready for review 😃

@pritidesai will you have some time to review this PR?

@pritidesai will you have some time to review this PR?

Thank you @souleb for all the changes and your patience 🙇♀  Yes I will spend some time today to review this PR and continue early next week after long weekend. Thanks a bunch again for all your efforts 🙏

/retest

/test pull-tekton-pipeline-integration-tests

@souleb please rebase this PR, its ready to merge. Thanks again for all the hard work. 🙏

/retest

@vdemeester can reset the lgtm? The bot unset it following my rebase.

/retest

/retest
🤔


Enable cargo-spellcheck for vm/message/

Issue summary

Make sure that cargo spellcheck --cfg ../../.config/spellcheck.toml can run without errors in vm/message/. For details, see the tracking issue.
Other information and links

Issue resolved under PR #1787


[Snyk] Upgrade @apollo/client from 3.8.6 to 3.8.7

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @apollo/client from 3.8.6 to 3.8.7.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-11-02.

Release notes

Package name: @apollo/client

3.8.7 - 2023-11-02Patch Changes

#11297 c8c76a522 Thanks @ jerelmiller! - Add an explicit return type for the useReadQuery hook called UseReadQueryResult. Previously the return type of this hook was inferred from the return value.

#11337 bb1da8349 Thanks @ phryneas! - #11206 used the TypeScript syntax infer X extends Y that was introduced in TS 4.8.
This caused some problems for some users, so we are rolling back to a more backwards-compatible (albeit slightly less performant) type.

3.8.6 - 2023-10-16Patch Changes

#11291 2be7eafe3 Thanks @ ArioA! - Fix a bug that allows to only call loadErrorMessages without also calling loadDevErrorMessages.

#11274 b29f000f3 Thanks @ jerelmiller! - Start the query ref auto dispose timeout after the initial promise has settled. This prevents requests that run longer than the timeout duration from keeping the component suspended indefinitely.

#11289 b5894dbf0 Thanks @ phryneas! - MockedProvider: default connectToDevTools to false in created ApolloClient instance.
This will prevent the mocked ApolloClient instance from trying to connect to the DevTools, which would start a setTimeout that might keep running after a test has finished.

#11206 dd2ce7687 Thanks @ phryneas! - cache.modify: Less strict types & new dev runtime warnings.

    from @apollo/client GitHub release notes

Commit messages

Package name: @apollo/client

8329f07 Version Packages (#11298)
bb1da83 work around TS 4.8-specific `infer X extends Y` (#11337)
0de3b29 bump engines field to include npm 10 (#11335)
fe12ef4 chore(deps): update cimg/node docker tag to v21 (#11310)
d998550 chore(deps): update actions/setup-node action to v4 (#11324)
dcf82ec chore(deps): update all dependencies - patch updates (#11320)
7d39e26 chore(deps): update peter-evans/create-or-update-comment action to v3.1.0 (#11309)
0b46ed0 Replace lit-element with Lit in integrations.md (#11311)
00fd096 chore(deps): update all dependencies - patch updates (#11307)
0bcbb2c Bump @ babel/traverse from 7.22.5 to 7.23.2 in /integration-tests (#11294)
946b0e9 update circleci secops orb (#11302)
c8c76a5 Add an explicit return type for `useReadQuery` (#11297)
89374ca Add delay in docs for loading state testing (#11287)
7d3cac9 Update renovate.json to ignore glob (#11293)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Apollo is not used anymore.


Typecast issue when compiling on Particle

Fails to compile on Paticle:
invalid conversion from 'const uint8_t* {aka const unsigned char*}' to 'uint8_t* {aka unsigned char*}' [-fpermissive]

Fix:
// returns true if address is valid
bool DallasTemperature::validAddress(const uint8_t* deviceAddress) {
	return (_wire->crc8((uint8_t*)deviceAddress, 7) == deviceAddress[DSROM_CRC]);
}

Fixed in #183

LGTM
As it is part of #183 I propose to close this issue as it need no separate tracking.

I've tested this on my particles and works via the cloud IDE.


Clarify in v1beta1 Upgrade Guide which AWS role to use in the EC2NodeClass

Description
Can the v1beta1 Upgrade Guide be more clear about which AWS role to use in Step 8?

Step 4 said "Apply the new Karpenter policy and assign it to the existing Karpenter role".
Step 8 said, "Specify your AWS role where there is a your AWS role Here placeholder".

I was just following the instructions from top to bottom, so I guessed that Step 8 was referring to the AWS role created in Step 4. In my case, turns out I needed to use the existing role prepended with KarpenterNodeRole that was created following Getting Started with Karpenter.

Appears to be a duplicate of #4820


Auto Translator auto stopping after I say anything

please give some detail log error msg or something.

it gave no error msg or logs when I press F and speak and release the F key it just stops the process but in the Whisper AI logs it gives the 400 bad request error

probably something wrong with your connection, do you use vpn?

no I do not use a vpn

wait can you check if these audio ids or what ever are correct?

@SrijanSharma21 if audio problem that requests to Whisper will not run, check the logs on the colab.

this is the only thing I get

yes an http 400 request thing which I believe is the bad request one


Token Balances No Longer Displayed

The production version as of today is not displaying my token balances when online or offline. Older versions display my tokens offline without any trouble (for instance, v<IP_ADDRESS>).
This is frustrating because in this case the wallet is located on a Trezor device and I am NOT able to send my tokens. I can look, but I cannot touch.
Monte

I neglected to mention I am on macOS Sierra v10.12.5 and Chrome Version v60.0.3112.90 (Official Build) (64-bit)

Not sure why, but it turns out I needed to manually add token contract address. "Display All" would display none. Although I am able to work around the issue, it is still an issue. That is, something about either this wallet or the latest version of MEW is failing to fetch or mask the well-known list of token addresses.
Shouldn't matter but just in case, the wallet is managed by Trezor.
Thanks,
Monte

I was going to open an issue...
(you were 2 hours earlier)
Something not right.

The same happens to me, version 3.10.2 was OK but with <IP_ADDRESS> it is no more possible to view or send tokens. Both tested with Chromium and Firefox under Ubuntu 16.04.3 LTS 32 bits. Seems not being platform dependent.

I can confirm this, also.

I was working today on merging mew old commits (+-5days) into myTokenWallet.com, and during this work I had to fix many problems and I remember that I had before some ours the same problem. But at moment mtw works for me and shows all tokens.
If you want to test it your are welcome at: https://mytokenwallet.com
On the toolbox menu I have a Mnemonic Code Converter (it is set by default to ETH) so you can play and test if it is network problem. Remember: Security is hard!
Now I will dig into mew also to find the bug

They are appearing for me now (under v<IP_ADDRESS>).
Thanks for looking into it.
Monte

The tokens were taken down due to an exorbitant amount of calls due to Ox ico irresponsibly handling their deployment.

https://ethereum.stackexchange.com/questions/24428/using-trezor-my-list-of-tokens-is-gone-and-my-alt-coins-do-not-appear-in-myethe/24429#comment25567_24429


add optional option to auth hook

It's really great that the new authentication populates the user. However, often-times it's nice to know if the user is authenticated, and have access to that data, but the authentication itself is not required. What do you guys think of a simple optional boolean exposed when using the hook?
As an example, we use the presence of an authenticated user to filter the properties returned to the client in find and get requests, but it is not required that the request be authenticated.
exports.before = {
  all: [],
  find: [
    auth.hooks.authenticate(['local', 'jwt'], {optional: true}),
  ],
  get: [
    auth.hooks.authenticate(['local', 'jwt'], {optional: true}),
  ],
  create: [],
  update: [
    auth.hooks.authenticate(['local', 'jwt']),
  ],
  patch: [
    auth.hooks.authenticate(['local', 'jwt']),
  ],
  remove: [
    auth.hooks.authenticate(['local', 'jwt']),
  ],
};

Hey @marbemac! Thanks for the PR. Unfortunately, this would have bigger implications than just changing the hook. You would also need to change the socket and express middleware implementation to support the same thing.
With that said, auth can fail for all sorts of reasons (including user lookup) so I don't think this is a good idea. If you are looking for the functionality you can always implement your own strategy or custom verifier (see any of the auth plugin docs) or you could likely wrap the hook, catch the error, and ignore it.
Couple ways you could support what you want I think. Hope you don't mind if I close this. Thanks again for taking the time! 😄


x/pkgsite: package removal request for [go.appmanch.org/commons]

Hello Team,
Requesting you to please remove the package as it is no longer available.
Thank you

Done, and should take effect within the next few hours. Please comment if not.


fix most scala deprecation warnings rooted in this build

remaining warnings will be handled in follow-up PRs

PPrinter regexes
overflowdb codegen

Needs one more scalafmt ;-)

Needs one more scalafmt ;-)


Black artifacts and FF crash

Hi, whenever I enable the extension my firefox is basically unusable. I'm getting black artifacts and if I continue using it ultimately crashes

What is your:

OS
Extension version

Do you have a screenshot of the artifacts ?
Can you have access to the web console and tell me if do you see something ? (Ctrl+Maj+J)
In Preferences, do you have auto save enabled for bookmarks ? If yes, does it the same if disabled.

Since v0.4.1: bookmark auto-save disabled; Next version will be totally different.

Hi, sorry I really didn't have the time to comment on this.
I was using arch 4.14, were you able to reproduce it? Should I try the next version?

Don't worry, I just closed it because I changed the code a lot last weeks. Moreover, it is a problem that is probably from the bookmark auto-save. Especially, it was doing lots of troubles on Linux distribution.
The reasons:

I was bookmarking every tabs from scratch at each save
Bookmarking is a very heavy operation

With too many tabs/groups, it could often finish by crashing. Since v0.4.1, this functionality is totally disabled until I will make it better.
You can try it again, sure. If you don't want to disturb your classic session do firefox -ProfileManager -no-remote & from the terminal and create a new profile. You will have a Firefox window not related to your other tabs and bookmarks.
If you have again the problem, can you try to report those information:

Do you have a screenshot of the artifacts ?
Can you have access to the web console and tell me if do you see something ? (Ctrl+Maj+J)
In Preferences, do you have auto save enabled for bookmarks ? If yes, does it the same if disabled.


.Nuget updates / NET Core 3.2 Preview 1

.Nuget updates / NET Core 3.2 Preview 1 and Nuget Package updates.
For some reason if I have .Nuget updates / NET Core 3.2 Preview 1 in Blazor Boilerplate and use 2.0.1 with .Net Core 3.1 my menu is messed up.  Maybe this will help you out.
Basically delete Startup.cs and move some of that to the Program.cs

Yeah! Thank you! This will be in release at Monday.

Hello, does this mean we can't use the latest version of MatBlazor if we wish to stay on .NET Core 3.1 ?

I don't understand why dependencies were made on a preview of .NET Core. Now none of us can use this and deploy on production .NET Core. I think this was a bad call.

Ok, today we will move back

@SamProf My opinion is we should be on the leading edge of releases.  I know there has to be a lot of people who want the latest release as well as other's who in production.
I know I had MatBlazor issues in 3.1 and my app was on Preview 3.2 Preview 1.  Either way it will cause issues for some group of people.

@SamProf I was just about to suggest that. Can do a preview release. Not sure how much more work it is?

@enkodellc I understand the desire, but some of us are using this in production on Azure and also need the incremental fixes. It's too bad 3.2 apparently introduces some sort of breaking change. I really hope we can figure out something to support both.

from now we will have 2 releases.
https://www.nuget.org/packages/MatBlazor/2.2.0 (for stable DotNetCore)
https://www.nuget.org/packages/MatBlazor/2.2.0-preview (for Preview version)
Preview will be here https://github.com/SamProf/MatBlazor/tree/preview

@RonPeters @enkodellc Can you check your versions? Thank you!

Just tested and pushed the preview. Thanks @SamProf

from now we will have 2 releases.
https://www.nuget.org/packages/MatBlazor/2.2.0 (for stable DotNetCore)
https://www.nuget.org/packages/MatBlazor/2.2.0-preview (for Preview version)
Preview will be here https://github.com/SamProf/MatBlazor/tree/preview

nuget does not understand that 2.2.0 is not an upgrade of 2.2.0-preview, it would be better to increase version, e.g. 2.2.1-preview


update release_tag

just to test the new static binary builds in pre-release master-2e4f8c1f

🎉 No seg fault errors!
I didn't update the cached SHA files for unit testing, so the unit test for get_sha_checksum() failed, but all downloaded static binaries had no problem executing --version.

it is fixed (for now). See my other comment


TP posts showing up in FP queries

It would seem that the is_fp column in p_posts is not updated when the fp feedback is cleared. It could be the same for naa and tp, but I haven't had time to dig into it properly.
I came across this while looking at the Autoflag FP Dashboard, and I noticed that there is one FP with 7 reasons, and a weight of 692. This is the post. It has no FP feedback visible, but on clicking clear it shows that two users had their feedback cleared or automatically deleted.
Glorfindel has written a query to show the Top 20 FPs.
When I came to write this issue, the three posts I had found (1, 2, 3) had stopped showing up in queries. However three others are now in their place.
So it could be something else (other than the is_fp column not updating).

CC'ing @thesecretmaster on this, I think you were the last person to poke this code.

Yep, see #469 and #470. I bet we're just missing a call to #update_feedback_cache somewhere.


Misleading Font Installation Dcoumentation

When following the documentation for installing the fonts needed for powerlevel9k it seems that the 4 different options are not independent.
To be precise, I need to install the powerline font irrespective of the other options as a base before I can go ahead and install the other fonts such as the FontAwesome icons.
This can be misleading and a clean up of the documentation would be nice especially for newbies to ZSH.

Hi @varunagrawal ! Thanks for the report.
Actually these methods are independent, but which one you have to choose depends on your configuration and personal preference ;)
This is a pretty confusing topic. I'll try to elaborate more (in a general way, so that we can copy & paste it to the wiki; so you might already know a lot of things I describe here).
------------8<-------------------
General informations about fonts
First off:
This theme uses special characters that require to use special fonts. To use these fonts properly, you have to install them (see below) and to set it in your terminal emulator.
So, basically there are two* different methods how to use fonts:

Use a font that already contains all necessary characters. That is called using a "patched" font. The advantage is, you just need to install one font. Disadvantage is that you are limited to the icons of that font.
Use a method of falling back from one font to another, if the character you want to display is not contained. This requires the use of an additional program ( e.g. "fontcache"). The advantage is flexibility. You can basically configure every font you want. Disadvantage is that the setup is more complicated.

*Some terminal emulators (like iTerm2) allow it to configure text and non-ASCII fonts separately. If you want this is a third way.
Examples of icons in P9K
The most basic form of a powerline optic looks like this:

So, you can see there are only the segment separators (the triangles) and a git-branch icon present.
But there are fonts out there that contain a lot of icons, like Nerd-Fonts or Awesome-Terminal-Fonts. Here is a demonstration of Awesome-Terminal-Fonts in a prompt:

Choosing a default icon set in P9K
If you prefer such a look for your prompt, you need to install such a font and set the appropriate POWERLEVEL9K_MODE, which pre configures a set of default icons for the according font. Remember to set the POWERLEVEL9K_MODE before you apply/specify the theme in your ~/.zshrc.
The different POWERLEVEL9K_MODEs are for:

default
awesome-fontconfig
awesome-patched
nerdfont-fontconfig

regular powerline optic
Awesome Terminal Fonts (fontconfig variant)
Awesome Terminal Fonts (pre patched)
Nerd-Font (fontconfig)

Actual installation of font
Pre patched font
Installing a pre patched font is fairly easy. Just download the desired font and double click it. A dialog should pop up, guiding you through the installation process.
After you installed the font, you have to configure your terminal emulator to use that font.
Setup fontconfig
A good installation instruction that topic can be found in the Awesome-Terminal-Fonts README for Linux and in the wiki for OSX
------------8<-------------------
I did not go into detail into why there are different variant of the fonts (even within one font), and did not explain what code points, glyphs and double-width glyphs are. I am planning to write a separate wiki page for that.
What do you think? Is that explanation clearer than the original wiki page? It is a quite complicated and confusing topic..

Hi @dritter. Thanks for the awesome response!
However my problem is that in order to get Option 3: FontAwesome to work, I had to also install Option 1: Powerline fonts as well. Thus saying that I can pick and choose which option I wanted based on my preferences didn't seem to sit well with me. If I only install FontAwesome font then I don't get the powerline triangles in my shell and it looks terrible. Installing the powerline font fixes that immediately.
I assumed it was an issue with patching that @V1rgul had worked on and fixed and which was going to be released in the next version. Hence I thought if the documentation needed to be updated, I'd be more than glad to submit a PR for it.

So this should help explain my issue.
If I don't install the PowerlineSymbols.otf font as detailed here, then I get the below terminal prompt:

If I download and move the PowerlineSymbols.otf file to the ~/.fonts directory, the terminal prompt automatically fixes to this:

Nowhere in the Awesome-Terminal font installation README does it mention to install the Powerline Symbols font. Maybe I am missing something here, but the Awesome-Terminal font is option 3 and the Powerline Symbols font is option 2 and hence my confusion.

Yes, that is because you need one text font and add the special icon fonts to it. Some of the text fonts (especially the terminal/developer facing fonts like PragmataPro and SourceCodePro) already include powerline icons (they are pretty popular among the users of that fonts). There a few traces that @gabrielelana used PragmataPro as base font. That is IMHO the reason why he did it that way.
He could have included a font that only has the powerline icons in it (like the one included in powerline itself). Btw. even the official powerline/font repo offers just pre patched fonts..

Btw. Just a wording thing: FontAwesome is a special icon font that is included in Awesome-Terminal-Fonts.

Btw. I just updated the description in my comment to clarify that a user has to download the powerline icons if Awesome-Terminal-Font was used, but no powerline-enabled base font.

@dritter you are right, I use PragmataPro, I strongly suggest to everyone to use a font that already has the powerline glyphs as a base font because it's really hard to patch a pre existing font with those glyphs in a way that looks right, I'm pretty obsessive for this kind of thing, a shift of few pixels for me is unacceptable 😅
@varunagrawal I should also clarify things better on [https://github.com/gabrielelana/awesome-terminal-fonts](my side)

@dritter and @gabrielelana thank you for the clarification. I'll close this issue.

Just used @dritter's awesome response in this thread to create a new wiki page!
https://github.com/bhilburn/powerlevel9k/wiki/About-Fonts


Deleted Messages Disappear

Describe the bug
When i click on someone's reply and the telegram go to that message up there and then i go back to the chat the deleted messages Disappear.
Steps to reproduce

go to any group chat
click on someone's reply that's up in the chat view
click the arrow to go to the recent/new messages

Expected behaviour
the deleted messages should not be deleted when i move through users replies.
Actual behaviour
deleted messages disappear.
Configuration

Operating system: Windows 10 Pro x64
Version of Telegram: 2.1.6

Logs
Insert ArLog.txt here. (if necessary)
It's in the same directory as the version.dll file.

A message that jumps to long ago will cause Telegram to directly clear all messages from the current memory and then download messages near the jump location.
That's why the plugin prevents messages from being deleted but they may still disappear after a while.
This problem may be fixable, but it will cause Telegram to use more memory, require the plugin to do more patches to Telegram, and make future updates more troublesome.
I don't think it's worth it, so there are no plans to fix it for now.
Thank you for the report!

A message that jumps to long ago will cause Telegram to directly clear all messages from the current memory and then download messages near the jump location.
That's why the plugin prevents messages from being deleted but they may still disappear after a while.
This problem may be fixable, but it will cause Telegram to use more memory, require the plugin to do more patches to Telegram, and make future updates more troublesome.
I don't think it's worth it, so there are no plans to fix it for now.
Thank you for the report!

can you create something like deleted messages log in a file ? or can i forward deleted messages to saved messages chat ? it would be very helpful to take a look at unseen deleted messages if you can make the plugin log them in a file with chat id and user first name ..etc  because sometimes the telegram updates or the pc shuts down so i lose all the deleted messages on all chats
note: also some chats like groups i really don't know how to make it automatically download the images and files before it gets deleted because some videos or images are marked as deleted but i can't view them or they are deleted from the telegram server ?

can you create something like deleted messages log in a file ? or can i forward deleted messages to saved messages chat ? it would be very helpful to take a look at unseen deleted messages if you can make the plugin log them in a file with chat id and user first name ..etc because sometimes the telegram updates or the pc shuts down so i lose all the deleted messages on all chats

#1

note: also some chats like groups i really don't know how to make it automatically download the images and files before it gets deleted because some videos or images are marked as deleted but i can't view them or they are deleted from the telegram server ?

Telegram automatically downloads images or videos only in the currently opened group/channel.
They may have been deleted on the Telegram server before you downloaded them.

If the whole chat is deleted, only the chat that is loaded on telegram is available to copy, not the past messages that loads when you scroll up. Is there a way to keep all messages?

If the whole chat is deleted, only the chat that is loaded on telegram is available to copy, not the past messages that loads when you scroll up. Is there a way to keep all messages?

No way


Add recipe for tiramisu

I would like to add a recipe for traditional Italian tiramisu.

Sure, just submit a PR for it, if you have a good one :+1:


Add detail key to haskell ghc handler

Add detail key to the dict returned from the haskell ghc handler

If you're going to create a pull request, you should at least write a short description explaining what it is you're trying to do.

I see your point about the number of lines in ALEDetail could get long and require pagination. To be honest, I just copied the way detail was defined in other linters.
There's two issues which would make it a bit hard to trim the output

I don't think there's a good heuristic for selecting which parts of the output are the most pertinent

The most recent version of GHC improved error formatting https://downloads.haskell.org/~ghc/8.2.1/docs/html/users_guide/8.2.1-notes.html#highlights
Overall that's a good thing but unfortunately neither of the linters I use (hdevtools + ghc-mod) work with ghc 8.2 right now and so it'd be difficult to be sure that any changes I make now to omit content selective won't break the handler in the future since I can't easily test the new output format with those linters.

Overall, if you'd prefer not to include any detail fields that could get too long then I'm happy to close the PR and handle this in some other way

I have handled this now. The lines of output starting from the error message are added to the detail key.


Expected to not be in Angular Zone, but it is!

I am trying run this example but i am getting following error.
Do i need to make any other changes to avoid this error ?

I made my own test project from scratch and I'm getting the same error.  Please see README from my test repository at https://github.com/andyolivares/single-spa-test for more details of what I did.
Thank you!

When this issue occurs and I am in the process of reading a cookie using the Cookie Service (ngx-cookie), the service appears to be trying to find a cookie with a path of "/" instead of a path matching the base href of my child app.  I can prove this by deliberately writing my cookie to the root path (which causes the read to succeed).  This suggests to me that the zone issue is really a symptom of the active zone not being set properly when transitioning between (mounting) apps.
I don't have a simple example to share as I'm wiring up a production application.  My steps include the following:  I added my site to the sample portal solution.  I added a menu icon to call my app.  I open the portal and click my icon from the menu.  My app launches, the URL changes to show my new base href, my login form is displayed, and the console shows one zone error which was thrown from the Menu bundle.   Note that I am already switched to my app, but it is the Menu bundle which threw the error.  Similarly, if I use a button on the Home page, the error will be thrown from the Home bundle.

I'm having the same issue as well. How did you solve it?

I was not able to correct the Angular CLI Zone error.  Since my child site
works once it gets mounted without throwing any additional zone errors, I'm
guessing the zone error is a timing issue, and/or something that happens
only during the mounting process.
On Mon, Dec 10, 2018 at 4:06 AM Fergardi<EMAIL_ADDRESS>wrote:

I'm having the same issue as well. How did you solve it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PlaceMe-SAS/single-spa-angular-cli/issues/33#issuecomment-445759441,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AqvFBiYJBzoJkMw5vTUyaUl7zJCtnINfks5u3jIXgaJpZM4Wncco
.

--
John Lewis Consulting
618-444-5131
<EMAIL_ADDRESS>

For me, the error is totally random.

Once I mount my child application, I am not using the home page or menu
from the sample application.  I have changed the "matchRoute" for menu to
"/home/".   In your browser, add a watch for singleSpaAngularCli (this
object is provided by the SingleSpaAngularCli library).  Once your child
app is opened, check the the singleSpaAngularCli object and see which apps
are mounted.  I only see the zone error if more than one app is mounted.
On Mon, Dec 10, 2018 at 10:56 AM Fergardi<EMAIL_ADDRESS>wrote:

For me, the error is totally random.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PlaceMe-SAS/single-spa-angular-cli/issues/33#issuecomment-445888744,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AqvFBl_srrKv5GeCSI01BRK2loaYWN_hks5u3pIkgaJpZM4Wncco
.

--
John Lewis Consulting
618-444-5131
<EMAIL_ADDRESS>

@Smallwood19 this may be my issue, because I notice an incredibly lag and random errors from unmounted apps, timeouts exceeded, extremely long waiting periods, non cached apps and so on. I used the template as an scaffolder but I clearly messed up with it, just dont know how.

Has anyone had any movement on this issue? I am running into the same thing. Error occurs when navigating between apps.

the best solution to deal with zone.js issue so far is
1, comment out import 'zone.js/dist/zone'; from the pollyfill.ts in all the apps
2, add  into the index.html for angular apps
since angular will compile zone.js into the bundle so this way the spa framework won't only load/unload zone.js from the child, and only the shell has one zone.js file acting.

@nickfeng1982 The error described in this thread still exists if this method is used.

After a long long time of this issue plagueing us, I have found and implemented a fix!
How to use the fix
For those using single-spa-angular-cli, try upgrading to<EMAIL_ADDRESS>with npm install --save<EMAIL_ADDRESS>For those using single-spa-angular@beta, the fix was released in 3.0.0-beta.12. Follow the instructions at https://github.com/CanopyTax/single-spa-angular.
Example repo
https://github.com/joeldenning/coexisting-angular-microfrontends
Other gotchas with multiple angular apps on the same page

Make sure to set the APP_BASE_HREF with the url prefix in the single-spa activity function.
Make sure to have a unique selector in app.component.ts for each of your applications.

I'm closing this issue, but feel free to reopen for further discussion if this doesn't work for you.

For anyone bumping into this who might not even be using single-spa framework: there's another solution of passing down parent ngZone to angular's bootstrapModule function as described here https://github.com/angular/angular/issues/31870.
Potential drawback of this is that child apps might trigger change detection cycles on parent app.

Any solution for this error? I am getting this error on the Production server but not on local or any test server. What might be causing ?

I have met this problem too, when I upgrade angular 15 to angular 17, and this worked for me.
version
"single-spa-angular": "9.1.2",
"zone.js": "0.14.10",
"typescript": "5.4.5"

Update polyfill.ts
import 'zone.js';
Update angular.json build
"polyfills": ["src/polyfills.ts"],
"tsConfig": "tsconfig.app.json",
 "allowedCommonJsDependencies": [
          "zone.js"
]

I have met this problem too, when I upgrade angular 15 to angular 17, and this worked for me. version
"single-spa-angular": "9.1.2",
"zone.js": "0.14.10",
"typescript": "5.4.5"

Update polyfill.ts import 'zone.js';
Update angular.json build
"polyfills": ["src/polyfills.ts"],
"tsConfig": "tsconfig.app.json",
 "allowedCommonJsDependencies": [
          "zone.js"
]

This worked for me like charm. I was tired of looking for solutions. Thanks buddy.


findViewById always return null after BottomBar attach

TextView before = (TextView) findViewById(R.id.tv_toolbar_title);
bottomNavigation = BottomBar.attach(this, savedInstanceState);
TextView after = (TextView) findViewById(R.id.tv_toolbar_title);

before is not null, after is null.


[Feature Request] Can setCurrentNode/setCurrentKey focus?

Existing Component
Yes
Component Name
el-tree
Description
setCurrentNode/setCurrentKey choose correctly but never focus

@Glenfiddish sorry, we have no plan to support it.

这个是很常见的需求，圈子太小了额


Data Source filtering syntax

Continuing a discussion from issue #695:
@nilshoerrmann:

Well, we started on the premise "Only cosmetics!" ... When we start working on JSON integration we should also have a look at negation and filter options in general ("Field X is not of the value of {$y}").

@brendo:

That's more a field's responsibility to implement the filtering, unless your taking about having a better selector instead of 'mode: {value}' in which I guess it's related.

You're right. But in this case we should make sure that all core fields allow negations (e. g. not: {$title}). Furthermore it would be nice if we could improve variable replacements. Currently it's possible to do this, which represents a if not then statement:
{$title:my-default-title}

What about improving/extending this syntax:
{$title?my-default-value}

If $title exists, use my-default-value instead.
This would be helpful for cross field relations because I could us this syntax not only on the title field but also on a subtitle field.

Continued in #757.


[Feature Request] Logging

Great work! It'd be great to log when someone comes online. Cheers!

This is easy.
Go to MainContent.js file.
In line# 106, change  L(account.phoneNr + ' is online') to below:
L(account.phoneNr + ' is online at' + Date().toLocaleString().slice(3, 24))
Save the file. Compile again using the steps given here.
Now, the log in the Chrome Console would have timestamps.

@muskw I changed that line , and saved it , now for compiling , the steps written are :
Clone the repo git clone https://github.com/LoranKloeze/WhatsAllApp.git
Make sure Node.js is installed
Run npm install or yarn install
Run npx webpack or npx webpack --watch to pack the code from /src to /dist
Load the unpacked extension from /dist in Chrome

So yes, for that I installed Git and Git Hub , and was able to clone it in a different folder ( originally whatsallapp extension is in a different folder )
See the CMD lines
Microsoft Windows [Version 10.0.17134.765]
(c) 2018 Microsoft Corporation. All rights reserved.

C:\Users\admin>cd C:\Users\admin\Documents\www

C:\Users\admin\Documents\www>git clone https://github.com/LoranKloeze/WhatsAllApp.git
fatal: destination path 'WhatsAllApp' already exists and is not an empty directory.

C:\Users\admin\Documents\www>git clone https://github.com/LoranKloeze/WhatsAllApp.git
fatal: destination path 'WhatsAllApp' already exists and is not an empty directory.

C:\Users\admin\Documents\www>git clone https://github.com/LoranKloeze/WhatsAllApp.git
Cloning into 'WhatsAllApp'...
remote: Enumerating objects: 25, done.
remote: Counting objects: 100% (25/25), done.
remote: Compressing objects: 100% (18/18), done.
remote: Total 447 (delta 10), reused 15 (delta 7), pack-reused 422
Receiving objects: 100% (447/447), 3.72 MiB | 1.66 MiB/s, done.
Resolving deltas: 100% (192/192), done.

C:\Users\admin\Documents\www>npm install
npm WARN saveError ENOENT: no such file or directory, open 'C:\Users\admin\Documents\www\package.json'
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN enoent ENOENT: no such file or directory, open 'C:\Users\admin\Documents\www\package.json'
npm WARN www No description
npm WARN www No repository field.
npm WARN www No README data
npm WARN www No license field.

up to date in 8.223s
found 0 vulnerabilities

C:\Users\admin\Documents\www>npx webpack
npx: installed 324 in 84.175s
One CLI for webpack must be installed. These are recommended choices, delivered as separate packages:
 - webpack-cli (https://github.com/webpack/webpack-cli)
   The original webpack full-featured CLI.
We will use "npm" to install the CLI via "npm install -D".
Do you want to install 'webpack-cli' (yes/no): yes
Installing 'webpack-cli' (running 'npm install -D webpack-cli')...

><EMAIL_ADDRESS>postinstall C:\Users\admin\Documents\www\node_modules\webpack-cli
> node ./bin/opencollective.js

                            Thanks for using Webpack!
                 Please consider donating to our Open Collective
                        to help us maintain this package.

                 Donate: https://opencollective.com/webpack/donate

npm WARN saveError ENOENT: no such file or directory, open 'C:\Users\admin\Documents\www\package.json'
npm WARN enoent ENOENT: no such file or directory, open 'C:\Users\admin\Documents\www\package.json'
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN www No description
npm WARN www No repository field.
npm WARN www No README data
npm WARN www No license field.

+<EMAIL_ADDRESS>added 208 packages from 127 contributors and audited 1053 packages in 136.984s
found 0 vulnerabilities

{ Error: Cannot find module 'webpack-cli'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:581:15)
    at Function.Module._load (internal/modules/cjs/loader.js:507:25)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at runCommand.then (C:\Users\admin\AppData\Roaming\npm-cache\_npx\5900\node_modules\webpack\bin\webpack.js:143:5)
    at process._tickCallback (internal/process/next_tick.js:68:7) code: 'MODULE_NOT_FOUND' }

Did I do right ?

I managed to export the logs for further analysis, check it out
https://github.com/visika/WhatsAllApp/commit/6cf519d75db77b25712cc9d50cc49aeb4488cfbc


Failing to install any CIA files

Since i updated to version 2.5.1 i can't dowload any CIA files whatsoever. I tried using QR codes, Urls or the programs own titledb, but none of it worked. The same error message keeps appearing. It states the following:
Failed to install from URL.
->especific program URL
Result code: 0xD963F99C
Level: Permanent (27)
Summary: internal (11)
Module: Application (254)
Desc: HTTP 404: Not Found (412)
Is this problem related to the update or is the issue on my own 3ds? I would appreciate if i could get a response, thanks in advance!

I get the same error

Same. Can't install from files or a URL. Had to revert back to 2.5.0 to fix it.

I don't know how to revert... tried installing 2.5.0 but it says a new version already exists... am waiting for 2.5.2... (2.5.0 gave me a weird problem, it wasn't opening the TitleDB also)

Fixed in 2.5.2, thanks for the quick find.


Remove optimize_children and replace with map_children

Which issue does this PR close?

Closes https://github.com/apache/arrow-datafusion/issues/5762.
Rationale for this change
NA

What changes are included in this PR?

Removed optimize_children()
Replaced optimize_children() withmap_children() in aggregate_statistics.rs
(Also changed argument from config to _config to fix linter error)

Are these changes tested?
This is a refactor inside optimize(), this function is already covered by existing tests

Are there any user-facing changes?
No

👍


Operator security: for tls to work there is a dependance on securityConfigSecret.

For operator to add tls setting as follows:
    tls:
      transport:
        generate: true
      http:
        generate: true

There is a dependance on
  security:
    config:
      securityConfigSecret:
##Pre create this secret with required roles and security configs
       name: <secret_name>

If only TLS is added
Error
ACTIONGROUPS, CONFIG, ROLES, ROLESMAPPING, TENANTS, NODESDN, WHITELIST, AUDIT] (index=.opendistro_security)
[2022-03-30T17:47:02,622][ERROR][o.o.s.c.ConfigurationLoaderSecurity7] [my-first-cluster-masters-2] Failure no such index [.opendistro_security] retrieving configuration for [INTERNALUSERS, ACTIONGROUPS, CONFIG, ROLES, ROLESMAPPING, TENANTS, NODESDN, WHITELIST, AUDIT] (index=.opendistro_security)
[2022-03-30T17:47:02,622][ERROR][o.o.s.c.ConfigurationLoaderSecurity7] [my-first-cluster-masters-2] Failure no such index [.opendistro_security] retrieving configuration for [INTERNALUSERS, ACTIONGROUPS, CONFIG, ROLES, ROLESMAPPING, TENANTS, NODESDN, WHITELIST, AUDIT] (index=.opendistro_security)
[2022-03-30T17:47:02,622][ERROR][o.o.s.c.ConfigurationLoaderSecurity7] [my-first-cluster-masters-2] Failure no such index [.opendistro_security] retrieving configuration for [INTERNALUSERS, ACTIONGROUPS, CONFIG, ROLES, ROLESMAPPING, TENANTS, NODESDN, WHITELIST, AUDIT] (index=.opendistro_security)
[2022-03-30T17:47:02,622][ERROR][o.o.s.c.ConfigurationLoaderSecurity7] [my-first-cluster-masters-2] Failure no such index [.opendistro_security] retrieving configuration for [INTERNALUSERS, ACTIONGROUPS, CONFIG, ROLES, ROLESMAPPING, TENANTS, NODESDN, WHITELIST, AUDIT] (index=.opendistro_security)
[2022-03-30T17:47:02,622][ERROR][o.o.s.c.ConfigurationLoaderSecurity7] [my-first-cluster-masters-2] Failure no such index [.opendistro_security] retrieving configuration for [INTERNALUSERS, ACTIONGROUPS, CONFIG, ROLES, ROLESMAPPING, TENANTS, NODESDN, WHITELIST, AUDIT] (index=.opendistro_security)
[2022-03-30T17:47:02,622][ERROR][o.o.s.c.ConfigurationLoaderSecurity7] [my-first-cluster-masters-2] Failure no such index [.opendistro_security] retrieving configuration for [INTERNALUSERS, ACTIONGROUPS, CONFIG, ROLES, ROLESMAPPING, TENANTS, NODESDN, WHITELIST, AUDIT] (index=.opendistro_security)
[2022-03-30T17:47:02,622][ERROR][o.o.s.c.ConfigurationLoaderSecurity7] [my-first-cluster-masters-2] Failure no such index [.opendistro_security] retrieving configuration for [INTERNALUSERS, ACTIONGROUPS, CONFIG, ROLES, ROLESMAPPING, TENANTS, NODESDN, WHITELIST, AUDIT] (index=.opendistro_security)
[2022-03-30T17:47:02,622][ERROR][o.o.s.c.ConfigurationLoaderSecurity7] [my-first-cluster-masters-2] Failure no such index [.opendistro_security] retrieving configuration for [INTERNALUSERS, ACTIONGROUPS, CONFIG, ROLES, ROLESMAPPING, TENANTS, NODESDN, WHITELIST, AUDIT] (index=.opendistro_security)
[2022-03-30T17:47:03,001][ERROR][o.o.s.a.BackendRegistry  ] [my-first-cluster-masters-2] Not yet initialized (you may need to run securityadmin)
[2022-03-30T17:47:03,004][ERROR][o.o.s.a.BackendRegistry  ] [my-first-cluster-masters-2] Not yet initialized (you may need to run securityadmin)
[2022-03-30T17:47:05,500][ERROR][o.o.s.a.BackendRegistry  ] [my-first-cluster-masters-2] Not yet initialized (you may need to run securityadmin)
[2022-03-30T17:47:05,503][ERROR][o.o.s.a.BackendRegistry  ] [my-first-cluster-masters-2] Not yet initialized (you may need to run securityadmin)
[2022-03-30T17:47:08,001][ERROR][o.o.s.a.BackendRegistry  ] [my-first-cluster-masters-2] Not yet initialized (you may need to run securityadmin)
[2022-03-30T17:47:08,004][ERROR][o.o.s.a.BackendRegistry  ] [my-first-cluster-masters-2] Not yet initialized (you may need to run securityadmin)

Background:
OpenSearch once TLS is added for Node Transport and HTTP rest API, the embedded security plugin creates.opendistro_security index to enable security settings, for this the securityadmin.sh has to run to load new settings, else the demo install_demo_configuration.sh file will run by default if the TLS setting is not added (If you do not configure anything opensearch will use included demo TLS certificates that are not suited for real deployments.)
curl -k https://localhost:9200/_cat/indices -u admin:admin
green open security-auditlog-2022.03.29 SHZ_xtRBTGub4NFhbtugSw 1 1 7 0 116.4kb 96.8kb
green open .kibana_1                    UOntE6z9Soa73BSdk3JI5Q 1 1 0 0    416b   208b
green open .opendistro_security         RYmlNkB5RgWAKMZU3_S05Q 1 2 9 0 178.1kb 59.3kb

With the current setup from the PR https://github.com/Opster/opensearch-k8s-operator/pull/61/files#diff-190387233823a104ed9004f0cba248cf0aa504090c923cad3be1a901bd01e99f
the securityadmin.sh  will be called by a kubernetes batch job.
securityadmin.sh need to run when we add tls or custom secrets and securityadmin.sh should also run when we add new config files.
With just adding TLS setting does not run the batch job, the following is seen in logs, as once TLS is added to operator opensearch.yml is already modified with Security settings, so the Demo Installer will quit
OpenSearch Security Demo Installer
 ** Warning: Do not use on production or public reachable systems **
Basedir: /usr/share/opensearch
OpenSearch install type: rpm/deb on NAME="Amazon Linux"
OpenSearch config dir: /usr/share/opensearch/config
OpenSearch config file: /usr/share/opensearch/config/opensearch.yml
OpenSearch bin dir: /usr/share/opensearch/bin
OpenSearch plugins dir: /usr/share/opensearch/plugins
OpenSearch lib dir: /usr/share/opensearch/lib
Detected OpenSearch Version: x-content-1.2.3
Detected OpenSearch Security Version: <IP_ADDRESS>
/usr/share/opensearch/config/opensearch.yml seems to be already configured for Security. Quit.
sed: cannot rename /usr/share/opensearch/config/seddRF6sR: Device or resource busy
Enabling OpenSearch Security Plugin

To move forward, we need to add securityConfigSecret for the security plugin to pick up TLS and passed in roles example as https://github.com/opensearch-project/security/tree/main/securityconfig
A Readme doc on configuring this setup would be helpful.
Once added
security:
   config:
     securityConfigSecret:
##Pre create this secret with required roles and security configs
      name: securityconfig-secret
   tls:
     transport:
       generate: true
     http:
       generate: true

To job runs to call securityadmin.sh

And now I can see all pods coming up

As discussed on Slack I think we should only mount the secrets if there is user provided config for the files, otherwise we should use the defaults from the Opensearch image.
If we do decide to maintain our own copy of the default files these should be embedded into the Operator, and we should have unit tests to confirm that the secrets are created correctly.

Fixed


Fix sparse array construction

Closes #141.
Still requires tests.

This also fixes another bug where construction of a sparse array on Dual values would drop dual tracers if their primal value was zero, due to SparseArrays._iszero(d::Dual) calling iszero(d::Dual), which returns iszero(primal(d)).

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

Project coverage is 46.62%. Comparing base (ac94586) to head (bf7478c).

Files
Patch %
Lines

src/overloads/arrays.jl
0.00%
2 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (ac94586) and HEAD (bf7478c). Click for more details.
HEAD has 3 uploads less than BASE
| Flag | BASE (ac94586) | HEAD (bf7478c) |
|------|------|------|
||4|1|

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #142       +/-   ##
===========================================
- Coverage   91.20%   46.62%   -44.59%
===========================================
  Files          37       36        -1
  Lines        1456     1349      -107
===========================================
- Hits         1328      629      -699
- Misses        128      720      +592

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

Looks like _iszero was a recent addition in August 2022 and doesn't exist in 1.6: https://github.com/JuliaSparse/SparseArrays.jl/pull/177

The first Julia version that passes is 1.9.

Then add the fix for version 1.9, and the sparse constructors will be buggy below that. Do they error or silently return something?

https://github.com/JuliaSmoothOptimizers/ADNLPModels.jl/issues/247 has me thinking: should we overload Base.ismissing? After all, a global tracer can never be missing, can it?

https://github.com/JuliaSmoothOptimizers/ADNLPModels.jl/issues/247 has me thinking: should we overload Base.ismissing? After all, a global tracer can never be missing, can it?

Not really related to this PR, but there is no sane thing to return for ismissing on global tracers. We don't want to return true or false and accidentally hit a branch in some control-flow statement, leading to a non-global sparsity pattern.
I opened #143 since this topic keeps popping up repeatedly.


add forceupgrade settings to aks stable cli

Related command
add forceupgrade settings to aks stable cli
Description

Testing Guide
aks update --resource-group={resource_group} --name={name} --enable-force-upgrade
This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.


MFA Option

Hello and thank you for your time.
The wording of the section on MFA confuses me a little.
I have AADConnect. PHS is enabled as backup to AD FS

"... if your Azure AD Connect is configured to synchronize credentials (username and passwords..." Does this mean PHS is turned on? Does it matter if it is turned on just as a backup but federation is actually being used?
In my scenario, can I user Azure MFA cloud service directly or do I need to use AD FS 2016 MFA adapter?

@SB-o-matic - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further.
Thank you.

@SB-o-matic - We would like to follow up on this. May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can make the necessary changes to improve the MS document for accuracy.

@SB-o-matic - We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Asgardia.space should be "impossible"

The current level "hard" is wrong for Asgardia because when me and my friend asked them about deleting our account, they didn't delete them.
They will "accept" your request(I've forward your request. please wait 3 days[1]), but they don't delete anything
about you(we waited 1 month+.) We can still login to our account, and, when I asked about my deletion
status, they told me same thing again([1]).

https://asgardia.space/en/forum/forum/general-discussion-14/topic/how-can-i-optout-from-your-mapping-5087/
They also use your "confidence" information publicly.


added styling for video element and fixed issue with safari image capture

Greetings,
I've added image styling for the video element, which would let you for example set width 100% or maybe have border radius for the video element , very useful when you have full width video element
i've also fixed an issue with Ios safari, it doesn't have ImageCapture object by default, I've passed the video element to the capture method and extracted the image from there ,
I've tested this method on another project with the exact same code and it works perfectly

alright, that seams reasonable
but i have a few questions what is the difference between cpp version and js version?
and would it be enough if i did this?
public async Task<string> Capture(ElementReference canvas, ElementReference? video = null)
and used image capture if video element isn't present in the js file?


Search by ISIN

Thanks for the amazing library!
Is it possible to get data for instruments by ISIN and not by a symbol?
Thank you.
Regards!

Same here

Unfortunately it's not possible as the ISIN code can be the same for multiple ticker symbols. The ticker symbol is at exchange level and it is different depending on the market: IBM ticker is not the same in NYSE and in Buenos Aires yet ISIN is same

Hello,
Seems legit. Thanks for the reply :)

Not Possible.


Azure container registry repo addition error using Helm 3

I am having issues in adding the Azure container registry as repo in azure CI/CI pipeline using Helm 3 .
Getting below error
Error: looks like "https://xxx.azurecr.io" is not a valid chart repository or cannot be reached: failed to fetch https://xxx.azurecr.io/index.yaml : 404 Not Found
Syntax using for adding the repo
helm repo add  https://xxx.azurecr.io --password $(acr_password)  --username $(acr_user)
Set HELM_EXPERIMENTAL_OCI=1 in azure pipeline
With helm 2 commands, i am able to add the repo without issues.
As we are migrating from helm 2 to 3, I am not able to add the ACR as repo using helm 3

@puneethkgowda Thank you for your query!!!
Can you please help me out wit documentation you are following so that I can help you out better on this?
Thanks

I have reference the below doc.
I am able to save and push the chart to repo.
I can even pull and extract to local cache.
But if i try to add the repo locally, its giving error. As a workarouf , i am indivually pulling each chart to local and trying to deploy

@puneethkgowda Thank you for responding back.
Apologies I am not able to see doc in your comment so can you please send it over again so I have better idea.
Thanks

https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/container-registry/container-registry-helm-repos.md

@puneethkgowda Thank you for the information let me check on this and i will get back to you with update.

@psmanhas @puneethkgowda The helm 3 CLI manages Helm 3 charts, and also is backward-compatible with Helm 2 repositories.
The helm repo, including helm repo add, is a command set dealing with Helm 2 repositories, although the Helm CLI is in version 3. To add a Helm 2 repository via Helm 3 client, you should run az acr helm repo add or run
helm repo add https://xxx.azurecr.io/helm/v1/repo/ --username ${acr_user} --password ${acr_password}

Instead of pushing / pulling helm 2 charts, we suggest you to pull / push helm 3 charts. For more detials, please check the documentation Push and pull Helm charts to an Azure container registry.
Additionally, helm 3 charts are artifacts like docker images. You do not to add the repository explicitly.

@shizhMSFT I am performing push/pull on helm 3 charts. When i push it to ACR, i could see the charts as artificats in ACR repository.
To deploy the chart, i am pulling each chart to local cache, extract it and then installing it.
When using helm 2 charts, i was adding repo using  az acr helm repo add. then was able to access without fetching it to local.
Below are the steps performed in helm 2

Devops build pipeline
-Helm package
-az acr helm push

Devops release pipeline
-az acr helm repo add -n $(acr_user) --password $(acr_password)  --username $(acr_user)
-helm install $(acr_user)/chartName

Below are the steps performed in helm 3

Devops build pipeline
-helm chart save
-helm chart push ( charts are stored as artifacts in the path mwhChart/${chart} )

Devops release pipeline
-helm registry login  --password "$(acr_password)" -u $(acr_user) $(acr)
-helm chart pull $(acr)/mwhChart/${chart}:${version}
-helm chart export $(acr)/mwhChart/${chart}:${version} --destination ./$(acr_user)
-helm install $(acr_user)/${chart}

I modified pipeline to
-helm registry login  --password "$(acr_password)" -u $(acr_user) $(acr)
-helm repo add $(acr_user) https://xxx.azurecr.io/helm/v1/repo/ --username ${acr_user} --password ${acr_password}
-helm install $(acr_user)/${chart}
Getting below error
Error: looks like "https://xxx.azurecr.io/helm/v1/repo/" is not a valid chart repository or cannot be reached: failed to fetch https://xxx.azurecr.io/helm/v1/repo/index.yaml : 401 Unauthorized

@shizhMSFT Can you please provide your inputs on mentioned ask?
Thanks

@puneethkgowda @psmanhas Sorry for the late reply. The response shows 401 Unauthorized, which means that

Incorrect username / password are provided
Or, incorrect type of credential is provided. Please note that helm repo add accesses remote servers using basic auth. For ACR, only admin user credentials or service principal credentials can be accepted for basic auth.

Please verify your credential and its type accorrdingly.
Here is an example output using admin credentials:
$ helm repo add $name https://$name.azurecr.io/helm/v1/repo/ --username $username --password "$password"
"<name>" has been added to your repositories

@shizhMSFT I am using the admin account to connect to ACR. I am using the username and password from below screen in ACR access keys.

Admin user credentials should work. Could you double check the credentials and login server name for the helm command? Please also check that you have double quoted the password variable in case there’s special characters in it.
If you still have difficulties on the authentication, please provide the helm version and the login server name if possible. Again, we suggest you opening a support ticket in Azure so that we can help you with privacy preserved.

@puneethkgowda Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


Windows Build for NetExec

Update pyinstaller configuration for a working Windows build

@NeffIsBack @zblurx ready for review!


Menu: add new theme variables

Issue Description
In the current Menu component, certain style attributes are not themeable. Additionally, some interaction states aren't accounted for in the current component. The items to be updated are:

Menu item Padding (made themeable)
Menu item Margin (made themeable)
[EDIT] Menu item disabled color (made themeable)
Keyboard Focus State (state defined/added, and made themeable)
Pressed State (state defined/added, and made themeable)

Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
The Menu component should have themeable variables for all of the desired attributes.
Current Behavior
"N/A"

We are looking to have this functionality implemented as well. Beyond the things listed above, we would also like to be able to define a disabled color for the chevron also.


groovy-mode: install license file in prefix instead of doc directory

This changes where gplv2.txt is installed, as I brought up here.

Thanks! Merged as e2bf336761be38220c7e61f6df6069b3d75ac9c0.


can't write full ip into commander

Hello,
Under ubuntu 18.04.03, i can't type the full ip but when inspect the character are well written.

huh, that doesn't look like a valid IP address.

si si the address i want to type is <IP_ADDRESS> and i don't see the 9 into the commander zone so of course i type several times to ensure there is a bug. ok ?

Oh, let me have a look.

Weiiiird, works for me if I type it:

wait a minute please not sure i'm working with the latest version...

Oups sorry with the latest version no problem


Slash-menu positioning bug in Safari

Issue Summary

Steps to Reproduce

Open the editor, type / in an empty paragraph

Technical details:

Ghost Version: https://github.com/TryGhost/Ghost-Admin/commit/60ca9e8aced21cca5779c7264534adfa6ac955f0
Browser/OS: Safari 10.1

Thanks @kevinansfield ,
This is a bug in Safari and Safari mobile where it won't give the position of a range which is collapsed. In this case the editor tries to figure out where to place the menu based on the paragraph that the cursor is in.
It's not ideal, the only reasonable workaround that I can think of is to create a new one character range where possible and measure that.
Ironically this bug makes the automatic scrolling to match the cursor location feel nicer in Safari as it jumps paragraph by paragraph rather than line by line. 😄


Generate GitHub releases for gem changes

What
We want to use the GitHub releases function to improve visablity to automatically generate the release notes for each version of a gem
Why
To improve the visibility of the changes in each version and make it easier for everyone to access if to upgrade
How
We need to use a GitHub Action in the pipeline to create the release using the version from the version.rb file.
https://github.com/actions/create-release
ACs

[ ] We have a release with a version that matches the version.rb file
[ ] It has auto generated release notes based on included PRs

https://github.com/simplybusiness/approved-github-actions/pull/15

Initial working solution:
Need to manually specify:

master to the default branch
Gem name instead of Example in Example::VERSION

name: Create GitHub Release

on:
  push:
    branches:
      - "master"

jobs:
  create-release:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v2
      - name: Set up Ruby
        uses: ruby/setup-ruby@v1
        with:
          bundler-cache: true
      - name: Get Ruby Version
        id: get-ruby-version
        run: echo "::set-output name=RUBY_VERSION::$(bundle exec ruby -e 'puts Example::VERSION')"
      - name: Release
        uses: softprops/action-gh-release@v1
        with:
          generate_release_notes: true
          tag_name: ${{ steps.get-ruby-version.outputs.RUBY_VERSION }}


[pigment-css][material-ui] Grid throws an error

Steps to reproduce
Link to live example: (required)
Steps:

Use the official nextjs material-ui pigment-css template
run dev to see the error

Current behavior

Expected behavior
There should be no errors
Context
No response
Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: material-ui pigment-css grid

duplicated with #43635


Azure.Identity Conceptual Docs (with language-specific code samples)

This is a tracking item to finish writing Conceptual Documentation for Identity before the November SDK update.
This is something @ellismg began last week, but it needs to be expanded with language-specific code samples as well.
Per offline discussion, this conceptual docs will be PR'd to all 4 repos and each language-specific package owner will fill-in the language-specific portion in the markdown / docs
Related: Azure/azure-sdk-for-java#6062 https://github.com/Azure/azure-sdk-for-net/issues/8488

Removing blocking-release as this is purely documentation that does not impact the shipping package.

Reactivating based on planning discussion in late-December.  @ellismg is driving Identity conceptual documentation push across all azure-sdk languages

@joshfree - to reactive, should we reopen issue?


Native query method in Repository Interfaces

Possibility of add methods for to execute native queries, adding the query as method notation.
https://docs.spring.io/spring-data/jpa/docs/current/reference/html/#jpa.query-methods.at-query

Closed this issue because the purpose of this framework is to avoid write any SQL query.


Is it necessary to modify RUSTFLAGS in cargo-3ds?

libctru is included twice

I think there's something to this stemming from the fact that cargo-3ds adds RUSTFLAGS for linking libctru, and ctru-rs also does in its build script. I'll open an issue to investigate on cargo-3ds
Originally posted by @ian-h-chamberlain in https://github.com/Meziu/ctru-rs/pull/46#issuecomment-1034901000

More context: RUSTFLAGS get updated here in cargo-3ds, but there is already a build script for ctru-sys which specifies that it links against libctru (as well as the links field in Cargo.toml, although that seems to be more about convention and less functional).
If rustc will automatically link to the right libraries via the build script, is it necessary to provide via RUSTFLAGS the same linker args? I have noticed when linking fails there seem to be several duplicated references to ctru, e.g. (edited for clarity):
error: linking with `arm-none-eabi-gcc` failed: exit status: 1
  |
  = note: "arm-none-eabi-gcc"
"-specs=3dsx.specs"
"-mtune=mpcore"
"-mfloat-abi=hard"
"-mtp=soft"
... lots of object files ...
"-Wl,--as-needed"
"-L"
"/Users/ianchamberlain/Documents/Development/3ds/ctru-rs/target/armv6k-nintendo-3ds/debug/deps"
"-L"
"/Users/ianchamberlain/Documents/Development/3ds/ctru-rs/target/debug/deps"
"-L"
"/opt/devkitpro/libctru/lib"
"-L"
"/opt/devkitpro/libctru/lib"
"-L"
"/Users/ianchamberlain/.rustup/toolchains/horizon/lib/rustlib/armv6k-nintendo-3ds/lib"
"-lctru"
"-Wl,-Bstatic"
... rlib files ...
"-Wl,-Bdynamic"
"-lctru"
"-lctru"
"-lc"
"-lm"
"-lctru"
"-lc"
"-lm"
"-lctru"
"-Wl,--eh-frame-hdr"
"-Wl,-znoexecstack"
"-L"
"/Users/ianchamberlain/.rustup/toolchains/horizon/lib/rustlib/armv6k-nintendo-3ds/lib"
"-o"
"/Users/ianchamberlain/Documents/Development/3ds/ctru-rs/target/armv6k-nintendo-3ds/debug/deps/common-238f93ba962b2c97.elf"
"-Wl,--gc-sections"
"-no-pie"

I think it's probably harmless to have -lctru multiple times like that, but I wonder if we are doing extra work unnecessarily for the linker invocation. If so, the fix would probably be to simply remove the RUSTFLAGS modification from cargo-3ds.

Reopened as of #40. Seems like network related functions won't link without using RUSTFLAGS.

I ran into a problem with this today, when linking ctrud via the build script from ctru-sys (but cargo-3ds still passes an explicit -lctru while building std), resulting in duplicate definitions between libctrud and libctru at link time.
I think part of the problem here is that -Zbuild-std builds the standard library with "plain" RUSTFLAGS, but Cargo might end up passing different flags to ctru-sys when it builds it (based on --release, [profile."*"] etc), resulting in different link flags between std and the final executable build.
From doing a bit of searching, perhaps we can somehow use the --as-needed flag https://github.com/rust-lang/rust/issues/99424 ? (i.e. something like RUSTFLAGS="-Clink-arg=-Wl,--as-needed"). A little bit of testing hasn't gotten me far with it, but maybe there's some way to use it to solve this...
I am not sure of a general way to unify the linker flags (specifically -lctru[d]) between build-std and the downstream libraries, unless there is some magic rustc/cargo flag to say --link-but-only-if-nothing-else-did=ctru. Maybe something like https://doc.rust-lang.org/cargo/reference/unstable.html#profile-rustflags-option ?
Thus far I haven't found a relevant issue in https://github.com/rust-lang/wg-cargo-std-aware/ but that repo is rather large in scope so :shrug:

If none of the above flags produce what we need, it's possible we could try to parse the cargo flags and figure out whether we should link ctrud or ctru (similar to https://github.com/rust3ds/ctru-rs/blob/a636722b4978d36531e5c9e2a866cdec3387877e/ctru-sys/build.rs#L27-L33 except we'd have to figure out the PROFILE based on the command invocation :sob:) but this doesn't seem ideal.


Fix issue causing fck-nat instances to be randomly terminated and not replaced

By default, EC2 spot instance requests are "one-time". When the requested spot instance is interrupted, it will be terminated without replacement. terraform-aws-fck-nat does not change this setting.
Therefore, terraform-aws-fck-nat users who enable use_spot_instances will see that, after an unpredictable length of time, their instances get terminated, and require manual intervention to restore.
This PR changes spot instance requests to be "persistent", which means that after a termination, AWS will restore them automatically. It also sets the spot instance_interruption_behavior to "stop" because setting it to either "stop" or "hibernate" is a requirement for persistent requests. I didn't choose hibernation because it's probably unnecessary, and it limits which instance types can be used.

I closed this because it can create problems when a new fck-nat instance is created by Terraform. The previous spot requests remain active and spawn additional unwanted instances. I'll reopen when I have a fix.


This Month in Rust OSDev (January 2021) does not appear in the status updates section

Thanks for reporting! Seems like our script accidentally starts a new year with month number 0 :smile::
https://github.com/phil-opp/blog_os/blob/9d381723ed7075d6e1405e7ae2a0ae17680f5cfa/blog/before_build.py#L72
I'll push a fix in a minute.

Fixed in https://github.com/phil-opp/blog_os/commit/15ae425a6b99ad7132a0da65baa267f6327c3d68.


[BUG] nav item is overlaping with the toggle icon

Description

"Getting Started" nav item is overlaping with the toggle icon at the of reloading.
Expected Behavior

It should behave like normal.
Screenshots

Contributor Guides and Handbook
The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.

📚 See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Hello @anikdhabal , Thanks for raising this issue however its a duplicate for the issue #1162


Make a shard's description nillable

We grab the description from the GitHub repo, and it might not be set.
The migration simply removes the NULL constraint on the database. The
reverse migration sets the default to be an empty string, then re-adds
the NULL constraint.
I needed to change the type for description from String to String?
and finally, ensure that if there is no description the ShowPage
doesn't crash.
Fixes #17

Hmm, having read the title again, I think I assumed Shard#description not Repo#description. Should I have just made the Repo description nillable and kept the Shard description and database requirement?

The issue was originally around Repo#description throwing an exception when it received a nil description but Shard would have to be updated anyway. This is exactly what I was thinking!


tests: use testify to check the test results

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area tests
What this PR does / why we need it:
Use testify to assert the test results to make more clean and avoid writing more code
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

@cpanato: The label(s) area/tests cannot be applied, because the repository doesn't have them

In response to this:

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area tests
What this PR does / why we need it:
Use testify to assert the test results to make more clean and avoid writing more code
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign issif after the PR has been reviewed.
You can assign the PR to them by writing /assign @issif in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

f95b60f fix alermanager test

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

testfy catchs a broken test :)

LGTM label has been added.  Git tree hash: a7252c423abc8b33830410c7112a20d013157c4c

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

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


Cleanup diagnostics

Codecov Report

Merging #1364 (0ff8bf4) into master (2c0f786) will decrease coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1364      +/-   ##
==========================================
- Coverage   41.27%   41.14%   -0.13%
==========================================
  Files         158      155       -3
  Lines       12849    12712     -137
==========================================
- Hits         5303     5230      -73
+ Misses       6806     6753      -53
+ Partials      740      729      -11

Impacted Files
Coverage Δ

pyramid/tier_fs.go
58.77% <0.00%> (-3.95%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #1364 (0ff8bf4) into master (2c0f786) will decrease coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1364      +/-   ##
==========================================
- Coverage   41.27%   41.14%   -0.13%
==========================================
  Files         158      155       -3
  Lines       12849    12712     -137
==========================================
- Hits         5303     5230      -73
+ Misses       6806     6753      -53
+ Partials      740      729      -11

Impacted Files
Coverage Δ

pyramid/tier_fs.go
58.77% <0.00%> (-3.95%)
:arrow_down:

Continue to review full report at Codecov.

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


extractor, utils: dedupe bitcode paths before linking

This is my attempt at fixing #53.
Let me know what you think -- Go isn't my strongest language, so I wasn't sure how idiomatic my implementation here is.

Thanks. Go isn't my strongest language either, so I will rely on the grade it gets after I merge...

Thanks! I'll test these changes out.


ENV property deprecation warnings

Using the latest seed, I'm seeing numerous warnings like this:
Warning: the "ENV" property is deprecated. Use "BUILD_TYPE" instead.
I've done a global search through everything under /tasks and all I can find is:
Search "ENV" (5 hits in 4 files)
  tools\config\seed.config.ts (2 hits)
	Line 65:   get ENV() {
	Line 66:     util.log(util.colors.yellow('Warning: the "ENV" property is deprecated. Use "BUILD_TYPE" instead.'));
  tools\env\dev.ts (1 hit)
	Line 4:   ENV: 'DEV'
  tools\env\prod.ts (1 hit)
	Line 4:   ENV: 'PROD'
  tools\utils\seed\template_locals.ts (1 hit)
	Line 38:     ENV_CONFIG: JSON.stringify(Object.assign(baseConfig, config))

So basically I can't tell what is trying to use Config.ENV and generating these warnings. Obviously, I can omit the warnings by just commenting out the log message, but I didn't really want to do that without understanding what's trying to use ENV in the first place. It seems like nothing is using it, so the fact that the warnings are still appearing is making me think I'm missing something.

To add: just to see what happens, I commented out the get ENV() method in seed.config.ts, and everything still seems to compile and run correctly. Which seems to make this even more baffling.

https://github.com/mgechev/angular-seed/pull/1519

The lodash templetes are accessing ENV which causes the warnig. The ENV property together with the deprecation warning will go away in the next few days.


Increase timeout

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Describe your test plan
Write there any instructions and details you may have to test your PR.

Will do, just wanted to create the PR to get the .msi first 😅


Better animation
What is the current behavior?
Items show up sooner than animation ends.
What are the steps to reproduce?

What is the use-case or motivation for changing an existing behavior?
Its a bug
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular: 5.2.5
Material: 5.2.1
OS: Mac 10.13.3
Typescript 2.6.2
Chrome 64

Why "Sidenav" in title? This seems to be an expansion or something like that. Could you provide a stackblitz minimum reproduction?

True, i use the expansion panel as a sidenav so that i made the title mistake.

Closing as duplicate of #8754


Support Multi-Agent applications?

很不错的工作，看给的例子都是简单的单Agent例子，请问有多Agent的示例吗？最好是关于RAG的，谢谢！

Thanks a lot for your recognition! We are also developing and testing multi-agent instances and we will keep this repo updated. Suggestions and pull requests are always welcome!


⚠ Bofrost.nl has degraded performance

In f913b3e, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 921 ms

Resolved: Bofrost.nl performance has improved in ea42fa6.


Fix indentation on sample smcp v2

/retest

no idea why it failed twice. It passes locally...

/retest

no idea why it failed twice. It passes locally...

https://github.com/maistra/istio-operator/pull/649 needs to be merged

no idea why it failed twice. It passes locally...

https://github.com/maistra/istio-operator/pull/649 needs to be merged

/retest

/retest

/retest

/retest


Prefetch count doesn't work

let ch = rabbit.open_channel(None).unwrap();
ch.qos(0, 20, true).unwrap();

This code doesn't work.
let queue = ch.queue_declare(
            "some_queue",
            QueueDeclareOptions {
                exclusive: false,
                auto_delete: true,
                ..QueueDeclareOptions::default()
            },
 ).unwrap();

But in rabbitMq it's still have prefetch count 0

I think you might be looking in the  wrong place because you passed true for global. Can you try two things:

With the code you have now, can you look at the channel settings (not the queue consumer settings)? If I use your code I see

Try changing global to false and check the consumer screen you're looking at, and see if that does it.

I'm going to close this; please reopen if I was wrong and there's a problem here.


Hasura actions template

Since you are already providing the pulumi aws provider I think the best way to do this is by provisioning an aws lambda function and api gateway endpoint that is used by a hasura action.
Let me know if you need a hand with this, I have some generic hasura / lambda boilerplate that might be helpful.

Agreed. I like that. I can handle this. What’s a simple example for an
action with a Todo app?
On Tue, Nov 17, 2020 at 10:14 PM Austin Rivas<EMAIL_ADDRESS>wrote:

Since you are already providing the pulumi aws provider I think the best
way to do this is by provisioning an aws lambda function and api gateway
endpoint that is used by a hasura action.
Let me know if you need a hand with this, I have some generic hasura /
lambda boilerplate that might be helpful.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tiagob/create-full-stack/issues/53#issuecomment-729513173,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH32YS37RZX7K54LS63H2DSQN66VANCNFSM4NMNLLPA
.

The easiest action I could think of would be to just provide an echo function that sends back whatever you send it, but if you want something specific to the todo app maybe an email notification on item completion?
I've written some code to extract and validate hasura action body and headers using the codegen types, but I think it might be a little to abstract for a simple example.


added provider flag for config generation

defaults to aws

Don't merge yet but feel free to comment - I am going up to update the documentation to reflect this flag

I am currently working on another branch where several internal parts of generateCmd have been extracted to make testing easier. This new branch will include the same features as this PR, but it will look a little different and will include a test.

Okay, ready to review.  Test was added to check that there is a default provider of 'aws' if provider flag not passed, that providerConfig and generateConfig variables are set in preRun, and that the specified provider will grab the correct default config file.  I was thinking of changing config.yaml to aws-config.yaml, does anyone have any opinions on that?

I vote yes on changing the config to aws-config.

@leahnp The file created is still config.yaml no matter what since k2cli is looking for that file, but I was thinking that down the road as we have more providers and more default configs, it would be good to have aws-config, bare-metal.config, gke-config, joyent-config in the filesystem to make it more obvious. Not really an issue though until we get more provider support.

@DStorck agreed!


Brave CDI Integration

What kind of issue is this?

[ ] Question. This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use Gitter
or Stack Overflow. https://gitter.im/openzipkin/zipkin https://stackoverflow.com/questions/tagged/zipkin

[ ] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests
get fixed and stay fixed. If you have a solution in mind, skip raising an issue and open a
pull request instead.

[ X ] Feature Request. First, look at existing issues to see if the feature has been requested
before. If you don't find anything, tell us what problem you’re trying to solve. Often a
solution already exists! Don’t send pull requests to implement new features without first
getting our support. Sometimes we leave features out on purpose to keep the project small.

I'd like to propose Brave shipping with out of the box CDI support.  CDI, Contexts & Dependency Injection for Java (previously for Java EE) is a programming model akin to Spring providing dependency injection support.  Having Brave support CDI would allow anyone who uses Brave to use it both in their EE applications as well as plain java applications leveraging CDI 2.0, as well as those adding support for Servlet containers.
I've done a bit of a prototype, adding support for JAX-RS integration.  I'd like to push it forward, adding base servlet filter and some AOP type interactions.
You can see what I've done here: https://github.com/hammock-project/hammock/tree/master/util-brave/src/main/java/ws/ament/hammock/brave

@johnament you know me.. rule-of-three :) can you help make this more popular? maybe see if users of CDI are interested and can comment back that they's use it?

@adriancole actually the idea is spawned from a discussion here - https://groups.google.com/forum/#!topic/microprofile/YxKba36lye4 - adding CDI integration basically makes this work, and would positive Brave to be heavily used across MP implementations.

gotcha, looking at your code, the CDI feature would be quite small, though it is hard to say how big it might become? what sort of systems do you see ending up needing CDI'ified? would this be reasonable as a single jar, or would it end up needing buddy modules in many places.

Hi, @johnament I don't know enough about CDI to know what to do. Can you help? Brave 4.3 is being released now and has a new README etc. We can release 4.3.1 if there's something quick to add to support CDI. Brave 4.3 won't be announced until Monday

I just experiences this exception when trying to inject brave.Tracing:
org.jboss.weld.exceptions.DeploymentException: WELD-001410: The injection point has non-proxyable dependencies:
[UnbackedAnnotatedField] @Inject private TracingTest.tracing at TracingTest.tracing(TracingTest.java:0)
StackTrace at org.jboss.weld.bootstrap.Validator.validateInjectionPointForDeploymentProblems(Validator.java:399)
...
Caused by: org.jboss.weld.exceptions.UnproxyableResolutionException:
WELD-001480: Bean type class brave.Tracing is not proxyable because it contains a final method public final brave.Clock brave.Tracing.clock(brave.propagation.TraceContext) - Producer Method [Tracing] with qualifiers [@Any @Default] declared as [[BackedAnnotatedMethod] @Produces @ApplicationScoped TracingProducer.produceTracing()].
	at org.jboss.weld.util.Proxies.getUnproxyableClassException(Proxies.java:223)

When producing javax.enterprise.context.Dependent scope it works, but it should be a "singleton", right? At least it is for spring applications.

not sure why Tracing needs to be proxied.. there's no circular deps.
I'm open to helping with this, if someone can make me an example app like https://github.com/openzipkin/brave-webmvc-example

Why CDI is proxying: https://weld.cdi-spec.org/documentation/#13

We can remove final on there.. I'd prefer to see a JEE spec that says types can't be final, but this vendor-specific one can work I suppose. @mojavelinux @aslakknutsen I know it has been a while, but do you remember if it is an implementation detail that singleton types MUST not be final, or something in the CDI spec?

If I don't get an official link about this, I'll remove final anyway with a comment about the link you gaev @svenhaag .. thanks!

It certainly has been awhile :wave:, but I don't remember anything about a singleton type needing to be final.

If I don't get an official link about this, I'll remove final anyway with a comment about the link you gaev @svenhaag .. thanks!

Actually it should be possible according to the legal-bean-types spec?!
But then it also says what is unproxyable.
WELD mentions it too, see proxying_classes_with_final_methods.
The thing with proxies is, they need to inherit the object to be proxied.

not sure why Tracing needs to be proxied.. there's no circular deps.
I'm open to helping with this, if someone can make me an example app like https://github.com/openzipkin/brave-webmvc-example

Created PR https://github.com/openzipkin/brave-webmvc-example/pull/39


Handle toast timeouts better

Cancel timeouts if they are running and set new ones

This issue was moved to RiotGear/rg#8


Enhance Anyway reports

Create two selection dropdown boxes:

"Select report type:": Accidents near school, Pedestrian accidents by city, more TBD
Select specific report, if accidents near schools was selected, choose choose between these reports, if other report type was selected choose between flavors.

Added pedestrians by city to the drive, please add it to the reports

#Hackaton_February_2018


Update es-ES.coffee

Added some new translates and changes.

Hello, @duhow! Thank you for contributing. Can you sign our CLA so that I can merge your pull request?

@Imperadeiro98 Done!

@nwinter, can you merge this PR?

Hi Duhow!
Looks like we had the same idea at the same time. I think it is easier to provide smaller patches, it makes rollback faster and can be easily reviewed.
Also, I've found some 'wording' issues in this patch. E.g. on certain pages 'payments' has already been translated as 'Pagos', and you are using 'Facturas'. Worths to check previous commits or other sentences to build a common wording guide.
I was about to start doing it and then I saw your PR. Do you want to split tasks?

Thanks @duhow!