Various outstanding issues from PR 8084

Problem
The code that landed in #8084 has various unresolved issues, none of which are caught because it is neither compiled nor run in CI:

[x] Is not tested in CI
[x] Probably doesn't compile, since kAnyNodeId is not a thing anymore.
[x] Never shuts down the DeviceCommissioner after calling Init on it.  It should be shut down after RunEventLoop returns.
[x] DeviceCommissioner::Shutdown never frees mUdcServer, so repeated init/shutdown cycles will leak more and more memory.
[x] Once the previous item is fixed, we should probably delete mUdcTransportMgr before deleting mUdcServer, so we don't have a dangling pointer for a bit there, with potential use-after-free.
[ ] UserDirectedCommissioningClient::SendUDCMessage should probably be setting the (required) exchange id in the message header.  Possibly this should just get sent via an actual exchange instead of via a raw call on a transport manager....  @pan-apple any thoughts?

Proposed Solution
Fix issues.

Issues addressed in 8739, assigning to Boris to close, thanks

The last bullet point does not look fixed to me.  That is, this bit:

UserDirectedCommissioningClient::SendUDCMessage should probably be setting the (required) exchange id in the message header. Possibly this should just get sent via an actual exchange instead of via a raw call on a transport manager....

The last bullet still looks to be un-addressed in the code on master.

Requesting secondary triage since it's not clear how important fixing that is.

Do we want our UDC messages to look kind of like what the spec says?
That said, I don't know what the practical impact of sending every single UDC message with an exchange id of 0 is.

These UDC messages are fire-and-forget with no accommodation for responses so I'm not sure what the value in setting a specific exchange id would be.

The main idea of setting it would be so the other side doesn't consider them as part of an existing exchange.  Exchange id 0 is valid, last I checked, so there is potential for confusion and incorrect dispatch there....

But maybe the fact that these are not operational messages and hence the source node id is not present means that does not matter in practice?
In that case, what we have is technically a spec violation (@turon can you please check whether that claim is true?) but might not matter in practice.

SDK Spec Review: We do not believe this is a spec conformance issue. Second, we believe this also complete. Closing.


Clean up yarn install warnings

Closing in favor of #196. Warnings are relevant only to outdated tslint


imporve refactor runtime_filter_mgr with bloomfilter and fix bug in change_to_bloom_filter

Proposed changes

Reduced the granularity of the lock. In the past, the entire map was locked
map(string) --> map(int)
The bf does not need to init_with_fixed_length
init_with_fixed_length in change_to_bloom_filter

pick from https://github.com/apache/doris/pull/21715
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 53.27 seconds
stream load tsv:          534 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size:<PHONE_NUMBER>3 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230713144826_clickbench_pr_177888.html


Require changes to run parallel builds in TC

so exciting!

cc @mgummelt

greping <IP_ADDRESS> looks like it's also used in tests/unit/data/node.txt, tests/fixtures/node.py, tests/fixtures/service.py

Those shouldn't be a problem.

Ping @mgummelt @tamarrow . PR is ready for your review!!

LGTM.  Thanks for fixing


Replace -r with --recipe-url

