Failover host allocation for service

Rancher Versions:
Server: 1.3.0
healthcheck: 0.1.0
ipsec: 0.7.5
network-services: 0.2.15 (metadata 0.6.8)
scheduler: 0.4.0
kubernetes (if applicable):
Docker Version:
1.12.6
OS and where are the hosts located? (cloud, bare metal, etc):
RHEL 7.3, baremetal
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node, external DB
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Not sure exactly how to make it happen by design, but I'll describe what was going on.
When upgrading a service (the scheduler in this case), the response from docker daemon when attempting to the remove the old container failed ("Error response from daemon" is the error message in rancher UI), rancher is unable to bring the service back online, on this host.
In this case, docker daemon failed to remove the container due to "Device busy" for the container file system. So it was an underlying docker issue.
Results:
The service remains unhealthy.
We would liked to have seen rancher mark this host as unusable for this service, and move on to deploy the service on the next available host. Rancher should go out of its way to bring the service back online, despite malfunctioning docker daemons.
As we see it, in this scenario, a single faulty docker daemon can fail an entire service regardless of the number of potential nodes to deploy to.
We may have missed critical points here, but I'd rather have this discussion lead to a resolution on our end, than not having it at all ;)
We've been so quiet for a long time now, only because we're so busy implementing stuff around rancher - without too much issues :)

have you seen this recently @kaos? I think this affected older versions of rancher and has since been fixed.

No, I haven't. So I hope it's been fixed ;)
fre 19 maj 2017 kl. 05:05 skrev Ahmad E<EMAIL_ADDRESS>
have you seen this recently @kaos https://github.com/kaos? I think this
affected older versions of rancher and has since been fixed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rancher/rancher/issues/7521#issuecomment-302596754,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEdBV5nMTUEmYaEr5t0EAiIy08uhdaQks5r7QbcgaJpZM4LsApN
.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Refine settings driver

write the actual setting's length into flash

add getAlignLength() for address alignment

mark original setting block as deleted before setting the same one

Current coverage is 73.71% (diff: 80.00%)

Merging #847 into master will increase coverage by 0.32%

@@             master       #847   diff @@
==========================================
  Files           111        111
  Lines         16520      16525     +5
  Methods        2260       2261     +1
  Messages          0          0
  Branches       1984       1985     +1
==========================================
+ Hits          12125      12182    +57
+ Misses         3720       3669    -51
+ Partials        675        674     -1

Powered by Codecov. Last update 87f9cb0...ad97879


[DO NOT MERGE] Build gem with GitHub Actions

Takes inspiration from https://github.com/alphagov/rubocop-govuk/pull/45
I've saved the API credential under 'Secrets' in this repo, as
'GEM_HOST_API_KEY', which is referenced by the GitHub Action.
The logic is conditional so that a gem is only built and published
if the version doesn't match the latest version on rubygems.
As we haven't been able to release 0.11.1 yet, this should mean
that gem is released when this PR is merged.

Hi @MatMoore - we're planning to use GitHub Actions to publish this gem, following the approach taken in other GOV.UK repositories. We've just noticed middleman-search-gds is under your ownership, so our publish action is unlikely to work.
Would you mind transferring ownership of this gem to GOV.UK? I think this is the command to use: gem owner middleman-search-gds -a<EMAIL_ADDRESS>

Sorry @ChrisBAshton, I don't have access to that account anymore. 😞
It belongs to my cabinet office email address, which I no longer control.
I suggest contacting RubyGems directly and see if they can help you take over the account. Sorry for the inconvenience. At the time we forked this, the tech docs gem wasn't owned by GOV.UK, and I don't think there was a GDS shared account, so that might be why I published it seperately (and then forgot about it).

Thanks @MatMoore. I have raised a support ticket with them.
Adding DO NOT MERGE label to this PR, in favour of #7 for the time being.

I think we can probably close this for now since we're going to use Jenkins. We can always re-open it if we need to.


Scrolls functionally, but no scrollbar + empty space at bottom of Div

I'm implementing tiny-scrollbar in a wordpress theme I'm building. I was pumped to get it working but I'm having two issues:

The scrollbar it self is not showing up. The div will scroll via mousewheel, but I can't actually manipulate the scrollbar on the screen.

There is a bunch of empty space at the bottom of the div when I scroll with the mousewheel.

Any advice?

I think the css isn't properly applied. Please check if the css file is loaded and the container div meets the requirements.
Thanks.

Thanks for getting back to me so quickly!
I have applied the requirements to the div that I need to scroll. However, I'm not sure if I understand the last two requirements regarding the scrollbar's position... how do I make those changes? What selector do I use in my css to meet these requirements?
As for loading the css... Can I just copy the css to the bottom of my main css file? That doesn't seem to change anything.
Thanks again.

For the last two, perfect-scrollbar.css contains them, so you don't need to worry. Also for loading CSS, surely you can just copy the code.
If it still doesn't work, please let me know, preferably with an example page I can look into to narrow down the reason of the problem?
Thanks.

I have a similar problem in a project. The element scrolls fine with the mouse-wheel but for some reason the scrollbars are not visible and missing from the DOM. In this case I have three scrollable elements on the page and only one of them has this bug. If I can't solve it soon I'll make a fiddle or something.

I'm getting the same issue. After getting content via ajax and init PS, scrolling works fine but there is no scrollbar rails in DOM

If your scrollbar elements are missing, it must be because they are removed when the content is refilled, likely with .html() or .empty() or something like these.
In the latest version, when the scrollbars are missing, there's no way to show them again, but you can manually call .perfectScrollbar('destroy') and .perfectScrollbar() again and then they'll show up.
However, in the new version, even if the scrollbars are missing, they'll show up again when .perfectScrollbar('update') is called.

you are not loading the css file


Is there any documentation available by which we can be build a prediction model as  a "if else rule form" from the lightgbm model saved as txt file?

At the serving end, we have to convert the output of the lightgbm model to an if-else rule. Is it possible to map the model to if-else-rule with the feature names?

@jameslamb  can you tag someone who can answer my questions.


Expose a Gauge for Ingestion Paused

We need to expose a metric to identify when a table is paused.

Codecov Report
Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (18639b9) 61.42% compared to head (7c8b447) 34.92%.
Report is 36 commits behind head on master.

Files
Patch %
Lines

.../core/realtime/PinotLLCRealtimeSegmentManager.java
0.00%
12 Missing :warning:

...g/apache/pinot/common/metrics/ControllerGauge.java
0.00%
2 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12000       +/-   ##
=============================================
- Coverage     61.42%   34.92%   -26.51%
+ Complexity     1147      943      -204
=============================================
  Files          2378     2309       -69
  Lines        128894   125491     -3403
  Branches      19929    19448      -481
=============================================
- Hits          79177    43823    -35354
- Misses        44001    78552    +34551
+ Partials       5716     3116     -2600

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
34.92% <0.00%> (-26.39%)
:arrow_down:

skip-bytebuffers-false
34.90% <0.00%> (-26.51%)
:arrow_down:

skip-bytebuffers-true
34.90% <0.00%> (-26.37%)
:arrow_down:

temurin
34.92% <0.00%> (-26.51%)
:arrow_down:

unittests
46.78% <0.00%> (-14.64%)
:arrow_down:

unittests1
46.78% <0.00%> (+0.15%)
:arrow_up:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


fix all yamllint issues

There are several yamllint issues reported by stylecheck workflow. Let's try to get this CI check passing.
Let's update yamllint configuration https://github.com/buildtesters/buildtest/blob/devel/.yamllint.yml if needed. We can disable option that doesn't make sense.

i did go through and disable some checks which were quite annoying


silver-searcher: add -fcommon workaround

