Multiple MySQL connections. Error in initializing tenant, prompting that the tenant database does not exist

Bug description
After creating a tenant, copy the data of one tenant to another tenant, and prompt that the tenant database does not exist.
My function is to copy data from another tenant to the current tenant after the tenant is created successfully.
When there is only one MySQL connection, the function is OK. However, when multiple MySQL connections are made, the program prompts that the tenant database does not exist.
tenancy()->initialize($anotherTenant);
ERROR
Log display:
SELECT SCHEMA_NAME FROM INFORMATION_SCHEMA.SCHEMATA WHERE SCHEMA_NAME = 'saas_1'
Database saas_1 does not exist

Steps to reproduce
Tenant::create(['tenancy_db_connection'=>'tenant1',...]);

ERROR
Database saas_1 does not exist
In database.php
'central' => [
            'read' =>['host'=><IP_ADDRESS>],
            'write' =>['host'=><IP_ADDRESS>],
            'sticky' => true,
            'driver' => 'mysql',
            'port' => 3306,
            'database' => 'central',
            'username' => 'homestead',
            'password' => 'secret',
            'unix_socket' => '',
            'charset' => 'utf8mb4',
            'collation' => 'utf8mb4_unicode_ci',
            'prefix' => '',
            'strict' => true,
            'engine' => null,
        ],
'tenant' => [
            'read' =>['host'=><IP_ADDRESS>],
            'write' =>['host'=><IP_ADDRESS>],
            'sticky' => true,
            'driver' => 'mysql',
            'port' => 3306,
            'database' => '',
            'username' => 'homestead',
            'password' => 'secret',
            'unix_socket' => '',
            'charset' => 'utf8mb4',
            'collation' => 'utf8mb4_unicode_ci',
            'prefix' => '',
            'strict' => true,
            'engine' => null,
        ],
'tenant1' => [
            'read' =>['host'=><IP_ADDRESS>],
            'write' =>['host'=><IP_ADDRESS>],
            'sticky' => true,
            'driver' => 'mysql',
            'port' => 3306,
            'database' => '',
            'username' => 'homestead',
            'password' => 'secret',
            'unix_socket' => '',
            'charset' => 'utf8mb4',
            'collation' => 'utf8mb4_unicode_ci',
            'prefix' => '',
            'strict' => true,
            'engine' => null,
        ],

In the tenants table, the table data is as follows:：
Tenant 1's data:{"tenancy_db_name": "saas_1","tenancy_db_connection": "tenant"}
Tenant 2's data:{"tenancy_db_name": "saas_2","tenancy_db_connection": "tenant1"}
In App\Providers\TenancyServiceProvider
public function events(){
	return [
		...
		Events\TenantCreated::class => [
                JobPipeline::make([
                    Jobs\CreateDatabase::class,
                    Jobs\MigrateDatabase::class,
                    Jobs\SeedDatabase::class,
                    Tenant\Jobs\CreateSmsGateway::class,// init sms gateway
                ])->send(function (Events\TenantCreated $event) {
                    return $event->tenant;
                })->shouldBeQueued(false),
         ],
         ...
	];
}

In Tenant\Jobs\CreateSmsGateway
public function handle(){
    {
        //tenancy()->end(); // This is invalid

        /** @var Tenant $anotherTenant */
        $anotherTenant = tenancy()->query()->whereKeyNot($this->tenant->getKey())->oldest()->first();
        if (!$anotherTenant) return;

        // init tenant ERROR,database saas_1 does not exists
        tenancy()->initialize($anotherTenant);

		....

  }

Expected behavior
Multiple MySQL database connections, and multiple tenants can be set.
And the tenant switch is correct
Laravel version
6.20.44
stancl/tenancy version
3.4.6

Is this related to databases you're creating using the package? Could this be related to #515?

Is this related to databases you're creating using the package? Could this be related to #515?

Hi.
In Stancl\Tenancy\Database\DatabaseManager
/**
     * Create the tenant database connection.
     */
    public function createTenantConnection(TenantWithDatabase $tenant)
    {
        $this->app['config']['database.connections.tenant'] = $tenant->database()->connection();
    }

In Stancl\Tenancy\Bootstrappers\DatabaseTenancyBootstrapper
public function bootstrap(Tenant $tenant)
    {
        /** @var TenantWithDatabase $tenant */

        // Better debugging, but breaks cached lookup in prod
        if (app()->environment('local')) {
            $database = $tenant->database()->getName();
            if (! $tenant->database()->manager()->databaseExists($database)) {
                throw new TenantDatabaseDoesNotExistException($database);
            }
        }

        $this->database->connectToTenant($tenant);
    }

It is wrong to set only one tenancy_db_connection field.
Multi-connections,When the tenant connection is modified, reinitializing other tenants will fail.
Solution:

Write the host, username and password of the database into the data field

In a non local environment

Tenant 1's data:{"tenancy_db_name": "saas_1","tenancy_db_host": "<IP_ADDRESS>","tenancy_db_username":"homestead","tenancy_db_password":"secret"}
Tenant 2's data:{"tenancy_db_name": "saas_2","tenancy_db_host": "<IP_ADDRESS>","tenancy_db_username":"homestead","tenancy_db_password":"secret"}
Although it solved my problem. But I still have questions

Under the multi database connection, it should be possible to save only tenancy_db_connection fields in data

Why do you want to check whether the database exists,only in the local environment? This will cause errors in switching tenant connections

Thank you very much:)

Please answer the questions I asked in the comment above

I mean the linked issue can't solve your problem here 😄 but if your issue is solved I'm closing this.


Multiple Anaconda installations after updating Visual Studio

Environments
I have both Visual Studio Enterprise 2017 Preview and Visual Studio Enterprise 2017 (2) installed on my machine running Windows 10. I don't actually believe I've used Preview on Windows 10 (I've opened it up, but haven't worked in it), but I used to use it on Windows 7 before the Data Science workload was available in non-Preview, so I went ahead and installed it. On January 23, I updated my Visual Studios and their Data Science workloads. Today, I was creating a new Python project when I got confused and frustrated about not finding packages that I was installing on my OS.
Turns out I was looking in C:\Program Files\Anaconda2\, when I was supposed to be looking in C:\Program Files (x86)\Microsoft Visual Studio\Shared\Anaconda2_64. I only installed Python 2 (and Anaconda2) while updating Visual Studio, because I'm just now starting work on a project that uses Python 2-only packages. However, now I've got:
PS C:\Program Files> ls .\Anaconda[23]\

    Directory: C:\Program Files

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
d-----        1/23/2018   7:52 PM                Anaconda2
d-----        9/26/2017   4:52 PM                Anaconda3

PS C:\Program Files> ls C:\Python*\

    Directory: C:\

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
da----        1/23/2018   8:12 PM                Python27amd64

PS C:\Program Files> ls '..\Program Files (x86)\Microsoft Visual Studio\Shared\*\'

    Directory: C:\Program Files (x86)\Microsoft Visual Studio\Shared

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
d-----        1/23/2018   8:48 PM                Anaconda2_64
d-----        1/23/2018   8:40 PM                Anaconda3_64
d-----        1/23/2018   8:09 PM                Common
da----        1/24/2018  12:15 PM                Python36_64

Moreover, the Python versions are (almost) all different:
PS C:\Program Files> .\Anaconda2\python.exe --version
Python 2.7.13 :: Anaconda 4.4.0 (64-bit)
PS C:\Program Files> .\Anaconda3\python.exe --version
Python 3.6.1 :: Anaconda 4.4.0 (64-bit)
PS C:\Program Files> & '..\Program Files (x86)\Microsoft Visual Studio\Shared\Anaconda2_64\python.exe' --version
Python 2.7.13 :: Anaconda, Inc.
PS C:\Program Files> & '..\Program Files (x86)\Microsoft Visual Studio\Shared\Anaconda3_64\python.exe' --version
Python 3.6.2 :: Anaconda, Inc.
PS C:\Program Files> & 'C:\Python27amd64\python.exe' --version
Python 2.7.14
PS C:\Program Files> & '..\Program Files (x86)\Microsoft Visual Studio\Shared\Python36_64\python.exe' --version
Python 3.6.3

Finally, I could have sworn my previous Python 3 installation was at C:\Program Files\Python3, but that seems to have disappeared entirely.
Questions

What are the Visual Studio-expected locations for each of these environments? I understand that I can add environments in Visual Studio, but, say, what's the default on clean install? C:\Anaconda2\, C:\Anaconda3\, C:\Python3, etc. made sense to me, but I can understand them all being in the same Shared folder now. It's just the C:\Python27amd64 that's throwing me off.

Can I reconcile my new Anaconda3_64 environment with the old C:\Anaconda3\ one? I would have expected Anaconda to update, not to get a whole new installation, but maybe it couldn't find it a la #2530? That being said, I have code that I'm still working on that uses extra libraries installed in the C:\Anaconda3\ environment and a virtual environment created from there, so I'd rather the two Anaconda 3s get rolled into one.

Thank you in advance!