Otherwise Chef complains with:
WARN: -r MUST be changed to --recipe-url, the -r option will be changed in Chef 13.0 at
[...]/chef-12.9.41/lib/chef/application.rb:56:in `run'

@dividedmind what's missing here is a changelog entry


Add Viewport in usecase.

Viewport sample in https://github.com/kaixinjxq/demo-express/tree/master/samples/Viewport
New pull request on demo-express is https://github.com/crosswalk-project/demo-express/pull/147
BUG=https://crosswalk-project.org/jira/browse/XWALK-7312

LGTM

This doesn't fix anything.  The bug should not have been closed.


squares

When first loading, I will see squares with X's through them. Is this an issue with delayed font loading?

Update Possibly related to https://github.com/material-foundation/google-fonts-flutter/issues/151

It's a flutter font loading issue. If you have tips I would appreciate it. Maybe it is just slow in Debug?

It's only in Chrome, MacOS looks good. Just flutter wasm slowness I guess.


Add CloudSQL backup/restore to incident playbooks

The purpose is to have all documentation in one place during the incident.

What's the intended course of action when the database is unable to be backed up (e.g. due to corruption) and the most recent backup is older than 1 hour?

What's the intended course of action when the database is unable to be backed up (e.g. due to corruption) and the most recent backup is older than 1 hour?

If we cannot accept to restore to the last backup (b/c it's too old), then imho we would have to perform point in time recovery - but this is way harder, as we need to switch applications to use new instance.
@daxmc99 @danieldides what is your suggestion here ^^ ?

Honestly, it depends on the circumstances. In the vast majority of cases, I expect we would choose the data loss over needing to reconfigure the application and the time a point-in-time recovery might need.
Number 1 use-case here is to replicate an incident exactly. Database state before something occurred.


Set the default GKE cluster type for ray to GKE Autopilot.

Also add instructions to use a standard cluster if preferred.

/gcbrun


Document that transactions may be lost in the event of unclean shutdown

This is not standard behavior for databases, although it makes sense in the context of ParityDB, where lost data can be restored from other nodes.

Documented here: https://github.com/paritytech/parity-db/blob/master/src/options.rs#L31
Most embedded key-value stores, such as LevelDB or RocksDB also have fsync-durability off by default.


The buttons don't work

Describe your bug here.
Help, my psych engine doesn't work, the buttons just press and don't perform their functions
Crash Log or logcat logs (if existing)
No response
Are you modding a build from source or with Lua?
I am not modding
What is your build target?
Android
Which psych engine version you're on?
1.0 (Final)
Did you edit anything in this build? If so, mention or summarize your changes.
No

fully reinstall


Display hypervisor type for VM snapshot

Description

The "hypervisor" field in listvmsnapshot response will
be used in primate to enable/disable creating a snapshot
from VM snapshot functionality.
Creating snapshot from VM snapshot will be enabled only if
the hypervisor is KVM
Types of changes

[X] Enhancement (improves an existing feature and functionality)

Screenshots (if appropriate):
How Has This Been Tested?

Cloudmonkey output
(local) 🐵 > list vmsnapshot
{
  "count": 2,
  "vmSnapshot": [
    {
      "account": "admin",
      "created": "2020-08-18T11:12:11+0000",
      "current": true,
      "displayname": "i-2-5-VM_VS_20200818111211",
      "domain": "ROOT",
      "domainid": "f47-c10d-11ea-8715-063381",
      "hypervisor": "KVM",
      "id": "d34f9-df5a-4510-985a-e7f508ad",
      "name": "i-2-5-VM_VS_20200818111211",
      "state": "Ready",
      "tags": [],
      "type": "DiskAndMemory",
      "virtualmachineid": "52cfd-e8d6-4c85-a7e0-f603f151",
      "zoneid": "d0802-be7b-4345-8b55-1612c0b"
    },
    {
      "account": "admin",
      "created": "2020-08-17T15:48:08+0000",
      "current": true,
      "description": "second",
      "displayname": "second",
      "domain": "ROOT",
      "domainid": "f4702-c10d-11ea-8715-0623381",
      "hypervisor": "KVM",
      "id": "c953-0836-4d75-83a7-f805f8ad",
      "name": "i-2-8-VM_VS_20200817154808",
      "state": "Ready",
      "tags": [],
      "type": "DiskAndMemory",
      "virtualmachineid": "3c34-99d2-491c-b1a1-228ec87cb",
      "zoneid": "d0802-be7b-4345-8b55-1612fe0b"
    }
  ]
}

@rhtyd @davidjumani Please review

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✖centos8 ✔debian. JID-1787

(centos8 failure is ignorable as 4.14 does not support centos8 pkg support)

Explicit smoketest is not needed as fix is presentation on API layer and we can merge as soon as Travis passes.


Add option to use ethgasstation.info for gas price estimation

Hi!
First of all, thanks to the team for creating and developing this amazing project.
I've been running a noether node for almost a month now and I've suffered several failed transaction due to high volatility/fluctuation of Ethereum gas price.
During those hiccups the gas price set by Infura would be too low for transactions to be processed quickly enough by Ethereum network to fit in the block producing eligibility window.
Some transactions (not linked for privacy but can do through private channel if needed) would take up to an hour to get validated by Ethereum network and, once approved, would be refused by the contract for producing new blocks (as intended as they were delayed too much).
The solution I am bringing to the code base is to fetch the gas price from ethgasstation.info before sending the transaction. I tested it for several weeks now and found the best way to ensure no transaction fails is to use the fast gas price provided by the service, as I got failed transactions with both the cheap and average gas price profiles.
The API URL is configurable (GAS_STATION_API_URL) so that a similar services or privately hosted stations could be used.
A timeout (GAS_STATION_API_REQUEST_TIMEOUT_MS), set to 10 seconds by default, ensures that the node produces new blocks in case the API becomes unreachable.
The whole "fetch gas from gas station" feature can also be disabled in the config (GAS_STATION_API_ENABLED).
The GAS_MULTIPLIER config has been split into two different constants:

GAS_LIMIT_MULTIPLIER
and GAS_PRICE_MULTIPLIER

; so that the gas price multiplier could be configured independently from the gas limit multiplier.
The gas price multiplier is used when the gas station feature is disabled or when the API is unreachable.
The gas station profile (fast by default) is also configurable (GAS_STATION_API_PROFILE).
The gas price is fetched only for the GAS_STATION_API_CHAIN_ID (1 by default). So for other chains it uses Infura gas price along with the GAS_PRICE_MULTIPLIER.
The developped code is tested with mocha so I had to add some new dev dependencies:

mocha, chai, sinon for testing ;
moxios to easily mock axios ;
waffle to mock the provider.

I had to add git in the Dockerfile because some dependencies of the new packages can only be fetched through git.
I also added a tsconfig.prod.json to prevent building test files in the production build.
If this PR gets accepted we could in the future add those tests to the CI pipeline.
I hope my explanations were clear enough, please tell me if you have any question or you would like to see more improvement to the work I've done. Thanks!

Hello @alexandre-abrioux ! Thank you very much for your PR.
A better gas price estimator is a really nice addition to noether.
Testing with mocha is very welcome too.
The basic algorithm of using the fast predictor, and failing back to the provider gas is good.
I'd like to suggest a few changes to the code structure, which I believe will help maintenance in the future.
Consider the following idea:

create a typescript interface called GasPriceProvider.

interface GasPriceProvider {
    getGasPrice(): Promise<BigNumber>;
}

create a class implementation of that interface called ProviderGasPriceProvider. Its constructor receives a ethers provider and the multiplication factor.

create an alternate class implementation called GasStationGasPriceProvider that does the job of talking to the gas station API.

create a class implementation that tries one GasPriceProvider  and falls back to another GasPriceProvider  in case the first one fails, and wire the two together.

Let me know what you think about this design, and if you need more clarification or help with the code.
Thanks again

Hello @tuler, thanks for your review, this makes sense! I'll find some time during the week to try and implement those changes and I'll get back to you :)

@tuler Just finished implementing the requested changes, it feels much cleaner now! Tell me what you think about it.
I'm testing the new code on my production node, did not get any error so far. I'm gonna wait for a few blocks to get produced and I will get back to you if needed :)

Added the copyright headers!

Hello @alexandre-abrioux , as described in our contributing guidelines I'd like to ask you to sign the contributing agreement that is there and send over to<EMAIL_ADDRESS>Thank you again!

@tuler I've filled up the Google Form "Enter E-mail Below to Request DocuSign". Is that OK? I haven't received anything yet.

@tuler I've filled up the Google Form "Enter E-mail Below to Request DocuSign". Is that OK? I haven't received anything yet.

That's it. We'll send you the doc soon. Thank you!

OK thanks :)

Hi again! I don't know if you got back the signed document from DocuSign, please tell me if I can help.

Hey @alexandre-abrioux , we got it, thanks!
We are doing final tests to prepare the release.

Your contribution has been integrated to the codebase through a rebase along with other changes.
I'll close this PR (instead of merging it.
Thank you again.

Nice! Thanks.


Facebook

How can i havck a facbook

There are different ways to search on YouTube, my suggestion is
https://youtu.be/V99i8xI-QoQ
در تاریخ چهارشنبه ۶ مهٔ ۲۰۲۰، ۰:۴۱ minuku10<EMAIL_ADDRESS>نوشت:

How can i havck a facbook
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chinoogawa/fbht/issues/23#issuecomment-624280383, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AI3TIXMSITYAU4PQ7Z6DGOLRQBXGLANCNFSM4MVJXCZQ
.

How can i havck a facbook

Hello, This is Shohag. I am a professional Ethical Hacker,
Security Specialist as well as Penetration Tester. I can test the following vulnerability and I will provide you with a perfect assessment of your website security
which makes your website safe and sound.
My Experience:
💗. Recovering Hacked WordPress Website
💗. Full Vulnerability Scan and WordPress website malware removal
💗. Vulnerability Assessment
💗. Antispam protection
💗. Analysis of database servers
💗. Restore a hacked website
💗. Ethical Hacking(White Hat Hacker)
💗. Hacked account recovery
💗. Password recovery
💗. Decode encrypted files
💗. Vulnerability Testing
💗....my  WhatsApp number +8801314956025


Trunc shader function is breaking on Android with GLES2

Godot version
3.4.stable
System information
Android 10, GLES2, PowerVR Rogue GE8322
Issue description
Had a major issue with the 'trunc()' function on Android with the GLES2 renderer, where the CanvasShaderGLES2 gives me the following error:
   at: _display_error_with_code (drivers/gles2/shader_gles2.cpp:126) - CanvasShaderGLES2: Vertex shader compilation failed:
Compile failed.
ERROR: 0:282: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
ERROR: 0:286: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
ERROR: 0:286: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
ERROR: 0:290: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
ERROR: 0:290: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
ERROR: 0:290: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
ERROR: 0:294: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
ERROR: 0:294: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
ERROR: 0:294: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
ERROR: 0:294: '<' : Wrong operand types. No operation '<' exists that takes a left-hand operand of type 'in invalid float' and a right operand of type 'const int' (and there is no acceptable conversion)
10 compilation errors. No code generated.

From reading the log file that my game created, we can see the GLSL code before compiling and where the error is happening, turns out that the 'trunc' function is breaking when using it inside GLES2. The function compares an int & a float but that isnt compatible inside said renderer.
The 'trunc()' funtion in GLSL:
 highp float trunc(highp float x) {
 	return x < 0 ? -floor(-x) : floor(x);
 }

 highp vec2 trunc(highp vec2 x) {
 	return vec2(x.x < 0 ? -floor(-x.x) : floor(x.x), x.y < 0 ? -floor(-x.y) : floor(x.y));
 }

highp vec3 trunc(highp vec3 x) {
 	return vec3(x.x < 0 ? -floor(-x.x) : floor(x.x), x.y < 0 ? -floor(-x.y) : floor(x.y), x.z < 0 ? -floor(-x.z) : floor(x.z));
 }

 highp vec4 trunc(highp vec4 x) {
 	return vec4(x.x < 0 ? -floor(-x.x) : floor(x.x), x.y < 0 ? -floor(-x.y) : floor(x.y), x.z < 0 ? -floor(-x.z) : floor(x.z), x.w < 0 ? -floor(-x.w) : floor(x.w));
 }

Possible solution:
Change every '0' inside all the 'trunc()' functions to '0.0', i did a clone of one of those functions with that change and it works fine! I don't know why inside the built-in function isn't like that, maybe it has something to do with compatibility(?), but I'm not sure, if someone know's why, please answer so we can fix this issue!
Steps to reproduce
1.Create a GLES2 project with an Android preset inside the export window.
2.Create a new Control node & a Label node as it's child. Type something in the label node's text variable.
3.Add a shader material to that Label node.
4.In the shader inside the material, write:
shader_type canvas_item;

void fragment(){
	COLOR.rbg = trunc(texture(TEXTURE, UV).rgb);
}

The text should apear with a high contrast.
5.Export that to android (you can use One-Click Deploy if you want to) with Debug/Deploy with remote debug turned on.
Done! It will apear nothing in the screen & there will be some errors on both the Debugger & Output tabs!
Minimal reproduction project
MOBD UI FX TEST.zip

Change every '0' inside all the 'trunc()' functions to '0.0',

This is the right solution. Please go ahead and open a PR with your changes! :)

This is the right solution. Please go ahead and open a PR with your changes! :)

I tried searching for the file where this GLSL code is located but it's nowhere to be found....

Here you go:
https://github.com/godotengine/godot/blob/e937963007ed005e2564a8b574f7437fed173193/drivers/gles2/shaders/stdlib.glsl#L259-L277

Reopening, as this bug isn't fixed until https://github.com/godotengine/godot/pull/56061 is merged.


make clean not clean depends/include/bpf_helper_defs_ext.h

What happened:
When using enhanced kernel, when i run make build the depends/include/bpf_helper_defs_ext.h will be modified. But after make clean, this file is not recovered.

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Kmesh version:
Others:

/assign, should optimize make clean

should optimize make clean


Use the signature_if_final method for ALIndividuals in generated attachment blocks

Fix #327

This does use the right method name even if original PR + the commit used the wrong one

Good call, yes. I'll change before merging
On Tue, Apr 13, 2021, 5:08 PM Bryce Willey @.***> wrote:

@BryceStevenWilley approved this pull request.
LGTM, but could be even more straightforward
In docassemble/assemblylinewizard/interview_generator.py
https://github.com/SuffolkLITLab/docassemble-assemblylinewizard/pull/345#discussion_r612777691
:

@@ -409,8 +409,12 @@ def attachment_yaml(self):
elif hasattr(self, 'field_type') and self.field_type == 'number':
format_str += r'${{ "{{:,.2f}}".format(' + self.variable + ') }}\n'
elif self.field_type_guess == 'signature':

 comment = "      # It's a signature: test which file version this is; leave empty unless it's the final version)\n"

 format_str = comment + format_str + '${{ ' + self.final_display_var + " if i == 'final' else '' }}\n"

 if self.final_display_var.endswith('].signature'): # This is an ALIndividual

   # We don't need a comment with this more explanatory method name

   format_str += '${{ ' + self.final_display_var[:-9] + 'signature_if_final(i) }}\n'

If it ends with ].signature, you can just do self.final_display_var +
'_if_final(i), right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SuffolkLITLab/docassemble-assemblylinewizard/pull/345#pullrequestreview-635027661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2KTSORV33IGGPYWZ6SRGDTISXGBANCNFSM42373ROA
.


Allow external host tags from agent checks to be configurable

Some agent checks (vSphere, Ambari, Cisco ACI and OpenStack) collect metrics for external hosts from a single machine. Those checks also submit specific host tags for those external hosts but the agent only submit them to the Datadog backend once every 10min.
Because of this, in the case of vSphere, when a VM is migrated from host A to host B, the datadog backend will still apply the tags from host A to the VM. This PR makes the submission rate configurable to help customers decrease the length of the discrepancy.

Tested with the following configuration
agent_checks_metadata_refresh_rate: 15

Works as expected:
florianveaux@VM-Ubuntu:~/.omnibus/pkg$ sudo -u dd-agent DD_LOG_LEVEL=debug datadog-agent run | grep 'Sending host metadata payload' | cut -c 1-23
2019-07-31 17:24:38 EDT
2019-07-31 17:25:38 EDT
2019-07-31 17:26:38 EDT
2019-07-31 17:27:38 EDT

FYI https://github.com/DataDog/datadog-agent/pull/3939


Update feature branch Phoenix-6387-4.x from 4.x

@swaroopak can you merge this PR to update the feature branch PHOENIX-6387-4.x from 4.x

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 29s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  3s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 28 new or modified test files.

_ PHOENIX-6387-4.x Compile Tests _

+0 :ok:
mvndep
5m 19s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
9m 29s
PHOENIX-6387-4.x passed

+1 :green_heart:
compile
1m 33s
PHOENIX-6387-4.x passed

+1 :green_heart:
checkstyle
10m 46s
PHOENIX-6387-4.x passed

+1 :green_heart:
javadoc
2m 25s
PHOENIX-6387-4.x passed

+0 :ok:
spotbugs
4m 15s
root in PHOENIX-6387-4.x has 1009 extant spotbugs warnings.

+0 :ok:
spotbugs
2m 57s
phoenix-core in PHOENIX-6387-4.x has 945 extant spotbugs warnings.

+0 :ok:
spotbugs
0m 51s
phoenix-pherf in PHOENIX-6387-4.x has 51 extant spotbugs warnings.

+0 :ok:
spotbugs
0m 42s
phoenix-tools in PHOENIX-6387-4.x has 3 extant spotbugs warnings.

-0 :warning:
patch
1m  9s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 22s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
5m 46s
the patch passed

+1 :green_heart:
compile
1m 33s
the patch passed

+1 :green_heart:
cc
1m 33s
the patch passed

-1 :x:
javac
1m 33s
root generated 1 new + 4 unchanged - 0 fixed = 5 total (was 4)

-1 :x:
checkstyle
10m 49s
root: The patch generated 1681 new + 22489 unchanged - 500 fixed = 24170 total (was 22989)

+1 :green_heart:
prototool
0m  1s
There were no new prototool issues.

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

+1 :green_heart:
shelldocs
0m  2s
There were no new shelldocs issues.

-1 :x:
whitespace
0m  1s
The patch 1 line(s) with tabs.

+1 :green_heart:
xml
0m 14s
The patch has no ill-formed XML file.

-1 :x:
javadoc
0m 49s
phoenix-core generated 1 new + 91 unchanged - 1 fixed = 92 total (was 92)

-1 :x:
javadoc
0m 17s
phoenix-pherf generated 25 new + 33 unchanged - 24 fixed = 58 total (was 57)

-1 :x:
javadoc
1m 15s
root generated 26 new + 124 unchanged - 25 fixed = 150 total (was 149)

-1 :x:
spotbugs
3m 29s
phoenix-core generated 7 new + 943 unchanged - 2 fixed = 950 total (was 945)

-1 :x:
spotbugs
1m 17s
phoenix-pherf generated 12 new + 42 unchanged - 9 fixed = 54 total (was 51)

-1 :x:
spotbugs
1m  2s
phoenix-tools generated 5 new + 0 unchanged - 3 fixed = 5 total (was 3)

-1 :x:
spotbugs
4m 55s
root generated 24 new + 995 unchanged - 14 fixed = 1019 total (was 1009)

_ Other Tests _

-1 :x:
unit
145m 28s
root in the patch failed.

+1 :green_heart:
asflicense
2m 47s
The patch does not generate ASF License warnings.

224m 11s

Reason
Tests

FindBugs
module:phoenix-core

org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.SCHEMA_VERSION_BYTES is a mutable array  At MetaDataEndpointImpl.java: At MetaDataEndpointImpl.java:[line 344]

org.apache.phoenix.mapreduce.index.IndexVerificationResultRepository.SHOULD_RETRY_BYTES is a mutable array  At IndexVerificationResultRepository.java: At IndexVerificationResultRepository.java:[line 62]

Load of known null value in org.apache.phoenix.monitoring.TableMetricsManager.getInstance()  At TableMetricsManager.java:in org.apache.phoenix.monitoring.TableMetricsManager.getInstance()  At TableMetricsManager.java:[line 88]

Write to static field org.apache.phoenix.monitoring.TableMetricsManager.options from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:[line 57]

Write to static field org.apache.phoenix.monitoring.TableMetricsManager.isMetricPublisherEnabled from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:[line 69]

Write to static field org.apache.phoenix.monitoring.TableMetricsManager.isTableLevelMetricsEnabled from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:[line 58]

Write to static field org.apache.phoenix.monitoring.TableMetricsManager.tableClientMetricsMapping from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:[line 61]

FindBugs
module:phoenix-pherf

Possible null pointer dereference in org.apache.phoenix.pherf.workload.mt.handlers.RendezvousingWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at RendezvousingWorkHandler.java:org.apache.phoenix.pherf.workload.mt.handlers.RendezvousingWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at RendezvousingWorkHandler.java:[line 76]

Return value of org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.getPhoenixUtil() ignored, but method has no side effect  At RendezvousingWorkHandler.java:but method has no side effect  At RendezvousingWorkHandler.java:[line 83]

Possible null pointer dereference in org.apache.phoenix.pherf.workload.mt.handlers.TenantOperationWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at TenantOperationWorkHandler.java:org.apache.phoenix.pherf.workload.mt.handlers.TenantOperationWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at TenantOperationWorkHandler.java:[line 65]

Return value of org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.getPhoenixUtil() ignored, but method has no side effect  At TenantOperationWorkHandler.java:but method has no side effect  At TenantOperationWorkHandler.java:[line 67]

input must be non-null but is marked as nullable  At IdleTimeOperationSupplier.java:is marked as nullable  At IdleTimeOperationSupplier.java:[lines 51-74]

input must be non-null but is marked as nullable  At PreScenarioOperationSupplier.java:is marked as nullable  At PreScenarioOperationSupplier.java:[lines 51-85]

A prepared statement is generated from a nonconstant String in org.apache.phoenix.pherf.workload.mt.operations.PreScenarioOperationSupplier$1.apply(TenantOperationInfo)  At PreScenarioOperationSupplier.java:from a nonconstant String in org.apache.phoenix.pherf.workload.mt.operations.PreScenarioOperationSupplier$1.apply(TenantOperationInfo)  At PreScenarioOperationSupplier.java:[line 74]

input must be non-null but is marked as nullable  At QueryOperationSupplier.java:is marked as nullable  At QueryOperationSupplier.java:[lines 54-94]

Impossible cast from RuntimeException to java.sql.SQLException in org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.executePreRunOpsForTenant(TenantOperationFactory$TenantView, TenantOperationInfo)  At TenantOperationFactory.java:java.sql.SQLException in org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.executePreRunOpsForTenant(TenantOperationFactory$TenantView, TenantOperationInfo)  At TenantOperationFactory.java:[line 354]

input must be non-null but is marked as nullable  At UpsertOperationSupplier.java:is marked as nullable  At UpsertOperationSupplier.java:[lines 58-161]

org.apache.phoenix.pherf.workload.mt.operations.UpsertOperationSupplier$1.apply(TenantOperationInfo) may fail to clean up java.sql.Statement  Obligation to clean up resource created at UpsertOperationSupplier.java:up java.sql.Statement  Obligation to clean up resource created at UpsertOperationSupplier.java:[line 107] is not discharged

input must be non-null but is marked as nullable  At UserDefinedOperationSupplier.java:is marked as nullable  At UserDefinedOperationSupplier.java:[lines 43-47]

FindBugs
module:phoenix-tools

Found reliance on default encoding in org.apache.phoenix.schema.SchemaSynthesisProcessor.getQueriesFromFile(String):in org.apache.phoenix.schema.SchemaSynthesisProcessor.getQueriesFromFile(String): new java.io.FileReader(File)  At SchemaSynthesisProcessor.java:[line 193]

org.apache.phoenix.schema.SchemaSynthesisProcessor.getQueriesFromFile(String) may fail to close stream  At SchemaSynthesisProcessor.java:stream  At SchemaSynthesisProcessor.java:[line 193]

org.apache.phoenix.schema.SchemaTool.conf should be package protected  In SchemaTool.java: In SchemaTool.java

Possible null pointer dereference of cmdLine in org.apache.phoenix.schema.SchemaTool.parseOptions(String[])  Dereferenced at SchemaTool.java:cmdLine in org.apache.phoenix.schema.SchemaTool.parseOptions(String[])  Dereferenced at SchemaTool.java:[line 114]

Write to static field org.apache.phoenix.schema.SchemaTool.conf from instance method org.apache.phoenix.schema.SchemaTool.run(String[])  At SchemaTool.java:from instance method org.apache.phoenix.schema.SchemaTool.run(String[])  At SchemaTool.java:[line 73]

FindBugs
module:root

org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.SCHEMA_VERSION_BYTES is a mutable array  At MetaDataEndpointImpl.java: At MetaDataEndpointImpl.java:[line 344]

org.apache.phoenix.mapreduce.index.IndexVerificationResultRepository.SHOULD_RETRY_BYTES is a mutable array  At IndexVerificationResultRepository.java: At IndexVerificationResultRepository.java:[line 62]

Load of known null value in org.apache.phoenix.monitoring.TableMetricsManager.getInstance()  At TableMetricsManager.java:in org.apache.phoenix.monitoring.TableMetricsManager.getInstance()  At TableMetricsManager.java:[line 88]

Write to static field org.apache.phoenix.monitoring.TableMetricsManager.options from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:[line 57]

Write to static field org.apache.phoenix.monitoring.TableMetricsManager.isMetricPublisherEnabled from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:[line 69]

Write to static field org.apache.phoenix.monitoring.TableMetricsManager.isTableLevelMetricsEnabled from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:[line 58]

Write to static field org.apache.phoenix.monitoring.TableMetricsManager.tableClientMetricsMapping from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:from instance method new org.apache.phoenix.monitoring.TableMetricsManager(QueryServicesOptions)  At TableMetricsManager.java:[line 61]

Possible null pointer dereference in org.apache.phoenix.pherf.workload.mt.handlers.RendezvousingWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at RendezvousingWorkHandler.java:org.apache.phoenix.pherf.workload.mt.handlers.RendezvousingWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at RendezvousingWorkHandler.java:[line 76]

Return value of org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.getPhoenixUtil() ignored, but method has no side effect  At RendezvousingWorkHandler.java:but method has no side effect  At RendezvousingWorkHandler.java:[line 83]

Possible null pointer dereference in org.apache.phoenix.pherf.workload.mt.handlers.TenantOperationWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at TenantOperationWorkHandler.java:org.apache.phoenix.pherf.workload.mt.handlers.TenantOperationWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at TenantOperationWorkHandler.java:[line 65]

Return value of org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.getPhoenixUtil() ignored, but method has no side effect  At TenantOperationWorkHandler.java:but method has no side effect  At TenantOperationWorkHandler.java:[line 67]

input must be non-null but is marked as nullable  At IdleTimeOperationSupplier.java:is marked as nullable  At IdleTimeOperationSupplier.java:[lines 51-74]

input must be non-null but is marked as nullable  At PreScenarioOperationSupplier.java:is marked as nullable  At PreScenarioOperationSupplier.java:[lines 51-85]

A prepared statement is generated from a nonconstant String in org.apache.phoenix.pherf.workload.mt.operations.PreScenarioOperationSupplier$1.apply(TenantOperationInfo)  At PreScenarioOperationSupplier.java:from a nonconstant String in org.apache.phoenix.pherf.workload.mt.operations.PreScenarioOperationSupplier$1.apply(TenantOperationInfo)  At PreScenarioOperationSupplier.java:[line 74]

input must be non-null but is marked as nullable  At QueryOperationSupplier.java:is marked as nullable  At QueryOperationSupplier.java:[lines 54-94]

Impossible cast from RuntimeException to java.sql.SQLException in org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.executePreRunOpsForTenant(TenantOperationFactory$TenantView, TenantOperationInfo)  At TenantOperationFactory.java:java.sql.SQLException in org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.executePreRunOpsForTenant(TenantOperationFactory$TenantView, TenantOperationInfo)  At TenantOperationFactory.java:[line 354]

input must be non-null but is marked as nullable  At UpsertOperationSupplier.java:is marked as nullable  At UpsertOperationSupplier.java:[lines 58-161]

org.apache.phoenix.pherf.workload.mt.operations.UpsertOperationSupplier$1.apply(TenantOperationInfo) may fail to clean up java.sql.Statement  Obligation to clean up resource created at UpsertOperationSupplier.java:up java.sql.Statement  Obligation to clean up resource created at UpsertOperationSupplier.java:[line 107] is not discharged

input must be non-null but is marked as nullable  At UserDefinedOperationSupplier.java:is marked as nullable  At UserDefinedOperationSupplier.java:[lines 43-47]

Found reliance on default encoding in org.apache.phoenix.schema.SchemaSynthesisProcessor.getQueriesFromFile(String):in org.apache.phoenix.schema.SchemaSynthesisProcessor.getQueriesFromFile(String): new java.io.FileReader(File)  At SchemaSynthesisProcessor.java:[line 193]

org.apache.phoenix.schema.SchemaSynthesisProcessor.getQueriesFromFile(String) may fail to close stream  At SchemaSynthesisProcessor.java:stream  At SchemaSynthesisProcessor.java:[line 193]

org.apache.phoenix.schema.SchemaTool.conf should be package protected  In SchemaTool.java: In SchemaTool.java

Possible null pointer dereference of cmdLine in org.apache.phoenix.schema.SchemaTool.parseOptions(String[])  Dereferenced at SchemaTool.java:cmdLine in org.apache.phoenix.schema.SchemaTool.parseOptions(String[])  Dereferenced at SchemaTool.java:[line 114]

Write to static field org.apache.phoenix.schema.SchemaTool.conf from instance method org.apache.phoenix.schema.SchemaTool.run(String[])  At SchemaTool.java:from instance method org.apache.phoenix.schema.SchemaTool.run(String[])  At SchemaTool.java:[line 73]

Failed junit tests
phoenix.end2end.index.LocalIndexIT

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1236

Optional Tests
dupname asflicense shellcheck shelldocs javac javadoc unit xml compile spotbugs hbaseanti checkstyle cc prototool

uname
Linux 266bc5800772 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
PHOENIX-6387-4.x / 67620b1

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

javac
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/diff-compile-javac-root.txt

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/diff-checkstyle-root.txt

whitespace
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/whitespace-tabs.txt

javadoc
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/diff-javadoc-javadoc-phoenix-core.txt

javadoc
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/diff-javadoc-javadoc-phoenix-pherf.txt

javadoc
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/diff-javadoc-javadoc-root.txt

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/new-spotbugs-phoenix-pherf.html

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/new-spotbugs-phoenix-tools.html

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/new-spotbugs-root.html

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/artifact/yetus-general-check/output/patch-unit-root.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/testReport/

Max. process+thread count
5739 (vs. ulimit of 30000)

modules
C: phoenix-core phoenix-pherf phoenix-tools . U: .

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1236/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.7.0 spotbugs=4.1.3 prototool=1.10.0-dev

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Relocate C.W.Jormungandr.BlockHeaders to C.W.Network.BlockHeaders

Issue Number
#711
Overview

[x] Relocate Cardano.Wallet.Jormungandr.BlockHeaders to Cardano.Wallet.Network.BlockHeaders. Cardano.Pool.Metrics needs them!

Comments

bors r+

bors r+

bors r+


[GLUTEN-1336][VL] UT: add GlutenAdaptiveQueryExecSuite and GlutenJsonSuite for spark3.3

What changes were proposed in this pull request?
add GlutenAdaptiveQueryExecSuite and GlutenJsonSuite for spark3.3
How was this patch tested?
UT

Run Gluten Clickhouse CI


Guide not found

At the time of this writing the link to the guide leads to a 404 Page Not Found GitHub page

Guide is back to normal. Thank you for submitting this issue! 😄


Update dependencies for Kafka 3.3

closed via https://github.com/miracum/kafka-fhir-serializer/pull/15


Segmentation violation in FitDiagnostics when not using verbose

I have encountered a reproducible segmentation violation when running FitDiagnostics without the verbose option (at least -v 3). I obtained this stack trace from gdb:
#0  0x0000003db4443ee9 in vfprintf () from /lib64/libc.so.6
#1  0x0000003db444efc8 in fprintf () from /lib64/libc.so.6
#2  0x00007ffff7df53a9 in utils::anyParameterAtBoundaries (params=..., verbosity=verbosity@entry=0)
    at /uscms_data/d3/pedrok/SUSY2015/analysis/CMSSW_10_2_13/src/HiggsAnalysis/CombinedLimit/src/utils.cc:977
#3  0x00007ffff7d613ff in CascadeMinimizer::minimize (this=this@entry=0x7fffffff03a0, verbose=0, cascade=cascade@entry=true)
    at /uscms_data/d3/pedrok/SUSY2015/analysis/CMSSW_10_2_13/src/HiggsAnalysis/CombinedLimit/src/CascadeMinimizer.cc:473
#4  0x00007ffff7d3e313 in FitterAlgoBase::doFit (this=this@entry=0x568640, pdf=..., data=..., rs=..., constrain=..., doHesse=doHesse@entry=true, ndim=1,
    reuseNLL=true, saveFitResult=true) at /uscms_data/d3/pedrok/SUSY2015/analysis/CMSSW_10_2_13/src/HiggsAnalysis/CombinedLimit/src/FitterAlgoBase.cc:231
#5  0x00007ffff7dd48a8 in FitDiagnostics::runSpecific (this=0x568640, w=0x29bf690, mc_s=0x5fa4e70, mc_b=0x2ee5330, data=...,
    limit=@0x7fffffff5100: 0.75691106323917268, limitErr=@0x7fffffff5108: 0, hint=0x0)
    at /uscms_data/d3/pedrok/SUSY2015/analysis/CMSSW_10_2_13/src/HiggsAnalysis/CombinedLimit/src/FitDiagnostics.cc:259
#6  0x00007ffff7d3d70c in FitterAlgoBase::run (this=0x568640, w=0x29bf690, mc_s=0x5fa4e70, mc_b=0x2ee5330, data=...,
    limit=@0x7fffffff5100: 0.75691106323917268, limitErr=@0x7fffffff5108: 0, hint=0x0)
    at /uscms_data/d3/pedrok/SUSY2015/analysis/CMSSW_10_2_13/src/HiggsAnalysis/CombinedLimit/src/FitterAlgoBase.cc:194
#7  0x00007ffff7e6f330 in Combine::mklimit (this=this@entry=0x7fffffff5820, w=w@entry=0x29bf690, mc_s=mc_s@entry=0x5fa4e70, mc_b=mc_b@entry=0x2ee5330,
    data=..., limit=@0x7fffffff5100: 0.75691106323917268, limitErr=<optimized out>)
    at /uscms_data/d3/pedrok/SUSY2015/analysis/CMSSW_10_2_13/src/HiggsAnalysis/CombinedLimit/src/Combine.cc:227
#8  0x00007ffff7e75062 in Combine::run (this=this@entry=0x7fffffff5820, hlfFile=<incomplete type>, dataset="data_obs",
    limit=@0x7fffffff5100: 0.75691106323917268, limitErr=@0x7fffffff5108: 0, iToy=@0x7fffffff50ec: 35, tree=0x2318990, nToys=<optimized out>)
    at /uscms_data/d3/pedrok/SUSY2015/analysis/CMSSW_10_2_13/src/HiggsAnalysis/CombinedLimit/src/Combine.cc:1096
#9  0x000000000041063c in main (argc=<optimized out>, argv=<optimized out>)
    at /uscms_data/d3/pedrok/SUSY2015/analysis/CMSSW_10_2_13/src/HiggsAnalysis/CombinedLimit/bin/combine.cpp:315

The version of combine is v8.0.1.
I believe the underlying cause of this error is similar to #281. However, while that issue was fixed, there is another line where dup() may be used too frequently:
https://github.com/cms-analysis/HiggsAnalysis-CombinedLimit/blob/0e793441b6e1e1443805ca44ad412a8c95f4b9a8/src/CloseCoutSentry.cc#L85
I tried protecting this line like:
if(fdOutDup_ == 0) fdOutDup_ = dup( fdOut_ );

but this caused some other file access issues, so I am not sure of the appropriate fix.

@kpedro88 This may be too long ago now, but I think this issue should be solved by #651 . Are you by any chance able to confirm that?

@kcormi unfortunately this is too long ago for me to recall what datacard produced the crash. It looks like #651 addresses the apparent cause of the problem, so I think it's okay to close this issue, and reopen or make a new issue if the problem recurs in the future.


WIP: ngclient constants and configuration

Fixes #1402
Description of the changes being introduced by the pull request:
Adds a new public config module ngclient/config.py.

containing a dataclass UpdaterConfig with all client settings
Initializes updater with the default settings if no other config is provided.

Please verify and check that the pull request fulfills the following
requirements:

[ ] The code follows the Code Style Guidelines
[ ] Tests have been added for the bug fix or new feature
[ ] Docs have been added for the bug fix or new feature

Rebased on develop.
Also reworked all commits after the review suggestions so another one will be needed.


Causal inference example/tutorial

As requested by Zoubin. Everything works with #445 and #459, just need to fill in some text.
Not a real introduction to causal inference, only to Pyro's intervention mechanism pyro.do by way of a disease diagnosis example.

I also suggest we make sure whenever this is merged that the documentation in advanced features also exists for the do-poutine, currently this is not the case.

Great job!  A few comments:
The figure demonstrating that curing the cough doesn't cure the cold might throw people off.  The suggestion that someone might have a cold conflicts with the apparent low probabilities in the figure, and the reader has to backtrack to see the probability of a cold was set to set to low value of .02.  It might be a bit clearer if you also condition on having a fever so the probability of a cold is higher.
If you do end up making a causal inference tutorial, I suggest including a bandit algorithm (eg Bareinboim and Pearl 2015 or Lattimore and Reid.  I find that bandits and other sequential decision-making methods are good motivating examples, relative to the canonical Bayes net examples.

I know I'm late to the game, but please bear with me - why did this never get merged? Is this example outdated now?

I know I'm late to the game, but please bear with me - why did this never get merged? Is this example outdated now?

@talesa this code is pretty old and uses a bunch of obsolete APIs. If you're looking for more up to date examples of this sort, see @robertness's lecture notes here: https://github.com/robertness/causalML

Thank you @eb8680 @robertness 🙏


Error with the configuration of the plugin Motion

Describe the bug
Hello,
An error appears when I try to set the plugin @capacitor/motion.
To reproduce
Steps to reproduce the behavior:

Install an blank ionic project + add www and ìos`
Install the plugin using these commands: npm install @capacitor/motion npx cap sync
Open the file app.component.ts
Add the imports on the top of the file
In the function ngOnInit(), add the method to request the permission `await DeviceMotionEvent.requestPermission();``
See the error (red underline below requestPermission): Property 'requestPermission' does not exist on type '{ new (type: string, eventInitDict?: DeviceMotionEventInit): DeviceMotionEvent; prototype: DeviceMotionEvent; }'

Screenshot

Expected behavior
Don't have any error when I try to setup the plugin following the instructions.
Browser and OS (please complete the following information)

OS: MacOs Monterey 12.6
Browser: Chrome
Version: 105.0.5195.125

Thank you for your help,
Loïc

Hello,
I notice that this question was asked in the Ionic forum here.
So it looks like this is not a recent issue. Could you please help us to use this plugin with Ionic?
Thanks,
Loïc

PR to improve this: https://github.com/ionic-team/capacitor-plugins/pull/1614
Essentially you want to call:
if ((DeviceMotionEvent as any).requestPermission) {
        const result = await (DeviceMotionEvent as any).requestPermission();
        // This should return a result of 'granted'
}

The DeviceMotionEvent.requestPermission API is available on some Safari/Mobile Safari/iOS versions so you need to check if the API exists before calling it.


add content filtering to the website content listing page

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots
[x] Mobile width screenshots

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
for  #1020
What's this PR do?
this PR implements a basic sort UI on the content listing pages. I pulled out the logic I wrote for the same type of sort on the /sites page into a hook and then added it to the content page, so it works the same way.

Screenshots (if appropriate)

Codecov Report

Merging #1046 (690637e) into master (934657b) will decrease coverage by 2.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1046      +/-   ##
==========================================
- Coverage   91.10%   88.92%   -2.18%
==========================================
  Files         200      104      -96
  Lines        7652     5029    -2623
  Branches     1333      784     -549
==========================================
- Hits         6971     4472    -2499
+ Misses        596      472     -124
  Partials       85       85

Impacted Files
Coverage Δ

static/js/components/RepeatableContentListing.tsx

static/js/pages/SitesDashboard.tsx

static/js/test_util.ts

static/js/lib/redux_query.ts

static/js/util/dom.ts

static/js/components/Header.tsx

static/js/lib/urls.ts

static/js/types/websites.ts

static/js/context/Website.ts

static/js/components/SiteCollaboratorDrawer.tsx

... and 86 more

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 934657b...690637e. Read the comment docs.

why just "Pages" ?

@Ferdi it's not just present on pages, I just neglected to screenshot them :)
Here's what it looks like elsewhere:


Add windows supprot (arm)

Target architectures:

[ ] aarch64
[ ] armv7

Will be a 0.2.0 update, because containers will need to be reworked off of linux-libc.base


Cannot use module if the Control Tower & AFT accounts are in the same region

If when installing AFT for the first time and you terraform apply with the configuration options:
ct_home_region              = "us-east-1"
tf_backend_secondary_region = "us-east-1"

the apply fails because the regions are the same, and regions specified in resources within the module conflict with rules from the AWS provider.  This module should not fail if these regions are the same or at least there should be some way to handle this.
╷
│ Error: error creating Lambda Event Source Mapping (arn:aws:dynamodb:us-east-1:000000000000:table/aft-request/stream/2022-01-27T00:36:16.589): InvalidParameterValueException: Cannot access stream arn:aws:dynamodb:us-east-1:000000000000:table/aft-request/stream/2022-01-27T00:36:16.589. Please ensure the role can perform the GetRecords, GetShardIterator, DescribeStream, ListShards, and ListStreams Actions on your stream in IAM.
│ {
│   RespMetadata: {
│     StatusCode: 400,
│     RequestID: "26e643d1-bd4d-4f74-9302-d1c455b9f714"
│   },
│   Message_: "Cannot access stream arn:aws:dynamodb:us-east-1:000000000000:table/aft-request/stream/2022-01-27T00:36:16.589. Please ensure the role can perform the GetRecords, GetShardIterator, DescribeStream, ListShards, and ListStreams Actions on your stream in IAM.",
│   Type: "User"
│ }
│
│   with module.aft.module.aft_account_request_framework.aws_lambda_event_source_mapping.aft_account_request_action_trigger,
│   on .terraform/modules/aft/modules/aft-account-request-framework/lambda.tf line 75, in resource "aws_lambda_event_source_mapping" "aft_account_request_action_trigger":
│   75: resource "aws_lambda_event_source_mapping" "aft_account_request_action_trigger" {
│
╵
╷
│ Error: error initially creating DynamoDB Table (aft-backend-000000000000) replicas: error creating DynamoDB Table (aft-backend-000000000000) replica (us-east-1): ValidationException: Cannot add, delete, or update the local region through ReplicaUpdates. Use CreateTable, DeleteTable, or UpdateTable as required.
│       status code: 400, request id: EF57U9342DNOBCGGLVKI0JSRIJVV4KQNSO5AEMVJF66Q9ASUAAJG
│
│   with module.aft.module.aft_backend.aws_dynamodb_table.lock-table,
│   on .terraform/modules/aft/modules/aft-backend/main.tf line 221, in resource "aws_dynamodb_table" "lock-table":
│  221: resource "aws_dynamodb_table" "lock-table" {
│
╵
╷
│ Error: error creating KMS Alias (alias/aft-backend-000000000000-kms-key): AlreadyExistsException: An alias with the name arn:aws:kms:us-east-1:000000000000:alias/aft-backend-000000000000-kms-key already exists
│
│   with module.aft.module.aft_backend.aws_kms_alias.encrypt-alias-secondary-region,
│   on .terraform/modules/aft/modules/aft-backend/main.tf line 275, in resource "aws_kms_alias" "encrypt-alias-secondary-region":
│  275: resource "aws_kms_alias" "encrypt-alias-secondary-region" {
│
╵

Thanks for bringing this up, I've gone ahead and added this feature to our backlog. I'm going to go ahead and resolve this issue in the mean time.

Hi @balltrev. Is your backlog of work visible anywhere? If not, is this expected to be added in the near future? Do you accept PRs for us to add the functionality?

I've re-opened this for tracking purposes. At this time we're not accepting pull requests.

Hi @witchbutter,
AFT 1.10.0 introduced single-region deployments (by omitting the secondary region, which is now optional). I believe that meets the scope of this request, so I'm going to close this issue as resolved.
Please feel free to open a new issue if there is additional scope to your feature request. Thanks!


Locales 404 responses for other languages than english

Use Case (delete as appropriate): Using Portainer at Home
Have you reviewed our technical documentation and knowledge base? Yes

Question:
Hello, I have an issue with portainer in docker container behind a reverse proxy (nginx-proxy-manager) on my raspberry
In fact, at every reload of the portainer page, logged in or not, my french browser will try to fetch /locales/fr/translation.json and /locales/fr-FR/translation.json files that aren't present in the container and it send me back 404 responses after finally finding the default /locales/en/translation.json file.

Is there any way to add a custom translation file or forcing the loading of english local file directly without sending 404 responses ?
It is quite annoying because I have a fail2ban rule who ban after to many 404 responses.
I've tried to add LANGUAGE, LC_ALL and LANG to my docker compose file for portainer but it doesn't work

I've tried to search into the docker folder tree but didn't found the location of locale files

Found that this is i18next nodejs module who is in charge to load these files, does it is related ?

Thanks in advance.

@moniika21
I wanted to follow up on this request.  This is a result of ongoing work as part of the transition from Angular to React, and is not fully functional at this stage.
Thanks!

I have the same issue. In the default installation the containers view crashes because the translations,json returns 404. Is there any workaround?


Can this be used on a grid of Div elements

I am using Foundation and have a grid of staff. Each staff member has a .memberitem class. Can I use HideSeek to filter that structure? Any examples?

You can use hideseek for any container which contains list of items. If you want to use specific search string collection use "attribute" option. Please see other my answers for details e.g.: https://github.com/vdw/HideSeek/issues/52


add_script_s3_bucket_deletion

The main purpose of this script is to delete a bucket associated with a cluster
after the cluster is uninstalled

/verified

Moved https://github.com/RedHatQE/cloud-tools/pull/4


Update comment form interactions.

Highlight form tab on select
Show text input on dependent input when no values

[Resolves https://github.com/eregs/notice-and-comment/issues/272]

Functionally looks good to me. 👍

Please don't merge until https://github.com/eregs/notice-and-comment/pull/278#issuecomment-218905280 is resolved

My head's on straight now. Good to go!

Mind adding some unit tests around this?


fix(ui): removing aria-labelledby from payment methods component

What's the purpose of this pull request?
The payment methods component was returning an accessibility error on storybook:

To solve it, aria-labelledby was removed.

It isn't a problem to leave the payment method div ID. Overall LGTM

Maybe this won't work either :/
Can't we use section tag instead of div for data-payment-methods-flags ?
and change
{title}
 for
{title}


Add leaf group "erasure" function

This is needed to fix tests in https://github.com/rjl493456442/go-ethereum/pull/12 : Gary's stuff will call DeleteAccount if the account is non-existent. While this is quite unclean and should be fixed at the geth layer, I reckon that it is much more difficult to do than designing the verkle library to handle this.
As a result, I introduce an EraseStem method, which is used to replace InsertAtStem when deleting a function: it will go through the tree, find a group if it exsits, and overwrite all values present in that group. This is less invasive than the current method of calling InsertAtStem with 256 non-zero values.
It has the following drawbacks:

This behavior needs to be specified in the spec, since nil values are not overwritten
It also breaks the replay code, which uses the trick of checking that all values are 0 in order to simulate the "pre-verge" behavior of account deletion.

Fixes #448


:construction_worker: migrate to maven

Fixes drogatkin/mediautil#3
Here comes a simple maven pom.
That builds an executable jar.
I used a pretty old JDK (1.7) because the library is quite old.
This will allow to publish to maven central and favor reuse.

Selection of JDK 1.7 not because it's old. It's because it's required to run on Android including the Amazon flavor which is JDK 7 based. If you need a fresher bytecode, you can always recompile the library with required target.

Ok @drogatkin , my bad :bow:
I changed the Description :wink:
Do you consider merging this one and pusblishing to maven :grimacing: ?

I am not familiar with Maven, otherwise I have no problem to merge.


Update version to 2.10

Update AirVPN to version 2.10

Thank you for the contribution. It was merged directly as f6120ab16ed238ad6ddd60c36450cbcf53d6c910 to keep commit history cleaner. Your contribution is still credited to you.


MacOS Anaconda/Spyder installation.

I have downloaded the newest version of Anaconda and want to use Spyder. I am having issue regarding installation of Spyder. For some reason the spyder hasn't been installed and the icon for is also some generic logo. I can only open Spyder tru Anaconda and when close is it closes Spyder as well. I would appreciate the help. I am new to Python and have no clue what to do and how to check and install everything and see even if the libraries have been installed.

If you can run Spyder then everything was installed correctly.

the icon for is also some generic logo

There's no solution to that problem right now.

I can only open Spyder tru Anaconda and when close is it closes Spyder as well

To avoid that I think you have to open a terminal, and run Spyder like this
spyder &

Then you should be able to close the terminal. If that's not possible, then you need to maintain it open.


Close side menu clicking on another button

Hi all,
I have read these issues, but I didn't find an answer to my question.
I am using this component, that made my day, and I am using this with Router Flux.
So in my App.js as you can see I have the sidemenu and the router:
<ScrollView>
   <SideMenu
    menu={menu}
    isOpen={this.state.isOpen}
    onChange={(isOpen) => this.updateMenuState(isOpen)}
    menuPosition={'right'}
    scrollsToTop={false}
    openMenuOffset={Dimensions.get('window').width * 0.85}
    >
 <View style={{marginTop:20, height: 40, backgroundColor: 'white'}}>
      <Button style={styles.button} onPress={() => this.toggle()}>
        <Icon name="menu" />
      </Button>
    </View>
    <View style={styles.containerNoPad}>
      <Router hideNavBar={true}>
        <Scene key='root'>
          <Scene key='homepage' component={HomePage} title='Homepage' initial={true} />
          ..........
        </Scene>
      </Router>
    </View>
  </SideMenu>
</ScrollView>

Then in Menu.js I have the render() of the menu. I'd like that clicking on a menu item, the menu will be closed automatically with its effect. Unfortunately I am not able to do this. Any help?
This is my menu item:
<View>
  <Button onPress={() =>  {
      Actions.anotherPage({hideNavBar: true, type: 'reset'})
    }}>
    <View>
      <Icon name="paging" />
    </View>
    <View>
      <Text>Another page</Text>
    </View>
  </Button>
</View>

Sorry for the indentation problem.....

Hi @IronTony,
Unfortunately, I don't have enough time to make an example for you, but the general idea would be like this:
When you click on button, emit an action that re-render menu with two props:

isOpen={false} to close the menu
`onChange={processPendingTransition} in order to process your pending transition to the next screen