Workaround build failure on -fno-common toolchains like upstream
gcc-10. Otherwise build fails as:
ld: src/zfile.o:/build/source/src/log.h:12: multiple definition of
  `print_mtx'; src/ignore.o:/build/source/src/log.h:12: first defined here

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 175252 run on x86_64-linux 1

2 packages built:

memo
silver-searcher


Create "Intro to API-X document"

This is a public-facing doc, authored by the stakeholders, intended to give an overview of API-X for those interested in assessing its suitability for meeting business needs.

@rduerr wrote the first draft of this document, it is up here:
https://docs.google.com/document/d/1XzkD6OZOrEblBjsWXuWuzwPSgvgfv7igQlBSbGAo-Ls/edit

Final text can be found in: https://wiki.duraspace.org/x/7YEpB


notes: add proper storage and rendering for codeblocks

Still working on the wrapParagraphs issue, so I branched off of there (codeblock formatting would get messed up otherwise).
This should fix #1616
I imagine we need to do some kind of state transition to account for the new key? cc: @arthyn

Been attempting to write the state transition this morning, will definitely need to pair with someone on it.

Apparently state transition is unnecessary, nevermind.

lgtm, does the language class automatically provide a background?

no, it's dark either way. I'll add white for dark mode now.

I just ended up using tailwind's invert utility and set a white background for dark mode:


Octal numerals

When in LDPL you execute the statement
DISPLAY 0123

the value 83 is printed instead of 123. One would expect 123 to be printed, but this other number is printed instead because C++ asumes numerals that start with 0 (even 0 itself) to be encoded in octal base.
The same happens with vector accesses, for example if you store 1 in vector:0012, if you read vector:12 you'll find a 0, for the index that was accessed is not 12 but octal 12.
This should be fixed by checking if numerals have leading 0s and removing them (except, of course, if they are followed by a . and decimals).

Fixed in ad0b77e538058d61b8af844a2026cc96d4b09218.


fix dry-run error

Fix
https://github.com/istio/istio/issues/19998
https://github.com/istio/istio/issues/19997

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

Test name
Commit
Details
Rerun command

lint_operator
897fb3f45dfb0bec9de4383b9ab5cec1e73496c6
link
/test lint_operator

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

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

Test name
Commit
Details
Rerun command

lint_operator
897fb3f45dfb0bec9de4383b9ab5cec1e73496c6
link
/test lint_operator

e2e_operator
897fb3f45dfb0bec9de4383b9ab5cec1e73496c6
link
/test e2e_operator

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

/test e2e_operator

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

Test name
Commit
Details
Rerun command

lint_operator
897fb3f45dfb0bec9de4383b9ab5cec1e73496c6
link
/test lint_operator

e2e_operator
897fb3f45dfb0bec9de4383b9ab5cec1e73496c6
link
/test e2e_operator

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

e2e error is caused by https://github.com/istio/istio/issues/20021

@irisdingbj: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
edb86ce1f65c18987be73ee25f96e62ef9a6680c
link
/test e2e_operator

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

@irisdingbj: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
b9bace81a28ae8826cf66228fd57824df560cada
link
/test e2e_operator

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


docs(datasync): add docs for conflict resolution

I plan to add some more fixes to this PR including #1871 and #1869
Pushed for review of conflict resolution docs

Thanks for the review @craicoverflow ! I have incorporated your inputs from the review in the last push

CC @wtrocki

@machi1990 Thanks for the review!

This is not strict related to this PR but a general comment that goes along side this #1823, maybe we should move all the API docs in the API reference section.

Is this to be done in this PR? The issue was in triage so I am wondering if it is to be done now or after some discussion?

@machi1990 Thanks for the review!

This is not strict related to this PR but a general comment that goes along side this #1823, maybe we should move all the API docs in the API reference section.

Is this to be done in this PR? The issue was in triage so I am wondering if it is to be done now or after some discussion?

Preferably in a separate PR. But, if you have a few cycle, you can go ahead and move some of the docs to API Reference at least in this PR let's not add API docs outside the API Reference.


Convert heroku/heroku-buildpack-scala to GitHub Actions

Pipeline migrated from Circle CI :tada:
Manual steps
Perform the follow steps to complete the migration:
heroku/heroku-buildpack-scala/default-ci-workflow

[ ] Ensure a runner with the following labels exists: sfdc-hk-ubuntu-latest

Will be implemented by #214


Stream URLs for direct viewing

Is there an API in rustube that allowed me to get the stream URL directly, so I can view it directly in a <video> tag?
Youtube-dl supports this with the -g flag.

@Darksecond this is defeinetly something that should exist, and I will implement it within the next few days.
As a temporary fix you can run rustube fetch <ID> and look for the  signature_cipher field of the stream you want. There you will find the subfield url, which contains a host, a path, and a query. Using those fields, you can build an URL.

I should add that the output of the CLI is currently just the debug representation ("{:?}") of the Video. This will change with the next version to yaml and JSON, so you can use it with scripts.

so I can view it directly in a  tag
If your plan is to serve the videos using your own server, I highly recommend using the rustube library instead of the CLI. It gives you much more control and also allows you to access the URL.

@Darksecond took a bit longer than expected, but with rustube-cli 0.3.0 it's now possible to get only the stream-urls by running
rustube check -l url <identifier>

Running only rustube check <identifier> will also print out the URLs together with some additional information. It's a shorthand for rustube check -l "url general video-track audio-track" <identifier>.
rustube check --help gives more information about all the options.


Modifying options for the SASS loader

Hey guys,
Some time ago, I created a plugin for enabling CSS Modules for React builds. It doesn't do a whole lot, but add some babel config, and inject options into any css-loader it can find. The code is available here.
The plugin was built for Mix v2, but also works in v3. v4 is generally fine, except that it doesn't work with SASS correctly. If a user sets up the SASS importer in webpack.mix.js, it'll correctly convert everything to CSS, but it won't correctly activate the CSS modules functionality.
I have noticed that SASS is now optionally included in the Webpack config. This apparently also means that I cannot modify any SASS-related rules in my plugins webpackConfig() function, since it's not part of the base Webpack config.
Is this assumption correct? And is there a way of modifying the SASS-related rules, or would I have to inject any SASS-rules directly from my plugin? I would preferably just modify the options for any css-loaders I can find, so that it can work with any CSS preprocessor that is required.
I trust you can offer some guidance. Thanks in advance!
Lei

Is there a way to work around this by setting a custom webpackConfig in my webpack.mix.js file?

@evan-duncan Probably, but this means you'll have to be able to actually override the CSS loader that Laravel Mix provides. I have tried testing this, but my results have come up dry, as the webpackConfig option only allows for deep merges, and the loaders come as an array. Actually overwriting the config object is only supported as part of plugins. I hope you can work something out, but until this issue is fixed, I cannot really do anything.

@leinelissen It looks like you're not rewriting all of the css-loaders in your extension. I removed the length check on this line and it seems to be working https://github.com/leinelissen/laravel-mix-react-css-modules/blob/master/src/ReactCSSModules.js#L71


NIFI-2095: Introduce User/Group/Policy UI

NIFI-2095

Adding a page for managing users and groups.
Adding a page for managing access policies.
Renaming accessPolicy in entity to permissions to avoid confusion with the accessPolicy model.
Adding an Authorizable for access policies.
Refactoring access policies endpoints.

NIFI-2022:

Implementing site to site authorizations.

@mcgilman looks great! A whole lot of work has gone into this, and it looks fantastic. Was able to verify functionality of the different access policies. I create a handful of additional JIRA's that were minor but I didn't want them to hold up merging this PR. This is now merged to master. Thanks!


Guys you literally can't place attachments

Describe the bug
.
How can we reproduce this bug or crash?
install mod
go into game
attachments cant be placed
cry
What Minecraft version is this happening on?
Minecraft 1.16.5
What Forge version is this happening on?
36226
What Refined Pipes version is this happening on?
latest
Relevant log output
No response

Logs please

Wasn't there a conflict with the diet mod that caused this?

I have diet mod installed so will test without that. Will get you logs soon as I'm home from work


react-scripts test is not reporting non-zero exit code on test failures (when running inside a bash script)

Is this a bug report?
Possibly

Did you try recovering your dependencies?

Yes
Environment

System:
    OS: macOS High Sierra 10.13.6
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
  Binaries:
    Node: 8.12.0 - ~/.nvm/versions/node/v8.12.0/bin/node
    Yarn: 1.10.1 - /usr/local/bin/yarn
    npm: 6.4.1 - ~/.nvm/versions/node/v8.12.0/bin/npm
  Browsers:
    Chrome: 75.0.3770.80
    Firefox: 66.0.2
    Safari: 12.1
  npmPackages:
    react: ^16.6.3 => 16.6.3
    react-dom: ^16.6.3 => 16.6.3
    react-scripts: 2.1.8 => 2.1.8
  npmGlobalPackages:
    create-react-app: Not Found

Steps to Reproduce

Create a CRA project with tests
Create a bash script that encapsulates the react-scripts test command execution
Output the exit code result from within that wrapping bash script

Expected Behavior

I expect the exit code to be non-zero when tests fail
Actual Behavior

react-scripts test is reporting a 0 exit code
Reproducible Demo

None yet

You can try setting CI=true to get a non-zero exit code.

Duplicate of #7026 I think. Last time I checked we're doing everything right, but jest doesn't return the appropriate error code.

Setting the CI env variable works but the usage of that for local test runs doesn't seem right to me. We also lose the pretty output.
Been digging around for the past couple of hours and found that adding --bail in the CLI works as well.

I think its the same problem as #6944
I have a Husky configured and without CI env, even though tests are failing, Husky is not stopping commits from going through. Adding of CI works, but as @thedanchez pointed out, we are losing pretty output.

Confirmed, steps to reproduce:
npx create-react-app exittest
cd exittest
echo fail > src/App.js
npx react-scripts test --watchAll=false
echo $?
0

I think add --bail works as others pointed out

Imho, a proper exit code should be set by default and not only with obscure arguments like --bail.

Agreed, PRs welcome!

Also --bail will stop running tests on the first error, rather than running the full test suite then erroring.


Feature/event-image-uploads

Description
I added the basic code to allow users to send event images to the backend and also for the front end to request the image in the event resolvers. We can decide as a group how best to limit file size as I currently have it set to 15mb.
Type of change

[x] New feature (non-breaking change which adds functionality)

Change Status

[x] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?
I've tested the added photo field for the resolvers both locally in react and also in graphql playground.
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

 Get photo upload working on create-event forms


[Task] Change keys in Wasm from Base58-BTC strings to UInt8Array

Description
Change the public, private key representations used in the Wasm bindings from Base58-BTC encoded Strings to UInt8Array.
Motivation
Keys are not generally meant to be viewable and developers can encode them however they wish. UInt8Array is also the format expected by existing libraries such as TweetNaCl, so from an interoperability standpoint using UInt8Array would be more convenient.
May cause issues with the Stronghold Napi bindings though.
Resources
This is a sub-task extracted from:

https://github.com/iotaledger/identity.rs/issues/669

To-do list

[ ] Change keys in Wasm from Base58-BTC strings to UInt8Array

Change checklist
Add an x to the boxes that are relevant to your changes, and delete any items that are not.

[ ] The feature or fix is implemented in Rust and across all bindings whereas possible.
[ ] The feature or fix has sufficient testing coverage
[ ] All tests and examples build and run locally as expected
[ ] Every piece of code has been document according to the documentation guidelines.
[ ] If conceptual documentation (mdbook) and examples highlighting the feature exist, they are properly updated.
[ ] If the feature is not currently documented, a documentation task Issue has been opened to address this.

#734


feat(NumberTheory/LSeries/PrimesInAP): new file

This PR creates a new file Mathlib.NumberTheory.LSeries.PrimesInAP that will eventually contain a proof of Dirichlet's Theorem on primes in arithmetic progression.
As a first step, we provide two lemmas on re-writing sums (or products) over a function supported in prime powers. (We also add an API lemma for Multipliable/Summable that seems to be missing.)
See here on Zulip.

Thanks :tada:
bors merge


Plugin usage throws "Maximum recursive updates exceeded in component"

Reproduction
https://formkit.link/b1dc4539bfb84c25ea3702d75af5dc3e
Describe the bug
I want to use the proposed Formkit plugin for "Make number inputs return number values" as proposed at https://formkit.com/essentials/examples#plugins and https://formkit.link/b1dc4539bfb84c25ea3702d75af5dc3e
The example throws the following error/warning from version 0.17.x on up to latest 0.19.x:
[Vue warn]: Maximum recursive updates exceeded. This means you have a reactive effect that is mutating its own dependencies and thus recursively triggering itself. Possible sources include component template, render function, updated hook or watcher source function.

Environment
• OS: Ubuntu
• Browser Firefox + Chrome
• Version 0.17.x, 0.18.x, 0.19.x

You just need to add the hook inside the created event otherwise at first load it will try to mutate the value at the same time as the v-model does I believe. https://formkit.link/9fb566f6fc9a692452d478a6f25feb7c

@GustavoFenilli is correct, this is an interplay between FormKit and the v-model. I'm going to update the example to use #default slot restructuring to check the value which is what we recommend anyways — but wrapping in a created event works if your use-case requires v-model for some reason.
https://formkit.link/9d5ee95835fc654a2887b7c391f3e8e7


SlashableConditionException

Description
Since updating validators to 0.12.4, have been periodically seeing slashing protection kick in:
tech.pegasys.teku.core.signatures.SlashableConditionException: Refusing to sign attestation at slot 162110 with source epoch 5064 and target epoch 5065 because it may violate a slashing condition
	at tech.pegasys.teku.core.signatures.SlashingProtectedSigner.lambda$verifySigningAllowed$4(SlashingProtectedSigner.java:93)
	at java.base/java.util.concurrent.CompletableFuture.uniAcceptNow(CompletableFuture.java:753)
	at java.base/java.util.concurrent.CompletableFuture.uniAcceptStage(CompletableFuture.java:731)
	at java.base/java.util.concurrent.CompletableFuture.thenAccept(CompletableFuture.java:2172)

In normal operation of the beacon node, the validator should never attempt to sign anything that would be slashable - the slashing protection should just be a fallback. These errors indicate something unusual is happening with either the beacon chain or the validator client.

Haven't seen an instance of this since updating to 0.12.5. Will monitor for another day or two and close if no further instances occur.

When a reorg event occurs the scheduled duties map was cleared, however the duties objects weren't cancelled. So any events that were already pending but waiting for the duties scheduling to load would still be processed once the request for duties came back. This potentially creates a slashable offence as the recalculated duties would also be scheduled.
#2682 will fix it.

#2682 has been merged which fixes this.


webpack v5   Webpack-hot-server-middleware  Compatibility problems

Bug report

What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?

Other relevant information:
webpack version:
"webpack": "^5.74.0",
"webpack-cli": "^4.10.0",
"webpack-dev-middleware": "3.7.3",
"webpack-hot-server-middleware": "^0.6.1",
Node.js version:  16
Operating System:  mac  windows
Additional tools: vscode
I was using webpack webpack-dev-middleware Webpack-hot-server-middleware for Node SSR Webpack-hot-server-middleware Hot updates will report errors in Webpack V5.
error message:
client.8d64ed71.js:55443 Uncaught TypeError: Cannot read properties of undefined (reading 'push')
    at self.webpackHotUpdatewebpack_demo1 (client.8d64ed71.js:55443:46)
    at client.cbbc1ef178f03782094c.hot-update.js:2:38
self.webpackHotUpdatewebpack_demo1 @ client.8d64ed71.js:55443
(anonymous) @ client.cbbc1ef178f03782094c.hot-update.js:2
vendors.8d64ed71.js:29085 Uncaught TypeError: Cannot read properties of undefined (reading 'push')
    at self.webpackHotUpdatewebpack_demo1 (vendors.8d64ed71.js:29085:46)
    at vendors.cbbc1ef178f03782094c.hot-update.js:2:38

I think Webpack-hot-Server-middleware is not compatible with Webpack V5. I hope you can upgrade Webpack-hot-Server-middleware。
Also, when I use WebPack V5, Webpack-dev-middleware is not upgraded to the highest V5 version, otherwise, res.getHeader is undefined。
I an error making project address: https://github.com/qq281113270/react-ssr-lazy-loading

Please wait answer https://github.com/webpack-contrib/webpack-hot-middleware/issues/426, no need to duplicate, thank you


#28 Update frontend to bundle for CORS.

It uses the following techniques:

Deno(>=1.18.0) of supporting npm
npm esbuild

Thanks for the great PullRequest!
It seems to be working fine on my pc!
I am not familiar with esbuild so I have a few questions.

"denops/bufpreview/lib/filetype/markdown/client/markdown.bundle.js" looks like a very large file. I think it can be removed as it can be generated whenever you want by running $ deno task prepare.
In the original version, the browser window was automatically closed when the connection to deno was broken. However, the new implementation does not seem to work.
　- Maybe fixing this problem is a labor-intensive job. I want to get this PR in as soon as possible, so this problem may not need to be fixed this time.

I am so glad you fixed bufpreview.vim that was broken for so long!
Thank you.

"denops/bufpreview/lib/filetype/markdown/client/markdown.bundle.js" looks like a very large file. I think it can be removed as it can be generated whenever you want by running $ deno task prepare.

It's my mistake. I delete "denops/bufpreview/lib/filetype/markdown/client/markdown.bundle.js".

In the original version, the browser window was automatically closed when the connection to deno was broken. However, the new implementation does not seem to work.

I fix this behavior.

LGTM!!!!!! Thank you🎉


completely hide the hidden widget

https://msdata.visualstudio.com/Data Studio/_workitems/edit/1613458
when hidden, the elements are still in the DOM, we need to set display to none so that keyboard focus won't go into it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.02%) to 41.902%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-bindings/src/common/azureFunctionsUtils.ts
1
80.65%

extensions/sql-database-projects/src/common/httpClient.ts
1
66.18%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
2
50.42%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
28652

Relevant Lines:
63495

💛  - Coveralls


[SE-0155][WIP] Normalize Enum Case Representation

⚠ DO NOT MERGE ⚠
SE-0155 Requirements:

[x] Ban case Foo(), rewrite to case Foo(Void)
[x] Labels in associated value clauses are part of the case name
[x] Case name overloading on distinct full names
[x] Default arguments in associated value clauses
[ ] Consistent pattern matching behavior (see below)

A number of cleanups fell out of this patch:

EnumElementDecls owning a parameter list simplifies both serialization and AST printing
SourceKit now reports EnumElementDecls as owning a parameter list instead of a tuple
Function types in enum cases are implicitly marked @escaping and actually participate in semantic checks

Some open questions:

[ ] I would like the role of labels in patterns clarified before I finish off the work in TypeCheckPattern
[ ] The SILVerifier really doesn't like when uncurried applies and EnumInsts interact
[ ] The check for tuple shuffles in a CallSite feels very brittle

This will fail two tests to give y'all a feel for what my proposal in the linked SE thread would look like
@swift-ci please smoke test

@CodaFi Any progress to this PR? I'm really looking forward to this feature. :)

Got a pile of test failures with this patch.  Turns out changing SILGen changes SILGen
    Swift(macosx-x86_64) :: IRGen/enum_resilience.swift
    Swift(macosx-x86_64) :: SILGen/argument_shuffle_swift3.swift
    Swift(macosx-x86_64) :: SILGen/default_arguments.swift
    Swift(macosx-x86_64) :: SILGen/enum.swift
    Swift(macosx-x86_64) :: SILGen/errors.swift
    Swift(macosx-x86_64) :: SILGen/indirect_enum.swift
    Swift(macosx-x86_64) :: SILGen/opaque_values_silgen.swift
    Swift(macosx-x86_64) :: SILGen/toplevel_errors.swift
    Swift(macosx-x86_64) :: SILGen/tuples.swift
    Swift(macosx-x86_64) :: SILOptimizer/access_marker_verify.swift
    Swift(macosx-x86_64) :: SILOptimizer/definite-init-wrongscope.swift

I've rebased this PR last weekend: https://github.com/dduan/swift/tree/the-pitter-pattern-of-tiny-feet

In theory this passes the tests now
@swift-ci please smoke test

@swift-ci please smoke test Linux platform

Some of these failures look suspicious
@swift-ci please smoke test Linux platform

@slavapestov I think a combination of my removing the applyEnumElementConstructor path and your work on SILGen for nil is causing malformed code to be emitted after you apply this patch:
SILGen/objc_blocks_bridging.swift

SIL verification failed: instruction isn't dominated by its operand: Dominance->properlyDominates(valueI, I)
Verifying instruction:
%35 = partial_apply [callee_guaranteed] %34(%33) : $@convention(thin) (@noescape @callee_guaranteed (@in ()) -> @out ()) -> () // users: %60, %36
->   destroy_value %35 : $@callee_guaranteed () -> () // id: %60

Closed for #15418


chore(deps): update common-dev-assets digest to 497c14b

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
d224210 -> 497c14b

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

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


test(transformer/class-properties): failing test for super.prop as assignment target

Add failing test for syntax we can't support yet.

#8005  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 18, 2:03 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


0 balance with version 5!!!

This wallet has been a challenge... a hit or miss. I now have a zero balance where I should have<PHONE_NUMBER>1 ARRR

I’m having the same problem


chi_clickhouse_metric_fetch_errors switch to gauge

solution for https://github.com/Altinity/clickhouse-operator/issues/390#issuecomment-632722864
metrics-exporter collect metrics from clickhouse-server only when prometheus /metrics URL fetches, so gauge which show "errors is exist" case is enough
fix https://github.com/Altinity/clickhouse-operator/issues/390
Signed-off-by: Eugene Klimov<EMAIL_ADDRESS>

@alex-zaitsev please look https://github.com/Altinity/clickhouse-operator/blob/fix_metrics_exporter_fetch_status/tests/test_metrics_fetch_errors.sh
the test will PASS, before a merge with master


Update nano-server.rst

Minor typos fixed.

Hi @alexandair, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

:shipit:

:shipit:

What does that comment mean?

:shipit: === "Ship it". That means we have accepted the changes and someone with push access will merge it.

Thanks.

Thanks for the PR! Merged in 0c3d55a0753f0c30e91ac698b4c52154bd39764b


ERROR: 1049  Unknown database 'test'

Docker 1.9.1 on Ubuntu 14:
docker run --name some-mariadb -e MYSQL_ROOT_PASSWORD=my-secret-pw -d mariadb:10.1
Error:
ERROR: 1049  Unknown database 'test'
Extended Info:
Initializing database
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] /usr/sbin/mysqld (mysqld 10.1.11-MariaDB-1~jessie) starting as process 50 ...
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Using mutexes to ref count buffer pool pages
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: The InnoDB memory heap is disabled
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Memory barrier is not used
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Compressed tables use zlib 1.2.8
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Using Linux native AIO
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Using SSE crc32 instructions
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Initializing buffer pool, size = 256.0M
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Completed initialization of buffer pool
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: The first specified data file ./ibdata1 did not exist: a new database to be created!
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Setting file ./ibdata1 size to 12 MB
2016-02-01 20:06:04<PHONE_NUMBER>92576 [Note] InnoDB: Database physically writes the file full: wait...
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: Setting log file ./ib_logfile101 size to 48 MB
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: Setting log file ./ib_logfile1 size to 48 MB
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: Renaming log file ./ib_logfile101 to ./ib_logfile0
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Warning] InnoDB: New log files created, LSN=45883
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: Doublewrite buffer not found: creating new
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: Doublewrite buffer created
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: 128 rollback segment(s) are active.
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Warning] InnoDB: Creating foreign key constraint system tables.
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: Foreign key constraint system tables created
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: Creating tablespace and datafile system tables.
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: Tablespace and datafile system tables created.
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB: Waiting for purge to start
2016-02-01 20:06:05<PHONE_NUMBER>92576 [Note] InnoDB:  Percona XtraDB (http://www.percona.com) 5.6.26-76.0 started; log sequence number 0
2016-02-01 20:06:05<PHONE_NUMBER>91200 [Note] InnoDB: Dumping buffer pool(s) not yet started
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] /usr/sbin/mysqld (mysqld 10.1.11-MariaDB-1~jessie) starting as process 78 ...
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Using mutexes to ref count buffer pool pages
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: The InnoDB memory heap is disabled
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Memory barrier is not used
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Compressed tables use zlib 1.2.8
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Using Linux native AIO
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Using SSE crc32 instructions
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Initializing buffer pool, size = 256.0M
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Completed initialization of buffer pool
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Highest supported file format is Barracuda.
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: 128 rollback segment(s) are active.
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB: Waiting for purge to start
2016-02-01 20:06:07<PHONE_NUMBER>55648 [Note] InnoDB:  Percona XtraDB (http://www.percona.com) 5.6.26-76.0 started; log sequence number 1616799
2016-02-01 20:06:07<PHONE_NUMBER>76096 [Note] InnoDB: Dumping buffer pool(s) not yet started
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] /usr/sbin/mysqld (mysqld 10.1.11-MariaDB-1~jessie) starting as process 107 ...
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Using mutexes to ref count buffer pool pages
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: The InnoDB memory heap is disabled
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Memory barrier is not used
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Compressed tables use zlib 1.2.8
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Using Linux native AIO
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Using SSE crc32 instructions
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Initializing buffer pool, size = 256.0M
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Completed initialization of buffer pool
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Highest supported file format is Barracuda.
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: 128 rollback segment(s) are active.
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB: Waiting for purge to start
2016-02-01 20:06:10<PHONE_NUMBER>76736 [Note] InnoDB:  Percona XtraDB (http://www.percona.com) 5.6.26-76.0 started; log sequence number 1616809
2016-02-01 20:06:10<PHONE_NUMBER>98880 [Note] InnoDB: Dumping buffer pool(s) not yet started
ERROR: 1049  Unknown database 'test'
2016-02-01 20:06:10<PHONE_NUMBER>76736 [ERROR] Aborting

Was trying to follow instructions under 'Start a mariadb server instance' on dockerhub. but got the above error.  Had to add MYSQL_DATABASE=Somedatabase -e MYSQL_ROOT_PASSWORD=somepassword in order for it to work.

Same issue and output as above with 10.1, but adding -e MYSQL_DATABASE=Somedatabase did not help.
In the docker-entrypoint.sh, I had to add /etc/init.d/mysql restart after init in order for if echo 'SELECT 1' | "${mysql[@]}" &> /dev/null; to work.
I also had to fix the debian-sys-maint with:
pwd=($(cat /etc/mysql/debian.cnf | grep password | awk {"print \$3"}))
and
GRANT ALL PRIVILEGES ON *.* TO 'debian-sys-maint'@'localhost' IDENTIFIED BY '$pwd';
While I have modified the Dockerfile and docker-entrypoint.sh for using with Galera and xtrabackup 2.4 (see https://github.com/stuartz/mariadb-cluster), I had this issue before modifying.

I confirm that I'm also encountering this bug when trying to use both MYSQL_ROOT_PASSWORD and MYSQL_DATABASE.
There's also another report here: https://mariadb.com/kb/en/mariadb/mariadb-10112-mysql_install_db-aborts-on-unkown-file-test/
This makes the docker image unusable for me.

@jmatsushita The post that you mentioned is one I put up.  Since then I have been using a variation of the official docker/mariadb for a cluster.  I have posted thoughts and utilities at https://github.com/stuartz/mariadb-cluster.  Most of it is up to date, but I have made some changes that I have not updated yet.  It does abort, but starts up again.  One tricky area is putting the data in a volume for persistence.  I have had issues with SST transfer of clearing files to transfer and failing because they still exist on the cached layer.  In the case where I needed to do a full SST, I deleted the volume and restarted fresh.  Performance over the mysql 5.1 server we had is outstanding.
For connecting nodes on AWS, I had trouble with binding the listener and went with secured weave for ssl connections between AWS and our local node.

Seems that if I run docker-compose run command with sudo that it installs and connects as planned.

Also had the same issue, I fixed it by removing my local mariadb image and pulling it again.


Suggestion: toggle DAP repl instead of open

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
Thank you very much for adding DAP support. I have a very minor suggestion to improve the mappings.
There is currently a keymap to open the DAP REPL:
https://github.com/LazyVim/LazyVim/blob/6c86952a0e8be1fd06786f12a627a073982fa114/lua/lazyvim/plugins/extras/dap/core.lua#L85
I think it makes more sense to have it bound to a toggle instead, which has strictly more functionality IMO:
    { "<leader>dr", function() require("dap").repl.toggle() end, desc = "Toggle REPL" },

Describe the solution you'd like
N/A
Describe alternatives you've considered
N/A
Additional context
No response

Feel free to submit a PR

https://github.com/LazyVim/LazyVim/pull/663


Collect all coverage data at once

Description
Collect all coverage data at once - should reduce throttling by GitHub APIs when coverage data is proceeded
Testing
Manually verification against codecov website
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@peternied Should this be marked as ready for review?


Functions that make the cards animate left or right.

Another addition would be adding two functions that drag/animate the cards left or right. An example would be how tinder functions when you like someone the card goes right and when you dislike the card goes left. managed to get this by modifying your code a bit and reversing the forward and back functions animations.

Another addition would be adding two functions that drag/animate the cards left or right. An example would be how tinder functions when you like someone the card goes right and when you dislike the card goes left. managed to get this by modifying your code a bit and reversing the forward and back functions animations.

You can use controller's forward method and pass the SwipDirection argument  to achieve this. Check here use-a-controller-to-control


_episodes/06-sharing: Fix link to (now renamed) 05-examples

The previous PR changed this name to 05-examples, and I remembered
to check right after merging.
Review: minimal

Ah thanks a lot for this!


Some doc additions on handles and matrices

I updated the comment a bit based on the discussion above, better?

Yeah that's perfect! Sorry for being picky :D


SE-597 Upgrades Library Dependencies

Fixes #597 .
DO NOT MERGE It's a WIP yet.

Current coverage is 29.09% (diff: 19.19%)

Merging #598 into master will decrease coverage by 0.16%

@@             master       #598   diff @@
==========================================
  Files            50         48     -2
  Lines          1135       1155    +20
  Methods        1027       1032     +5
  Messages          0          0
  Branches         23         29     +6
==========================================
+ Hits            332        336     +4
- Misses          803        819    +16
  Partials          0          0

Powered by Codecov. Last update 5970164...695db5c


Fix for issue 777 "Training IMSDB hangs when raw data files are not present

When training an ion mobility database quite a lot of work is done before we even try to access the raw data files. When they turn out to be missing, it's a frustrating user experience. With this change, we check for the raw data before we do anything else.

The bug description for Issue 777 says that the LongWaitDlg did not go away on its own when the file could not be found. That makes me think that there is something else going on. Which file had you downloaded from Panorama where you saw the LongWaitDlg hang?

I wasn't the one reporting the error, so I don't know what file was in use. But my strategy here is to never even put up that LongWaitDlg if we can see that the operation is going to fail ahead of time. It's likely that yes, we'd still have that lingering LongWaitDLg issue if the file was somehow made to go away after the initial confirmation of existence, but I didn't think that worth pursuing.

I think users might already see the "data file could not be found" message coming from "ScanProvider.GetDataFile".

They do with the original code, but only after a fair bit of work has been done. My change performs the file existence check up front so there's less frustration.


Add maintainer targets to create source archive

Extend make release target to create json.tar.xz to be used by FetchContent. See #3255.

Coverage remained the same at 100.0% when pulling 18ab0be2da08c5c607a854a2632a98e874b1be92 on issue3255_xz into 293f67f9ff1a3c7d16cf0959c95761e89b9b64e9 on develop.


Foreign Key

Hi,
Maybe i am missing something, but in your EFCore configuration file, i didn't see any foreign keys, is it on purpose ?
My project has several tables with
entity.HasOne(e => e.Parent)
                .WithMany(p => p.Children)
                .HasForeignKey(d => d.ParentId);

types of configuration. How do i transform that ?

Hi @pendenaor,
Our implementation is missing the configuration that tells Entity Framework how to build the SQL foreign keys. This mapping needs to be added to AccountConfiguration.cs and CustomerConfiguration.cs and you are welcome to open a PR including this :)
Best

Thanks. I would try.


router guide: fix formatting

Sometimes ` characters are visible in the output (they should be <code>...</code>). An example: "You can use `redirectTo` ..."
Under "Sibling Outlets", the HTML code is visible (on a white background) at the bottom of the code box, whether the ES or HTML tab is selected.
Also, there are places where adjacent code snippets should be combined into a single tabbed view.
Under Sibling Outlets, there's an ES5/HTML example followed immediately (no text) by a ES5/TS example. Besides that problem, the template part of the TS example is blank (unescaped HTML?).

