Notify failure not working

Only the Notify Start is sending a notification to the channel. Notification for failure is not working.

I'm having the same problem. I can get an abort notification but failure notifications don't seem to work.

did you all get it working? it works for me.

Hi guys,
forgot to update. To remedy this issue you need to config the plugin to notify you of failures as well (why isn't this the default? I don't know).
Go to your project configuration, scroll down to Post-build actions, expand the Slack section if needed, and check the Notify failure, like here:

This is configured to notify only of success and failure. Hope it helps.


[Gekidou] new UI for no results found

Summary
Adds new UI for no results when searching.
This PR applies to:

Browse channels
Create Direct Message

Ticket Link
https://mattermost.atlassian.net/browse/MM-41761
NONE

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/21737
https://circleci.com/gh/mattermost/mattermost-mobile/21736

Artifact links:
https://output.circle-artifacts.com/output/job/64fa713c-0cdf-4fb0-ae07-5bb6ede1ecc4/artifacts/0/tmp/artifacts/Mattermost_Beta.ipa
https://output.circle-artifacts.com/output/job/43d4935c-523c-4860-8f19-dd1a2676cdc2/artifacts/0/tmp/artifacts/Mattermost_Beta.apk


do link shorteners count or not count as blocked?

Hi, a few of my friends and fans use your list. I was recently blindsided by a few of them could not access links I shared with a shortener. Sure enough, buff.ly is on the list, however I checked for other shorteners and just one example, bit.ly is not.
Anyway, forgive me getting to the point: what gives? Surely bitly is considered a more spammy link shortener than buffer/buffly? There are a lot of legitimate people using buffer to save time in getting their message out. I myself am an indie musician and I would waste all day online if I didn't have work to do, so I'm trying to stay out of that pit of despair.
At a cursory glance of the types of things that are blocked I'm not sure if shorteners are considered "social", I just want to understand what's going on and help improve the project with my feedback if I can.

@inversephase in commit lightswitch05/hosts@cb7ab86d7d8fd6319242786c5011de259dadf6f3 I removed buff.ly from my ads-and-tracking list and moved it to my aggressive list instead - which is not part of this project. Everyone maintains their own lists as they see fit. I don't have a specific stance on link shorteners but definitely don't like breaking sites for people. I don't use social media, so sometimes it can be hard to sort the trash from useful tools, especially regarding social media since all of it can fall into ads and tracking in some way or another. I hope this answers your question, please close the ticket if so.

Thanks @lightswitch05!
This is included in the latest release.
Closing.

thanks for the quick responses on this, sorry I was not so quick on my reply. appreciated!


Remove Flaky ConcurrentTaskUtilTests

These tests are too unpredictable to consistently pass, so they are to be removed.

Benchmarking with nanotime on the JVM is not a good idea anyway: https://shipilev.net/blog/2014/nanotrusting-nanotime/
If we ever want to do somehing like this again, we better use JMH:

https://github.com/openjdk/jmh
https://github.com/sbt/sbt-jmh
(also usefull for periodically test for performance regressions in frontends)

Good idea for the long term - especially if a bad query ends up slowing down post-processing


Check the rsync as the first preparation step

As the very first prepare action for each guest we need to ensure
that rsync is installed on the guest if needed. This fixes failing
/tests/run/worktree when run against virtual provision method.

This fix is necessary in order to make the new worktree feature correclty working with virtual provision. Please, review as soon as possible. Thanks.

You should use the method for installation we already have.

Yes, I was pondering about this for some time. But this would mean that the extra rsync prepare plugin would be always included under the prepare step, not brining much benefit I believe. I finished with this approach as not causing unnecessary spam to the user.

But theoretically these dependencies important for test execution, should be solved as part of provision step, what will install rsync or whatever as part of image/container init.

The prepare step seems like a logical place for the initial guest preparation to me. Why do you think provision step would be better for this?

But theoretically these dependencies important for test execution, should be solved as part of provision step, what will install rsync or whatever as part of image/container init.

The prepare step seems like a logical place for the initial guest preparation to me. Why do you think provision step would be better for this?

Because for me preparation is preparation for test execution, so there should be just items what are relevant to testplans, not some underlying tools, what are theoretically not anyhow important for system preparation steps. E.g. I can imagine to replace rsync, by using pure ssh like tar cf - | ssh user@hostname tar xf - and then rsync is unnecessary to install anyhow. So that each provision plugin should took care to be able to communicate with TMT. so just prepare plugin knows how, and what to install, so possible to be part of push/pull method as some check if system is prepared.