@Kureev - unfortunately if you want to maintain your state when also using gestures you have to dispatch an action from onChange too, and since changing isOpen causes onChange to be fired, you end up with two actions and possibly an unnecessary render in that case.
It's tricky to work around because onChange is fired before the parent has received the new isOpen state (eg from redux/flux), so it's hard* to detect from the onChange handler whether we're responding to a swipe and should dispatch an action, or responding to props-driven change and have already dispatched an action.
This seemed to be what @grabbou addressed in https://github.com/react-native-community/react-native-side-menu/pull/175 addressed but that's a while out of date now.
* componentWillReceiveProps in the parent provides a hacky workaround - assuming your parent is also props driven, the new prop will appear hear and you can compare with onChange to decide whether to dispatch an action.

@Kureev @rh389 I have an issue with what you said which is related to this. Hopefully you might be able to help, thank you for your time! If you don't mind, allow me to provide a verbose version so I'm clear.
I'm using react-native-side-menu and react-native-hamburger. the side menu is in a file called Nav.js, the hamburger button is used in a separate view, and is in its own file called HamburgerButton.js. I set up a simple observer pattern with the corresponding Actions.js, ActionTypes.js, Dispatcher.js, and Store.js.
I can confirm if I press the hamburger component located inside the homescreen view, it dispatches an event to update the state of "menuState" to true, the same event is listened to by Nav.js which then updates the side menu's isOpen prop. Both states react and update correctly. However, the side menu does not open when its state gets updated.
If you don't mind, here's the source code:
HamburgerButton.js:
'use strict';