I've updated the initial comment to reflect the current state of the page.

Obsolete issue.


Real db support for block query

Don't try to imagine all the code I wrote but then deleted because I realized it could be done better.
All the queries are hooked together and it runs with the ./block.sh in test/ with the blocks that are generated in dev mode that only contain coinbases.
Further testing will follow when more work is put into the testing harness for this thing

Having internet issues so I can’t push — note to reviewers. I made all changes suggested by @mrmr1993 and it is much nicer with the pk joins.


Able to deploy multiple charts with duplicate name.

If you are trying to resolve an environment-specific issue or have a one-off question about the edge case that does not require a feature then please consider asking a question in Devtron Discord channel.
Checklist:

[x] I've searched in the docs and FAQ for my answer
[X] I've included steps to reproduce the bug.

Describe the bug
Select any number of a chart by clicking on + sign. Deploy all the charts in same project and environment with same name for all copies of the charts. It will get deployed but it shouldn't. Also when I delete on of the instance, it shows not NOT FOUND for all the remaining instances. But it keep showing in Helm Apps section with no app details.
To Reproduce
[ordered list the process to finding and recreating the issue, example below]

Go to charts section
select any chart and any number of instances by clicking on + sign.
Click on  Deploy To... button present on lower-right corner.
Select Project and Environment
Give same name for all the chart instances.
Deploy charts.
Delete any one of the instances.
You'll not be able to delete any of the remaining instances.