I'd suggest deselecting Anaconda from the Visual Studio installer and then reinstalling (or upgrading) it manually in the location where you'd like it. Anaconda's installer doesn't keep track of existing installs, so that's why it didn't remove the old one (CPython does keep track, and so we won't try and install it unless we have a newer version, and it will clean up the old one as much as it can).
The installers in VS are largely a convenience for users who don't really care where they end up, as long as they are available. So if you'd rather have them somewhere convenient for you, go ahead and deselect them from VS and install it manually.

On the recent VS2017 update, I experienced on multiple machines Anaconda 2 and 3 installing via the updater, when previously the installation had been selected not to install Anaconda 2 or 3. So the update settings, whatever they are, are unintentionally installing Anaconda 2 and 3 into everyone's machines, at least from my experience with three Windows 10 machines.

@zooba this sounds like the bug you've mentioned recently, is it fixed now?

At this point, we can't fix anything related to this any more. The next version of VS won't include Anaconda in the installer as full installs, and the problem described here is not popping up frequently enough to push out a fix in the upcoming updates. So I'm closing this.


[3.50] private anim helpers

When using sprite.anims.create one would expect to find the other helpers such as generateFrameNumbers there as well.  Currently, generateFrameNumbers is found only under scene.anims, which works for private anims, just a bit wordier.

Thank you for submitting this feature request. We have implemented this and the feature has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


CMEK: now with 200% more Vault!

https://cockroachlabs.atlassian.net/browse/DOC-3387

verified vault+GCP KMS works in staging!

sorry, in production, not staging*


Fix attestation  processing bug where we would use wrong state

Seems like I'm on a bug hunt today.
Fixed:
warn: Failed to process attestation. Reason: crosslink committee retrieval: out of range epoch: 2
This will speed up e2e tests as we would reach finalization sooner.

yeah that's why I just see it recently, quite annoying 💯

yeah that's why I just see it recently, quite annoying

I have no idea how we didn't noticed earlier that we were using epochctx of last justified state :sweat_smile:


#35 Add support for Read operation for PersistentVolume.

After this PR, CITF will have support for  READ operation for Persistent Volume.
Changes committed:
utils/k8s/k8s.go

Codecov Report

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

@@           Coverage Diff           @@
##           master      #40   +/-   ##
=======================================
  Coverage   42.46%   42.46%
=======================================
  Files           5        5
  Lines         146      146
=======================================
  Hits           62       62
  Misses         83       83
  Partials        1        1

Continue to review full report at Codecov.

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

@sbcd90
Can we see following PR for reference and cast you PR similar to that.
https://github.com/openebs/CITF/pull/57/files
Additionally, please resolve the conflict.

Hello @sonasingh46 , I have followed the reference PR now & resolved conflicts. Can you please review again?


Replace error-chain with thiserror

Replaces the abandoned error-chain crate with thiserror, but does not clean up any expect() or unwrap() calls!

Closes #58

Lets see what CI thinks.

bors merge


Fix cache bug

Their keys should be string.
cc @guangxuli

Pull Request Test Coverage Report for Build 53

3 of 12 (25.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.008%) to 35.53%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

knitter-manager/models/logical_ipgroup.go
1
3
33.33%

knitter-manager/models/logical_network.go
1
3
33.33%

knitter-manager/models/subnet.go
1
3
33.33%

knitter-manager/models/port.go
0
3
0.0%

Totals

Change from base Build 48:
-0.008%

Covered Lines:
5633

Relevant Lines:
15854

💛  - Coveralls

test again


Updated 1 useful resource

Hey Team,
Just wanted to give you a heads up that I've added a cool resource to the Awesome Web Development Resources. Now under "Learning Platforms" section there is one new helpful resource link that is handpicked to make life easier for folks learning about web development. I think that'll be super useful for everyone using the guide. Let me know what you think!
Cheers,
Soham Mukherjee

No articles allowed.


Add whatsapp.net frontend and backend

Adding whatsapp.net backend and frontend to the proxy configuration to support whatsapp.net traffic including media.

I will also update the docker-compose.yml file with these ports.

Add whatsapp.net frontend

Sya minta Maff Sya nk cuba main thread ini tapi Sya tk reti
Pada Jum, 1 Nov 2024, 1:03 PTG alphachart @.***>
menulis:

Your controol me??!!!!!!!!!
—
Reply to this email directly, view it on GitHub
https://github.com/WhatsApp/proxy/pull/197#issuecomment-2451298378, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BMKUT5YONNHYWPFKLHI7YZLZ6MDTRAVCNFSM6AAAAABQGR53OWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRGI4TQMZXHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Oo n...vcfyj


Non-optional Link properties throw error

In Swift 2 on Xcode 7 beta 6, any link property that is not marked with optional throws an error as of d2bf93c27785df9046d31907b9d704437d9cbf1d. Are there any plans to bring back non-optional link properties?

This is actually an intentional change, which is called out in the CHANGELOG. Realm actually never supported non-optional link properties, and could make no guarantees that a non-optional link property would never be nil, and Xcode 7 Beta 6 was a reminder of the danger of returning nil from a non-optional properties (NULL dereferences in the Swift runtime), so we thought it would be better to make that an error upfront. If you'd like, you should be able to declare the property as an implicitly unwrapped optional, though we haven't tested that yet.

Hey @segiddins. Thanks for the quick response. Implicitly unwrapping the optional seems to be working for us.


write_verilog: emit casez as if/elif/else whenever possible

This results in a very significant improvement in readability when emitting Verilog from Amaranth code, and a slight improvement when emitting Verilog from Yosys-parsed Verilog code. Not sure about Verific.

Before
    (* src = "test.v:5.3-8.14" *)
    casez (a)
      /* src = "test.v:5.7-5.8" */
      1'h1:
          _0_ = 8'h55;
      /* src = "test.v:7.3-7.7" */
      default:
          _0_ = 8'haa;
    endcase

After
    (* src = "test.v:5.7-5.8" *)
    if (a)
      _0_ = 8'h55;
    else
      _0_ = 8'haa;

@povik All fixed, thank you

Say you have an
    casez (sel)
      2'b1?: ...
      2'b?1: ...
      default: ...
    endcase

and sel[0] can have an x-bit, does what we emit with this change have different semantics?

To illustrate the change better, this takes
module top(a, b, c, out);
	input wire a, b, c;
	output reg [3:0] out;
	always @(*) begin
		casez ({c, b, a})
			3'b??1: out = 0;
			3'b?1?: out = 1;
			3'b1??: out = 2;
			default: out = 3;
		endcase
	end
endmodule

and emits
  always @* begin
    if (\$auto$verilog_backend.cc:2239:dump_module$2 ) begin end
    _0_ = _1_;
    (* src = "test.v:0.0-0.0" *)
    if (a)
      _1_ = 4'h0;
    else
    (* src = "test.v:0.0-0.0" *)
    if (b)
      _1_ = 4'h1;
    else
    (* src = "test.v:0.0-0.0" *)
    if (c)
      _1_ = 4'h2;
    else
      _1_ = 4'h3;
  end
  always @* begin
      out <= _0_;
  end

does what we emit with this change have different semantics?

I'm going to counter this with: Do we care? Is 'x in this context a part of a contract we state anywhere or are expected to uphold?

Do we care? Is 'x in this context a part of a contract we state anywhere or are expected to uphold?

That's exactly what we were discussing on the dev JF. (I used the comments here as a scratchpad for our discussion.) Anyway I think we agreed on merging this, only please rebase this since there's conflict with #4111.

That's exactly what we were discussing on the dev JF.

Thanks for dealing with this! I was too sick to attend today's dev JF but I will be present on the next one.
I'll rebase.

More specifically, AFAICT the SV spec can have different 'x behavior for both variants, but they agree on all synthesizable cases (i.e. excluding behavior that's specified for 'x but not a sound approximation of the '0 and '1 cases, thus requiring the synthesis incompatible ability to tell non-constant 'x apart from '0 or '1).

Updated. GitHub seems to be having trouble right now, with lots of 500 returned.

Updated. GitHub seems to be having trouble right now, with lots of 500 returned.


Cookie improvements

related to https://github.com/getsentry/raven-ruby/issues/900

Seems to work decently, though, I guess it leaves the values in request.cookies inconsistent with the value in the Cookie header.  Thoughts on this approach?

Any updates on this? I'm tired of seeing the following error in our event processing...

@josh-m-sharpe thanks for the PR! can you add a test case for the fix as well?

I suspect I won't be re-opening this to test it at this point.

closing because the issue has been fixed in #983


ocamlPackages.reason-native: 2022-08-31-a0ddab6 -> -unstable-2024-05-07

Description of changes
Upstream doesn't use releases, branches, or tags, so this is the most recent commit: https://github.com/reasonml/reason-native/commit/20b1997b6451d9715dfdbeec86a9d274c7430ed8

ocamlPackages.reason-native.cli: init at 0.0.1-alpha-unstable-2024-05-07
ocamlPackages.reason-native.console: 2022-08-31-a0ddab6 -> 0.1.0-unstable-2024-05-07
ocamlPackages.reason-native.dir: 2022-08-31-a0ddab6 -> 0.0.1-unstable-2024-05-07
ocamlPackages.reason-native.file-context-printer: 2022-08-31-a0ddab6 -> 0.0.3-unstable-2024-05-07
ocamlPackages.reason-native.fp: 2022-08-31-a0ddab6 -> 0.0.1-unstable-2024-05-07
ocamlPackages.reason-native.frame: init at 0.0.1-unstable-2024-05-07
ocamlPackages.reason-native.fs: init at 0.0.2-unstable-2024-05-07
ocamlPackages.reason-native.pastel: 2022-08-31-a0ddab6 -> 0.3.0-unstable-2024-05-07
ocamlPackages.reason-native.pastel-console: 2022-08-31-a0ddab6 -> 0.0.0-unstable-2024-05-07
ocamlPackages.reason-native.qcheck-rely: 2022-08-31-a0ddab6 -> 1.0.2-unstable-2024-05-07
ocamlPackages.reason-native.refmterr: 2022-08-31-a0ddab6 -> 3.3.0-unstable-2024-05-07
ocamlPackages.reason-native.rely: 2022-08-31-a0ddab6 -> 4.0.0-unstable-2024-05-07
ocamlPackages.reason-native.rely-junit-reporter: 2022-08-31-a0ddab6 -> 1.0.0-unstable-2024-05-07
ocamlPackages.reason-native.unicode: init at 0.0.0-unstable-2024-05-07
ocamlPackages.reason-native.unicode-config: init at 0.0.0-unstable-2024-05-07
ocamlPackages.reason-native.utf8: init at 0.1.0-unstable-2024-05-07

The version numbers are taken from upstream's <package>.json file.
The constituent packages were in an almost-but-not-quite callPackage style. I've updated them so that they're all actual packages. There's a mild amount of duplication induced by doing so, but no more so than every package in pkgs/by-name.
I also fixed and updated ocamlPackages.reason-native.console.tests so that it builds and no longer includes its default.nix file.

#301014

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[x] Fits CONTRIBUTING.md.

Upstream doesn't use releases, branches, or tags

When I initially packaged this I wasn't aware of:

passthru.updateScript = unstableGitUpdater { };

Can be executed as:

nix-shell maintainers/scripts/update.nix --argstr package ocamlPackages.reason-native.paste

Things need to be tweaked a bit. But it could prove helpful in case you want to keep it fresh, not that upstream is changing much.

Things need to be tweaked a bit. But it could prove helpful in case you want to keep it fresh, not that upstream is changing much.

Thanks for the link and the information. I thought about how updates would happen, given that this update was a little manual... but that slow rate of changes made me think it wasn't going to be an issue. If I'm in the area again in another couple years, let's see if I think differently.


Return proper error when connection fails

Errors from n.createNewConnection are not getting returned properly because err is not in scope when instantiated with :=. This change fixes that and bubbles the error up properly.