import Expo from 'expo';
import React from 'react';
import {
  StyleSheet
} from 'react-native';
import Hamburger from 'react-native-hamburger';
import Actions from '../actions/Actions';
import ActionTypes from '../actions/ActionTypes';
import Store from '../store/Store';

export default class MenuButton extends React.Component {
  constructor(props) {
    super(props);
    // get initial state from Store
    this.state = {
      active: Store.getStore().menuState
    };
    this._onPress = this._onPress.bind(this);
    this._changeState = this._changeState.bind(this);
    console.log('Hamburger this', `${JSON.stringify(this)}`);
  }

  // state changed, get updated state from store
  _changeState() {
    this.state = {
      active: Store.getStore().menuState
    };
    console.log('Hamburger State Updated', `${JSON.stringify(this.state.active)}`);
  }

  // menu icon was pressed, fire action to update state
  _onPress() {
    console.log('Hamburger _onPress called');
    Actions.toggleMenu(!this.state.active);
  }

  // mount event listener that will talk to store and trigger this._changeState
  componentDidMount() {
    console.log('Hamburger componentDidMount called');
    Store.addChangeListener(this._changeState);
  }

  render() {
    return (
      <Hamburger
        active={this.props.active}
        type='spinCross'
        onPress={this._onPress} />
    );
  }
}