Expected behavior
Should not be able to deploy multiple instances of charts with duplicate name.
Current behavior
Able to deploy multiple instances of charts with duplicate name.
Screenshots

Logs
Paste any relevant application logs here.

AB#387

Proposal
If multiple instances of the same chart are added in multi-chart deploy flow:

Default app name: Append number in the default app name. Eg. airflow1, airflow2, airflow3 [Refer screenshot]
Validation: Do not allow deployment if same name is provided for multiple applications. Show error toast "Multiple applications cannot have same name"


Robotium testing on options menu item click

From<EMAIL_ADDRESS>on December 16, 2013 04:57:50
I want to write a test class where i have to test on click of optionsmenu item(I have 3 items in options menu). so onclicking the options menu item I m showing the list view with the data which I am retrieving from the sd card.
the application should crash if run time exception occurs.
Kindly help me with some code snippet/example.
Here is my code but and suggest me if any changes required.
private Solo solo;

@SuppressWarnings("deprecation")
public Mytest() {
	super("com.ge.contentmanagement.ui", ContentManagementActivity.class);
}

@Override
protected void setUp() throws Exception {
	super.setUp();
	solo = new Solo(getInstrumentation(), getActivity());
}

public void TestOptionsmenuItemclick() {
	solo.sendKey(Solo.MENU);
	solo.sendKey(KeyEvent.KEYCODE_MENU);
	solo.clickOnMenuItem("Review");
	solo.assertCurrentActivity("hai", getName());

}