Hi Ian,
I have merged your commit into PR #9 and added you to the README as a contributor. Thank you!

Thanks @lukebakken


☕  Refine JSR exports

SSIA

Summary by CodeRabbit

New Features

Enhanced module export capabilities with multiple entry points for easier access.

Bug Fixes

Updated import paths for error handling to reflect new file names.

Refactor

Restructured module organization for better clarity and management of exports.

Should we change jsr:@denops/core to jsr:@denops/core/type in these examples?
https://github.com/vim-denops/deno-denops-core/blob/4aae5ba73b388735d1cd31c9094cf2e009fffea6/README.md?plain=1#L15-L21
https://github.com/vim-denops/deno-denops-core/blob/4aae5ba73b388735d1cd31c9094cf2e009fffea6/mod.ts#L11-L17
https://github.com/vim-denops/deno-denops-core/blob/4aae5ba73b388735d1cd31c9094cf2e009fffea6/type.ts#L139-L144
https://github.com/vim-denops/deno-denops-core/blob/4aae5ba73b388735d1cd31c9094cf2e009fffea6/type.ts#L150-L161

I think it's OK to keep the current example code while most of developers should NOT use @denops/core anyway.


Archive the repo so no more Issues and PR can be made, and close existing ones

@joe27g, updates has been stopped anyway, so might as well prevent people from wasting time pull requesting and making issues here.

Also mentioning the developer (<@246107833295175681>) on Discord, in-case they aren't very active on Github.


Not reconnecting in all cases