Nav.js (has the side menu):
'use strict';

import Expo from 'expo';
import React from 'react';
import {
  SideMenu
} from 'react-native-elements';
import {
  StyleSheet,
  View,
  Text,
  App,
  Dimensions
} from 'react-native';
import HomeScreen from './HomeScreen';
import MenuContent from '../components/MenuContent';
import Actions from '../actions/Actions';
import ActionTypes from '../actions/ActionTypes';
import Store from '../store/Store';

// dimensions
const SCREEN_WIDTH = Dimensions.get('window').width;

export default class Nav extends React.Component {
  constructor (props) {
    super(props);
    this.state = {
      isOpen: Store.getStore().menuState
    };
    this._changeState = this._changeState.bind(this);
    this._onChange = this._onChange.bind(this);
    console.log('Nav this', `${JSON.stringify(this)}`);
  };

  // state changed, get updated state from store
  _changeState() {
    this.state = {
      isOpen: Store.getStore().menuState
    };
    console.log('Nav State Updated', `${JSON.stringify(this.state.isOpen)}`);
  }

  // menu has changed, fire action to update state
  _onChange () {
    console.log('Nav _onChange Called');
    Actions.toggleMenu(!this.state.isOpen);
  }

  // mount event listener that will talk to store and trigger this._changeState
  componentDidMount() {
    console.log('Nav componentDidMount called');
    Store.addChangeListener(this._changeState);
  }