@Override
public void tearDown() throws Exception {
	solo.finishOpenedActivities();
}

     }

Help is always appreciated!
Thanks
Original issue: http://code.google.com/p/robotium/issues/detail?id=566

From<EMAIL_ADDRESS>on December 17, 2013 00:01:20
Please keep this issue tracker for defects.
Status: Invalid
Owner<EMAIL_ADDRESS>


Use native --watch for node

Node has added a native watch mode in v18.11.0 (https://nodejs.org/en/blog/release/v18.11.0/)
Would it be possible to implement this and would it maybe fix the problem of spinning up multiple node sessions in the debugger?

It actually doesn't help or change the underlying issue. The native --watch mode also just simply creates a new child process. I just tried it...
But I think that we actually don't need to completely nuke the process and start everything from scratch. Wouldn't it be enough to remove the old definitions (pages/layouts/compnents), register the new ones and load or override the (express) route handler functions?


Network policies for address spaces

why
Kubernetes supports creating NetworkPolicy resources to control who can send/recv traffic to/from namespaces. For an EnMasse service admin, it is desirable to configure network policies for the created infrastructure. I.e only allow send/recv traffic to the infrastructure related to the address space endpoints.
See https://kubernetes.io/docs/concepts/services-networking/network-policies/ for more information about network policies in Kubernetes.
Proposal
The StandardInfraConfig and BrokeredInfraConfig resources should be extended to allow network policy to be defined. One important question for discussion, is the granularity of what we would allow. The finer granularity, the more complex it will be, but it will also allow for more flexible policies. Since an advanced policy would need to have knowledge of the UUID assigned to the labels of the infra when creating the policy, some care needs to be taken.
The answer could be to design something that allow both the simple and the advanced form to be specified. For the below simple alternative,  one alternative is to use the none policy and require the policies to be created explicitly to get the advanced features.
Simplest
The simplest alternative is to add another field networkPolicy indicating what network policy to apply to the created infrastructure. Example:
kind: StandardInfraConfig
...
spec:
  networkPolicy: none | allowAll | ownerOnly

none - don't create any network policies and rely on the cluster defaults
allowAll - create policies that allow all address spaces to be accessed from all other namespaces
ownerOnly - create network policies so that only the namespace where address space resource is created can access the namespace

The policy can be overridden when creating the address space in the same manner:
kind: AddressSpace
...
spec:
  networkPolicy: none | allowAll | ownerOnly

Advanced
The other extreme is flexible but also less user-friendly alternative is to expose more details of the underlying NetworkPolicy resource. The syntax could then look something like (with ${INFRA_UUID} being some kind of templating parameter we require to be set to match the infrastructure):
kind: StandardInfraConfig
...
spec:
  networkPolicies:
    - apiVersion: networking.k8s.io/v1
      kind: NetworkPolicy
      metadata:
        name: test-network-policy
      spec:
        podSelector:
          matchLabels:
            role: router
            infraUuid: ${INFRA_UUID}
        policyTypes:
        - Ingress
        - Egress
        ingress:
        - from:
          - ipBlock:
              cidr: <IP_ADDRESS>/16
              except:
              - <IP_ADDRESS>/24
          - namespaceSelector:
              matchLabels:
                project: myproject
          - podSelector:
              matchLabels:
                role: myapp
          ports:
          - protocol: TCP
            port: 6379
        egress:
        - to:
          - ipBlock:
              cidr: <IP_ADDRESS>/24
          ports:
          - protocol: TCP
            port: 5978

Testing

Deploy an OpenShift cluster with ovs-networkpolicy plugin enabled
Create infra resources with different network policy types
Ensure only allowed address spaces are allowed to access endpoints

Documentation

Service admin guide should have instructions on the network policy settings

Tasklist
Enumerate the tasks required to complete the feature

Personally I'd be happy with the "simplest" proposal - though is this really a per config or a per addressspace thing (with the infra config really only providing a default) ?

Question is: Should one be allowed to set a less-secure policy than the one configured by the service admin? My initial feeling is "no", and that one can only set a stricter policy than the default.

I'm not sure that this is just a strict security thing... Especially if the addressspace owner can still create a route anyway (which effectively allows anyone to make a connection to the endpoint).  As such I'd be tempted to say that the default can always be overridden by the addressspace owner, so the system admin can make "ownerOnly" the default, but an individual app can decide to allow all.

@rgodfrey Ok, so the initial syntax might be too simplistic for the ownerOnly policy. Basically the matching rules require either to match on pod labels or namespace labels. On OpenShift instance, users are not allowed to label their namespaces by default, so I'm thinking maybe we should support a subset of the NetworkPolicy syntax of the :
networkPolicy:
  from:
    - namespaceSelector:
        matchLabels:
           user: alice
      podSelector:
       matchLabels:
           role: db
  to:
    - namespaceSelector:
        matchLabels:
           user: alice
     - podSelector:
       matchLabels:
           role: db

See https://kubernetes.io/docs/concepts/services-networking/network-policies/#behavior-of-to-and-from-selectors for more info about this.


FIX: add oneCCL environment variables

What does this PR do?
Fixes issue https://github.com/huggingface/accelerate/issues/2340 and adds the missing LOCAL_WORLD_SIZE environment variable.

@sgugger

@faaany can you do pip install -e .[quality] and do make style; make quality please?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@faaany can you do pip install -e .[quality] and do make style; make quality please?

done. Could you pls review it? thanks!


VK page

VK (vk.com, m.vk.com) page looks wrong on a mobile device!
Is it possible to add to VK m.vk.com?
Add social network ok.ru

I fixed problems with vk. Can I help you or send a corrected version of the tool?

How can we proceed ?? Tell me the benefit ,? Tell me the cost ,? Apart from
that do you have a means of reaching out to client means ??? Thanks allot I
do appreciate your email,
.
On Wednesday, January 2, 2019, kalibri232<EMAIL_ADDRESS>wrote:

I fixed problems with vk. Can I help you or send a corrected version of
the tool?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/UndeadSec/SocialFish/issues/88#issuecomment-450849426,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Arz8arGn8cSApB8h5UpZeVqeGTef-YKLks5u_KLKgaJpZM4ZGZZS
.

I don't think it's worth the money, I just wanted to help you because I really like your product

you can contact me in the telegram @kalibri2323

You can also contact me<EMAIL_ADDRESS>


Rethinking 2 - Chapter 5 Edits

Thank you for opening a pull request!
Please check our style guide, and also make sure that the notebooks you've modified pass the pre-commit checks. If, for example, you modified notebook1.ipynb and notebook2.ipynb, you could do this by running:
pre-commit run --files notebook1.ipynb notebook2.ipynb

You may need to run it a couple of times before all the checks pass.

@canyon289 from what I can tell, it looks like all of the notebooks under Rethinking2 have been converted to PyMC v4. As for your two questions, I certainly can change the env file. Is that as simple as just changing "pymc3" to "pymc"? And yes, I intend to convert all of them as I progress through the text. Should I open a single PR for this initiative and commit as I go along? I wasn't sure what the right approach was.
Thank you for your patience with me. Two of my goals this year are to learn more about Bayesian statistics and to contribute to open source. I thought this looked like a good exercise to get experience in both of those areas. I am still early in my learning journey.

certainly can change the env file. Is that as simple as just changing "pymc3" to "pymc"?

Thanks so much! What I would suggest right now is creating a second environment.yml file labeled environment_v4.yml while were switching from v3 to v4. I would make it the versions of all the packages youre using right now, This will be more than just v4 as the aesara and aeppl versions will ned to be changed as well.
If you're going through the book checking the other notebooks will be a great way to learn as well.
As for this PR the precommit failed. Please run the isort and nblack commands to ensure all the formatting is correct. Thanks much for the PR!

@yahrMason  Thanks for the edits, it really helped.
I'm not sure, however, that all of the notebooks under Rethinking2 have been converted to PyMC v4.

Thank you!


[WIP] Use single precision floats

Fixes https://github.com/picnicml/doddle-model/issues/104.

Just double checking to be sure, this fix removes doubles because (most of the time) there's no need for that high precision, right? Or is there some other bug in the library that is caused by having doubles?
I'll take a closer look tomorrow (I guess it's mostly about making sure every case of using doubles is found), but at the moment I've noticed (besides the failed test that's seen in log of CI) that the gradient tests for linear/logistic/Poisson regression and softmax are failing - I assume it's because the tolerance is too strict.