We have observed that the module isn't reconnecting in all cases, but still haven't found the specific problem.
Logs from one issue (sorry for the mixture of formats, only the connection manager had a custom logger applied):
{"level":"info","ts":"2022-12-16T04:05:23.661704901Z","caller":"worker/worker.go:109","msg":"Handled message"} <-- last message
2022/12/16 04:59:48 gorabbit ERROR: attempting to reconnect to amqp server after cancel with error: ctag-/bin/worker-1
2022/12/16 04:59:48 gorabbit INFO: waiting 5s seconds to attempt to reconnect to amqp server
2022/12/16 04:59:48 gorabbit INFO: rabbit consumer goroutine closed
2022/12/16 04:59:53 gorabbit WARN: successfully reconnected to amqp server after cancel
2022/12/16 04:59:53 gorabbit INFO: successful recovery from: ctag-/bin/worker-1
2022/12/16 04:59:53 gorabbit ERROR: error restarting consumer goroutines after cancel or close: declare queue failed: Exception (404) Reason: "NOT_FOUND - queue 'new-builds' in vhost 'platform-development' process is stopped by supervisor"
2022/12/16 04:59:53 gorabbit ERROR: attempting to reconnect to amqp server after close with error: Exception (404) Reason: "NOT_FOUND - queue 'new-builds' in vhost 'platform-development' process is stopped by supervisor"
2022/12/16 04:59:53 gorabbit INFO: waiting 5s seconds to attempt to reconnect to amqp server
2022/12/16 04:59:58 gorabbit WARN: error closing channel while reconnecting: Exception (504) Reason: "channel/connection is not open"
2022/12/16 04:59:58 gorabbit WARN: successfully reconnected to amqp server
2022/12/16 04:59:58 gorabbit INFO: successful recovery from: Exception (404) Reason: "NOT_FOUND - queue 'new-builds' in vhost 'platform-development' process is stopped by supervisor"
{"level":"error","ts":"2022-12-16T05:00:41.13624477Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"attempting to reconnect to amqp server after connection close with error: Exception (501) Reason: \"write tcp <IP_ADDRESS>:49792-><IP_ADDRESS>:5672: write: connection reset by peer\"","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:85"}
{"level":"info","ts":"2022-12-16T05:00:41.136325279Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"error","ts":"2022-12-16T05:01:16.137948493Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"error reconnecting to amqp server: dial tcp: lookup rabbitmq.rabbitmq.svc.cluster.local: i/o timeout","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).reconnectLoop\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:115\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:86"}
{"level":"info","ts":"2022-12-16T05:01:16.138040023Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"error","ts":"2022-12-16T05:01:51.141472319Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"error reconnecting to amqp server: dial tcp: lookup rabbitmq.rabbitmq.svc.cluster.local: i/o timeout","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).reconnectLoop\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:115\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:86"}
{"level":"info","ts":"2022-12-16T05:01:51.141562769Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"error","ts":"2022-12-16T05:02:26.143127796Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"error reconnecting to amqp server: dial tcp: lookup rabbitmq.rabbitmq.svc.cluster.local: i/o timeout","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).reconnectLoop\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:115\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:86"}
{"level":"info","ts":"2022-12-16T05:02:26.143234192Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"error","ts":"2022-12-16T05:03:01.145103317Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"error reconnecting to amqp server: dial tcp: lookup rabbitmq.rabbitmq.svc.cluster.local: i/o timeout","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).reconnectLoop\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:115\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:86"}
{"level":"info","ts":"2022-12-16T05:03:01.145347574Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"error","ts":"2022-12-16T05:03:36.147081783Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"error reconnecting to amqp server: dial tcp: lookup rabbitmq.rabbitmq.svc.cluster.local: i/o timeout","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).reconnectLoop\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:115\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:86"}
{"level":"info","ts":"2022-12-16T05:03:36.147184473Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"error","ts":"2022-12-16T05:04:11.148442465Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"error reconnecting to amqp server: dial tcp: lookup rabbitmq.rabbitmq.svc.cluster.local: i/o timeout","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).reconnectLoop\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:115\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:86"}
{"level":"info","ts":"2022-12-16T05:04:11.148524676Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"error","ts":"2022-12-16T05:04:46.150196103Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"error reconnecting to amqp server: dial tcp: lookup rabbitmq.rabbitmq.svc.cluster.local: i/o timeout","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).reconnectLoop\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:115\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:86"}
{"level":"info","ts":"2022-12-16T05:04:46.150313791Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"error","ts":"2022-12-16T05:05:21.15146101Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"error reconnecting to amqp server: dial tcp: lookup rabbitmq.rabbitmq.svc.cluster.local: i/o timeout","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).reconnectLoop\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:115\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:86"}
{"level":"info","ts":"2022-12-16T05:05:21.151567677Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"error","ts":"2022-12-16T05:05:56.152881546Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:25","msg":"error reconnecting to amqp server: dial tcp: lookup rabbitmq.rabbitmq.svc.cluster.local: i/o timeout","stacktrace":"XYZ/golib/hrabbitmq.Logger.Errorf\n\t/go/pkg/mod/XYZ/golib/hrabbitmq@v1.7.0/logger.go:25\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).reconnectLoop\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:115\ngithub.com/wagslane/go-rabbitmq/internal/connectionmanager.(*ConnectionManager).startNotifyClose\n\t/go/pkg/mod/github.com/wagslane/go-rabbitmq@v0.11.0/internal/connectionmanager/connection_manager.go:86"}
{"level":"info","ts":"2022-12-16T05:05:56.15299162Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"waiting 5s seconds to attempt to reconnect to amqp server"}
{"level":"warn","ts":"2022-12-16T05:06:14.267313916Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:29","msg":"error closing connection while reconnecting: Exception (504) Reason: \"channel/connection is not open\""}
{"level":"warn","ts":"2022-12-16T05:06:14.267384733Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:29","msg":"successfully reconnected to amqp server"}
{"level":"info","ts":"2022-12-16T05:06:14.267430768Z","logger":"rabbitmq","caller":"hrabbitmq@v1.7.0/logger.go:33","msg":"successful connection recovery from: Exception (501) Reason: \"write tcp <IP_ADDRESS>:49792-><IP_ADDRESS>:5672: write: connection reset by peer\""}

<-- no further messages handled

Code:
conn, err := rabbitmq.NewConn(
  connStr,
  rabbitmq.WithConnectionOptionsLogger(hrabbitmq.NewLogger(logger)),
)
if err != nil {
  sugar.Fatalw("failed to create a rabbit connection", "err", err)
}
defer conn.Close()

consumer, err := rabbitmq.NewConsumer(conn, func(d rabbitmq.Delivery) rabbitmq.Action {
  // omitted handler logic
  return rabbitmq.Ack
},
  "new-builds",
  rabbitmq.WithConsumerOptionsRoutingKey("organization.*.addBuild"),
  rabbitmq.WithConsumerOptionsRoutingKey("organization.*.addVersion"),
  rabbitmq.WithConsumerOptionsQueueDurable,
  rabbitmq.WithConsumerOptionsQOSPrefetch(1),
  rabbitmq.WithConsumerOptionsExchangeName(conf.RabbitmqExchange),
  rabbitmq.WithConsumerOptionsExchangeKind("topic"),
  rabbitmq.WithConsumerOptionsExchangeDurable,
)
if err != nil {
  sugar.Fatalw("failed to start consumer", "err", err)
}
defer consumer.Close()

Used version github.com/wagslane/go-rabbitmq v0.11.0
If you have any pointers what we could look into, that would be appreciated.

Debugging this further it seems that the connection is successfully recovered (it shows up in the rabbitmq management interface), but the channel / consumer isn't 🤔

Thank you for all the tests and this error log. After playing with the tests and looking through your logs, I don't think there is a good solution on the library side, except to maybe improve the logs so it's more obvious what's happening. This error:
declare queue failed: Exception (404) Reason: "NOT_FOUND - queue 'new-builds' in vhost 'platform-development' process is stopped by supervisor

is a queue declaration failure, which is something I don't think we want to be trying to recover from on the library side. You should be debugging your declarations for your application.

is a queue declaration failure

Yes and no. A 404 can also happen while a queue is being moved between queue leaders and can be a temporary failure.

want to be trying to recover from on the library side

This would be fine for us as long as the library returns an error in such cases, so we can exit the process (and have it restarted).

It feels somehow fairly implicit that a library user needs to implement logic that decides if the library can handle certain errors or not.
Couldn't there be some kind of unrecoverableError hook, that is called in such situations so I can hard-fail, panic the process?
This way the logic if something is recoverable could be located inside the library. Unrecoverable errors could be all kind of channel failures or failures that have been generally retried more than X times.


No "username" field in PRIVMSG event

Steps to reproduce
chat.on(chatConstants.TAG_COMMANDS.PRIVATE_MESSAGE, event => {
    console.log(event);
});

Your environment

Operating system(s):
TwitchJS version<EMAIL_ADDRESS>NodeJS version: v8.11.4

Duplicates #165


Update readme to mention how to require gem

The main entry point of this gem is actually within lib/kaminari/neo4j.rb which requires us to require 'kaminari/neo4j'. This PR just adds that to the readme so that it's clear.

That's actually really strange if gem 'kaminari-neo4j' didn't work for you.  Bundler is supposed to automatically require gems, and I'm pretty sure it respects the Ruby conventions for such things:
http://guides.rubygems.org/name-your-gem/
But if it didn't work for you then it should definitely be in the README.  Thanks!


Added k-NN algorithm [Python]

By submitting this pull request I confirm I've read and complied with the below requirements.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the coding guidelines for this project.
[x] My code follows the skeleton code structure.
[x] This pull request has a descriptive title. For example, Added {Algorithm name} [{Language}], not Update README.md or Added new code.

Closes #41.

As i can see this is related to machine learning and not to Basic algorithms.

As i can see this is related to machine learning and not to Basic algorithms.

It is a basic machine learning algorithm.


Removed ability to pass a promise to the load functions

Fixes #5475
@pjcozzi can you review?

@lilleyse, thanks for the pull request! Maintainers, we have a signed CLA from @lilleyse, so you can review this at any time.
I am a bot who helps you make Cesium awesome! Thanks again.


DNN-8532 - Messaging Fails for users with an Apostrophe in their name

Fixed username encoding so now can contains apostrophe without errors on messaging tab. Also applied same fix in portal name variable.

Code review done. It looks good.


[0.5pt] Create a Book class (Team member #1)

Create a Book class in a separate .rb file.

Hi @Zuhaib042
Please check my pull request


Sharing configurations across multiple workspaces

It would be useful to have a feature where we can share common configuration files across multiple workspaces easily.

You can easily do this yourself today. decK takes multiple state files as an input (-s/--state flag). Move all of your common configurations into a single file (or multiple) and then reference those files for each workspace that you wish to sync to.


batch  write reqs on a per-metric bases at the output level

This should be more efficient when there are concurrent writes to a single metric with different series.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Joshua Lockerman seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


How to query PSTN usage report using PowerBI or Powershell

Is this report accessible via PowerBI or Powershell so we can automate the delivery to managers and stakeholders without the need of going into the admin console?

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

ID: 0ad0c889-56d5-35fe-de15-177b6d922d07
Version Independent ID: e4363647-33e1-b665-5881-169fb619a5b3
Content: Microsoft Teams PSTN usage report
Content Source: Teams/teams-analytics-and-reports/pstn-usage-report.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@floydpreyes Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Appreciate this Manuela @scanum :) our data team has informed me that accessing it via API will also work for them, if MS can somehow allow that as well.

@floydpreyes  Thank you for raising this issue. I will be looking into it.

thank you @tokyoscarab, please let me know how you go so I can inform our team to start working on it once accessible. We moved from on-premises PSTN to Teams and we used to have an automated reporting solution that tracks individual calls received/made per staff, we need to provide the same functionality to our internal managers as they are requesting for it.

@LanaChin From what I've been able to find, it doesn't appear that Teams PSTN usage reports are accessible through PowerShell or PowerBI as of yet. Can you verify that? Do you know if it's being worked on or will be implemented in the future?

@LanaChin Hi Lana, could you please help with this issue? Thanks!

This is also something of interest to me.  Until it was deprecated in 2018, it was possible to extract PSTN usage metrics using the Powershell cmdlet "Get-CsPSTNUsageDetailReport", and from what I've gathered there is no equivalent/replacement.

Pinged PMs.

@floydpreyes, @TokyoScarab, @darque808 - thanks for your question. I reached out to the PM and received confirmation that the PSTN usage report isn't available through PowerShell. And, currently, there's no plans for availability through PowerShell.

Hello. I think what the community needs is a method to extract the PSTN usage data grammatically, rather than through the console.
What does Microsoft recommend as a way to achieve this, ideally also as a schedule?

Beta version of PSTN usage report in Graph is now available - see https://docs.microsoft.com/en-us/graph/api/callrecords-callrecord-getpstncalls?view=graph-rest-beta


Grace Periods for newly provisioned machines

Addresses: https://github.com/macadmins/nudge/issues/328
  "userExperience": {
    "allowGracePeriods": false,
    "gracePeriodInstallDelay": 24,
    "gracePeriodLaunchDelay": 1,
    "gracePeriodPath": "/private/var/db/.AppleSetupDone"
}

Once the age of gracePeriodPath is greater than both the gracePeriodInstallDelay and gracePeriodLaunchDelay the requiredInstallationDate is going to revert to the original date in the profile/json. From a user experience perspective, nothing will change as in both cases, the user is now passed the original requiredInstallationDate or new requiredInstallationDate.

I'm not too worried about this

Each time nudge runs during the gracePeriodInstallDelay, the requiredInstallationDate is going to keep moving for the user. This is because allowButtons is set to false only when the user is passed the allowedDeferrals count or passed the requiredInstallationDate.

This is hugely problematic, and the only way I can think to solve this issue is to start adding the requiredInstallationDate to the user's preference file and using it for this logic. I will then have to remove this key if it exists post this behavior.

So I've redone the logic entirely and came away with the following conclusion - it doesn't matter if you are before or after the requiredInstallationDate

if you are either passed requiredInstallationDate OR (gracePeriodInstallDelay + gracePeriodLaunchDelay) > the number of hours remaining in a nudge event

if you are under the gracePeriodLaunchDelay, nudge exits out
If you are over the gracePeriodLaunchDelay, your requiredInstallationDate is the created date of gracePeriodPath + (gracePeriodInstallDelay + gracePeriodLaunchDelay in hours). By default that gives you 24 hours to install the required update, which is within the imminentWindow by default, giving the user a way to exit out.

I think rather than solving the various amounts of edge cases, I would rather document the following for the admin

the total grace period is gracePeriodInstallDelay + gracePeriodLaunchDelay, which defaults to 24 (23 + 1)
the imminentWindow defaults to 24, and you should test the user experience should you you increase the gracePeriodDelay values to something that's higher as Nudge's user experience may change.

Example: If you were to set it to the values to 1 and 71 respectively, you Nudge would show the approachingWindow logic so long as the default was kept of 72

As mentioned, this code is prime for Unit testing

As mentioned, this code is prime for Unit testing

I hate you. It's done.


Upgrade Rails to 4.2.2

Even if this not an active app, it shouldn't turn up with grep.

I think it's still worth making this change, but if we don't plan to use it, it could be moved to gds-attic?

We'll discuss whether to finish it or move it to the attic "later".


Unable to access 'getModel' function in typescript

Version
5.2.2
Link to Minimal Reproduction
No response
Steps to Reproduce

Create any chart
on the chart finish event . I am trying to access
chart.getModel().getComponent('grid')
So while running the application we are getting below issue

Property 'getModel' is private and only accessible within class 'ECharts'.
Current Behavior
It is working as expected but need to make the getModel function as public
Expected Behavior
It is working as expected but need to make the getModel function as public
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
If any workaround is there to get the component other than getModel function

getModel is not ready to be public yet. We only suggest using this method when it's the only workaround to solve the issue and you can accept the missing doc and potential breaking change of this method in the future.

Please close this issue if you don't have further questions. Thanks!
@RavalikaP

@pissang

I'm a bit confused, getModel() appears frequently, both in the documentation and in git. For example, https://echarts.apache.org/en/faq.html#baidu-map and https://github.com/apache/echarts/tree/5.3.3/extension-src/bmap both mention  chart.getModel().getComponent('bmap').getBMap();, which specifically accesses the private method. If this is not allowed to be used directly, should we update the documentation?

@pissang @Ovilia but if not using getModel(), how can I get the component value when zooming in and zooming out?


[FLINK-2226][YARN] fail container on failed single-job cluster job

Failing jobs executed in the YARN cluster mode leave the application
container in the "SUCCEEDED" final state. While for long-running Flink
YARN clusters where multiple jobs are run, this is fine, for single jobs
it is appropriate to mark the container as failed.

You are exiting the application with status FAILED and not the container. The container is finishing as expected.
The code (naming etc.) make it sound like you are failing the container (I don't know if that is possible). Other than that I think this is a good change.
A user found this confusing, because the job failed, but the YARN logs and UI show succeeded applications.

I agree with Ufuk. You are setting the status of the app to failed, not of the container.

Agreed, a bit misleading name. Changed it to failAppllication.

The travis run for the previous commit (before your var name update) failed  with
java.lang.RuntimeException: Error while stopping YARN Application Client
	at scala.concurrent.impl.Promise$DefaultPromise.ready(Promise.scala:219)
	at scala.concurrent.impl.Promise$DefaultPromise.ready(Promise.scala:153)
	at scala.concurrent.Await$$anonfun$ready$1.apply(package.scala:95)
	at scala.concurrent.Await$$anonfun$ready$1.apply(package.scala:95)
	at scala.concurrent.BlockContext$DefaultBlockContext$.blockOn(BlockContext.scala:53)
	at scala.concurrent.Await$.ready(package.scala:95)
	at scala.concurrent.Await.ready(package.scala)
	at org.apache.flink.yarn.FlinkYarnCluster.shutdown(FlinkYarnCluster.java:420)
	at org.apache.flink.client.FlinkYarnSessionCli.stop(FlinkYarnSessionCli.java:461)
	at org.apache.flink.yarn.YarnTestBase$Runner.sendStop(YarnTestBase.java:514)
	at org.apache.flink.yarn.YarnTestBase.runWithArgs(YarnTestBase.java:440)
	at org.apache.flink.yarn.YARNSessionFIFOITCase.testClientStartup(YARNSessionFIFOITCase.java:103)

Your change should actually not trigger this issue ... but it is happening in the shutdown() method

I think this is by chance. Here it passes: https://travis-ci.org/mxm/flink/builds/66877782

And here as well: https://travis-ci.org/mxm/flink/builds/66881439


Create 1466. Reorder Routes to Make All Paths Lead to the City Zero.py

File(s) Modified: 1466-reorder-routes-to-make-all-paths-lead-to-the-city-zero.py
Language(s) Used: python
Submission URL: _https://leetcode.com/problems/reorder-routes-to-make-all-paths-lead-to-the-city-zero/submissions/1175173370/_

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

@luarss Thanks for your contribution 👍


can use pl33 filter with dask-backed DataArrays

Ok, but can I ping you to review still?

Sure. I owe some other reviews but this one will be quicker :grimacing:

@ocefpaf Should I merge?

I'll try to get a new release out soon. Probably next Tuesday.


math/big: printing float is not respecting RoundingMode

What version of Go are you using (go version)?
go version go1.7 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH=""
GORACE=""
GOROOT="/usr/local/go1.7"
GOTOOLDIR="/usr/local/go1.7/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
What did you do?
Rounded a big.Float using formatting and using SetPrec to the same precision and using the same RoundingMode.
https://play.golang.org/p/wqWAnOzf45
What did you expect to see?
I expected formatting of numbers should also respect the rounding mode.
Not sure if this was intentional, but putting it out there.
What did you see instead?
It does not.

Note that the behavior for big.Floats is the same as for float64 with respect to printing: https://play.golang.org/p/z_Y_hIg9Cq
I'm not saying there's no issue here, but if there is, it's also in strconv. Haven't had time to investigate in more detail.

Background; I'm adding this ability to x/text repo for number formatting. Right now I have the unfortunate situation that I can't rely on much of the code in core to do number formatting (or formatting in general). I can also show you what I end up with and then see how to retrofit it in core if there is a desire to not have duplicate code.

The "issue" here is that the rounding mode for formatting is always RoundToNearestEven. This matches the strconv library.
If you need to round than do it before formatting and you get the desired result: https://play.golang.org/p/iSo7VROcPj


add keyboard shortcuts, fixes #5

cc @nerik
Sorry, this simulateClick stuff is a bit weird - didn't quite find a better way to do that without adding something like jQuery. And, we should definitely chat about #15 :-)
Hopefully this is roughly okay to merge.

Smart hack, that definitely works. Indeed, with a more structured codebase we wouldn't have to go through this.


delete the orderBy

Before
TransactionBenchmark,Tue Jun 22 15:06:28 EDT 2021
Threads,Memory GB,Operations/s/t,Pass,Operations/s,Total,Transactions (M),Run,Fail
1,0.31,35.87,10000,35.87,10000,0.01,1,0
TransactionBenchmark,Tue Jun 22 15:11:09 EDT 2021
Threads,Memory GB,Operations/s/t,Pass,Operations/s,Total,Transactions (M),Run,Fail
1,0.24,37.70,10000,37.70,10000,0.02,1,0
After
TransactionBenchmark,Tue Jun 22 15:40:25 EDT 2021
Threads,Memory GB,Operations/s/t,Pass,Operations/s,Total,Transactions (M),Run,Fail
1,0.34,54.97,10000,54.97,10000,0.01,1,0
TransactionBenchmark,Tue Jun 22 15:44:20 EDT 2021
Threads,Memory GB,Operations/s/t,Pass,Operations/s,Total,Transactions (M),Run,Fail
1,0.21,46.01,10000,46.01,10000,0.02,1,0

DO NOT MERGE
I think we still need the orderBy. Actually, in the services file it does:
.setOrder(new foam.core.PropertyInfo[] {foam.nanos.crunch.CapabilityCapabilityJunction.PRIORITY})
So the order should be the default anyway.
Instead, let's add an index.
Commit 285021ce062299647b30767a373b8a6c59da5772
Author: Kevin Greer<EMAIL_ADDRESS>Date:   Thu Jun 24 12:05:13 2021 -0400
Add PRIORITY index on CapabilityCapabilityJunctionDAO.

@hchoura, can you retest with the above commit?

Also, does this cache need to be per-user?

285021c
Before
Threads,Memory GB,Operations/s/t,Pass,Operations/s,Total,Transactions (M),Run,Fail
1,0.24,41.08,1000,41.08,1000,0.00,1,0
1,0.17,41.96,1000,41.96,1000,0.00,1,0

After
done
TransactionBenchmark,Mon Jul 05 13:57:52 EDT 2021
Threads,Memory GB,Operations/s/t,Pass,Operations/s,Total,Transactions (M),Run,Fail
1,0.25,86.66,1000,86.66,1000,0.00,1,0
done
TransactionBenchmark,Mon Jul 05 13:58:14 EDT 2021
Threads,Memory GB,Operations/s/t,Pass,Operations/s,Total,Transactions (M),Run,Fail
1,0.23,83.56,1000,83.56,1000,0.00,1,0


document and support logging in via an auth token

Logging into android devices quickly will trigger a serverside rate limit which logs out all android devices. This causes problems for some plugin users that get loaded and unloaded very often (eg https://github.com/diraimondo/gmusicproxy/issues/34).
We should document this in mobileclient.login, and also support the workaround where client code keeps track of the auth token.

https://github.com/simon-weber/gmusicapi/issues/620#issuecomment-439731287 will fix this.

That's out now in 11.1.0: see the docs for details.


Any working example available?

Hi I am new to batch programming. Can you tell me if there's any working example is available for using easy-batch?
I am looking to create a batch job that stores file locations in DB and retrieve the file locations from DB to process and store the information on to DB.
Thanks

Can you tell me if there's any working example is available for using easy-batch?

You can find complete tutorials here: https://github.com/j-easy/easy-batch/tree/master/easy-batch-tutorials.


Updated README to contain example usage of using proxies

Thanks adding the pass through for proxies, I've updated the docs to contain example usage.

Usage examples have been added.


Typography updates case consistency headers, size etc as specified in Design Review

Changes in this PR:
Typography updates case consistency headers, sizing, spacing etc as specified in Design Review
Trello card / Rollbar error (etc)
https://trello.com/c/6SUqtJGI/1444-eui-design-review-of-current-site
Screenshots of UI changes:
Bold headings details,submitted,assigned
Before

After

CSS to render case consistently in title links -capitalise
Before

After

Update to 19px headings-details
Before

After

Reduce space between key value pairs in listing
Before

After

Reduce space between key value pairs in metadata panel text
Before

After

[ ] Requires env variable(s) to be updated

 [EUI] Design Review of current site

I'd be tempted to take even more space out of the key-value pairs -- I think the important width might be "end of longest key to start of value" which is very wide on the left hand side due to being short words (Name vs. Judgment Date). But all looks good and the case-fixing is very nice indeed.


cilium connectivity test hand on Waiting for CiliumEndpoint

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
hang here:
[anvil-vpi-sandbox] Creating namespace for connectivity check...
✨ [anvil-vpi-sandbox] Deploying echo-same-node service...
✨ [anvil-vpi-sandbox] Deploying same-node deployment...
✨ [anvil-vpi-sandbox] Deploying client deployment...
✨ [anvil-vpi-sandbox] Deploying client2 deployment...
✨ [anvil-vpi-sandbox] Deploying echo-other-node service...
✨ [anvil-vpi-sandbox] Deploying other-node deployment...
⌛ [anvil-vpi-sandbox] Waiting for deployments [client client2 echo-same-node] to become ready...
⌛ [anvil-vpi-sandbox] Waiting for deployments [echo-other-node] to become ready...
⌛ [anvil-vpi-sandbox] Waiting for CiliumEndpoint for pod cilium-test/client-57544b6d4-r8674 to appear..
Cilium Version
ilium-cli: v0.11.5 compiled with go1.18.1 on linux/amd64
cilium image (default): v1.11.3
cilium image (stable): v1.11.4
cilium image (running): v1.11.4
Kernel Version
5.10.43-flatcar
Kubernetes Version
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.4", GitCommit:"e6c093d87ea4cbb530a7b2ae91e54c0842d8308a", GitTreeState:"clean", BuildDate:"2022-02-16T12:38:05Z", GoVersion:"go1.17.7", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.15",
Sysdump
cilium-sysdump-20220507-162850.zip
Relevant log output
No response
Anything else?
ran different options of helm template cilium cilium/cilium --version 1.11.4 but always hang
also, the policies are not enforced.
also, the nodes have calico running:
$ ps -ef | grep calico
root        8454    8452  0 20:28 ?        00:00:00 svlogd -tt /var/log/calico/bird6
root        8455    8451  0 20:28 ?        00:00:00 svlogd /var/log/calico/confd
root        8456    8453  0 20:28 ?        00:00:00 svlogd -tt /var/log/calico/bird
root        8457    8450  0 20:28 ?        00:00:00 svlogd /var/log/calico/felix
root        8462    8452  0 20:28 ?        00:00:02 bird6 -R -s /var/run/calico/bird6.ctl -d -c /etc/calico/confd/config/bird6.cfg
root        8463    8453  0 20:28 ?        00:00:02 bird -R -s /var/run/calico/bird.ctl -d -c /etc/calico/confd/config/bird.cfg
root        8464    8450  1 20:28 ?        00:01:26 calico-felix
root        8465    8451  0 20:28 ?        00:00:01 confd -confdir=/etc/calico/confd -interval=5 -watch --log-level=info -node=http://<IP_ADDRESS>:2379 -client-key= -client-cert= -client-ca-keys=
$ cat /etc/calico-node.yml

apiVersion: v1
kind: bgpPeer
metadata:
node: anvil-vpi-sandbox-10-54-213-159
peerIP: <IP_ADDRESS>
scope: node
spec:
asNumber: 65012
apiVersion: v1
kind: bgpPeer
metadata:
node: anvil-vpi-sandbox-10-54-213-159
peerIP: <IP_ADDRESS>
scope: node
spec:
asNumber: 65012
apiVersion: v1
kind: ipPool
metadata:
cidr: <IP_ADDRESS>/22
spec:
nat-outgoing: true

but 05-cilium.conf is present.
there were calico files in /etc/cni/net.d but cilium replaced them.
maybe cilium is still taking precedence.
$ find /etc/cni
/etc/cni
/etc/cni/net.d
/etc/cni/net.d/05-cilium.conf
$ k exec -ti cilium-4qkvc -- cilium-health status
Defaulted container "cilium-agent" out of: cilium-agent, mount-cgroup (init), clean-cilium-state (init)
Probe time:   2022-05-07T23:08:18Z
Nodes:
anvil-vpi-sandbox/<IP_ADDRESS> (localhost):
Host connectivity to <IP_ADDRESS>:
ICMP to stack:   OK, RTT=258.639µs
HTTP to agent:   OK, RTT=348.72µs
Endpoint connectivity to <IP_ADDRESS>:
ICMP to stack:   Connection timed out
HTTP to agent:   OK, RTT=519.282µs
anvil-vpi-sandbox/<IP_ADDRESS>:
Host connectivity to <IP_ADDRESS>:
ICMP to stack:   OK, RTT=379.134µs
HTTP to agent:   OK, RTT=607.787µs
Endpoint connectivity to <IP_ADDRESS>:
ICMP to stack:   Connection timed out
HTTP to agent:   OK, RTT=909.174µs
anvil-vpi-sandbox/<IP_ADDRESS>:
Host connectivity to <IP_ADDRESS>:
ICMP to stack:   OK, RTT=502.021µs
HTTP to agent:   Get "http://<IP_ADDRESS>:4240/hello": dial tcp <IP_ADDRESS>:4240: connect: connection refused
Endpoint connectivity to <IP_ADDRESS>:
ICMP to stack:   Connection timed out
HTTP to agent:   OK, RTT=7.141852468s
Code of Conduct

[X] I agree to follow this project's Code of Conduct

needed to chain the CNIs properly.


Add missing code to identify the complete UKIT (MKIT+EKIT+WKIT+other...)

Task Description
The code to identify the global assembly (UKIT) to execute the upgrade is missing.
Definition of Done

[x] The documentation contains the upgrade identification code
[x] Each part (MKIT, EKIT, etc) that constitutes the UKIT is described and correctly identified
[x] The scope of the upgrade is clearly specified (for which platform)

@Fabrizio69 now the documentation of UKIT should be more nice-looking from your standpoint.
Give it a glance and report here any feedback you may have, please.

I am not dealing with UKIT on a daily basis, so I may suggest something that it may not have sense, but if the UKIT code is the main entry point and identified for a ukit, I think it could make sense to add it to the summary and the menu to access the UKITs:

Because I imagine that in the future the number of UKITs will grow, and opening all the pages of the UKITs to understand to which human readable name corresponds the UKIT 003 may be complex.

I'll do that 👍🏻

Done @traversaro 👍🏻

The information you required is now available @Fabrizio69.
Closing for now, you could still point us at any improvements you may have.


Upgrade Instructions for Docker Compose

I think it's just:
docker compose down
docker pull counterparty/counterparty:master
docker compose up

done here https://github.com/CounterpartyXCP/Documentation/pull/205


NPM registry not updated with latest version

useragent has some fixes merged but NPM registry is not updated with latest version.
https://github.com/3rd-Eden/useragent/pull/144#issuecomment-735786435

Hi @3rd-Eden , as mentioned here, can you please publish a new version to NPM registry?
There is a security finding with severity high on your repository regarding ReDoS (Regular Expression Denial of Service), which was fixed by @ChALkeR on your merge to master on May 2019 (see here). Unfortunately the NPM package was last updated 3 years ago, can you please do a quick npm publish? That would be a great help to us all.
Thanks in advance!

When can we expect this to be published. This is a high priority security vulnerability, So Can you please publish the updated package ASAP.

When can we expect this to be published. This is a high priority security vulnerability, So Can you please publish the updated package ASAP.

+1

+1


Job Batch compatibility with proxy sql

Add a transaction on Job Batch dispatch to send insert + select query on the database master slave.
With a proxy sql splitting queries, update/alter/transaction on master/select on slave database, use of Job Batch thrown exception :
the insert query will be sent to master database, the select query will be sent to slave. The slave database is not synchronised and can't return inserted data.

A proxy sql to split query is an external stack. For dev ops, i don't want Laravel to manage different database connection.
All queries are sent to the master database, it's the proxy sql who manage where to redirect them.
Without transaction here or a change to not insert+select immediately, Laravel will be incompatible with this type or product.
Here the doc of Maxscale on the subject for routing queries : https://mariadb.com/docs/xpand/architecture/components/maxscale/routers/readwritesplit/route-statements-maxscale-read-write-split-router/

Laravel is not inherently incompatible, but you've gotta understand the trade-offs. Many features of laravel require read your own writes, you can't use the read-write split routing feature with many features of Laravel.


bug: inline option for embedfields doesn't do anything

Describe the bug
setting an embed field as inline doesn't do anything, and it appears as though it isn't inline
To Reproduce
attempt to set an embed field as inline
Expected behavior
the field is properly inline
Screenshots

no difference between inline and non-inlinefields

Ah.. well that's actually only a problem on the example code not being properly representative of how to do inline fields. You're missing the inline-index property.
This will work:
				<discord-messages>
					<discord-message profile="skyra">
						<discord-embed slot="embeds">
							<discord-embed-fields slot="fields">
								<discord-embed-field field-title="Inline field title" inline inline-index="1"> Some value here </discord-embed-field>
								<discord-embed-field field-title="Inline field title" inline inline-index="2"> Some value here </discord-embed-field>
								<discord-embed-field field-title="Inline field title" inline inline-index="3"> Some value here </discord-embed-field>
							</discord-embed-fields>
						</discord-embed>
					</discord-message>
				</discord-messages>

The reason you have to set the index manually is because there is simply no reliable way to determine which field goes where, because each DiscordEmbedField is distinct and doesn't have knowledge of how many there are in total to then divide that amount by 3.

Thanks, worked perfectly!


"Installing packages..." stays in the status bar after installing OmniSharp and its dependencies

VSCode Version: 1.40.1
OS Version: Windows 10, Version 1903 (OS Build 18362.476)

Steps to Reproduce:

I opened a .cs file and Visual Studio Code started downloading the C# dependencies
Despite the fact that the installation process finished (?), the status bar is stuck with "Installing packages...". Hovering that item, it displays the following tooltip: Installing package 'Razor Language Server (Windows / x64).

Installing C# dependencies...
Platform: win32, x86_64
Downloading package 'OmniSharp for Windows (.NET 4.6 / x64)' (32059 KB).................... Done!
Validating download...
Integrity Check succeeded.
Installing package 'OmniSharp for Windows (.NET 4.6 / x64)'
Downloading package '.NET Core Debugger (Windows / x64)' (47489 KB).................... Done!
Validating download...
Integrity Check succeeded.
Installing package '.NET Core Debugger (Windows / x64)'
Downloading package 'Razor Language Server (Windows / x64)' (50372 KB).................... Done!
Installing package 'Razor Language Server (Windows / x64)'
Finished

Please file the issue against the C# extension.


RNCryptor in C# and Swift, RNCryptorError error 2

When I encrypt some plain text in Swift,I can decrypt it in Swift correctly without any error. And in C# too. But when I encrypt some text in C# and then I want to decrypt it in Swift,I got an error " The operation couldn’t be completed. (RNCryptorError error 2.)"
Thank you so much.
My code in C# :
public static string EncryptQRCode(string qrCodeString){
        var qrEncryptor = new Encryptor ();
    return  qrEncryptor.Encrypt (qrCodeString, "password");

    }
public static string DecryptQRCode(string qrEncryptedString){
        var qrDecryptor = new Decryptor();
      return qrDecryptor.Decrypt (qrEncryptedString, "password");
    }

My Code in Swift:
func Encrypt(msg:String, pwd:String) -> String{
    let data = msg.dataUsingEncoding(NSUTF8StringEncoding,      allowLossyConversion: false)
    let chiper = RNCryptor.encryptData(data!, password: pwd)
    let base = chiper.base64EncodedDataWithOptions(NSDataBase64EncodingOptions(rawValue: 0))
    let stringBase = String(data: base, encoding: NSUTF8StringEncoding)
    return stringBase!
}
func Decrypt(msg:String, pwd:String) -> String{
    let encodedData:NSData = NSData(base64EncodedString: msg, options: NSDataBase64DecodingOptions(rawValue: 0))!
    do{
        let text = try RNCryptor.decryptData(encodedData, password: pwd)
        let k = String(data: text, encoding: NSUTF8StringEncoding)
        return k!
    }
    catch let error as NSError{
        print(error.localizedDescription)
        print(error.localizedDescription)

    }
    return "AN ERROR OCCURED"

}

Im very glad that nobody here couldn't help me to find the solution. At last after 3 days trying to solve this problem,I find the solution. In C#,When you want to encrypt your data,you have to set Schema to V3. Now Swift decrypt your data without any error.


Update CQD-Power-BI-connector.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/5966
Removed wording

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Add Insufficient Forensic Visibility in GCP Storage

https://www.mitiga.io/blog/mitiga-security-advisory-insufficient-forensic-visibility-in-gcp-storage

I don't think this should be added as it seems more like something that would be nice to have, but does not seem to be a security mistake.  As per the response of Google, it is not a vulnerability.  I'm not even sure if this is all that valuable of a wishlist item for defenders, as an attacker could just as easily download data locally and then copy it externally in order to circumvent the logging that AWS is said to do better.


are there intrinsics for the whole register load/store instructions?

Hi,
I'm looking for these intrinsics and I am not finding them. Do we have them or are they in an unexpected place?
-rich

I stumbled across this issue while I was looking for intrinsics for whole-register moves (vmv<nr>.v), which also don't exist. Regarding these moves, @eopXD wrote:

We currently have = with semantic of a whole vector register move (which is also vmv.v.v of VL=VLMAX).

https://github.com/riscv-non-isa/rvv-intrinsic-doc/issues/74#issuecomment-1202231257
I don't think this has been documented yet.
I've only given it about five minutes of thought, but I think this is a reasonable API design decision.
Regarding your question about whole-register loads/stores, I don't recall if the omission was intentional. I'm struggling for a use-case as an intrinsics programmer. The only use-case I see is context save/restore code, but we've already agreed to ignore this use-case in our API design (e.g., we already don't expose R-type vsetvl).

Hi Rich,
I brought this up at this week's task group meeting. The conclusion is that the API intentionally omits whole-register loads, stores, and moves. The rationale is that the usual loads/stores/moves provide the same functionality, and a compiler could instead generate whole-register versions if it thought it would be profitable. If a compelling use-case arises in the future, we could introduce new intrinsics in a backwards-compatible way.
Best,
Nick


Change the way ioreqs metric is handled

Indeed, it refers to the in_flight field in diskstat file and is the only field that should/might go to zero. I think it's better to report the metric as it, without modification.
It might also be worth to rename the metric name.
Source :

https://www.kernel.org/doc/Documentation/block/stat.txt
https://www.kernel.org/doc/Documentation/iostats.txt

I'm fine with both. :)


Implement Decentralized Wallet Restore

[ ] check design is ready before start, https://github.com/horizontalsystems/bank-wallet-ios/issues/139
[ ] add choose methods controller, as a first step in restore process
[ ] enable to choose restore method between fast restore (via third party API) and slower decentralized  restore (restoring directly from blockchain)
[ ] the third party restore option should provide multiple public APIs for selection

design for the restore screen: https://github.com/horizontalsystems/bank-wallet-ios/issues/248

Issue Merged Into https://github.com/horizontalsystems/bank-wallet-ios/issues/62


Allow changing multiple key values atomically

Have you thought about having that in libkv? Simple transactions are possible with Consul, Zookeeper, etcd (v3) and Redis. Looks like pretty common use case?

Yes, thought about this and ACLs, for now I'll focus on improving the tests and make the foundations more solid. Then if we can add more features such as transactions that would be nice indeed.
Let me know if you are interested in getting started with the implementation. It doesn't have to be for every store at first, we can throw an store.ErrCallNotSupported  for other stores.

Yes, I am. Would be great if you could propose some API for that. I would probably start with etcd.

@abronan do you have any idea how the API could look like?

@mthenw Sorry for the very long wait, was pretty busy with interviews recently. I think you could take a look at how etcd v3 API does transactions (I really like their API), specifically with Put/AtomicPut and how we use the client.Txn method. I think it gives a nice idea of what this might look like on our side.
For example:
ctx, cancel := context.WithTimeout(context.Background(), 5 * time.Second)
txn := client.Txn(ctx)
txn.Op(libkv.OpPut("firstKey", []byte("value")))
txn.Op(libkv.OpPut("secondKey", []byte("value")))
result, err := txn.Commit()
cancel()

We would get Op*** methods for Get/Put/AtomicPut, etc.
Will have to think about the AtomicPut case which might be a little trickier to rightly implement (I don't think the solution of etcd v3 with Compare is very friendly, although I'm not sure yet if there is a better way).


Search - Room and Workspace does not displayed for invited user

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to URL https://staging.new.expensify.com/
in maim and incognito windows and login with different Expensify users (A and B)
Create a workspace with user A
Go to the Manage members tab of the workspace, invite user B
Log in with account B
Verify that the invited member (B) can see the workspace and room

Expected Result:
The workspace and room should be available
Actual Result:
the workspace and room is not available
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/194730725-f6d581ea-72ff-47ca-b227-001a0ea33161.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Can you confirm you can see the rooms once you do a refresh and/or log in again? (I do see the new rooms in this case) It's possible this isn't connected to pusher yet.

So interestingly, I see the room as the "User B" but not "User A"

@AndrewGable here is the retest result
https://user-images.githubusercontent.com/43995119/194947024-5f57256d-4dac-4141-9413-f23683004654.mp4

Sorry I was OOO yesterday but can look into this today. Looks like this was assigned on a Saturday.

@marcaaron I think my PR is going to fix this too. I'd like to group them together, so I will grab this from you.


ssh disconnections

Type: Bug
Latest version was installed last night via snap.  Now I cannot connect to any remote host via the Remote Development module.  SSH connects successfully, according to sshd logs on the remote.  But VScode disconnects from the remote.  There are no error message on the remote sshd log that would indicate any reason for the disconnection.
This happens on all 3 remote hosts that I typically develop on.  Before the snap update, these connections worked fine.
I needed to revert to the previous version to regain remote development:
jjaroker@puma:~$ snap list code --all
Name  Version   Rev  Tracking       Publisher  Notes
code  6c3e3dba  137  latest/stable  vscode✓    classic
code  8b617bd0  138  latest/stable  vscode✓    disabled,classic

VS Code version: Code 1.82.0 (8b617bd08fd9e3fc94d14adb8d358b56e3f72314, 2023-09-06T22:07:18.759Z)
OS version: Linux x64 5.15.0-83-generic snap
Modes:
Connection to 'SSH: puppet.staging' could not be established  Canceled

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-2630 v3 @ 2.40GHz (32 x 1200)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 1, 0

Memory (System)
125.79GB (122.92GB free)

Process Argv
--no-sandbox --force-user-env --unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
plasma

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
x11

Connection to 'SSH: puppet.staging' could not be established  Canceled
Extensions: none

Same problem, fixed by switch "Remote - SSH" extension to Pre-Release Version.

Same problem, fixed by switch "Remote - SSH" extension to Pre-Release Version.

👍it works for me.

yes, it was fine in pre release version, now it reports Resolver Error
always "Failed to parse remote port from server output"

I had a similar problem and the ssh connection would hang with the message
Failed to set up socket for dynamic port forward to remote port ...:

I was able to ssh successfully from my terminal, but not from VSCode. I rolled back the remote ssh extension to a few different versions, but eventually that worked.
I first tried the prelease version, but that didn't work. I had to roll it back two versions before it worked.

I would be grateful if you could grab the "Remote - SSH" log from the "Output" tab in the bottom panel of VS Code when you get into this broken state. Note that we've turned off the problematic functionality in stable for the moment, but it can be turned back on via the "remote.SSH.useExecServer": true setting. Please turn that on for a few minutes and see if you can still get the failure. Thanks!

I have the same issue. Please find logs attached.
Issue is fixed switching to pre-release extension.
VScode_SSH_Log_Level.txt

@webjema your issue is the same as https://github.com/microsoft/vscode-remote-release/issues/8942#issuecomment-1714674704 which will be fixed when you update to the latest extension version (0.106.3)

Hello @connor4312
I attempted to get the additional information you requested, but there is no output generated.
Here is my environment:
jjaroker@puma:~$ snap list code --all
Name  Version   Rev  Tracking       Publisher  Notes
code  6c3e3dba  137  latest/stable  vscode✓    disabled,classic
code  65091741  139  latest/stable  vscode✓    classic

Enabling the option "Remote.SSH:Use ExecServer" and then making a remote connection resulted in several loops of "Downloading VS Code..." and finally an error "Could not fetch remote environment".
Here is a screenshot of what I observe.  The Output tab is blank.

Hi, you'll want to use the select box in the upper right side of the bottom panel and switch to "Remote - SSH". That should have the output. Thanks!

Sorry about that.
Attached are files from the ssh remote output and the error messages that appear on vscode console.
sshremote.log
vscode-app-1694631240522.log
vscode-app-1694631415189.log

Enabling "remote.SSH.useExecServer": true fixed it for me (before I was getting "Error opening exec server" when trying to launch a dev container via a remote SSH workspace

Enabling "remote.SSH.useExecServer": true fixed it for me (before I was getting "Error opening exec server" when trying to launch a dev container via a remote SSH workspace

Me too. Helped to fix reopening in the dev container command execution.

THat is tracked in https://github.com/microsoft/vscode-remote-release/issues/9064, release will be out shortly

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[Bug] PR from community is moved to In Progess instead of In Progress Communtiy

Bug Description

PR from community is moved to In progress which creates a bad credentials error .
To Reproduce

1.) Create a Pull request from a non cc staff .
Expected behavior

PR should be moved directly to In progress Community
Screenshots

Additional context

I think this is coming after merging pr for issue #305 .

This is a known issue that will be fixed by https://github.com/creativecommons/ccos-scripts/issues/23

However, this should not block PRs. To fix this, we need to add the continue-on-error configuration to the PR moving workflow, similar to line 21 on https://github.com/creativecommons/cccatalog-frontend/blob/develop/.github/workflows/project-automation.yml. If someone wants to fix that, that would be great.

I will work on it


Different extraction result on the same input

I tried to run the following extraction snippet on the same HTML input multiple times, but every 4th extraction returned shorter extraction.
link = '...'
html = scrape(link)
prev_extraction = None

for x in range(10):
    extraction = trafilatura.extract(html, include_comments=False,
        include_tables=False, no_fallback=True, target_language='en')
    if prev_extraction:
        if prev_extraction != extraction:
            print('Extraction looks weird!')
    prev_extraction = extraction

Is there any parameter I should be using or is this a bug to be fixed?
Thanks!

Hi, this happens because a deduplication algorithm is activated by default on element, paragraph and document levels. To make sure not to run it you can just pass an additional deduplicate=False argument to trafilatura.extract().
This additional feature will ship with the next package version, until then please use the latest code from the repository: pip install -U git+https://github.com/adbar/trafilatura.git

The new version (0.5.2) is out, deduplication is now off by default.


Switch back to mainline svg-react-loader

I've forked svg-react-loader to @benwiley4000/svg-react-loader which resolves outstanding npm audit vulnerabilities. Once the same is merged and released for the upstream svg-react-loader (https://github.com/jhamlet/svg-react-loader/pull/105), we should switch back.

Closed by https://github.com/benwiley4000/cassette/commit/33adeba2a98ba62590a109527c176cf6012d0633.


fix typo s33:GetAccountPublicAccessBlock

It seems this is typo
https://github.com/capitalone/cloud-custodian/blob/master/c7n/resources/account.py#L1119
s33:GetAccountPublicAccessBlock -> s3:GetAccountPublicAccessBlock

Thanks for the pr @cahn1, If you don't mind could you sign the CLA at the bottom of the README? Direct link here: https://docs.google.com/forms/d/e/1FAIpQLSfwtl1s6KmpLhCY6CjiY8nFZshDwf_wrmNYx1ahpsNFXXmHKw/viewform


optimize: spring boot compatible with file.conf and registry.conf  (#6811)

[x] I have registered the PR changes.

Ⅰ. Describe what this PR did
This PR implements a new class SeataFileConfigurationProvider in the seata-spring-autoconfigure-core module. This change ensures that both seata-spring-boot-starter and seata-spring-autoconfigure-server can read legacy configuration files (file.conf and registry.conf), addressing an issue where users upgrading from Spring to Spring Boot would only have their configuration loaded, causing confusion.
Ⅱ. Does this pull request fix one issue?
fixed #6811
This PR addresses the issue where seata-spring-boot-starter fails to load file.conf and registry.conf when users upgrade from Spring to Spring Boot.
Ⅲ. Why don't you add test cases (unit test/integration test)?
org.apache.seata.spring.boot.autoconfigure.SeataCoreAutoConfigurationTest
Ⅳ. Describe how to verify it

Use the seata-spring-boot-starter in a Spring Boot application.
Ensure that configurations from file.conf and registry.conf are correctly loaded alongside configurations.
Verify the behavior in both seata-spring-boot-starter and seata-spring-autoconfigure-server environments.

Ⅴ. Special notes for reviews

This change is backward-compatible and allows users upgrading from Spring to Spring Boot to continue using their legacy configuration files without disruption.
It would be helpful to review the relocation of SeataPropertiesLoader to the seata-spring-autoconfigure-core module, ensuring that legacy configuration files (file.conf and registry.conf) are still properly loaded in both seata-spring-boot-starter and seata-spring-autoconfigure-server environments. Additionally, please verify that this change maintains backward compatibility and ensures a smooth transition for users moving from Spring to Spring Boot.

请合并最新的2.x分支
Please merge the latest 2.x branch


Aggregation in vertical direction

[ ] Aggregation of multiple rows into one

[ ] By default on highest possible level

Will close this issue and create a new one that reflects our refined concept.


Upgrades OCI to 2.41.0 on helidon-2.x

Signed-off-by: Laird Nelson<EMAIL_ADDRESS>

This PR as it exists cannot be merged; there are test failures:
org.opentest4j.AssertionFailedError: Timed out after 10 sec. waiting for testMetricCount
	at org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:39)
	at org.junit.jupiter.api.Assertions.fail(Assertions.java:117)
	at io.helidon.integrations.oci.metrics.OciMetricsSupportTest.validateMetricCount(OciMetricsSupportTest.java:271)
	at io.helidon.integrations.oci.metrics.OciMetricsSupportTest.validateMetricCount(OciMetricsSupportTest.java:260)
	at io.helidon.integrations.oci.metrics.OciMetricsSupportTest.testMetricScope(OciMetricsSupportTest.java:158)

@barchetta Please look at dependencies/pom.xml. I've inlined the contents of the OCI BOM at the bottom of the file.  I had to comment out oci-objectstorage's entry while so doing because we already dependency manage it separately above.  Lastly, I've left in all the redundant <optional>false</optional> elements because copying and pasting these entries is burdensome enough without having to edit them severely with every subsequent upgrade.

Test failures are present on my local machine prior to this PR, so I'll go ahead and undraft it.


<EMAIL_ADDRESS>

Hello @Vertexwahn, modules you maintain (fmt) have been updated in this PR. Please review the changes.


Fix documentation of dynosaur conversion

Given the current implementation of schemaToCodec as an implicit conversion, you cannot summon a Codec from a Schema (even if you fix the example to make the schema value implicit), so this example demonstrating conversion is what works (in normal use, presumably you would make it implicit val bookCodec: Codec[Book] = bookSchema. I think this PR should be merged and the site updated to work with the current implementation.
Separately, I do wonder if schemaToCodec should not be an implicit conversion, but instead should take the schema argument implicitly: implicit def schemaToCodec[A](implicit schema: Schema[A]): Codec[A] = {..., then something like the original version of this doc would work if you did implicit val bookSchema...

hi @jrduncans, sorry for late reply. You are right, it is a bug. The intended design is to have:
import meteor.dynosaur.formats.conversions._

case class Book(id: Int, content: String)
object Book {
  implicit val schema: Schema[Book] = Schema.record[Book] { field =>
    (
      field("id", _.id),
      field("content", _.content)
    ).mapN(Book.apply)
  }
}

object App {
  val jClient = DynamoDbAsyncClient.create()
  val table =
    SimpleTable[IO, String]("books", KeyDef("id", DynamoDbType.N), jClient)
  val book = Book(1, "hello world")
  table.put[Book](book)
}

such that the Encoder is auto-derived using the implicit from import and having a Schema in scope. Please feel free to raise a PR as you proposed. Thank you.

Ok, opened #228


Step 4 uses incorrect syntax

Current implementation uses incorrect syntax of "POST -v -X". Updated to "curl.exe -v -X POST", which is also in sync with previous steps.

@PatrickFarley
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@PatrickFarley  - can you help us advance this aging PR that's assigned to you?  Should the changes be made or discarded?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
Thanks, PR review team

#sign-off


inference result is unstable

when i do inference use the same input at several times,  the inference result will be error at some times, this is terrible.  I don't know why, and anyone can help me? Thank you.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Add Indicator and option in manifest for Beta extensions/themes

Would be good for dribblish as the text is a bit much

You can just use a "beta" tag for this purpose.


fix issue554, a build error under win10 environment

Closes #554

Coverage remained the same at 97.727% when pulling aad6dd6a3c455564d7bd99c5a0270c9cbb400db6 on Omega-Ariston:fix-issue544 into 8fc5b926846cd4ef9a2dbccd411705e0c110a708 on Leonidas-from-XIV:master.

Problem fixed. thanks  👍


Wrap select HTML elements for easier styling

Came across the same problem as in #30.

Will release in 1.4.2, thank you!


Option name and value are reversed

The sample code in README.md shows how the displayed option string and the corresponding answer string can be different.  But running the example seems to have the two strings reversed.
For instance:
    {
      key: 'a',
      name: 'Overwrite this one and all next',
      value: 'overwrite_all'
    },

I would expect this to display as Overwrite this one and all next.  But here is what I get when I run that sample:
? What action should be taken on file.js?  (yadXh)
  overwrite
  overwrite_all
  diff
 ────────
  abort
  Help
  Answer:

Your issues are awesome! thank you. Well written and very clear, which makes it easy to act on and fix the problems described. again, thank you! I'm on it

No problem.  This one I thought I might have miscategorized.  I went digging and I think part the issue is actually all the way down in prompt-choices, but I don't understand enough of the layers to trace it properly.
In general it looks like value is initialized from name, but then their usage gets muddled, e.g. in Choice.prototype.render:
Choice.prototype.render = function(idx, options) {
 // ...
    return this.value;

And Choice's line property:
Object.defineProperty(Choice.prototype, 'line', {
  // ...
  set: function(line) {
    if (typeof line === 'string') this.value = line;

And up in prompt-checkbox at Checkbox.prototype.renderAnswer:
Checkbox.prototype.renderAnswer = function() {
  var keys = this.choices.checked.map(function(choice) {
    return typeof choice === 'string' ? choice : choice.value;

I'm sure you've got a better grasp than I do of the implications here, but it does look like this particular one is pretty pervasive.

I already have a fix for this I think. Somewhere along the way I guess I removed support for the .format option, which was passed to each choice to format the line.
fwiw, I probably did this because we've been refactoring incrementally to get closer to using a redux-like stateless object for rendering. Soon, when I have enough time, I'll just refactor everything to do that at once. The APIs for Enquirer and prompts shouldn't change much if at all, but it will make it much easier to understand how the prompts work internally, how to customize them, and how to debug for stuff like this.


About the improvement of README

@synesthesiam
First of all, thank you for the detailed introduction to the README. Is there an online version of this software? If there is a link to the online version at the beginning of the README, it can increase the user experience.

I have a link to a demo video now, but there is no "online" version of the software that I host anywhere. Anyone could host a version of the Larynx server, though, and provide access to it.

I watched the demo video and it was very detailed.Thank you very much for your reply.


Add p-m-1 to p-m-5

We have p-1 to p-5 to padding and we have m-1 to m-5 for margin set in Bootsstrap , But Usaly We use both of them . Bootstrap can have new class named p-m-1 to p-m-5 and have both of margin and padding

We'll pass on this—no need to add additional utilities for utilities.


Adding linter to circle CI.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@56d9c03). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #176   +/-   ##
=========================================
  Coverage          ?   66.97%
=========================================
  Files             ?       35
  Lines             ?     2695
  Branches          ?        0
=========================================
  Hits              ?     1805
  Misses            ?      659
  Partials          ?      231

Continue to review full report at Codecov.

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

@michelvocks Hi. :) The changes in here are because of the new linter. :D I had to rename the GRPC struct and added a comment. The renaming affected some files though. I checked, and they shouldn't be used by anything else in Gaia like the SDK for example.

Closes #160


Android - infinite buffering after seek or long watch time

Bug
I am streaming MP4 videos from an S3 bucket. When I try to seek forwards or backwards, after a few times the video enters an infinite buffering state. Even if I try to seek further or backwards, the video position never changes. Sometimes the infinite buffering also happens just after watching a video for a certain amount of time.
The issue is worse on higher quality videos, but I have tried reducing the bitrate with the maxBitrate property with no effect. I have tried altering every available property and none of it seems to make a difference.
I swapped out this package for Expo AV and the issue did not occur
Platform
Which player are you experiencing the problem on:

Android

Environment info
Library version: 6.0.0-beta.3
Device: Samsung S20 - Android 11
Steps To Reproduce

Watch a streamed MP4 video
Seek forwards or backwards by a few seconds at a time OR wait for around 20 seconds of video time

Expected behaviour

Buffering eventually resolves

Code sample
<Video
          style={videoStyle}
          ref={ref}
          source={{ uri: src }}
          resizeMode={resizeMode}
          posterResizeMode={posterResizeMode}
          automaticallyWaitsToMinimizeStalling
          disableFocus
          useTextureView={false}
          playWhenInactive
          paused={isPaused || isSeeking}
          ignoreSilentSwitch={IgnoreSilentSwitchType.IGNORE}
          controls={!!showControls}
          repeat={!!shouldRepeat}
          muted={!!isMuted}
          poster={loadingImage}
          onReadyForDisplay={onReady}
          onLoad={handleLoad}
          onBuffer={(ev) => setBuffering(ev.isBuffering)}
          onError={(err) => handleError(err.error)}
          onEnd={handleFinish}
          progressUpdateInterval={50}
          onProgress={handleProgress}
          bufferConfig={{
            minBufferMs: 2500,
            maxBufferMs: 3000,
            bufferForPlaybackMs: 2500,
            bufferForPlaybackAfterRebufferMs: 2500,
            maxHeapAllocationPercent: 0.5,
          }}
        />

Resolved by increasing the maxBufferMs to longer than the video


Why preferGlobal?

There is both a programmatic API and a command-line API. I think the preferGlobal option really only makes sense when there is nothing to require / import / use in another program. (See also this comment.)
Do we still need / want this or is it just old left-overs?
https://github.com/zemirco/json2csv/blob/1d6e18d0e2eec6486932559c572b3cd7c71c916e/package.json#L3
Using preferGlobal appears to be deprecated now.

Probably legacy.
It's also wrong because it should be a boolean and not a string.
It makes sense to install globally when using json2csv as a CLI tool, and as a local dependency when using it in your code.
I'll remove it!


Create a FAQ page

Description of changes
This page takes the /faq route and displays a FAQ page. Questions are generated from questions.json which is included inline with the faq scene and can be added to and changed dynamically.
Issue Resolved
There was no issue to create an FAQ page, but if there was, this would solve it.
Future Issues

Migrate the json data to the backend
Edit the questions and answers to remove dead links and inaccurate info
Style it and optimize for mobile

I mentioned in Slack that I branched late and didn't realize it. I will fix it up.


Add BIOS 1.08 for all EliteBook 8x0 G4 models

The fix list of this BIOS version is so long, it should have been called 2.0
http://ftp.hp.com/pub/softpaq/sp82001-82500/sp82278.html

Version 3.3.0 includes this. Closing issue.