  render () {
    const MenuComponent = (
      <View style={styles.menuView}>
        <MenuContent></MenuContent>
      </View>
    );

    return (
      <SideMenu
        ref='menu'
        isOpen={this.state.isOpen}
        onChange={this._onChange}
        menuPosition='right'
        openMenuOffset={SCREEN_WIDTH * 0.85}
        menu={MenuComponent}>
        <HomeScreen
          style={styles.HomeScreen}
        />
      </SideMenu>
    );
  }
}

const styles = StyleSheet.create({
  menuView: {
    flex: 1,
    backgroundColor: '#ededed',
    paddingTop: 20
  }
});

The console logs show everything happening correctly:

Any ideas where this is going wrong? Why the menu isn't opening as soon as isOpen has its state updated?


mitmproxy crashes

When the 'victim' tries to get a non existent file from a remote location, mitmproxy crashes.
Here's the log:
(venv)srv@ubuntu:~/mitmnbdf$ mitmproxy --version
mitmproxy 0.13
(venv)srv@ubuntu:~/mitmnbdf$ mitmproxy -T --host -s "mitmnbdf_inline.py mitmnbdf.cfg"
Traceback (most recent call last):
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/libmproxy/console/__init__.py", line 499, in run
    self.loop.run()
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/main_loop.py", line 278, in run
    self._run()
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/main_loop.py", line 375, in _run
    self.event_loop.run()
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/main_loop.py", line 678, in run
    self._loop()
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/main_loop.py", line 715, in _loop
    self._watch_files[fd]()
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/raw_display.py", line 392, in <lambda>
    event_loop, callback, self.get_available_raw_input())
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/raw_display.py", line 492, in parse_input
    callback(processed, processed_codes)
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/main_loop.py", line 399, in _update
    self.process_input(keys)
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/main_loop.py", line 496, in process_input
    event, button, col, row, focus=True ):
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/libmproxy/console/window.py", line 22, in mouse_event
    k = super(self.__class__, self).mouse_event(*args, **kwargs)
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/container.py", line 1169, in mouse_event
    event, button, col, row-htrim, focus )
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/listbox.py", line 1538, in mouse_event
    focus=True)
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/urwid/listbox.py", line 342, in calculate_visible
    focus_widget, focus_pos = self.body.get_focus()
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/libmproxy/console/flowlist.py", line 270, in get_focus
    f = ConnectionItem(self.master, self.state, f, True) if f else None
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/libmproxy/console/flowlist.py", line 106, in __init__
    w = self.get_text()
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/libmproxy/console/flowlist.py", line 114, in get_text
    marked=self.state.flow_marked(self.flow)
  File "/home/srv/mitmnbdf/venv/local/lib/python2.7/site-packages/libmproxy/console/common.py", line 399, in format_flow
    contentdesc = netlib.utils.pretty_size(len(f.response.content))
TypeError: object of type 'HTTPFlow' has no len()