@matejklemen yeah, we should sacrifice precision (I think ~7 digits is plenty for most cases) for a lower memory consumption (should be halved). The problem with failing tests was that we were using breeze's function for computing approximate gradients which we compared to the analytical solution and that one used the one-sided version of the finite difference formula. I now changed that to the two-sided estimation and also had to lower the tolerance for comparisons. The integration tests for linear models show a negligible change in performance. The tests are passing now but I want to test this on larger datasets to confirm I didn't break anything.


Cant go get v4

Trying to get the latest jwt-go
go get github.com/dgrijalva/jwt-go@a58974ee87a68abcf6cfb2315224e89135cf3311
results in the following error.
go get github.com/dgrijalva/jwt-go@a58974ee87a68abcf6cfb2315224e89135cf3311
go: finding github.com/dgrijalva/jwt-go a58974ee87a68abcf6cfb2315224e89135cf3311
go: finding github.com/dgrijalva/jwt-go a58974ee87a68abcf6cfb2315224e89135cf3311
go get github.com/dgrijalva/jwt-go@a58974ee87a68abcf6cfb2315224e89135cf3311: <EMAIL_ADDRESS>ee87a6: invalid version: go.mod has post-v1 module path "github.com/dgrijalva/jwt-go/v4" at revision a58974ee87a6

Please try go get github.com/dgrijalva/jwt-go/v4@a58974ee87a68abcf6cfb2315224e89135cf3311 and let me know if that works

That works but does not solve #355


Validation comparison

Made some changes so the branch works :)

you don't have to merge my changes, but you can take a look at what I changed if you're wondering why something doesn't work :)


Proposal: arbitrary hooks

Motivation
Helm hooks are the most valuable tools to ensure the quality and desired state of a chart release. In my opinion, it is what makes Helm really different from any other templating/packing tool.
Some applications eventually needs to executes life-cycle tasks as Jobs. It would take advantage to trigger those jobs as "arbitrary hooks", something as powerfull as an operator, but much simpler to implement.
Proposal
Example of an arbitrary hook named "clear-log", intended to do some application's life-clycle task.
apiVersion: batch/v1
kind: Job
metadata:
  annotations:
    "helm.sh/hook": clear-log
    "helm.sh/hook-delete-policy": hook-succeeded

Using it from command line, following the same helm test behavior and command structure
helm exec my-release-name clear-log

Likewise any other hook, it will be rendered using values.yaml presented during the install/upgrade.
The new helm exec command would allow the user to run (or re-run) any existent hook, like pre-install, post-install, etc. It would be very convenient when the chart is installed or upgraded using --no-hooks flag.

How does this differ from kubectl run or another command to run pods in the cluster?

The new helm exec command would allow the user to run (or re-run) any existent hook, like pre-install, post-install, etc.

Why would you wan to run pre-install hooks post-install? THat seems like incredibly dangerous behaviour. This would break current assumptions that pre-install or post-install hooks will only run once during a helm install or helm upgrade.

Hi Matthew,

How does this differ from kubectl run or another command to run pods in the cluster?

The command kubectl run can be used to run any pod, but not with all chart's logic managed by Helm.
The proposed helm exec command should be able to run the entire hook (jobs and all resources marked with the same helm.sh/hook annotation) but not restricted only to the pre-defined events.
For example, an arbitrary foo hook shoud be executed by helm exec my-release foo.
  annotations:
    "helm.sh/hook": foo

I think it shoud behave in the same way the helm test command works:

Run manually by the user
Run anytime the user needed it

Why would you wan to run pre-install hooks post-install? That seems like incredibly dangerous behaviour. This would break current assumptions that pre-install or post-install hooks will only run once during a helm install or helm upgrade.

Agreed ... it was a bad example. But in the same way --no-hooks flag will break this assumption too; I think that both resources should be used only for strict maintenance purposes.
On the other hand, helm test can be invoked many times as needed, without side efects; The same concept must be valid for "arbitrary hooks".

Accepting a list of arbitrary hooks means that we cannot reserve any hook names for future use without breaking backwards compatibility. For example, there’s been proposals discussing the introduction of a post-ready hook. If we implemented arbitrary hook names and a chart defined their arbitrary hook name as “post-ready”, the behaviour of that hook may change when they upgrade helm.
How do you suggest we fix that?

A couple thoughts...
First, I noticed the command would need to be altered to be:
helm exec my-release-name clear-log repo/chart

The chart package would need to be available to have the asset to render.
This leads to issues of version compatibility. What if the chart package is for a different version from the one currently running? Is there an error? Is there a --force feature to run anyway? How does compat work?
Encapsulating these sorts of things could be a way to enable more from operation systems like KUDO. I wonder what the KUDO devs or others think of this?

@bacongobbler A hook might not be the right terminology for this. A hook is a way to hook into the install process. These are more like npm scripts.
Might it make more sense to have a new thing similar to hooks but separate? That way hooks don't run into backwards compat issues and the name is more befitting what is being executed.

Hi folks,

@bacongobbler
Accepting a list of arbitrary hooks means that we cannot reserve any hook names for future use without breaking backwards compatibility.

That is true, we could create another annotation for arbitrary hook's name and use the reserved word arbitrary only for the hook type.
For example:
  annotations:
    "helm.sh/hook": arbitrary
    "helm.sh/hook-name": clear-log

@mattfarina
First, I noticed the command would need to be altered to be:
helm exec my-release-name clear-log repo/chart

All rendered hooks are stored on the release's secret, you don't need the chart again to get them.
For example using the command helm get hooks my-release

@mattfarina
Encapsulating these sorts of things could be a way to enable more from operation systems like KUDO. I wonder what the KUDO devs or others think of this?

Exactly, it is very like an operator, but the action performed is only related to an specif chart release.

@bacongobbler A hook might not be the right terminology for this. A hook is a way to hook into the install process. These are more like npm scripts.
Might it make more sense to have a new thing similar to hooks but separate? That way hooks don't run into backwards compat issues and the name is more befitting what is being executed.