Because for me preparation is preparation for test execution, so there should be just items what are relevant to testplans, not some underlying tools, what are theoretically not anyhow important for system preparation steps.

Thanks for the detailed explanation. I see your point. We can polish this in the future. But let's not block the 1.4 on this now.

Because for me preparation is preparation for test execution, so there should be just items what are relevant to testplans, not some underlying tools, what are theoretically not anyhow important for system preparation steps.

Thanks for the detailed explanation. I see your point. We can polish this in the future. But let's not block the 1.4 on this now.

Sure, it is generic RFE, not blocking this PR.


Fix 'disconnect' banner on Tanzu Explorer

make sure the "disconnect" button disappears upon clicking it

Update #1 after testing:

the banner does disappear after logging back in
-if you cancel the login after clicking the button, you cannot click it again. working to fix this


feat: add changelog information

I created a new experimental cli to repost changelog information to static site generator like HUGO.
I am opening this pullrequest in draft mode to see what it would look like
Description

Test
//
Additional Information
Tradeoff

Potential improvement

Well to make the test pass, I need to fix typos in all previous release note from version 0.16.0 because any changes done to files generated by releasepost will be reset next time releasepost is executed...


Using cue to generate deterministic example data that satisfies constraints

Is your feature request related to a problem? Please describe.

Describe the solution you'd like
if I have a configuration like this,
msg:   "Hello \(place)!"
place: "World" | "Earth" | "Toronto"

I would like to be able to execute some command that automatically chooses one of the options in some deterministic fashion.
cue generate myscript.cue
msg: "Hello Earth"
place: "Earth"

Describe alternatives you've considered
I can imagine that I could write some Golang code that would do this. I wanted to check if there is a way to do this out of the box.
Additional context
I'm trying to see if I could use cue to generate objects from something like factories in ORM. There are tools like Faker.js that have constraints but they are very limited. Also, they require writing JavaScript which can be unattractive to some opinionated developers who're not JavaScript users. CueLang seems like a good neutral alternative for a cross-platform API.

This would also be useful for fuzzing.

This issue has been migrated to https://github.com/cue-lang/cue/issues/717.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Use targetPort for probes if set

Description of the change
If targetPort is set to a different port than the exposed service port the probes should check the container targetPort instead of the service port.

Benefits

Possible drawbacks

Applicable issues

fixes #

Additional information

Checklist

[x] Chart version bumped in Chart.yaml according to semver.

Thanks for taking the time to submit this PR 👍 I'll see if we can get this merged into an upcoming version along with a fix for #69.


operator project-quay (3.9.0-nightly.20230321)

Add 3.9.0-nightly.20230321 to community operators

/merge possible

/retest

/merge possible

/merge possible


Switch to oci-spec-rs

We should switch to the oci-spec crate rather than having our own implementation of the runtime spec.

I can do that as part of #548, moving the code into crates necessitates that I touch the existing oci spec code anyway.

Sounds good @Furisto.


Unable to get array when posting multiple select values.

I have a form with a multiple select dropdown. When selecting more than 1 value, res.body does not give me the array of selected values, like it does for req.files.
Is this a bug?

I'm having a similar problem. Did you ever figure out how to get around this?

Sounds like a bug. But can some give some actual code examples.
Thanks! :100:

It should work now with mutiples: true. Feel free to reopen an issue if you can provide an example of it not working

It should work now with mutiples: true. Feel free to reopen an issue if you can provide an example of it not working


Update UsageWithReact.md

At this point in the tutorial, there is no id on a Todo instance, and the TOGGLE_TODO action is using the array index of todos -- so pass that index down to the Todo component.

Thanks!


Attempted to fix #180

Delete Usage section from README until we can update it

Codecov Report

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

@@           Coverage Diff           @@
##           master     #248   +/-   ##
=======================================
  Coverage   25.25%   25.25%
=======================================
  Files          20       20
  Lines         681      681
=======================================
  Hits          172      172
  Misses        509      509

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 2bbde84...912f49d. Read the comment docs.

Excellent, thanks @shaniphankar! :tada:


Allow API diagnostic overrides

Closes #672.

HI, @guythetechie
Can we use now below systax for yaml configuration file ?
I still cannot override diagnostics assignments for API's

name: prodcut-cognitive-search-apis
properties:
path: "indexes"
diagnostics:

name: applicationinsights
properties:
loggerId: "/subscriptions//resourceGroups/test-product-api-00001-rg/providers/Microsoft.ApiManagement/service/test-product-api-01-apim/loggers/product-api-01-appi"
verbosity: "information"