mitmproxy has crashed!
Please lodge a bug report at:
	https://github.com/mitmproxy/mitmproxy
Shutting down...

The victim's request is a simple curl --write-out "%{time_total} - %{url_effective}\n" --silent --remote-name http://my.remote.host/non-existent-file.exe.
It works fine if the file exists.

I'm digging, this may not be related to mitmproxy core but my inline script (yet can't find a reference in the stack trace).

Yes, confirmed: it's a bug on my script. Closing the issue. Thanks


[Wallet SDK] Mention all SEPs supported by the Anchor class

This PR updates the Wallet SDK documentation to make it clear about all SEPs the Anchor class currently supports.

Preview is available here:http://developers-pr925.previews.kube001.services.stellar-ops.com


No useful output from lxc commands after downgrade

Required information

Distribution: Ubuntu
Distribution version: 22.04
The output of "lxc info" or if that fails:

Kernel version: 5.15.0-1012-kvm
LXC version: 4.0
LXD version: 4.0
Storage backend in use: dir

Issue description
If the database has been initialized, even if lxd init has not been run, using "snap refresh" to downgrade will make most or all lxc commands (and lxd init) useless.
It seems like virtually any lxc command, not just lxd init, will initialize the database.  Once it has been initialized, downgrading lxd will mean that the schema is invalid for that downgraded version.  Having the wrong schema will prevent any communication on the socket.    In particular, they do not break downgrading.
when the DB schema is invalid, lxc commands produce an error message saying that the DB schema is invalid.  (This implies that lxd starts up in a very limited mode when the DB schema is invalid, instead of falling over.) Ideally they say which version of LXD is compatible with the DB.
when /usr/bin/snap refresh --channel=4.0/stable lxd would produce a broken configuration, it does nothing and exits with an error.  Presumably there would be a --force option to override.

Steps to reproduce
(recommended to do this in a vm launched with lxc launch --vm ubuntu:22.04)

sudo /usr/bin/snap install  --channel=5.0/stable lxd
sudo lxc ls
sudo /usr/bin/snap refresh --channel=4.0/stable lxd
sudo lxc ls

Information to attach

[ X ] Main daemon log (at /var/log/lxd/lxd.log or /var/snap/lxd/common/lxd/logs/lxd.log)
[ X ] Output of the client with --debug

lxd.log
debug-output.txt

It would also be nice if there were a way to run lxd init to recover from this situation, e.g. lxd init --overwrite --preseed.

I'm not sure there's a lot we can do about this. The reason is that:

There is no such thing as "initializing LXD", lxd init isn't special in any way, it just uses the normal REST API to setup storage, network, default profile, ...
The LXD database is needed for any of the API to function so it's automatically initialized on daemon startup
Similarly schema updates must be applied prior to any DB access, so they're applied extremely early on startup
LXD is socket activated on Ubuntu, so will start up when any lxc command is run or when anything else hits the unix socket
The lxc tool is just a REST API client, when the REST API isn't available because the daemon refused to start, it cannot connect and can't tell why

What we do to try and help with those situations is:

A clear downgrade error should be visible in the LXD log (/var/snap/lxd/common/lxd/logs/lxd.log)
A similar error should also be visible in journalctl -u snap.lxd.daemon
On DB upgrades, LXD makes a backup of the DB at /var/snap/lxd/common/lxd/database/global.bak, this can be restored should a downgrade be needed

@ru-fu we probably ought to add a doc page on the upgrade behavior that would more directly cover this

There is no such thing as "initializing LXD", lxd init isn't special in any way, it just uses the normal REST API to setup storage, network, default profile, ...

Is there a reason why it can't be special?  There's nothing to stop lxd --overwrite from deleting the database, is there?

The LXD database is needed for any of the API to function so it's automatically initialized on daemon startup

Until there's mutation, you have a choice about whether to preserve the database after an operation.

The lxc tool is just a REST API client, when the REST API isn't available because the daemon refused to start, it cannot connect and can't tell why

As I mentioned, supporting a sane error message implies that the daemon doesn't just fall over when the schema is invalid.  You have the option of allowing the daemon to run with an invalid schema, and on every connection report that the schema is invalid.

What we do to try and help with those situations is:

A clear downgrade error should be visible in the LXD log (/var/snap/lxd/common/lxd/logs/lxd.log)
A similar error should also be visible in journalctl -u snap.lxd.daemon
On DB upgrades, LXD makes a backup of the DB at /var/snap/lxd/common/lxd/database/global.bak, this can be restored should a downgrade be needed

@ru-fu we probably ought to add a doc page on the upgrade behavior that would more directly cover this

Is there a reason why it can't be special? There's nothing to stop lxd init --overwrite from deleting the database, is there? Since it's not an lxc command, it seems perfectly reasonable to support local-only operations.

lxd init is run as an unprivileged user and so doesn't have write access needed to wipe the database, nor does it now what init system you're using and how to restart LXD afterwards. So this kind of thing is actually better done by the user.
To properly reset LXD, the procedure usually is:

rm -Rf /var/snap/lxd/common/lxd
reboot

The reboot part also takes care of wiping any kernel state, disks, networks, ... which may be in place as merely getting rid of the database doesn't handle that.

As I mentioned, supporting a sane error message implies that the daemon doesn't just fall over when the schema is invalid. You have the option of allowing the daemon to run with an invalid schema, and on every connection report that the schema is invalid.

LXD requires database access to setup the network listeners and requires the daemon config to be read from database to setup the API handlers. The easiest way we could do something like this would be to have a completely separate listener and API handler just for this one case.


WIP: [BEAM-9623] Support for SQL TableProviders in Python SqlTransform

Currently this just hard-codes support for DataCatalogTableProvider. We should instead make it possible to specify and configure HCatalog instances as well.

Run XVR_Flink PostCommit


No module named 'distutils'

Got this error on macOS when I was trying to setup my project. My python version was 3.12.5. Apparently distutils has been removed in python 3.12.
Managed to fix it by adding setuptools==74.1.2 in requirements.txt and ran docker compose up --build again.

Hi,
Thanks. It looks like a week ago https://github.com/docker-library/python/pull/954 the official Python 3.12.5 image was changed to remove this. Prior to that it was installing that package. This project wasn't built in CI within a week.
When I build the image locally with docker compose build --no-cache I am able to reproduce the problem.
I just pushed https://github.com/nickjj/docker-django-example/commit/ea3ce5c451df49eff67ad3603a4315e1e785cb8e to fix it.

Wow, that was fast! Thanks 🚀

No problem, thanks to this issue I'm going to look into using GitHub Action's schedule feature to run CI every day so things like this get caught in an automated way since I'd get emailed if a build fails.

I was curious whether it was a broken dependency of this project that was missing the setuptools dependency (and if so, was going to report that upstream), however, trying out a build locally I see the issue was from a first-party usage of distutils's strtobool:
worker-1    |   File "/app/src/config/settings.py", line 16, in <module>
worker-1    |     from distutils.util import strtobool
worker-1    | ModuleNotFoundError: No module named 'distutils'

Checking https://peps.python.org/pep-0632/#migration-advice it seems there isn't an official replacement for strtobool - but if you wanted to avoid having to have the entirety of setuptools as a dependency you could either:

Switch to the third-party str2bool package: https://github.com/symonsoft/str2bool
Vendor the strtobool function from distutils directly (since it's <10 lines): https://github.com/pypa/distutils/blob/bddd2591d35cf59b2a0d54156b195c334ab7dc8a/distutils/util.py#L329-L342

Thanks, I remember thinking about this back when 3.12 dropped and I was undecided.
That str2bool package isn't backwards compatible with the one that's in distutils because it doesn't include on and off and uses a different function name. I also wasn't crazy about introducing a lib/ directory to drop in nothing more than a strtobool function so I left it be for now.
But maybe the lib/ directory idea isn't bad because it does provide an opinion on where to keep general non-project specific code that's not pulled in from a third party dependency.


Add a DisableEPSV configuration option.

This is to deal with cases where PASV requests lead to a successful connection
but EPSV requests to do not, possibly because of server-side configuration, IPv6,
NAT or firewall issues.
The DisableEPSV option is added in the spirit of the ftp:prefer-epsv option
provided by LFTP for similar reasons.
Signed-off-by: Jon Seymour<EMAIL_ADDRESS>

Thanks!


Bootstrap CDN and Static files configuration

@t0xicdream  It works without the changes to the settings.py for me by running python manage.py collectstatic. The settings you have added are also the default settings so they shouldnt matter at all, and it therefore is cleaner without them. So could you try testing it with my commit and see if you can get it to work? @maximekabel could you test it again? Ive re-requested your review

I think the top bar, side bar and footer can be a separate pr, as this is the basis for all other plugins, as all will need bootstrap and potentially statics files. Otherwise it will be a mess merging all the different implementations of bootstrap.


[5.8] Replace first occurrence instead of last for event discovery

This PR fix the previous PR about event discovery : #28421.
The goal is to replace only the first occurrence instead of the last one because this is not the correct behavior.
Here is an example where the previous PR will create a bug :
/app is the project directory.
/app/app/Listeners/app/FooListener.php is the listener file path.
In this example, the resolved class path was /app/app/Listeners/FooListener.php with the previous PR. This is incorrect as a namespace part is missing.
With this PR, the class is correctly resolved as /app/Listeners/app/FooListener.php.

Thx @taylorotwell for the quick merge.


pull request for e2e test at 12/10/2016 13:26:03

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Turn spotbugs back on in PR.

This PR removes the condition on spotbugs so it runs on PRs, this to catch issues that have been failing builds in CI.

Merging after a quick chat with @JonathanGiles


lines_cop.rb: Disable OS.mac? audit for Linuxbrew

Since brew 1.3.4, each tile OS.mac? is used, we get the following audit error:

Don't use OS.mac?; Homebrew/core only supports macOS

This new test only makes sense on mac OS, so we can disable it for Linuxbrew.

Failures:

  1) RuboCop::Cop::FormulaAudit::Miscellaneous When auditing formula with OS.linux? check
     Failure/Error: expect_offense(expected, actual)

       expected: not nil
            got: nil
     # ./test/support/helper/rubocop.rb:5:in `expect_offense'
     # ./test/rubocops/lines_cop_spec.rb:252:in `block (4 levels) in <top (required)>'
     # ./test/rubocops/lines_cop_spec.rb:251:in `each'
     # ./test/rubocops/lines_cop_spec.rb:251:in `block (3 levels) in <top (required)>'
     # ./test/spec_helper.rb:96:in `block (2 levels) in <top (required)>'
     # ./vendor/bundle/ruby/2.3.0/gems/rspec-wait-0.0.9/lib/rspec/wait.rb:46:in `block (2 levels) in <top (required)>'