I agree, it would be even better ... maybe we could add more "operator like" behavior, without care about compatibility issues.
I was trying reuse all hook's behavior in something more like a test, that is a hook but does not run into the install process.

To exemplify my use case, where did I get this proposal from:
I have an application packed into a Helm Chart, all install/upgrade/rollback tasks done by hooks.
It works really great and my Ops team loves it.
Many releases of the same Chart are deployed on my cluster (one release per customer), and eventually the Ops team needs to perform some maintenance tasks on a specific customer, like clear-log or compact-data.
So I had those options to solve the problem:

Starting a Job, but I have to manually create the correct manifest for each target release (including correct secrets, volumes, configMaps, etc)
Too complex to operate.

Running kubectl exec into the desired container and execute the command
Very simple, but not suitable for long running tasks.

Packing all the maintenance tasks into another Chart (this one was my choice)
It's not so bad, but I had to get the same values.yaml from the target application release to install the maintenance chart.
I managed to create a simple helm plugin to take care of it.
The Ops team just run helm clear-log my-relese-name where clear-log is the plugin name, on background it gets the values.yaml and install the maintenance chart.

Creating an Operator (using KUDO or OLM)
I started in that way, but it's sounded like overkill.
I could even replace Helm and deploy my application using the operator, but something wasn't right for me, is was just too complex to solve a much simpler problem.

Hi, @batiati - it sounds like you are trying to use Helm to carry out tasks that might be better carried out with Brigade - perhaps you will want to consider looking at that option?
If you do want to move forward with this proposal, please file a Helm Improvement Proposal so we can work through all the use cases and constraints - thanks!


[EWC/Volta] Places with Etherscan data instead of EWC

There is the list of places where the Etherscan data should be replaces with EWC data:

Another spot is the hover over blockexplorer links:

This fix is merged into the release branch in order to be deployed and tested in all environments


FR command in editor menu bar

With commands in Tree View / Status Bar / Quick Pick this extension has all places covered but the editor menu bar.
Please add this, allowing all command configuration be done with this extension.
There's a "kind of hard-wired" extension that does this single thing quite well so the general idea can be seen there https://open-vsx.org/extension/jerrygoyal/shortcut-menu-bar (the code can't be copied because it is GPL, but the public vscode API would be accessed the same way).
The configuration here would be similar to the Status Bar implementation (own sub-configuration and a "Toggle Editor Menu Bar" context menu entry to add/remove it).

The api is not flexible enough. It's going to require editing of package.json file, like the commands.populateCommandPalette setting.
Looking at open issues of that extension:

Would require editor reload for using fully custom items (using predefined actions is not great)
VSCode hides icons under a ...
Not possible to reorder items

And I personally don't like icons there (to the point of using Custom CSS extension to always hide them).

The api is not flexible enough. It's going to require editing of package.json file, like the commands.populateCommandPalette setting.

Agreed. I've actually found that hack quite cool btw. Was it your idea or copied from somewhere else?

and I personally don't like icons there

So I guess you may not implement it but would agree to work together on a PR to make this work and then integrated? Note: the reorder should be possible with the package.json hack, too.

Was it your idea or copied from somewhere else?

Most likely not mine. Not sure where it's from.

Yes, you can work on it. It would be great to use only built-in icons (codicons) and without any predefined actions.

No plans for this.


Publish release for 6.0.1.

@ColinHebert - Do you mind releasing master as 6.0.1 so that #161 is fixed in the release on Maven Central?
c8a5266bd57823da58885f56f383b65a900cf64d could also be cherry-picked into the 5.x maintenance branch. I skipped this due to the two-version from current maintenance policy since the server is now technically on protocol version 7. Version 5 is still supported, though, so it might make sense to release that as well.
Also, I'm not sure what the common workflow is for getting added as a publisher for Maven Central, but it might also be a good idea for me to go ahead and get that set up if you could point me in the right direction.
Thanks!

fyi @tkaemming is the guy at Sentry that we made learn Java ;)

If I were you, I'd release 6.0.1 with commit c8a5266 only. And after that release 7.0.0 with all new features support for Sentry 8. So project will respect semver.

Or release a 6.0.2 with latest version of everything
:+1:  for Sentry 8 support

Bump....   It seems that  the 6.0.1-SNAPSHOT has disappeared from maven repositories... https://oss.sonatype.org/#nexus-search;quick~raven-log4j

The 6.0.1-SNAPSHOT release should still be available in the Sonatype Snapshot repository: https://oss.sonatype.org/content/repositories/snapshots/net/kencochrane/raven/raven-log4j2/6.0.1-SNAPSHOT/
Details on how to use the snapshot repositories are here: https://github.com/getsentry/raven-java#snapshot-versions
We're going to be moving this over to com.getsentry very soon so that we have control over publishing and to clarify this is the officially supported Raven client for Java. We'll get back onto a normal release cycle at that point.

Closing this, as GH-182 is going to result in version 7.


Monitoring/Alerting Overview MVP

Adds the MVP with external links as described and mocked here
https://github.com/rancher/dashboard/issues/1064
https://github.com/rancher/dashboard/issues/1215

I moved the gauge styles to a global style so we don't have to duplicate these styles from the cluster explorer page.

@vincent99 We added the Prometheus and AlertManager CRD into the overview since it isn't in the side nav since most users would only ever have 1, but you would want to know if that CRD is up and running or not when troubleshooting.
Thoughts?

@lvuch This has all the right content, but it doesn't look pretty. Any thoughts on how to make it better?


Updated Props For Poco X2

Thank you for submitting this. As soon as life outside Android modding gets less hectic I will evaluate this print and look at getting it integrated into the module. Sorry for the delay...


Extracted ToolControl TabStripItem styles

Moved TabStripItem styles to the outer scope to make style modifications possible (currently the style always had higher priority that custom styles for the element)

@BAndysc Can you do the same for DocumentControl ?

Yeah, sure, in DocumentControl only base style is inside TabStrip.Style there. Should I also extract Template setter? I have left it in its place here, but wasn't sure if I should. Should it be possible to style manually Template?

Yeah, sure, in DocumentControl only base style is inside TabStrip.Style there. Should I also extract Template setter? I have left it in its place here, but wasn't sure if I should. Should it be possible to style manually Template?

Ah ok leave it as is for now


[Feature request] Sort friends by number

Friends are numbered by when they were added in order, would be nice to have a way to sort this.  It is really fascinating writing these all down.
Possibly a date tooltip if date added exists?

date tooltip
That's unfortunately not possible. We can only figure out the friend nr because VRChat "sorts" an array in a specific API endpoint this way.

sorry, but it is not enough to add this as a feature. 😅 Fortunately, there is a way to find out who the friend with the number is. Just look at the JSON on the more tab. Then expand the friends section, and the user id will be listed. The order of this is the order added. With this, you can search one by one in Direct Access.

Closing because inactive.


testing(state/core_accessor): add async submitPFB test case

Additional test case checks whether tx was included in the block after async submission

N/A


Investigate pro-hosted squarespace vs wp.com solutions.

Business Case
Limiting what changes we allow to cp.com allows up to ship fast and more confidently etc.
Investigate the pro-hosted solutions
So we need to investigate the wp.com vs Squarespace solutions. In particular but not exhaustively

[ ] Can we keep a flat site as home and redirect the rest of the site to them. (Not 100% necessary but would speed it up a lot)
[ ] Which ones easier to set up and a little bit flexible
[ ] Check with Rich once the first two are filled out that he can do the basic needs in it.

Update: There's no neat way to keep the flat site as homepage

Either we use a sub url "pages.etc" for the site
Or we move the homepage into the service (which would probably mean WordPress)

Might need to set up a theme in either and move the homepage in. Gonna grab rich and go over options with him and try to make a decision in the next 2 hours.

Ok so having hit issues with having a wordpress hosted & coolestprojects.org static page I spoke with @grega about what are options are regarding other wordpress hosting options.

WPEngine
This service offers us the most flexibility when it comes to managing a wordpress instance. dev, staging and production environments and the ability to deploy via a git origin. This would make working with the contractor easier and means we can utilise version control and deployment.

In terms of domains we can use a CNAME like blog.coolestprojects.org and then use the CloudFlare worker to mask it.

Additional domain
Greg noted that we could simply purchase an additional domain for the blog to live at e.g. coolestprojectsblog.org. This would be used to house the domain on wordpress.com. Requests to it would be masked by the CloudFlare worker as if its serving coolestprojects.org

Digital Ocean
We could use DO one click wordpress installation which would allow us to get started quickly, however we would need do additional work to allow for the environment to be managed via puppet like we do with our CodeClub DO instances. It would also need a deployment flow working out.

Recommendation
I think using WPEngine is the best option. This is because it will allow us to scale down the line. It also matches a deployment flow we already use.
If all that fails we can simply go with option 2 and buy a domain.

Thanks. This clarifies things.
I'm good with WPEngine. Also nice timing with new wp editing interface Gutenberg, WPEngine is launching it in a day which makes WP much nicer for non-devs to create pages in.
(side note We should have formalised the discussion here earlier rather than constantly jumping on calls, my fault!)

Not sure how 2) works to solve the issue with the web worker being unable to query a NS.

still needs to be tested out to see if it works, but the docs seems to inform it should (https://wpengine.com/support/cloudflare-best-practices/)

Can we close now ? That seems fairly engaged into WPEngine

yep good to close from me.


Improve Crashlytics Action Logging

Also hides the build secret when errors occur.
Before:

After:

LGTM - one suggestion about porting a helper method from the spec_helper of fastlane_core


Catch or release ObjectPool leaks

Problem
ObjectPools are sometimes destroyed while objects
remain live. For BitMapObjectPool case, such objects can't
be safely touched. For HeapObjectPool such objects can be used
but can't be released.
Some leaks have been fixed (#11983, #12031), but there are still
a few remaining.
Fixes #11880 Possible use of destroyed pool objects
Change overview

Prevent future leaks by adding an ObjectPool template argument
specifying what to do if it is destroyed with live objects: abort with
an error message (the default), release the live objects (which calls
their destructors), or ignore the condition (transitionally, until all
leaks are fixed).

Fix shutdown ordering in DeviceControllerSystemState.

For existing pools in SessionManager, InteractionModelEngine,
and GroupDataProviderImpl, ignore the condition.

Testing
CI; no changes to external functionality.

Filed issues for the remaining leaks:

#12503
#12504
#12505

⛔ Do not merge yet — there's a (non-merge) conflict with another PR that landed recently.


Issue related to notification doesn't open

Scenario 1

the app has been used recently to view an issue
a notification for a different issue is tapped

Expected

issue related to notification should open

Actual

nothing happens, we remain viewing the existing issue

Scenario 2

Tap a notification

Expected

issue related to notification should open

Actual

app opens to notification inbox only

Dup of #2171 ?
Sent with GitHawk

I believe we are waiting on #2105 and #1981 for this
Sent with GitHawk

I guess we can already pop the currently shown issue before tackling routing; would that be a good first step?
Sent with GitHawk

@BasThomas yes


A database operation failed with the following error: 'Could not find object 'Sales' or you do not have permission.'

Pipeline "IterateAndCopySQLTables" gives below error in Activity i.e "Copy data1"
"errors": [
    {
        "Code": 22301,
        "Message": "ErrorCode=SqlOperationFailed,'Type=Microsoft.DataTransfer.Common.Shared.HybridDeliveryException,Message=A database operation failed with the following error: 'Could not find object 'Sales' or you do not have permission.',Source=,''Type=System.Data.SqlClient.SqlException,Message=Could not find object 'Sales' or you do not have permission.,Source=.Net SqlClient Data Provider,SqlErrorNumber=15165,Class=16,ErrorCode=-2146232060,State=1,Errors=[{Class=16,Number=15165,State=1,Message=Could not find object 'Sales' or you do not have permission.,},],'"

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

ID: 125c8ae8-332f-8792-d81b-30bb0eb3976f
Version Independent ID: 2e2ac8cb-4208-6692-0f25-701e4906475a
Content: Copy data in bulk using Azure portal - Azure Data Factory
Content Source: articles/data-factory/tutorial-bulk-copy-portal.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

Do we need to have SCHEMA and TABLE definitions already created in Target Data Warehouse ?
I am not sure if Copy activity is also creating SCHEMA and TABLE definitions before it starts loading the data.
I am not able to sort out this even though user has all the permissions granted.

@ravidesai29 , Thanks for raising the issue. We will investigate and update you.

Yes , Its mentioned in the prerequisites steps .
https://docs.microsoft.com/en-us/azure/data-factory/tutorial-bulk-copy-portal#prerequisites
Please make sure that you have the schema and table created on the sink side and it should work just fine .
We will go ahead and close this issue now , you can always reopen this if you think its required .

ErrorCode=SqlOperationFailed,'Type=Microsoft.DataTransfer.Common.Shared.HybridDeliveryException,Message=A database operation failed with the following error: 'Could not find object '{"table":""}' or you do not have permission.',Source=,''Type=System.Data.SqlClient.SqlException,Message=Could not find object '{"table":""}' or you do not have permission.,Source=.Net SqlClient Data Provider,SqlErrorNumber=15165,Class=16,ErrorCode=-2146232060,State=1,Errors=[{Class=16,Number=15165,State=1,Message=Could not find object '{"table":""}' or you do not have permission.,},],'
I have tables and schemas created. They were auto created by datafactory. if I don't use pre-copy script It works fine . If I use pre-copy script (TRUNCATE) it fails. DataFactory has full permissions on my datawarehouse. I was using TRUNCATE before on other pipelines but those tables were created manually. These are new tables which were auto created. Could you please looking to this?
Thanks!


Master is broken

Please test the master if you push.

Sorry that's my bad, I work on API last week and I forget to delete that file, just delete the file api_urls.py and remove it from urls.py


--node-resource-group invalid parameter

Docs say to use --node-resource-group parameter to name 2nd resource group, which isn't a valid parameter.
unrecognized arguments: --node-resource-group
TRY THIS:
az aks create --resource-group MyResourceGroup --name MyManagedCluster
Create a kubernetes cluster with default kubernetes version, default SKU load balancer (Standard) and default vm set type (VirtualMachineScaleSets).
az aks create --resource-group MyResourceGroup --name MyManagedCluster --enable-managed-identity
Create a kubernetes cluster which enables managed identity.
az extension add --name anextension
Add extension by name
https://docs.microsoft.com/en-US/cli/azure/aks#az_aks_create
Read more about the command in reference docs

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

ID: d501ff67-9208-8bcb-e8df-f09e9b16a802
Version Independent ID: e3fc3851-ddc4-d26c-251d-6ae53d714081
Content: Frequently asked questions for Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/faq.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@ozzy92, thanks for the contribution. While you're correct that the linked reference page doesn't include the --node-resource-group flag, this is because the custom resource group behavior to which that flag is associated is part of the aks-preview Azure CLI extension, which that reference page does not document.
If you are getting an 'unrecognized arguments' error, please ensure you have the aks-preview CLI extension version 0.3.2 or later installed. As the instructions in the FAQ document are correct, we will be closing this issue.
#please-close


Why are assets compiled in to my source directory?

From the docs:
By default, once your assets are compiled, they will be placed in their corresponding directories, directly under the source folder:

What is the reasoning behind the built assets being duplicated in source and also in the build_local folder?
Ideally I would only have the built assets residing in the build_local or build_production folders which I can ignore from git.

This is because Webpack runs before the Jigsaw build.
Webpack needs an output path (mix.setPublicPath('source/assets/') in webpack.mix.js), but there's no build_local or build_production at that point in time.
You could simply gitignore the source/assets directory if you don't want it in version control.

Okay I get that, thanks.
Is it possible to build the client assets outside the source directory? I tried the following:
// webpack.mix.js

mix.setPublicPath('dist/');

// config.php

return [
    'build' => [
        'source' => 'dist'
    ],
    ...
];

The build was completing but the HTML files weren't being generated. It seemed like maybe the 'source' in config.php relates to where the view assets and webpack assets are?
It feels like I need a setting like:
// config.php

return [
    'build' => [
        'publicPath' => 'dist'
    ]
    ...
];

Thanks for your help, I really like the project so far!

The source key in config.php tells Jigsaw where your source files are located (layouts, partials, your .md files), so leave it as 'source' => 'source',.
publicPath won't do anything, the key you're looking for in there is destination.
So with that in mind:
// config.php
'build' => [
    'source' => 'source',
    'destination' => 'dist',
],

That would output the Jigsaw build where you want it.
But Jigsaw cleans up the destination directory when it builds, and because it runs after Webpack I think that would result in your dist/assets directory being removed. Haven't tested this though, try it out :)

Ah sorry, you're right, I did have 'destination' => 'dist'. I think the problem is I don't know how to configure where jigsaw looks for assets when it runs.
What I'm trying to achieve is:

Webpack runs first and outputs my built JS/SASS to /dist
Jigsaw should run over the views in source but look for the mix-manifest.json in /dist
Clears /build_local
Outputs the compiled views + copies built webpack assets from /dist in to /build_local

Do you know if this configuration is possible? The reason being that I only ever want /source to actually contain 'source' files to prevent confusion.

@hellocosmin's description of why it builds assets to source/assets is correct; Webpack runs first to compile assets, then Jigsaw runs its build process and simply copies the compiled assets to the build folder.
I can't think of a way to accomplish what you're hoping, at least without some additional Webpack build step after jigsaw build is run. In your step 2), Jigsaw only parses and copies whatever files it finds in source.
To help prevent confusion in /source/assets, I typically have a /build directory in there that non-static assets (JS, SASS, Less files, etc.) in /source/_assets get built to, and I gitignore that build directory. That makes it a little clearer that the generated files in /source/assets/build shouldn't be edited manually. If you do that, you'll need to also update your publicPath in webpack.mix.js to:
mix.setPublicPath('source/assets/build');

...and update Mix references in your templates:
<link rel="stylesheet" type="text/css" href="{{ mix('css/style.css', 'assets/build') }}">

Thanks @damiani, that looks pretty similar to the solution I have settled on for now.


Create Web App Bot using dependent resources - storage, application insights, app service, and app service plan

I'm trying to create a web app bot using CLI. The 'az bot create' command gives options to specify the app ID, password and storage account name. The 'az bot update' command gives option  --ai-api-key --app-insights-api-key to bind the application insights. I tried figuring out a way to configure an app service and its app service plan. Let me know please if there's a method in CLI and in AZ powershell as well.

@PrajithB and @KevinBrooke if you want to update the App Service and its App Service Plan, these are configurable using the following command modules:

az webapp: https://docs.microsoft.com/en-us/cli/azure/webapp?view=azure-cli-latest
az appservice: https://docs.microsoft.com/en-us/cli/azure/appservice?view=azure-cli-latest

Since these are distinct resources on Azure, az bot doesn't support updating them, it only provisions the resources on az bot create.

Closing this issue as the answer was provided in this comment. To upgrade/update the settings on the App Service or App Service Plan via the CLI, use az webapp or az appservice.


Add cache-endpoint automatically to allowed-endpoints

Solves #173

@varunsh-coder I resolved conflicts and created this new PR