Failures:

  1) Bash every `.sh` file has valid bash syntax
     Failure/Error: expect(path).to have_valid_bash_syntax

       expected that /home/linuxbrew/.linuxbrew/Homebrew/Library/Homebrew/cmd/shcmd.sh is a valid Bash file:
       /bin/bash: /home/linuxbrew/.linuxbrew/Homebrew/Library/Homebrew/cmd/shcmd.sh: No such file or directory
     # ./test/bash_spec.rb:29:in `block (4 levels) in <top (required)>'
     # ./test/bash_spec.rb:25:in `each'
     # ./test/bash_spec.rb:25:in `block (3 levels) in <top (required)>'
     # ./test/spec_helper.rb:96:in `block (2 levels) in <top (required)>'
     # ./vendor/bundle/ruby/2.3.0/gems/rspec-wait-0.0.9/lib/rspec/wait.rb:46:in `block (2 levels) in <top (required)>'

I believe this failure is intermittent. It depends on the order in which the tests are run. I'm rerunning CircleCI now.

CircleCI succeeded this time, so it is an intermittent failure.

Supersed by https://github.com/Linuxbrew/brew/pull/503

Thanks @sjackman


Add support for volume and volume mounts for runners

Currently there is no way to add different volume mounts to the runner pod, other than the default one that is created from the script spec. This PR adds support volumes and volume mounts.
The scenario we're aiming to utilise these in is when we're fetching the k6 script(s) and other artifacts from an external source (i.e. S3) through init containers, and copying them to the main containers using a Volume of type EmptyDir.
Closes #250

Thanks for the review @yorugac , i've reverted the Makefile changes and added a new sample k6 file in e2e.

@yorugac thanks for spotting the issue. I probably copied the wrong file 😅
Regards the CRD install, I believe I encountered the issue once and didn't keep the solution in the Makefile in the end.
The way I solved it was by adding the following flag to the kubectl apply in the  deploy command in the Makefile:
--server-side=true

This removes the "last-applied-configuration" annotation from what I understand.

OK, I see. Actually, --server-side=true seems to work only on clean slate, i.e. without existing k6-operator deployment. Otherwise, I get these errors:
error: Apply failed with 9 conflicts: conflicts with "kubectl-client-side-apply" using apiextensions.k8s.io/v1beta1:
- .spec.validation.openAPIV3Schema.properties.spec.properties.initializer.properties.initContainers.items
- .spec.validation.openAPIV3Schema.properties.spec.properties.initializer.properties.volumeMounts.items
- .spec.validation.openAPIV3Schema.properties.spec.properties.initializer.properties.volumes.items
- .spec.validation.openAPIV3Schema.properties.spec.properties.runner.properties.initContainers.items
- .spec.validation.openAPIV3Schema.properties.spec.properties.runner.properties.volumeMounts.items
- .spec.validation.openAPIV3Schema.properties.spec.properties.runner.properties.volumes.items
- .spec.validation.openAPIV3Schema.properties.spec.properties.starter.properties.initContainers.items
- .spec.validation.openAPIV3Schema.properties.spec.properties.starter.properties.volumeMounts.items
- .spec.validation.openAPIV3Schema.properties.spec.properties.starter.properties.volumes.items

But also, for 'cleaner' solution I think it's better to change make manifests with crd:maxDescLen=0 and not make deploy with  --server-side=true: so that there's an impact only on manifest generation during development and not on the bundle generation during deployments and releases. So, could you please add that crd:maxDescLen=0 to make manifests as well? Assuming that option works in your setup as well :slightly_smiling_face:

For whatever reason, adding crd:maxDescLen=0 results in the needed effect of not having descriptions, but changes the kind version to apiextensions.k8s.io/v1beta1, which results in the following error:
resource mapping not found for name: "k6s.k6.io" namespace: "" from "STDIN": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"

I had changed the manifests command to:
manifests: controller-gen
	$(CONTROLLER_GEN) $(CRD_OPTIONS) rbac:roleName=manager-role webhook paths="./..." output:crd:artifacts:config=config/crd/bases

Updating controlller-gen to v0.12.1 seems to have fixed it for me:
        ...
	go install<EMAIL_ADDRESS>;\
        ...

And then updating the CRD_OPTIONS to:
CRD_OPTIONS ?= "crd:maxDescLen=0"

As crd:trivialVersions is not an option anymore on that version and the default for crdVersions is v1.
Let me know how you'd like to proceed @yorugac. I'm happy to just change the version in the Makefile to what you suggested only if you think my case is an isolated case.

Added and tested the changes. Everything seems in order 👍🏼
Thanks @yorugac


Update Wordpress

Update to use official Wordpress image
Zeroconf setup for user and compatible with Wordpress Updates
Fixes https://github.com/akash-network/awesome-akash/issues/381
Uses standards and best practices from official Wordpress docker-compose.yml and Docker official Wordpress

Approving to move back to WordPress official image


Site cannot be served

I downloaded the repo and ran "make" and then "make serve"
I can connect to the webserver on port 4000 and see the main page but none of the linked pages can be accessed. I just see, for example "`/basics.html' not found." when going to "http://localhost:4000/basics.html"

@justinhj I'm sorry, it looks like you need another gem: jekyll-textile-converter


chore(pacmak/python): format generated code using black

Make the generated code a little more readable by running it through black.
This will make this a little nicer for Python developers using the "go to
definition" feature of their IDE whith libraries produced by jsii-pacmak.
Fixes #379
Fixes aws/aws-cdk#8532

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 4436a8c0b4aa8a3cc5f1979c0974ac29f0ba00bb
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 385e9d79dc9dee3e0bdc7e2c13095c17fe3359ef
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Calibration files

How are the calibration files 15*.txt created from the calibration files for the EMM microphones provided by Dayton?

I believe they're the same files that are downloaded directly from Dayton here using the microphone serial number.


Force a single active instance

The extension should only be allowed to have a single instance running. This might be a bit hard with the popup (action window) and tab, but one tab should be fine in combination with the popup. The popup is not controllable by JavaScript for security reasons.

After a few attempts and research on how to make it a single instance, this behavior won't be implemented as it's likely more work than continue to support multiple UI-instances. There are certain scenarios where user might end up doing something unexpected and have unexpected behavior, e.g. editing settings in two places at the same time.
Allowing multiple instances, will allow more advanced users to orchestrate their browsers and looking at and using different wallets at the same time.


fix authorised connection issue where .refresh() gets bypassed

APIObject.call calls self.connection.get() which bypasses the refresh code in AuthedConnection().call
The solution is to override get() in AuthedConnection() and put the refresh code in there, so that refreshes get carried out whether an APIObject or an AuthedConnection is called.

Coverage increased (+0.0%) to 99.6% when pulling 78b834f1378afaee88d4380bd3b1f6068c8ab19e on wtfrank:master into afbf4f9b518e4d4d85e5eca34c57aa264523557f on Dreae:master.


GC mem over limit, due to Quest mem leaks.

(Required) What is your Quests version / build number (do not say "latest")?
3.3.4-b316
(Required) Please describe the problem and how to replicate it. Any console errors?
https://paste2.org/44Adzawk
https://paste2.org/5kF6Wyc6

Commit should fix the first error, however the second paste appears to be caused by your server failing to provide Quests with various player-related events. Possibly fault of ViaBackwards?


aws: refactor completion sourcing logic

Clean up Homebrew detection and add comments.
Also changed some if flags.

I can test it


Update storage-account-key-note-include.md

I am a MSFT employee, I have added a Note section in the document with the changes that needs to be made. Let me know in case of any issue. Both Product Group and Supportability are informed about the change. We might also have to update the document later on since Product Group is working on changing the "Authentication type" to "Trusted Access" when using system key however we do not have a definite ETA for the same.

@tamram
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

@sudiptaC01 - I've created a separate PR to address this (linked above). You can close yours out now.

#please-close


filter_hybridization_rings fails on non-ring atoms

Describe the bug
The filter_hybridization_rings fails when it tries to correct a mapping involving an atom not part of a ring as the filter only stores info on ring atoms.
To Reproduce
See this example for the shp2 ligands
from gufe import LigandNetwork, SmallMoleculeComponent
from kartograf import KartografAtomMapper
from kartograf.filters.ring_changes import filter_hybridization_rings

# work around gufe issue 407
def node_to_rdkit(node: SmallMoleculeComponent) -> Chem.Mol:
    """Do some santization to make sure we have the atom hybridization"""
    rdkit_mol = node.to_rdkit()
    Chem.SanitizeMol(rdkit_mol, Chem.SANITIZE_ALL)
    Chem.AssignStereochemistryFrom3D(rdkit_mol)
    Chem.Kekulize(rdkit_mol, clearAromaticFlags=True)
    Chem.SetAromaticity(rdkit_mol, Chem.AromaticityModel.AROMATICITY_MDL)
    return rdkit_mol

mapper = KartografAtomMapper(additional_mapping_filter_functions=[filter_hybridization_rings])
with open("shp2.graphml", "r") as f:
    ligand_network = LigandNetwork.from_graphml(f.read())

for edge in ligand_network.edges:
    if edge.componentA.name == "lig_E6" and edge.componentB.name == "lig_E29":
        edge.componentA._rdkit = node_to_rdkit(edge.componentA)
        edge.componentB._rdkit = node_to_rdkit(edge.componentB)
        mapping = next(mapper.suggest_mappings(edge.componentA, edge.componentB))
        break
mapping

ai_only_arom_sys = atomA_ring_hyb_map[ai]
KeyError: 23

Expected behavior
This should work on molecules that mix ring and non-ring atoms.

@jthorton could you maybe add a figure of the 2 ligands here so it's easier to understand what is going on?

No problem see attached, atom 23 is the methyl carbon in the ligand on the left which the filter has no info about as its not in a ring.


Add image attachment to twitter connector & sample

Original issue - https://github.com/ballerina-lang/connectors/issues/95
Twitter connector and samples seem to be missing ability to attach images to the tweets.
Can someone please fix that?
Tweets without images are not nearly as effective as the ones with graphics attached.

Fixed in #23