Feature/370 add mTls support to docker endpoint

This PR add two options for docker endpoint authentication.

it is support of simple secured endpoint by TLS
it is support of client authentication via mTLS

PR contains 2 commits on for each option.
This PR is draft only to review approach of implementation.

I am not sure why few tests failed when locally run fine. I think that my code is out of test scope.
What this PR miss are:

[ ] Test strategy for new TlsEndpointAuthenticationProvider and MTlsEndpointAuthenticationProvider
[ ] Update build agents to be able to test these providers
[ ] Cleanup based on last PRs
[ ] Review of supportability in .NET Standard 2.1 (Do we test both .NET Standards?)

I would like to split this pull request into two smaller chunks. The first one should just contain the changes regarding ICustomConfiguration (reading the custom configuration values incl. tests). The second one the TLS implementation. It makes the review easier and we can focus just on the TLS part next. What do you think? In the meantime I'll setup the TLS test environment. Thanks again.

Ok, will try to prepare it soon.

We can use something like this to create a test instance:
public sealed class GitHub : IAsyncLifetime
{
  private const string CertsDirectoryName = "certs";

  private static readonly string ContainerCertDirectoryPath = Path.Combine("/", CertsDirectoryName);

  private static readonly string HostCertDirectoryPath = Path.Combine(Path.GetTempPath(), CertsDirectoryName);

  private readonly ITestcontainersContainer tlsContainer = new TestcontainersBuilder<TestcontainersContainer>()
    .WithImage("docker:20.10-dind")
    .WithPrivileged(true)
    .WithEnvironment("DOCKER_CERT_PATH", ContainerCertDirectoryPath)
    .WithEnvironment("DOCKER_TLS_CERTDIR", ContainerCertDirectoryPath)
    .WithEnvironment("DOCKER_TLS", "1")
    .WithEnvironment("DOCKER_TLS_VERIFY", "1")
    .WithBindMount(HostCertDirectoryPath, ContainerCertDirectoryPath, AccessMode.ReadWrite)
    .Build();

  [Fact]
  public Task PullRequest548()
  {
    return Task.CompletedTask;
  }

  public Task InitializeAsync()
  {
    return this.tlsContainer.StartAsync();
  }

  public Task DisposeAsync()
  {
    return this.tlsContainer.DisposeAsync().AsTask();
  }
}

This generates the certificates too.

Ok will try to use it. But I will update PR to latest master then will split it to two smaller chunks with tests.
Then we can deal with one feature in a time.

I updated mTls implementation based on implementation of Tls in PR #597  including inheritance to not repeat code.

@vlaskal Would you take a look at my recent changes? I think I just need to fix 2 tests, then we are good to merge. What do you think?

Done. I think we are close.

Closes #370.


enhancement make memtable memusage more accurate

Proposed changes

Add memtype to memtable, and save a weak ptr vector in memtable writer, so that we could get different memory usage by traverse the vector.
Using scoped memory usage to compute the mem usage of a memtable.


Monitor Group hierarchy not retained after restoring backup

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
After taking a backup and restoring on a new instance, the monitors are created on the new instance with the restore process. However, the monitors are not assigned to the monitor groups. The monitor groups are created, however, they are all empty and all monitors are at the same level with no parent / child relationship.
👟 Reproduction steps
Server 1:
Create monitor group
Create monitor and assign to monitor group
Perform backup
Server 2:
Restore backup using json on new server
All monitors and monitor groups are imported, however, when editing the monitor, the monitor group is set to none and no parent/child relationships are reflected in the dashboard.
The json file does not contain a MonitorGroup field
👀 Expected behavior
On the restored instance expect to see the parent / child relationship.  The monitor should be assigned to a monitor group as is defined in the instance where the backup was performed.
😓 Actual Behavior
All monitors and monitor groups are imported, however, the monitors are not assigned to any monitor groups.
🐻 Uptime-Kuma Version
1.23.9
💻 Operating System and Arch
RHEL 8.9 x86-64
🌐 Browser
Microsoft Edge Version 120.0.2210.91 (Official build) (64-bit)
🐋 Docker Version
No response
🟩 NodeJS Version
No response
📝 Relevant log output
No response

#4177

@asif-pspc We have dropped said feture in v2.0


Exit status of a Task added in Build Pipeline

I don't think that there is an option to get the Exit status of a particular task added in Build Pipeline.
For instance; i added task 1 and task 2 and task 3. Task 1 is success and task 2 is Failure. Task 3 is customized email task which will run even if the previous task has failed. I want to include the status of task1, task2 in the email body?

I'm closing this issue since it is not associated with a topic. To continue the discussion, please open a new issue that is associated with a topic by browsing to the topic on docs.microsoft.com and choosing Content feedback at the bottom. Thank you for your feedback.


add location matching

add location fields in solr index
calculate distance in solr index between needs
boost/filter matches according to distance

queries tested:
term query:
http://<IP_ADDRESS>:8983/solr/won/select?q=query({!dismax qf=_graph.http___purl.org_webofneeds_model_hasContent.http___purl.org_dc_elements_1.1_title v=test})&fl=score,id
distance query:
http://<IP_ADDRESS>:8983/solr/won/select?q= query({!func v=recip(geodist(need_location,48.209023,16.369851),1,100,100)})&fl=score,id
combined:
http://<IP_ADDRESS>:8983/solr/won/select?q=product(query({!dismax qf=_graph.http___purl.org_webofneeds_model_hasContent.http___purl.org_dc_elements_1.1_title v=test}), query({!func v=recip(geodist(need_location,48.209023,16.369851),1,100,100)}))&fl=score,id
combined with OR:
http://<IP_ADDRESS>:8983/solr/won/select?q=query({!dismax qf=_graph.http___purl.org_webofneeds_model_hasContent.http___purl.org_dc_elements_1.1_title v=test}) OR query({!func v=recip(geodist(need_location,48.209023,16.369851),1,100,100)})&fl=score,id
http://<IP_ADDRESS>:8983/solr/won/select?q=_graph.http___purl.org_webofneeds_model_hasContent.http___purl.org_dc_elements_1.1_title:test AND query({!func v=recip(geodist(need_location,48.209023,16.369851),1,100,100)})&fl=score,id
with defaults (doesnt work):
http://<IP_ADDRESS>:8983/solr/won/select?q=product(query({!dismax qf=_graph.http___purl.org_webofneeds_model_hasContent.http___purl.org_dc_elements_1.1_title v=tost},0.1), query({!func v=recip(geodist(need_location,48.209023,16.369851),1,100,100)}),0.1)&fl=score,id
=>  this one can be used to boost the term queries multiplicatively with the distance:
with multiplicative boosting:
http://<IP_ADDRESS>:8983/solr/won/select?q={!boost b=sum(recip(geodist(need_location,48.209023,16.369851),5,100,100),1)}_graph.http___purl.org_webofneeds_model_hasContent.http___purl.org_dc_elements_1.1_title:play&fl=score

some links:

https://cwiki.apache.org/confluence/display/solr/The+DisMax+Query+Parser#TheDisMaxQueryParser-Thebf(BoostFunctions)Parameter
https://lucidworks.com/blog/2011/12/14/options-to-tune-documents-relevance-in-solr/
http://stackoverflow.com/questions/13752735/how-do-i-combine-perform-a-solr-or-query-between-a-spacial-search-and-a-regular
https://cwiki.apache.org/confluence/display/solr/Common+Query+Parameters
https://wiki.apache.org/solr/FunctionQuery
http://wiki.apache.org/solr/SolrRelevancyFAQ#How_can_I_boost_the_score_of_newer_documents


Remove dependency on rancher-machine

It would be good to remove our dependency on rancher-machine. We currently use it in 1 place to get the tenant id:
https://github.com/rancher/aks-operator/blob/master/pkg/aks/client.go#L140

This can be closed


[Lightnet] Clean up only self resources

Today we are doing general Docker Engine resources cleaning up. This might lead to unpleasant user experience when, for example, users left some entities for later use by some other service, yet Lightnet CLI wiped everything.

False alarm. We are cleaning only lightnet resources.


Game immediately crashes upon startup when using VanillaFix.

Mods: https://prnt.sc/109nixo
Using 1.12.2 with the second latest version of the mod.

i had the same issue. i tried the fix on #265 but it didnt work, i tried unloading everything except for OptiFine, Mixin Booter and VanillaFix, didnt work (same error), the only thing that worked was uninstalling VanillaFix which stopped the game from crashing :(


Setup for selenium web tests

This pull requests adds the following tasks into usmqe role:

installation of Xvfb and x11vnc systemd services
opening of 5900 tcp port

Resolves https://github.com/Tendrl/usmqe-setup/issues/24

@dahorak This is not ready to be merged, we may tweak the role a bit first. Eg. we may not like to open vnc port by default. But it already works, see the updated readme file of qe-server role.

Based on discussion with @dahorak , I'm going to remove template feature from service ini file, specify dependencies properly, start both services and keep the firewall port 5900 open.

LGTM


fix incorrect example in documentation of Dict.get

:+1: LGTM

Thank you @joelgrus! And thanks @ajhager for the ping, seems like I am having trouble monitoring new issues through email notifications. Need to develop a more robust technique.

I was going to go through and mark all PRs that are obviously correct, but I didn't want to spam your inbox.

That would be very extremely helpful and appreciated!


Hardening binary & shared library

I'm one of the package maintainers for Arch Linux and I also maintain a few packages on the AUR, which mergestat can be found on.
Just wondering if there's any interest in RELRO/PIE being applied to the binary & shared library?
I generally try and apply these to all the Go-related packages that I maintain due to our Go package guidelines. I've found that mergestat seems to be working fine with these applied, as per this commit.

Hi @grawlinson thanks for reaching out! Yes - we do have interest in applying RELRO/PIE to the binary and shared library, thank you for sharing the background for those as well. It looks like line 57 in the linked commit there is the key bit? I will look into adding those flags to our Makefile and open a PR shortly

There’s a few parameters that need to be added to LDFLAGS, I’ll have a look at them and get back to you.


Add Python 3.9.2

Tool information

Tool name:  Python
Tool license:  PSF
Add or update?  Add
Desired version:  3.9.2
Approximate size:
If this is an add request:

Brief description of tool:
URL for tool's homepage:
Provide a basic test case to validate the tool's functionality:

Area for Triage:

Question, Bug, or Feature?:

Virtual environments affected

[x] Ubuntu 16.04
[x] Ubuntu 18.04
[x] Ubuntu 20.04
[x] macOS 10.15
[x] macOS 11.0
[x] Windows Server 2016 R2
[x] Windows Server 2019

Can this tool be installed during the build?

Tool installation time in runtime

Are you willing to submit a PR?

Hello, @kumaraditya303
We are planning to deploy new images with Python 3.9.2 in a week or two.

Python 3.9.2 is already available for GitHub Actions via https://github.com/actions/setup-python

https://github.com/microsoft/playwright-python/pull/543/checks?check_run_id=2020011803 See still Python 3.9.1 is used @maxim-lobanov

Try to specify exact version 3.9.2.
At first, the task tries to resolve version locally, it finds 3.9.1 that satisfies 3.9.x input and use it.
If you specify 3.9.2, it won't find it locally and will install in runtime (pretty quick).
As Aleks mentioned, 3.9.2 will be added to images very soon and after that task will resolve 3.9 correctly

Python 3.9.2 was added to all images recently. Closing the issue


chore: add fix-key-and-update.sh script to resolve cosign pub key issues

Thank you for contributing to the Universal Blue project!
Adds a script which can be executed by users who are on an image signed by an older cosign key pair.

Doing one final pass in a VM, so please do not merge yet :)


Update dependencies

v0.1.0 does not build due to a broken dependency tree. This PR updates all of the dependencies to the latest versions available on crates.io

@root670 Awesome! Thanks for the PR!


JsonConvert.SerializeObject returns indented json when called with Formatting.None

When JsonConvert.SerializeObject(object value, Formatting formatting, JsonSerializerSettings settings) is called with Formatting.None and settings that have Formatting.Indented I would expect the formatting to override the value in settings, but that is not the case. See the following failing test.
var obj = new { Formatting = "test"};

var settings = new JsonSerializerSettings { Formatting = Formatting.Indented };
string indented = JsonConvert.SerializeObject(obj, settings);

string none = JsonConvert.SerializeObject(obj, Formatting.None, settings);
Assert.AreNotEqual(indented, none);

it worked great!!


Changing the Combo Numbers and Rating sprites camera.

Describe your problem here.
I need help moving the Combo numbers and rating sprites to camGame instead of camHUD.
I tried using setProperty() and also runHaxeCode() but it wouldn't work for some reason or maybe because I'm just dumb.
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No

replace the assets in the mods folder
with the same name files from the assets folder

Forgot to mention that I also used setObjectCamera()

'rating', 'comboSpr', And 'numScore'. Im not sure if I'm using the correct tags

You can't afaik, you'll have to recreate them

Yeah, I experimented with those as well. Like frantastic24 says, yeah you might have to create some sort of artificial sprite system if you want to guarantee a sprite change, but for now you might have to keep looking in the source code to find what you need.

Alright, thanks for the help


Display Overall Connection Statuses

Add some sort of visual indication of connection statuses for both the Spotter and Gunner functionality.

Might be obsolete due to the addition of the connected users table on WitchyWork_UDP branch, but leaving this open for now.

Closing. Obsolete.


Basic Linting setup

Setup initial linting checks such as formatting and clippy style as part of the CI run. Optionally consider additional checks such as code coverage etc too.
Depends on #1

Resolved by https://github.com/rustyorg/rustygit/pull/7
We have rustfmt and clippy checks, will worry about code coverage at some later point.


Increase limit of feeds ?

I am testing Livemarks under different aspects, but when it reached a number 512 feeds : if  I click on"+Add livemark" : create a folder with the name of the website but that's all, it is empty and the contextual menu does not offer the possibility to reload...the following are no longer retained and included in the extension. On the other hand if I delete the last feed limit, and I recreate one then no problem...
note : the limit of 500 feeds are the operational maximums, the following 12 are stored by Livemarks but will be declared broken.
The limit may come from Firefox or from Livemarks ? but how to unlock it ? Or better : allow the user to select the limits himself !
By the way, it misses an element to count and visualize via a counter the number of feeds, it would be nice to have one !
Thank you for your attention.

Any limit seems like a Firefox restriction here to me. cc @Rob--W


Favicon Needs Updated

Asana changed it favicon and logo a while ago to be 3 red dot in a triangle.
Is it possible to update the asana plus favicon to match?
I think you're attempting to do something regarding notifications using the favicon. I that the same thing that Asana is already doing now with a black dot (which represents new items in the inbox)?

Perhaps it would be best to just remove the favicon notification feature from Asana Plus and go back to the default favicon.

Should be done :)
(0.16.0)

Thanks!


docs: fix broken links to docs.celestia.org

Closes https://github.com/celestiaorg/celestia-app/issues/2762
Testing
make markdown-link-check passes locally

Codecov Report

Merging #2763 (af8072e) into main (aadc323) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2763   +/-   ##
=======================================
  Coverage   19.62%   19.62%
=======================================
  Files         139      139
  Lines       16999    16999
=======================================
  Hits         3336     3336
  Misses      13341    13341
  Partials      322      322


Feature Request: search and zoom

Description
While I am quite fond of cities in Belgium, always being centered and zoomed in there makes it tedious to look at things in California.
Let people search by Divisions and 2 letter country code to find a place close to them and then let them click on the name and the map recenters to that division.
You could do it with place names. It would certainly be more likely to have terms a person would search, but it is a lot more to index.
What browser are you seeing the problem on? What version are you running?
No response
The OS you're using
No response
Steps to reproduce

Open the map
It is centered in Belgium
try to get to santa cuz on the map

The browser URL at the time you encountered the bug
https://explore.overturemaps.org

Totally understood. The release we're dropping tomorrow will fix this "always centered in belgium" problem a bit better- we have a navigator control that you can click on to zoom to different places/views. However, I think that some kind of geocoder/place hopper is for a great upgrade.


add ES6 implementation of a shuffle function

Explanation About What Code Achieves:

Add ES6 implementation of shuffle function to remove unnecessary variables

@AhmadShahid can you rebase the branch?

@pratu16x7  thanks for you response i have rebase the branch can u please check it

Sweet! Looks good 👍


Fallback not working when autoPlayAdBreaks off

Fallback ads work when autoPlayAdBreaks is on.  When autoPlayAdBreaks is off, code execution just halts.  I've created two codepens to demonstrate this.  Is there something else that needs to be done programatically when autoPlayAdBreaks are off?
With autoPlayAdBreaks off
https://codepen.io/davidjamesherzog/pen/eKELjN
With autoPlayAdBreaks on
https://codepen.io/davidjamesherzog/pen/dKVNRB

I'm having trouble with your samples - it looks like they're trying to load a controller.js that doesn't exist.

For both examples, click on the Play Video button below the player to start the ads.  The first link above does not do anything but it should play the fallback ad (which is why I opened this issue).  The second link should play the fallback ad successfully.

I removed the reference to controller.js.  That was a copy/paste error.

I think this issue goes beyond autoPlayAdBreaks - if I take your first example and just remove autoPlayAdBreaks: false, it still doesn't work. If the issue was with that mechanism, I would expect that change to fix it. It looks like you have a lot of other custom code going on in there - setting the ad tag and requesting ads on click instead of on page load, and adding your own CONTENT_RESUME_REQUESTED listener. Could you give me an idea of what you're trying to do with that code so we can see a) if it's doing what you want it to do, and b) if there's a better way to do it?

@shawnbuso - Sorry, but I was pulled off on other work and I am finally getting back to looking at this.  I took the advanced example from this repo.  I added autoPlayAdBreaks: false to the options and I added the following ad break listener:
var _this = this;
this.player.ima.setAdBreakReadyListener(function() {
  _this.player.ima.playAdBreak();
});

If I use the following sample URLs it works the same whether or not I set autoplay ad breaks on or off:
normal - http://pubads.g.doubleclick.net/gampad/ads?sz=640x480&iu=/124319096/external/ad_rule_samples&ciu_szs=300x250&ad_rule=1&impl=s&gdfp_req=1&env=vp&output=xml_vmap1&unviewed_position_start=1&cust_params=sample_ar%3Dpremidpostpod%26deployment%3Dgmf-js&cmsid=496&vid=short_onecue&correlator=
error - https://pubads.g.doubleclick.net/gampad/ads?sz=640x480&iu=/124319096/external/single_ad_samples&ciu_szs=300x250&impl=s&gdfp_req=1&env=vp&output=vast&unviewed_position_start=1&cust_params=deployment%3Ddevsite%26sample_ct%3Dredirecterror&nofb=1&correlator=
If I use the fallback URL, it works when autoplay ad breaks is set to true but not when it is set to false (no fallback ad plays).
fallback - https://pubads.g.doubleclick.net/gampad/ads?sz=640x480&iu=/124319096/external/single_ad_samples&ciu_szs=300x250&impl=s&gdfp_req=1&env=vp&output=vast&unviewed_position_start=1&cust_params=deployment%3Ddevsite%26sample_ct%3Dredirecterror&correlator=

Hmm the issue here may be that your fallback tag is a single ad, while the working tag is a VMAP response. Manual ad break playback only works with VMAP or ad rules responses, since you can start a single ad responses whenever you want by changing the timing of adsManager.start(). However I don't think this is supported yet with the IMA plugin - you can't delay the call to adsManager.start() with single ads. We can take that as an FR, though.

The fallback tag is the sample one Google provided.  I'll pass this info off to our Ad Ops team.  We do plan on using VMAP so hopefully they can setup an example that will work.  Thanks for the info!


Saving Joi schema

I have converted object using enjoi. How do I store it as file now?
const Enjoi       = require('enjoi')
const { convert } = require('json-to-json-schema')
const jsonSchema  = convert({ name:'Bob' })
const joiSchema   = Enjoi(jsonSchema)

// I want to store 'joiSchema' as file so that it would have content similar to this:
//
// const SCHEMA = Joi.object().keys({
//   name : Joi.string().required()
// })

Joi schemas are JavaScript objects so there is no practical way to serialize/deserialize to strings.
You can use describe to stringify it to a useful debugging format but nothing that I know of for converting it to a literal as used by compile.


Allow easily copying the buildnumber from the About dialog.

@aymanhab : We could add something to the scripting shell.

This has been folded into PR #950


Back button icon showing as a square box

Resources:
Before submitting an issue, please consult our troubleshooting guide (http://ionicframework.com/docs/troubleshooting/) and developer resources (http://ionicframework.com/docs/developer-resources/)
Please make sure you are posting an issue pertaining to the Ionic Framework. If you are having an issue with the Ionic Pro services (Ionic View, Ionic Deploy, etc.) please consult the Ionic Pro support portal (http://support.ionicjs.com)
Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x ] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[ x] bug report
[ ] feature request
Current behavior:

Back button icon in the header showing as an square box
Expected behavior:

Normal Back button icon should appear in the header

Sample App:
https://github.com/lordnino/sample-app.git

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having? Thanks for using Ionic!

I found the issue in the package.json and solved thank you

Excellent! Glad to hear!


IRCCloud incorrect notifications

When using Franz 5 13.316, the IRCCloud icon is showing unread notifications for all channels that I am in, not just direct pings/PMs. This appears to be a regression from Franz 4 and makes Franz essentially unusable.
Expected Behavior
Have the ability to only be notified about direct pings/PMs.
Current Behavior
Being notified about all new messages in all channels.
Your Environment

Franz Version used: 5 13.316
Operating System and version: OS X 10.13.1

This repository is only for the Franz client. Please report issues in the dedicated recipe repository.
https://github.com/albertomurillo/recipe-irccloud


export castpath function for repeated

Can lodash export castpath function as well?
I want to get same paths on different objects repeatedly.
Running benchmark
`var m = { 'a' : { 'b' : { 'c' : 2 } } };
var suite = new Benchmark.Suite();
suite
.add('lodash get', function() {   _.get(m, 'a.b.c', null);   })
.add('native', function() {  m.a.b.c; })
.add('lodash ++', function() {
var index = 0, length = 3, o = m, p = ["a", "b", "c"];
while (o != null && index < length) {  o = o[p[index++]];}
o
})
`
lodash get x 2,029,091 ops/sec ±3.71% (77 runs sampled)
native access x 56,556,627 ops/sec ±6.45% (67 runs sampled)
lodash pre-compiled castpath x 20,986,649 ops/sec ±4.16% (81 runs sampled)

Wring repos by mistake. Excuse me!


CI: move back Eirini smoke test to prod job

Is your feature request related to a problem? Please describe.
Due to HW problems in CI, we moved the Eirini smoke test to no-prod jobs https://concourse.suse.dev/teams/main/pipelines/kubecf?group=no-prod.
Describe the solution you'd like
Now the CI runs stable again, we could move the Eirini job among the prod ones, see : https://concourse.suse.dev/teams/main/pipelines/kubecf/jobs/smoke-tests-eirini
Describe alternatives you've considered
N/A
Additional context
N/A

Is it needed to bump smoke tests to consume https://github.com/cloudfoundry/cf-smoke-tests/pull/61 ?
Without it, smokes on Eirini on sle15 stack will fail.

Smoke tests have already been bumped in #920 and cherry-picked to release-2.2 in #921
However, eirini smoke tests on the release-2.2 pipeline are still completely red due to some logging problem:
https://concourse.suse.dev/teams/main/pipelines/kubecf/jobs/smoke-tests-eirini-release-2.2/builds/9
The eirini-smoke test is already on the non-experimental tab for each pipeline; it is just not configured as a required tests on Github for gating PRs right now.
So isn't this card already done? Fixing the failure of eirini smoke tests themselves is an independent issue.

Hmm, that particular build looked like the OOM issue:

Waiting for app to start...
Start unsuccessful
⋮
2020-05-26T05:55:16.32+0000 [API/0] OUT Process has crashed with type: "web"
2020-05-26T05:55:16.33+0000 [API/0] OUT App instance exited with guid ae09e5c9-8647-47dc-a506-de6b2bdac833 payload: {"instance"=>"smokes-10-app-c22275445dc25749-smoke-10--92a2b044e7-0", "index"=>0, "cell_id"=>"", "reason"=>"OOMKilled", "exit_status"=>137, "exit_description"=>"OOMKilled", "crash_count"=>0, "crash_timestamp"=>14, "version"=>"66088a94-a076-4a59-806f-f8c8e5fd3127"}

It only looked like a logging issue because the first test setup was to check for logging.
However, it does look like it has the bump for smoke tests: it is using smoke-tests-release 40.0.128, which should be using the sources with the appropriate merge.

This is weird, the OOM error is fixed on master, but still seems to happen on release-2.2 even though the bump to smoke-tests-release has been merged.

This is done


Multiple 'RunAfter' statements for a single action fails to produce valid mermaid code

Describe the bug
One action called Send_email in a LA has the following code
"runAfter": {
	"CalculateDigest": [
    	"Succeeded"
	],
	"Compose_2": [
    	"Succeeded"
	],
	"HTTP": [
    	"Succeeded"
	]
}

This will lead to a mermaid construct as follows
CalculateDigest Compose_2 HTTP --> Send_email

This will fail to render
Add error message
Parse error on line 7:
...CalculateDigest Compose_2 HTTP --> S
---------------^
Expecting 'SEMI', 'NEWLINE', 'EOF', 'AMP', 'START_LINK', 'LINK', got 'ALPHA'

Expected behavior
Multiple runAfter statements should be parsed to mermaid as follows:
CalculateDigest --> Send_email
Compose_2 --> Send_email
HTTP --> Send_email

Screenshots
NA
PowerShell version:
Run $PSVersionTable command in PowerShell host
e.g.
Name                           Value
----                           -----
PSVersion                      7.3.6
PSEdition                      Core
GitCommitId                    7.3.6
OS                             Linux
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

LogicAppDocs version

Version 1.1.1

Additional context
NA

Hi @SjoerdV,
Thanks for sharing this issue with me. Would it be possible to share the complete Log Analytics of PowerAutomate Flow code with me, so I can test this?
If so please remove any sensitive content. This can also be shared using a secret gist if you prefer.
https://docs.github.com/en/get-started/writing-on-github/editing-and-sharing-content-with-gists/creating-gists

I'll do you one better. The actual piece of code that needs adjustment is here: https://github.com/stefanstranger/logicappdocs/blob/91c6622c40c3a8e58df1325dccfd075b4a663a0b/src/New-LogicAppDoc.ps1#L192-L200
Needs a foreach to account for multiple runAfter keys.
example:
# Example Data
$runAfterObj = @'
{
    "runAfter": {
        "CalculateDigest": ["Succeeded"],
        "Compose_2": ["Succeeded"],
        "HTTP": ["Succeeded"]
    }
}
'@ | ConvertFrom-Json -AsHashtable

# Modified code
foreach ($Key in $runAfterObj.runAfter.Keys) {
  $mermaidCode += "    $($Key) --> $($object.ActionName)" + [Environment]::NewLine
}

Hi @SjoerdV ,
Thanks for the help and code🙏Let me find some time to update the code in the main branch.

@stefanstranger works like a charm! I tested both single and multiple runafter constructs.
Great work building this.

Merged changes into main branch. Closing issue.


Chrome 88: selector list argument of :not()

Fixes #5751.
https://tests.caniuse.com/?feat=css-not-sel-list:

as well as https://dabblet.com/gist/0fadd625c67da786579ac425eb8aa7cb:

Thanks!

Thanks!


Add overview command with image generation via canvas

I think It's will be very useful in groups to get overview information about some player in just one command. Maybe I will realize it in some future.

Woah image generation! Sounds really cool. But no hurry don't worry about it :)

Some progress.

Wow looking good! What's this canvas thing you're using?

Just node-canvas.

I think it's nearly release candidat :D

Beta 3.

wow you're really fast! 👍🏼

I think I will need some you help in "refactoring". I am not js-programmer, so write code like I know. Maybe violate some rules or somethink like that. Now I fix some minor bugs, then I think will do pull request.

Sure I can help you with that. But I'm a little busy these few weeks, exams and stuff, so I may not be able to work on the bot as quick.


IronLazyPagesBehavior should include IronSelectableBehavior

IronSelectableBehavior is now a dependency of IronLazyPagesBehavior
Fixed issues with various demos pulling in the wrong html file

this closes #28 and closes #29

Awesome, thanks a lot! :tada:


About migrations, pass argument

Hi @gi0baro
So far I used the database with auto_mgrate = True
The problem (solvable) is that in the first migration creates the entire structure of the database (which already exists) and if the application obviously produces an error. What I did to fix it is to modify the method up:
def up(self):
     pass

Apply the migration and then re-modify the method up to its original state. And from that point it works normal forward
I checked the new argument, which might work, but in the first migration it generates an error:

self._generate_migration_script(migration, source_rev.revision)
AttributeError: 'NoneType' object has no attribute 'revision'

I thought you could include a pass argument that tells the migration engine not to apply the migration, but is considered as applied.
It is understood?
Maybe something similar is already implemented, but not documented
Jose

@josejachuf I'm not sure I understood the problem correctly. What's wrong with the new command?

@gi0baro The situation is to start using migration in an existing database. The first migration will try to create the entire database, but the database already exists and when you try to apply the migration, an error will occur. I solved it like this:

comment all code in up
apply the migration
uncomment all the code in up
From then on there will be no problems

I think having a pass command or whatever you want to call it, would avoid doing all of the above. Because the migration would be considered applied, even though the application does nothing.
Sorry for not being able to explain it better

@josejachuf ok now I got it.
I'll think about a specific command to just override the migration status inside the database. But I will schedule this for 1.1

Do something like this:
$ weppy --app=myapp migrations generate -m "First migration"
$ weppy --app=myapp migrations pass

At this point we would already have synchronized the database (existing) and the migration system.

$ weppy --app=myapp migrations generate -m "Second migration"
$ weppy --app=myapp migrations up

This is implemented with https://github.com/emmett-framework/emmett/commit/ebd3f57cf6851ebf804bc2cb1b3f5bcb6b569542


Specify Aurora release in the gotchas and add a line about GTID lack of support

Aurora 1.x is based on 5.6..now that 2.0 is out and based on 5.7 the distinction needs to be clear
GTID is a 5.6 feature but is not supported in any Aurora releases.
A note on the maximum sized instance Aurora allows

Excellent contribution-- thanks Silvia!


Gulp dist does not create a distro

Using the current README, receive this error when running gulp dist:
[20:47:01] 'dist:berks-vendor' errored after 9.17 ms
[20:47:01] Error: Command failed: /bin/sh: berks: command not found

    at ChildProcess.exithandler (child_process.js:658:15)
    at ChildProcess.emit (events.js:98:17)
    at maybeClose (child_process.js:766:16)
    at Process.ChildProcess._handle.onexit (child_process.js:833:5)
[20:47:01] 'dist' errored after 590 ms
[20:47:01] Error in plugin 'run-sequence'
Message:
    An error occured in task 'dist:berks-vendor'.```

Looks like berkshelf isn't installed. It comes with the Chef DK, so you can install that to get berks, or you can just do gem install berks if you have ruby installed already.
Another thing is to make sure npm install has been run before you start doing anything. Unlike ruby, which will error out and tell you to run bundler, node just says it's missing a specific library.

Need to update the instructions to describe this in more detail (and perhaps run installchefdk.sh )


Upgrade ES Client to 7.x?

This is more of an enhancement request.  Just wonder if you have any plan to upgrade the Elasticsearch client from 6.4 to 7.3?  There are methods in 6.4 that are deprecated and not supported in 7.x version (ie: index call doesn't support the type field anymore)
Reason for this is that I'm writing a small library to handle SSE data inside a stream, using ES HighLevel Rest Client, my ES server is on 7.x and so I need my client to be on 7.x also.  When I used both my library and this plugin in jMeter, I ran into the incompatibility issue.  I'd love to use both.
Thanks

Hi @triportland ,
Have you upgraded the plugin to its latest version? We added support for ES 7.X a while back 😄 !

I understand and have been using it with ES 7.x.  The issue is if I'm in a situation where I have my own ES client with 7.3 version and it is incompatible with the 6.4 version.  I guess I can rewrite my code to use the LowLevel Rest Client just like you or just simply use your classes (with some modification since the data I'm capturing is not jMeter Sampler format). :)

That would be the way to😊


fixed access to deleted temporary string object

A have split the original commit in to two: This is the bug fix

Due to #10, that part of the code has been changed. I didn't notice. I didn't reproduce the same mistake https://github.com/kinow/gtest-tap-listener/commit/4f4318503b4fb3349ff8021b0d1b061c2ccb54db#diff-9095d050204b35b0221d0bf01f342102R239
This PR is then no more necessary. @BruceBlank, thanks again for having reported it to us. :+1:


Bigtable doesn't have a magical prefix range helper?

A common thing with Bigtable is the ability to scan based on a prefix, which is basically a range with start=prefix and end=someAlteration(prefix), but its tricky to do manually.
Any chance we can add a prefix filter that "just works"?

Go does this a bit trickily (see https://github.com/GoogleCloudPlatform/google-cloud-go/blob/master/bigtable/bigtable.go#L307), the gist seems to be (for those not familiar with go):

if the string is empty, the end range marker should just be empty string
otherwise, find the last character in the string that can be incremented (ie, abcde can increment position 4, abc\xff\xff can increment position 2.
using that pivot, take start.substr(0, n-1) and append the incremented character (start[n]+1)

I think -- in JS -- this would look something like....
var getPrefixEndRange = function(start) {
  var maxChar = String.fromCharCode(0xff);
  var position = start.length-1;

  // Walk backwards until we get to a character we can increment.
  while (start[position] == maxChar && position >= 0) position--;

  // If the position is -1, there is no reasonable end range for the prefix.
  if (position == -1) return '';

  var nextChar = String.fromCharCode(start.charCodeAt(position)+1)
  return start.substring(0, position) + nextChar;
}

Some test cases...
getPrefixEndRange('start'); // -> 'staru'
getPrefixEndRange('X' + String.fromCharCode(0xff)); // -> 'Y'
getPrefixEndRange('xoo' + String.fromCharCode(0xff)); // -> 'xop'
getPrefixEndRange(String.fromCharCode(0xff)); // -> ''
getPrefixEndRange(''); // -> ''

I can dig a prefix option. I might misunderstand, but couldn't we just use a row key filter?
Using the current implementation, I believe it would look similar to this
table.getRows({
  filter: {
    key: /^start/
  }
}, function(err, rows) {});

Implementing a prefix row scan via row key regex filter is inefficient, because it will scan the entire table, but return only the rows that match. A prefix row scan is typically implemented by computing the exact (start, end) rows, so it will only scan the subset of the table that will actually be returned back to the caller.

Gotcha, ok, I'll get started on this then :)

Yes -- what @mbrukman said. We want to look at keys only, and stop when we get to "the next one". The little snippet I jotted down gets us "where to stop" (ie, the first non-matching lexicographic key prefix)


Rotate image  and crop image on Canvas!! Need help

Is there any way like in android directly we can use Bitmap.createBitmap() so that we can easily perform image manipulations.

How can we rotate an image on canvas?
2.is there a way to crop image without any ui?

I have worked on image package but it is too slow on iOS! Need an alternative
Please help
Thank you Cheers!

https://stackoverflow.com/questions/44665720/how-do-i-crop-an-image-in-flutter mentions several methods.

what ????? he said on canvas!!! not widget


new jumbo version api

Jumbo uses v14 api, so the URL's need to change.
Could you perhaps set the URL as an env variable, or as parameter?  So we could easily change it in the configuration.

I have updated jumbo code to use the latest version of the API in #19

The current version of Jumbo API is v17. I have it working.
also
I suggest using different headers, as they probably redesigned security.
    HEADERS = {
        "User-Agent": "LuppieApp/1.1.28 Android/14.0 Mobile",
        "Accept": "*/*",
        "Accept-Encoding": "identity",
    }

LuppieApp is my name; feel free to use anything else. Mozilla 5 is blocked.


Change bundle-gen.py to account for new bundle format

xref: https://issues.redhat.com/browse/HIVE-1658

/unhold

/lgtm
Let's do this.

/test e2e-pool


[Godot 4] The editor crashed after opening a scene that has missing dependencies

Godot version
v4.0.alpha.mono.custom_build (9b78d6)
System information
Windows11, Vulkan, NVIDIA GT 1050 Ti
Issue description
It happens when I was trying to open a scene has missing dependencies in version 9b78d6.
I have built a newer version(f805ecf) on June 3rd, the bug also occurs in that version.
And, I got this log on console windows.

Steps to reproduce

Minimal reproduction project
Master Bug.zip
("SplashScreen" has missing dependencies because it was copied from another project, this is not a bug.)

@KoBeWi I did also use a regular build without C# support.

It seems the crash is caused by #61371, I reverted the commit and no longer get a crash

It seems to be caused by #61371, I reverted the commit and no longer get a crash.

I just tested it on latest master branch, It won't happen again.
I will close it.


Added tokenizers-remote

Using a remote tokenizer from huggingface isn't advisable in prod, and so the features to enable that aren't necessarily needed all the time. This PR introduces a feature flag tokenizers-remote which is on by default. When turned off, the Remote option for tokenizers is disabled.
The main win here for me at least is the removal of OpenSSL in a production environment. I deploy to various environments, many of which have no OpenSSL installation, and since I don't use a remote tokenizer, it's unnecessary.

Fantastic, thank you! I actually encountered this as well the other day, but was trying to fix it by making the tokenizers dependency entirely optional - in hindsight, I should have checked for an option on tokenizers itself 😅
I'll test this and merge it in if all's well.

I just need to fix one issue, I think the model crates are pulling in llm-base with it's default features, so I don't think there's a way to turn off  the tokenizers-remote feature yet from the outside.

Ok, fixed. I removed the feature as default from llm-base, but it's still turned on by default from llm, so the model crates don't need to change.
Now from the outside if you pull in the crate like so: llm = {version="1", default-features=false, features=["llama"]} no OpenSSL is included.

Thanks once again! Much appreciated, this will be handy for deployment on OpenSSL-less environments :)


🐛 BUG: Variables used in return in .astro files are ignored by TypeScript. '<variable>' is declared but its value is never read. ts(6133)

Describe the Bug
Astro's Typescript appears to be ignoring variables in the return value e.g. return redirect() in .astro files.
When using astro/tsconfigs/strictest the variables used in the return value the variables are highlighted with ts(6133): '<variable>' is declared but its value is never read.

I came across this issue whilst using a helper for my auth in .astro files, like so:
import { requiresSession } from "@/server/auth";

const { session, redirect } = await requiresSession(Astro);
if (!session) return redirect();

Where redirect is a function that returns Astro.redirect()
Steps to Reproduce
See my reproduction repo or follow below instructions.

npm create astro@latest

Empty
Typscript @ Strictest (Strict and base don't highlight, but still complain)

Install dependencies npm i
Open pages/index.astro and add (I'm using Math.random to get around other TS errors)

---
+const redirect = () => Astro.redirect('/')
+if (Math.random() > 0) return redirect()
---

		<meta charset="utf-8" />
		<link rel="icon" type="image/svg+xml" href="/favicon.svg" />
		<meta name="viewport" content="width=device-width" />
		<meta name="generator" content={Astro.generator} />
		<title>Astro</title>

Astro

TypeScript will highlight redirect with 'redirect' is declared but its value is never read.

Closing as a duplicate of https://github.com/withastro/language-tools/issues/476
(It's the same underlying issue)

Thanks! Yes, that'd be the same. 👍


Overwrite element converter function

Sometimes, I need to decorate transformed markdown element. For example,
# heading

->
heading

I used marked and it lets me do that. Would you want to expose a way to overwrite default converter function of all markdown element types?

@theking-laptop this is already possible. You can use two methods:

the documented one, by using a showdown (output) extension.
the undocumented one, by overriding a subparser completely.

You can override a subparser by redifining it. Check https://github.com/showdownjs/showdown/tree/master/src/subParsers
for the available parsers. If you need help with a specific question let me know.
The extension mechanism is also being rewritten and simplified. Those changes will prolly make it to the next release in the next couple of weeks.

Closing as there was no reply in 6 days


Multi Camera ambiguities warning and a dynamic-feature breakpoint issue

Bevy version
0.8.0
What you did
Create an Orthographic camera:
    commands.spawn_bundle(Camera3dBundle {
        transform: Transform::from_xyz(5.0, 2.0, 5.0).looking_at(Vec3::ZERO, Vec3::Y),
        projection: OrthographicProjection {
            scale: 3.0,
            scaling_mode: ScalingMode::FixedVertical(2.0),
            ..default()
        }.into(),
        ..default()
    });

And an ui camera:
commands.spawn_bundle(Camera2dBundle::default());

What went wrong
2022-08-20T11:16:29.212152Z  WARN bevy_render::camera::camera_driver_node:
Camera priority ambiguities detected for active cameras with the following priorities:
{(0, Window(WindowId(00000000-0000-0000-0000-000000000000)))}.
To fix this, ensure there is exactly one Camera entity spawned with a given priority for a given RenderTarget.
Ambiguities should be resolved because either
 (1) multiple active cameras were spawned accidentally, which will result in rendering multiple instances of the scene or
 (2) for cases where multiple active cameras is intentional, ambiguities could result in unpredictable render results.

How to set the priority of multiple cameras?
Additional Question
bevy_dylib_xxxx.dll and std_xxxx.dll cannot found in win11, when add breakpoint and debug in Clion.
I've run rustc --print target-libdir and set env like this:

when disable --features bevy/dynamic, the breakpoints goes well.
THX A LOT!

commands.spawn_bundle(Camera2dBundle::default());

Isn't a UI camera. You no longer need an explicit UI camera, I think?
To set the priorities, you set the priority field on the Camera component.


Can't Instantiate Client Object

Problem Description
I am developing within an AWS Cloud9 environment. I have Superset deployed inside of an EKS cluster via a helm chart per the documentation [1].
Steps to Reproduce
I expose Superset on port 8080:
kubectl port-forward service/superset 8080:8088 --namespace default

I then run the code suggested on this repository's README:
from supersetapiclient.client import SupersetClient

client = SupersetClient(
    host="http://localhost:8080",
    username="admin",
    password="admin",
)

This then results in the error:
Traceback (most recent call last):
  File "superset-api-client.py", line 6, in <module>
    password="admin",
  File "/home/ec2-user/.local/share/virtualenvs/environment-QZ1wNgYc/lib/python3.7/site-packages/supersetapiclient/client.py", line 64, in __init__
    self.charts = Charts(self)
  File "/home/ec2-user/.local/share/virtualenvs/environment-QZ1wNgYc/lib/python3.7/site-packages/supersetapiclient/base.py", line 106, in __init__
    for e in infos.get("edit_columns")
TypeError: 'NoneType' object is not iterable

Trouble Shooting Steps
In order to find out why infos.get("edit_columns") resulted in a NoneType object, I stepped through the Python debugger.
The error starts here in the client constructor. Charts which inherits from ObjectFactories in base.py and ultimately results in calling the api GET /chart/_info.
The specific url which we end up trying access  is 'http://localhost:8080/api/v1/chart/_info` via superset-api-client.
In order to validate this, we tried to access the same url with the requests library while the debugger was paused at base.py in line 103. We then entered an interactive shell and ran the following tests with pdb.
Test 1 (Duplicating the error)
import requests
import json
url = client.join_urls(self.base_url, "_info")  #self is the chart object
#url = 'http://localhost:8080/api/v1/chart/_info'
r = requests.get(url, headers = client._headers) #client is the superset client object
print(r.status_code) #r.status_code = 500
print(r.text) #r.text = '{"message":"Fatal error"}\n'

Test 2 (Adding Query)
In the UI for the Superset API, http://localhost:8080/swagger/v1,

A query object is added to the get request and it succeeds

We attempt to recreate while still in the debugger by modifying the request.
import requests
import json
url = client.join_urls(self.base_url, "_info")  #self is the chart object
#url => 'http://localhost:8080/api/v1/chart/_info'
query = {
  "add_columns": {
    "additionalProp1": {
      "page": 0,
      "page_size": 0
    },
    "additionalProp2": {
      "page": 0,
      "page_size": 0
    },
    "additionalProp3": {
      "page": 0,
      "page_size": 0
    }
  },
  "edit_columns": {
    "additionalProp1": {
      "page": 0,
      "page_size": 0
    },
    "additionalProp2": {
      "page": 0,
      "page_size": 0
    },
    "additionalProp3": {
      "page": 0,
      "page_size": 0
    }
  },
  "keys": [
    "add_columns"
  ]
}
#Query taken from UI example above
r = requests.get(url, headers =client._headers, params = {'q': json.dumps(query)})
print(r.text) # '{"add_columns":[{"description":"","label":"Slice Name","name":"slice_name","required":true,"type":"String","unique":false,"validate":["<Length(min=1, max=250, equal=None, error=None)>"]}]}\n'
print(r.status_code) #200
#Assuming there is a query response, the chart will still fail since it has no edit_columns key
infos = r.json()
output = infos.get('edit_columns')
#Output is of NoneType

Issues

The chart/_info API seems to need a query (q param)
There's no way to add a query the client object constructor
Even if a query was added, the Chart constructor will still fail since there's no "edit_columns"

#Question
Is there anything I can do to resolve this?

[1]
https://superset.apache.org/docs/installation/running-on-kubernetes

Hello, thank you for reporting the issue. There seems to have been breaking changes in the Superset API in newest version (3.x). I having found one additional issue to the one you mentioned (namely, strict policy on CSRF Token) and I implemented a quick fix which will be release soon.
I still need to some extra-tests to ensure compatibility depending on the version. Some extra development may be necessary in the future to allow API Client to be compatible with different Superset Versions. I'll keep you up to date.

thanks. I tested the changes and they seem to work for me.
A few more questions:

I don't see any documentation on how to create dashboards/charts. Is the preferred way of doing this something along the lines of:

from supersetapiclient.client import SupersetClient
from supersetapiclient.dashboards import Dashboard

client = SupersetClient(host="http://localhost:8080", username="admin", password="admin",)

dashboard_tile = 'automation-test-board'

testdash = Dashboard(id=None, dashboard_title='automation-test-board', published=False)

client.dashboards.add(testdash)

If so, it seems that defining testdash.slug is a prerequisite for this code to actually work as it errors out if no slug attribute is specified.
2. Are the create/delete operations  for superset resources on the road map to be developed/added?
Lastly, thanks again for your help.


Change of wording for MFA enablement on 2.4.0 client

Just validating the fix of the MFA enrollment.  When creating a new MFA required identity (through an auth policy),  when I click the MFA button I get this message:

Can we drop the d from enabled?

Gah typo. Thank you I'll fix that. Yes I made that change attempting to be more helpful to the user. It should say enable, not enabled

update artifact added to the release https://github.com/openziti/desktop-edge-win/releases/download/<IP_ADDRESS>/ thanks


Add job from new layout with ci-bootstrap on main branch

This job is the same as the staging branch one, but has a different parent defined in config repo[1],
which sets the ci-bootstrap main branch to be used.
The job based on ci-bootstrap repo should replace the code that we use from config repos, and allow us to test it as a role in its own repo. This job also takes a new network definition as input to configure the infrastructure needed to run our tests on zuul.
This patch doesn't wire up, or replace the current multinode-edpm job, but may do this in a next PR.
[1] https://review.rdoproject.org/r/c/config/+/51257/2/zuul.d/_jobs-crc.yaml
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running
[X] Appropriate documentation exists and/or is up-to-date:

[X] README in the role
[X] Content of the docs/source is reflecting the changes

recheck

lgtm and depends-on merged
but will you wire up the job here? or are we going to wait for testproject test on this before merge?

not really, this job is the same as the one wire up here https://github.com/openstack-k8s-operators/ci-framework/pull/1020
now that we moved all ci-bootstrap content from staging branch to main branch.
The difference between these 2 is only the branch that we do the checkout for ci-bootstrap

recheck

/approve

recheck

recheck

lgtm and depends-on merged
but will you wire up the job here? or are we going to wait for testproject test on this before merge?

not really, this job is the same as the one wire up here #1020 now that we moved all ci-bootstrap content from staging branch to main branch. The difference between these 2 is only the branch that we do the checkout for ci-bootstrap

i don't follow, i mean pull 1020 wires up some new pre-run plays for cifmw-extracted-crc-pre-bootstrap
my question was where are we going to run this new job you are adding here podified-multinode-edpm-deployment-crc but I see it is running in this PR there https://review.rdoproject.org/zuul/buildset/1efb03985d8049d591de5ac2c09afc3a
However I just checked and it looks like there are runs for this job from yesterday in other repos (edpm-ansible/install_yamls/dataplane-operator)  22nd but I don't see how if we are just adding it here. Did those run with depends-on? https://review.rdoproject.org/zuul/builds?job_name=podified-multinode-edpm-deployment-crc&skip=0

lgtm and depends-on merged
but will you wire up the job here? or are we going to wait for testproject test on this before merge?

not really, this job is the same as the one wire up here #1020 now that we moved all ci-bootstrap content from staging branch to main branch. The difference between these 2 is only the branch that we do the checkout for ci-bootstrap

i don't follow, i mean pull 1020 wires up some new pre-run plays for cifmw-extracted-crc-pre-bootstrap
my question was where are we going to run this new job you are adding here podified-multinode-edpm-deployment-crc but I see it is running in this PR there https://review.rdoproject.org/zuul/buildset/1efb03985d8049d591de5ac2c09afc3a
However I just checked and it looks like there are runs for this job from yesterday in other repos (edpm-ansible/install_yamls/dataplane-operator) 22nd but I don't see how if we are just adding it here. Did those run with depends-on? https://review.rdoproject.org/zuul/builds?job_name=podified-multinode-edpm-deployment-crc&skip=0

First of all, sorry Marios for not paying attention in your question. You are correct when asking about the name of the job and all the confusion. I choose by mistake the same name of the current multinode-edpm job, and that's why it was flat failing here. Thanks for pointing to that.
I changed the name to identify it as the job which uses ci-bootstrap role to configure the infra.
The idea is to replace the current edpm-multinode job, but in future PRs.
I will add testproject results as soon as reports success.

/hold
testing in testproject

podified-multinode-edpm-deployment-crc-bootstrap-testproject https://review.rdoproject.org/zuul/build/ceb1ae66061948bda033fc5b320d5b65 : SUCCESS in 1h 20m 47s

/test pre-commit

Ok thanks for update @viroel but question still stands - where are we going to wire it up to run (on pull requests?, periodic?)

This is going to be wired up in PRs, is focused on job with CRC based jobs running in a OCP env. The difference with current job is that is using ci-bootstrap role and is based on Pablo's new network definition. But the job for now is a mix of the new layout + old deployment. So it is needs more updates.

/hold cancel


demo issue

root@node6:~/data/wang_hao/lightweight-human-pose-estimation-3d-demo.pytorch# python demo.py --model human-pose-estimation-3d.pth --video dance.mov
Cannot load fast pose extraction, switched to legacy slow implementation.
: cannot connect to X server
I have not run step2,3 because it mistake, and I accept your advice on other issue only run the demo, and I get this mistake, how to solve it?

The error cannot connect to X server says, that no display available. Do you run the demo on the host with the display?

I get the problem, I run it on ssh

Ok, great.


Stabilization of ident_style?

Hi,
Could we stabilize this feature please?

The current implementation of the visual indent is kind of broken and I would like to fix it by using the visual indentation only when the whole expression or the type is simple.
Unfortunately, this is unlikely to happen in the near future because I don't have much time to work on it.

Closing in favor of #3346.


Only functions when given root privileges, GL Context Error

Which operating system does the issue occur on?
Gentoo Linux, Kernel v4.9.76-gentoo-r1, .config
If on linux, are you using X11 or Wayland?
X11 with the proprietary nvidia drivers "x11-drivers/nvidia-drivers".
Alacritty only launches when run as root, when run as a regular user it produces the following when using -vvv:
Alacritty encountered an unrecoverable error:
Error creating GL context; GL context creation failed

It doesn't work even when the x11 opengl implementation is selected using eselect, only when run as root. Any assistance is appreciated!

Can you run glxgears?

I can as root, functions fine. But as a regular user I get:
X Error of failed request:  BadValue (integer parameter out of range for operation)
  Major opcode of failed request:  153 (GLX)
  Minor opcode of failed request:  3 (X_GLXCreateContext)
  Value in failed request:  0x0
  Serial number of failed request:  31
  Current serial number in output stream:  32

Sounds like this is a configuration issue with your system. Once you can run glxgears as a regular user, alacritty should function fine.

Issue appears to be with the nvidia proprietary drivers and their associated permissions, I have a suspicion my udev rules weren't set up correctly. Switching my system to the nouveau drivers fixed the issue completely.


onpullingdown 事件在0.9.4 iOS版本还是不支持啊,是不是被遗漏了？

之前有文档说明 onpullingdown 事件在0.9版本会在iOS版本上支持啊。

貌似安卓已经有了，我跟进下:-)

This PR/issue doesn't receive response since Weex migrated to Apache two years ago and this repository is no longer active yet.
Feel free to ask question in new repository and thanks for your contribution.

This issue is outdated for a long time and will be closed now. You can create a new one if you still have questions.


Documentation (integration) how to use FDW (Foreign Data Wrappers) + pREST

What is FDW? read here
FDW connects to an external database (MySQL, MSSQL, Oracle, MongoDB, Kafka, HBase/BigTable, Elasticsearch, ...) and via PostgreSQL connection it is possible to consume data from the external database.
This integration should be written using WordPress, creating a Rest API using pREST
refs:

docker hub
docker source

@fabriziomello were you able to look at that integration (docs)?


Add adowair as owner, reviewer for Arabic content

This commit will add @adowair as an owner (approver) and reviewer for Arabic content on the website. This is based on the discussion here https://github.com/kubernetes/website/pull/44862#issuecomment-1995155214.

/assign @a-mccarthy @sftim

/unassign @sftim
I'm happy for anyone to review this.

/approve

@adowair this PR needs a rebase bc this pr https://github.com/kubernetes/website/pull/45793 was just merged.

/approve

/lgtm

/assign @katcosgrove

/unassign @katcosgrove
For this one, Prow doesn't know it, but an existing Arabic approver should approve the PR. @mboukhalfa is an Arabic approver, so:
/approve


Implement updateHeader function for Gcp Events Filter

Update Header functionality:
After receiving HttpRequest unbinded from SDK, update incoming HTTP request accordingly.
For PR#3, the detail logic for updateHeader function has not be implemented yet. It only returns absl::OkStatus() and do nothing.
Please checkout PR#3 for more detail.

For PR#6, the dependency of HttpRequest (boost beast) has been successfully imported into Envoy.
So the detail logic for updateHeader function has been implemented.
updateHeader takes an HttpRequest as input parameter and update the previous saved header accordingly.


added a new ENVIRONMENT_FILE variable

This adds a new Feature to sail which lets you optionally define a ENVIRONMENT_FILE.
See #368
-> https://docs.docker.com/compose/environment-variables/#using-the---env-file--option
Since the Option "--env-file" wasnt present before the default behaviour of docker is always parsing a ".env"-File if it's present.
So this change should not break anything - even if the --env-file ./.env-Option now always gets passed to docker when it's not overwritten.
See: https://docs.docker.com/compose/environment-variables/#the-env-file

To test:

Setup a new laravel install

curl -s https://laravel.build/example-app | bash
cd example-app

Ensure everything is still working like it did before with sail up
Copy a new env file: cp .env .env.test
Change i.e. the application port: echo "APP_PORT=8001" >> .env.test
Ensure the Application is running on the new Port when using this env ENVIRONMENT_FILE="./.env.test" sail up

I suggest exporting the file and making the changes you need 👍

So.. supporting multiple docker-compose files is okayish but supporting multiple env's isn't?
By supporting multiple docker-compose files #355 the big plus we get there is also supporting overrides for M1 macs or other platforms. By preventing custom env files there is no way to actually benefit from that without manually overwriting several variables?
We surely can manage our own repository with the changes applied but is there a reason to skip the env-support or the ability to use custom env-files on sail?

@taylorotwell I think this is a bug in Sail.
Laravel expects an APP_ENV variable and uses the corresponding .env file. But Sail does not seem to do that. I set APP_ENV=local for example and Sail/Docker does not use .env.local.


Storage SDK doesn't seem to implement Copy Blob

The Blob Service REST API offers a Copy Blob operation which is useful for copying blobs from one storage account to another or within the same storage account.
Currently the BlobClient doesn't implement this operation. It only implements the Copy Blob From URL operation which has a limit of 256MiB for the blobs it can copy.
We'd like to have an option to copy large blobs between storage accounts in the SDK.

Hi @neaorin, while investigating this it seems we actually implemented the CopyBlob rest call and not the CopyBlobFromUrl rest call. The naming is misleading though.
Have you attempted to copy file size > 256MB and it failed?


fix (build): Introduce Gradle-based build in Enola.dev

To simplify build, and for easier VSC IDE integration.
Partially motivated by frustration due to https://github.com/salesforce/bazel-vscode-java/issues/73.
TODO https://github.com/enola-dev/enola/compare/main...vorburger:enola:gradle3

With https://github.com/salesforce/bazel-vscode-java/issues/73 sorted out, I'm closing this (un-merged); at least for now.


SatelliteData class not encouraging immutability

This might not actually be desirable for other implementations right now, but it might be good to include in the next major version release.
Basically SatelliteData currently requires that you override setter methods if you implement the interface directly. If the calculators require this benefit of mutability, then there's not a nice way around this.
However, I'm trying to define my own class of data for a hex, and I want those objects to be immutable. If the board state changes, I want to be able to place newly constructed satellite data objects on those hexes in the grid.
So, this is barely a big deal since I can still just override them to have them do nothing anyway, but it does look like this:
public class MySatelliteData implements SatelliteData
{
    . . .

    public void setPassable( final boolean passable )
    {
        return;
    }

    public void setOpaque( final boolean opaque )
    {
        return;
    }

    public void setMovementCost( final double movementCost )
    {
        return;
    }

    . . .
}

Definitely just being nit-picky with this one. Immutability is definitely a good practice to be eliciting. It does result in a tad less flexibility from this library, but you are probably more than likely saving a consumer from potentially getting themselves into trouble via mutability.

I had a think about this as I made some other changes and came to the following conclusion:  #52 SatelliteData inteface should be optional.
WRT this issue: mutability is a user choice, do they wish to store a single instance of the data (and mutate it) or keep re-storing immutable classes.  This is a HUGE design decision in the modern multi-core world and depends on context that Mixite cannot know. I really like that Mixite is entirely linear, entirely read-only. Any threading concerns only relate to the storage backend and should be the user's problem to handle. This should be well documented in the readme / source.
Apart from those docs, IMO #52 supercedes this issue.


Setup project

Just the initial project setup, so that contributors can get started easily, and for SoC.

Composer;
PHPUnit;
Changelog and readme.
Github Actions.
This builds on PHP 5.5 - 8.0.

@Jean85, I added PHP 5.3 and 5.4 to the matrix, so that this matches the spec. Unfortunatley, as I explain in
2483bc1, it is not possible to make the build pass on PHP 5.3 right now. So I declared it as experimental, and it is allowed to fail.

@Jean85, I added PHP 5.3 and 5.4 to the matrix, so that this matches the spec. Unfortunatley, as I explain in
2483bc1, it is not possible to make the build pass on PHP 5.3 right now. So I declared it as experimental, and it is allowed to fail.

You can overcome that limitation by removing the dependency during the build of that specific job, before calling composer update. I did something similar here: https://github.com/getsentry/sentry-symfony/blob/b9fbbdf6492e0571a39014b9ecd547093d697d86/.github/workflows/tests.yaml#L62-L63

Ok, interesting workaround, thanks! Will try 👍

@Jean85, PHP 5.3 works now, but doesn't run PHPCS because PSR-12 does not exist there. But PHPCS on PHP 5.4 should be enough. Please take a look, and merge if all good 👍

@Seldaek, it would be cool if there was a Composer flag like -W but that allows any dependency to be up/downgraded, even if not a dependency of the package being required. It may not be all that safe, but would sure save a lot of hassle :)


Not an issue, just info

In scanner.c you have a function SCANNER_Stop(void).
That function (ignoring it's name) has no connection to any of the rest of the code in that file (scanner.c), the function is for stopping the other channel/frequency scanner rather than the freq/ctcss/cdcss scanner/search.
I've renamed the freq/ctcss/cdcss scanner/search to search rather than scanner so that my brain can tell the difference between the two, took me some time to work that one out lol. I've yet to rename the files to search.c and search.h

Here's a question, what does GPIOB_PIN_BK1080 actually do, does it simply power on/off enable/disable the BK1080 chip I wonder ?

Most likely... Just saw it and BK1080Init's bDoScan is really bEnable... I'll rename it. When I come back home, I'll check if I can trace the MVU pin going to the BK1080 for ena better name.

Based on it is being active low, I'd say it's the SEN pin (serial comms enable)

okey doke

no hurry, I'm not

I've updated your K5 spread sheet, I think you're missing the 7 VFO attribute bytes at 0x0E20 ..
UV K5 EEPROM.zip

Thanks, I will update.
Check your inbox!

oo oo oo !!
Thank you very much :)

hmmm, someone's shared some details/info with me today, just in time you might say, I was about to share a good chunk of 'unique' code and functionality into the project too, but what I've seen explains the past week or so.
Even as we get old(er), we still live n learn.

You've lost me on that one.


Add my first file

Created pull request to add newly created file from a new branch to the main branch.

:+1+ Everything looks great!


 I need to investigate but I don't wanna crop all the queued tests

weird af

thanks bud!


VOTE for MXNet's new logo!

We need a new logo to replace the current one since I received several negative comments about the latter.
William Wang from Amazon kindly provides us several candidates. Please thumb up the post below if you like it. Any comment and suggestion is also welcome.
Please vote with github's thumb up feature:

Ribbon 1

The idea of a continuous shape to form the logo I think communicates the concept of flexilbity well, and is conceptually similar to related competitors in the space. This gives us a logo mark that can be used alone for the “MX” or with the logotype below combined.

Ribbon 2

Same idea as above, but slightly different design.

MX polygon

Abstracted shape of “M” and “X” that is intended to look like a butterfly/crystal shape. It has some three dimensionality depending on the colors we use.

Continuous line

A single line forming MX, with a partial infinity loop at the end

Let't use github's new thumb up feature.
BTW, I like green better :)

Everything is available in green. Vote this post if you like blue.

Vote this post if you like green

Please feel free to submit any other logo candidates.

The blue one is actually a little different from the green one. I like the blue one's shape better. M should be a little smaller and X should be bigger.

MX polygon is definitely the best one.

The first one is concise and easy to recognize.

I prefer blue than green for software icon.

Hi everyone, William here. I wanted to share a few other color options since I feel strongly about avoiding the green design so that the MXNet logo can be more unique. I added some sample colors and reference to other brands out there to help the group reach a final color choice. To me, in order to stand out, the MXNet logo should use blue or purple as the main color.

Design for MXNet's new logo to embrace open source. Please thumb up the post below if you like it. Any comment and suggestion is also welcome.
http://photo.weibo.com/h5/comment/compic_id/1022:230597eecbfea942c091c899153f4667c126ff

Design for MXNet's new logo to embrace open source. Please thumb up the post below if you like it. Any comment and suggestion is also welcome.
http://t.cn/RfB1fqH

Design for MXNet's new logo to embrace open source. Please thumb up the post below if you like it. Any comment and suggestion is also welcome.
the logo site is: http://t.cn/RfB1fqH

I like the blue best but "MXNet" text could be better.


Add bevy_shader_hot_reloading feature for easier renderer development

Description
Having to recompile bevy and re-run test apps in order to iterate on shaders for bevy renderer features is slower than necessary. The goal of this change is to add a non-default feature that allows bevy renderer feature developers to edit core bevy renderer shaders and have them hot reload. This will enormously speed up development due to reducing iteration time. More changes may be needed to not replace shader assets if they in any way invalid, but this should be a good start.
Solution
NOTE: Builds on top of https://github.com/bevyengine/bevy/pull/3643 and watch for changes needs to be enabled using the resource as in that PR for this PR to work.
This was way more complicated than I expected:

Hot-reloading only notifies the handle which was created from AssetServer::load which is constructed from the AssetPath. This is convenient as the filesystem watcher for hot-reloading naturally and unsurprisingly produces events based on paths to files that changed.

To solve this I chose to add what I have called 'aliases' for handles.
Add aliases: HashMap<HandleId, HashSet<HandleId>> to Assets<T> to hold all handles that are aliases for the strong handle for an asset as obtained from AssetServer::load(). An alias handle is a handle made via other means, such as pub const PBR_SHADER_HANDLE: HandleUntyped = HandleUntyped::weak_from_u64(Shader::TYPE_UUID,<PHONE_NUMBER>767701046);.
Add alias_to_handle: HashMap<HandleId, HandleId> to Assets<T>
Add:

add_alias<H: Into<HandleId>, A: Into<HandleId>>(&mut self, handle: H, alias: A),
remove_alias<H: Into<HandleId>, A: Into<HandleId>>(&mut self, handle: H, alias: A),
and get_handle_from_alias<A: Into<HandleId>>(&self, alias: A) -> Option<&HandleId>
...to Assets<T> to be able to add/remove aliases, and for a given alias handle, get the strong handle. Assets<T>::get() uses the latter to attempt to get the asset using an alias handle if it exists. This allows materials to use PBR_SHADER_HANDLE and due to setting that as an alias handle for the strong handle, Assets<Shader>::get() 'just works'.

When using Assets<Shader>::set_untracked() with the const HandleUntyped (e.g. PBR_SHADER_HANDLE), the Shader is constructed and immediately chained with Shader::with_import_path() to define the shader import. When using AssetServer::load(), the Shader does not exist until loaded.

If a shader import path is necessary, use a system to check for when the shader asset is loaded, mutably get it and set the import path. I think this could be possibly improved somehow but it's good enough as a first shot perhaps.

Making use of these changes:

Move all the shader files into the assets/shaders/ directory in subdirectories named after their respective crates, e.g. assets/shaders/bevy_pbr/pbr.wgsl.
Add the non-default bevy_shader_hot_reloading feature
Where shaders are used in bevy rendering crates:

Move the current Assets<Shader>::set_untracked() code into a block for when the feature is not set.
Add a new block for when the feature is set, and use a pattern of:

let shader_handle = asset_server.load("path/to/shader.wgsl");
assets_shader.add_alias(shader_handle, SHADER_HANDLE);
app.world.insert_resource(Shaders {
    shader_handle,
});

https://user-images.githubusercontent.com/302146/149539258-98b9f832-9e59-40db-bbe7-d014539b7ae6.mp4

I only get a gray window with when running the sprite example on this branch.
I also checked on #3643 and there sprite shows up as it should.

I only get a gray window with when running the sprite example on this branch.

Fixed.

I also updated the usage example for shaders with entrypoints and for imports.

the new feature should be mentioned in https://github.com/bevyengine/bevy/blob/main/docs/cargo_features.md

Should this PR be closed now that #3966 has been merged?

Implemented differently by #3966


querystring validation/parsing

First off, thanks for creating this, I'm very excited to see where this goes, and it has been great to work with thus far.
Question about querystring in the schema, it doesn't appear to be doing anything. I'm hoping you could point out what I'm missing:
Here's an example:
fastify.route({
    "method": "GET",
    "url": "/list",
    "schema": {
      "querystring": {
        "limit": {
          "type": "integer"
        }
      },
      "out": {
        "type": "array",
        "items": {
          "type": "object",
          "properties": {
            "id": {
              "type": "string"
            },
            "count": {
              "type": "integer"
            }
          }
        }
      }
    }
  })

req.query outputs:

Valid query: /list?limit=10
{ limit: '10' } // Limit is a string

Invalid query param: /list?limit=a
{ limit: 'a' } // Invalid limit passed through as string

Extra query params: /list?limit=1&lookatme=true
{ limit: 'a', lookatme: 'true' }

Hopefully you can tell me what I'm doing incorrectly, thanks.

The syntax is slightly different. See https://github.com/mcollina/fastify/blob/master/test/get.test.js#L19-L28.
However, I like your syntax more.
Would you like to send a PR to add the support for either better reporting or key/value type of validation.
Maybe this can help http://json-schema.org/documentation.html (we can use the meta-schema to validate that the schema is valid JSON-schema).

Thanks @mcollina, worked like a charm. I'll see if I can get a PR in. I'm working off the reply-refactor branch, so I may wait until that PR is completed.

@brwnll that might be a good idea :)


[Improvement]: Avoid checking the annotation map when we know closure in question don't have any annotations

Description
Currently when we create any closure we are checking annotation map (ref). We can avoid this when we know the closure in question don't have any annotations attached (such as for closures created as part of desugaring process).
Potentially processFPValueAnnotations itself could be improved by avoiding early returning if key is not in the map without extracting the type.
Describe your problem(s)
No response
Describe your solution(s)
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Let's also update the processFPValueAnnotations method (https://github.com/ballerina-platform/ballerina-lang/blob/v2201.8.4/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/AnnotationUtils.java#L124) to avoid extracting the type unless required.


需求：希望增加自动备份功能或链接第三方网盘、本地/远程webdav等方式备份数据来增加数据系统的稳定可靠性。

Describe the solution you'd like
I hope to add an automatic backup feature or link to third-party cloud storage, local/remote WebDAV, etc., to enhance the stability and reliability of the data system
Type of feature
API
Additional context
因为这个项目是跑在软路由虚拟机的docker中，它并不像云服务器或物理机那样可靠，所以我想要增加一道数据安全保障来确保我的笔记不会丢失，这对我很重要，希望开发者能够做出备份功能，十分感谢您的付出！
I would like to emphasize the importance of data security for my project, as it runs on a software router virtual machine within Docker. Unlike cloud servers or physical machines, this environment may not be as reliable. Therefore, I am keen on incorporating a backup mechanism to ensure the safety of my notes. This is crucial to me, and I would greatly appreciate it if the developers could implement this backup feature. Thank you very much for your efforts!

hi. there are already plenty of options to setup your backup workflow
for example, you can create a cron job that backs up an sqlite database and then uploads it to a cloud provider of your choice using rclone. or you can use an external database like postgres and then setup a replication


Publish 8.3-20230715071551+0000

Publish 8.3-20230715071551+0000

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.


Add to up-for-grabs.net

Add to http://up-for-grabs.net/
How: https://github.com/up-for-grabs/up-for-grabs.net#add-your-project

Closed once up-for-grabs/up-for-grabs.net/pull/493 is merged.

up-for-grabs/up-for-grabs.net#493 has been merged so this issue can be closed.

Thx!


Added script for database creation

This looks good to me. We will just need to be sure that we make it difficult to accidentally run this file, but that can come later.

I'd imagine it wouldn't be, and it's not something that would be added into the features of the website itself. That first line is mostly there in the event we give the website bad data during testing and need a reset.


Unhandled exception in Vlc.DotNet.Wpf.VlcVideoSourceProvider.VideoFormat

I have an issue/question  about Vlc.DotNet. (both)
Generic information

Vlc.DotNet version : (3.0.0-develop322)
Vlc.DotNet project used : (WPF)
libvlc version :  (x64)
.net version : (Please fill)
Project language : (C#)
Project build architecture : (x64)
Operating system : (Windows x64)

Summary
This sometimes occur on systems with low RAM  (4GB) memory. It causes application crush down.
System.IO.IOException: The paging file is too small for this operation to complete. at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath) at System.IO.MemoryMappedFiles.MemoryMappedFile.CreateCore(SafeFileHandle fileHandle, String mapName, HandleInheritability inheritability, MemoryMappedFileSecurity memoryMappedFileSecurity, MemoryMappedFileAccess access, MemoryMappedFileOptions options, Int64 capacity) at System.IO.MemoryMappedFiles.MemoryMappedFile.CreateNew(String mapName, Int64 capacity, MemoryMappedFileAccess access, MemoryMappedFileOptions options, MemoryMappedFileSecurity memoryMappedFileSecurity, HandleInheritability inheritability) at Vlc.DotNet.Wpf.VlcVideoSourceProvider.VideoFormat(IntPtr& userdata, IntPtr chroma, UInt32& width, UInt32& height, UInt32& pitches, UInt32& lines)
How it could be handled?

Closing this for now since you didn't reply.
Feel free to reopen if needed.


求Docker版本，我想运行在CentOS的机器上，但是报错puppet缺少运行环境，已安装且复制环境，求协助解决。

报错信息：
22:18:46 ERR PuppetWeChatBridge start() exception: Error: Failed to launch the browser process!
/root/wechat-openai-qa-bot/node_modules/puppeteer/.local-chromium/linux-982053/chrome-linux/chrome: error while loading shared libraries: libatk-1.0.so.0: wrong ELF class: ELFCLASS32
用ldd chrome命令显示缺少依赖，下载并复制到/usr/lib64，仍不行。。折腾了好久，求大佬协助

    libatk-1.0.so.0 => not found
    libatk-bridge-2.0.so.0 => not found
    libcups.so.2 => not found
    libxcb.so.1 => not found
    libxkbcommon.so.0 => not found
    libX11.so.6 => not found
    libXcomposite.so.1 => not found
    libXdamage.so.1 => not found
    libXext.so.6 => not found
    libXfixes.so.3 => not found
    libXrandr.so.2 => not found
    libgbm.so.1 => not found
    libpango-1.0.so.0 => not found
    libcairo.so.2 => not found
    libatspi.so.0 => not found


fix: galoy-info component colors

Dark Mode
 Light Mode

can you add tag Andrej to those design PR? I don't have his github but you can ask on mattermost


Enabling two factor authentication breaks sign in

If a user has enabled two factor authentication for GitHub they are no longer able to sign in to the app. Existing sessions will still work.

Seeing this error again on Android 5.0. Latest Github app. Was able to auth on site, but cannot in app.

Are you running the latest master? If it's from the store, the last version
github uploaded wasn't able to handle their new auth scheme
On Sun, Jul 19, 2015 at 9:29 AM Serge Matveenko<EMAIL_ADDRESS>wrote:

Seeing this error again on Android 5.0. Latest Github app. Was able to
auth on site, but cannot in app.
—
Reply to this email directly or view it on GitHub
https://github.com/forkhubs/android/issues/404#issuecomment-122678650.

@hzsweers well, this means the only one thing. There is no working version released. Ok then.

Yes, we're pretty clear about that in the README.

@hzsweers Thank you very much for the answer! It is the great news. Looks like the original app should just be removed from the Google Play. What a shame for Github team.
Again, thank you and keep up the good work!

Thanks for the info.
GitHub app has been removed from my devices.

You can disable TFA, log into your github account on your phone and reenable TFA if you prefer to quickly fix things on your side.

This is still an issue for me!

Please include details of what you tried, that is not enough information to help.

Login with GitHub Android App fails when entering the 2 factor key (I use authy)
It accepts the password, then asks me for the PIN, I copy it from authy and it says "wrong password" throwing me back to the login-widget
Login on mobile website works without problems.

And this is built from master or the old GitHub app?

This is from play store. As the last message was from Nov '15, I thought this bug got forgotten.

Read the README and CONTRIBUTING.
On Wed, Jul 6, 2016, 1:20 AM Moritz<EMAIL_ADDRESS>wrote:

This is from play store. As the last message was from Nov '15, I thought
this bug got forgotten.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pockethub/PocketHub/issues/404#issuecomment-230708402,
or mute the thread
https://github.com/notifications/unsubscribe/ABTEvpEe7NPiiAlXpikVg2AvdqI6UREZks5qS2U8gaJpZM4A90O4
.

Authorize button is insensitive now, can't login. I didn't (and won't) try disabling 2FA but I wonder if it's related.

2017 - the bug that keeps on bugging. Cannot use the app since 2FA is on. This is still an issue, latest Android 8, Nexus 5X.

https://github.com/sessions/verified-device/mobile?auto=true


Choose the testing tools for React and the API

Because there are a lot of available testing tools for React, we should center on the ones we want to integrate. Some preference should probably be given to those that work with our current CI/CD pipeline.
Acceptance criteria:

We have identified what unit testing library we're going to use
Do we also want to include enzyme?
Do we want to add in some automated testing with node and puppeteer?
What kind of dynamic scanning on the API do we want to do

Regarding testing library: looking at this popularity poll (npm download trends for jest, mocha, react-testing-library); jest seems to be favored by the community.
It probably makes sense to integrate enzyme as well, but I think that is going to depend on the UT authors and how they want to do their validation.

Slightly skewed in that jest ships with create-react-app -- but for unit testing I could go either jest or mocha...

for e2e testing I would suggest cypress

@kevinhaube I'm assigning this to you for this sprint. Work with @Bisonburger to follow up on anything he's done in this space. But this is a big first step to getting the React site out.
fysa: @jlusds @Adrian-Brewster

@MauriceReeves-usds (et al)
Open questions:

Are we aiming to test state/props or just do DOM-based testing?
What is our back-end test suite built on?

Initially - DOM testing is sufficient - but i think we'll want to expand later to state
The back-end tests are ultimately based on JUnit
As I stated earlier - we've got some cypress tests in place - with some Jest for the unit level stuff...


Added Sardegna

Adding the data from most of Sardegna.
There are two files to be dowloaded from the page indicated  (strato03 and strato09) and a trivial join between house, street, location and municipality tables is needed.
I made the processing offline in QGIS and uploaded the processed file to dropbox. Let me know if another procedure is preferred.
Closes #1477.

this will close https://github.com/openaddresses/openaddresses/issues/1477

Re-run this please

I did the simple join here in QGIS GUI. Let me know if you want a script that will be able to do this automatically, won need to research a bit. Perhaps an intermediate SQLite form or something like that?

Format seems okay to me, and thanks for your recent contributions! I think it would make sense to make you a member of the OA Github organization, so that you can upload to S3 instead of relying Dropbox, and so that you can make branches directly in this repository.

I'm going to merge this but I do want to move data out Dropbox and into S3.


Can't output the data from the API

Hi,
I use the Harvest app API, which is a REST API. After it I use the JSON Input, FIlter Rows and Select Values blocks in PDI. At the end I try to output the data into Google Drive Spreadsheet.
It works fine for the 1 entity in the JSON. But when I have a list of entities, I receive the following error:

2019/03/22 17:48:53 - Google Spreadsheet Output.0 - ERROR (version <IP_ADDRESS>-12, build 1 from 2017-05-16 17.18.02 by buildguy) : Unexpected error
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - ERROR (version <IP_ADDRESS>-12, build 1 from 2017-05-16 17.18.02 by buildguy) : org.pentaho.di.core.exception.KettleException:
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - com.google.gdata.util.InvalidEntryException: Bad Request
[Line 1, Column 2279, element gs:colCount] The count must be specified.
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - Bad Request
2019/03/22 17:48:53 - Google Spreadsheet Output.0 -
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at org.ccci.gto.pdi.trans.steps.googlespreadsheet.GoogleSpreadsheetOutput.processRow(GoogleSpreadsheetOutput.java:140)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at org.pentaho.di.trans.step.RunThread.run(RunThread.java:62)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at java.lang.Thread.run(Unknown Source)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - Caused by: com.google.gdata.util.InvalidEntryException: Bad Request
[Line 1, Column 2279, element gs:colCount] The count must be specified.
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at com.google.gdata.client.http.HttpGDataRequest.handleErrorResponse(HttpGDataRequest.java:602)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at com.google.gdata.client.http.GoogleGDataRequest.handleErrorResponse(GoogleGDataRequest.java:564)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at com.google.gdata.client.http.HttpGDataRequest.checkResponse(HttpGDataRequest.java:560)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at com.google.gdata.client.http.HttpGDataRequest.execute(HttpGDataRequest.java:538)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at com.google.gdata.client.http.GoogleGDataRequest.execute(GoogleGDataRequest.java:536)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at com.google.gdata.client.Service.update(Service.java:1563)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at com.google.gdata.client.Service.update(Service.java:1530)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at com.google.gdata.client.GoogleService.update(GoogleService.java:597)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at com.google.gdata.data.BaseEntry.update(BaseEntry.java:639)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	at org.ccci.gto.pdi.trans.steps.googlespreadsheet.GoogleSpreadsheetOutput.processRow(GoogleSpreadsheetOutput.java:109)
2019/03/22 17:48:53 - Google Spreadsheet Output.0 - 	... 2 more

Could you help me with this, please?
It is quite urgent, because I do some stuff at work and have a deadline for the end of the month. But now I stuck and have no idea, what to change/check/do.
It looks like it produces more columns instead of more rows when the new entity on the list is processed.
When parsing the JSON I use something like:
$.*.id
When I use $.0.id the error is the same. I totally have no idea, what's going on.
Let me know, if you need any additional information about the flow.
Great thanks in advance and kind regards,
Arek

I forgot to mention. I use Pentaho Data Integration 7.1


Is there a way to know the topic when receiving a notification sent via topic messaging?

In my app, I subscribed to multiple topic. I need to know from which topic am I receive the notification. I guess this not possible atm?
In Android, there is RemoteMessage.getFrom() method which return the sender (senderId or topic): https://firebase.google.com/docs/reference/android/com/google/firebase/messaging/RemoteMessage.html#getFrom()
But I can't find an equivalent on iOS.

I don't think these is concept of topic in iOS so probably not. try passing the topic in payload?


⚠ Bofrost.at has degraded performance

In 21837b5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 862 ms

Resolved: Bofrost.at performance has improved in 72a62f2 after 11 minutes.


Draft: FI-1259: Fix invoice-canister example

Debug and fix invoice-canister example.

Closing since the invoice canister has now been relocated.


Not showing buttons(pause, next track) in notification panel [ANDROID]

Describe the bug
Not showing buttons in notification panel such as pause, next track etc...

Environment:
> react-native info

System:
OS: macOS Sierra 10.12.6
CPU: (2) x64 Intel(R) Pentium(R) CPU G2030 @ 3.00GHz
Memory: 174.85 MB / 6.00 GB
Shell: 5.2 - /bin/zsh
Binaries:
Node: 13.2.0 - /usr/local/bin/node
Yarn: 1.12.3 - /usr/local/bin/yarn
npm: 6.13.1 - /usr/local/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 11.2, macOS 10.13, tvOS 11.2, watchOS 4.2
Android SDK:
API Levels: 28
Build Tools: 28.0.3
System Images: android-28 | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.4 AI-183.6<IP_ADDRESS>22156
Xcode: 9.2/9C40b - /usr/bin/xcodebuild
npmPackages:
react: 16.11.0 => 16.11.0
react-native: 0.62.2 => 0.62.2
npmGlobalPackages:
react-native: 0.60.4

react-native-track-player: ^1.2.3
Running on real device, OS version: Android: 9.0
Code
app.js

import React, {useEffect} from 'react';
import {Button, StatusBar, View} from 'react-native';
import TrackPlayer from 'react-native-track-player';

const track = {
  id: 'unique track id', // Must be a string, required
  url: 'http://d.topmusic.uz/new/singles/vol119/Inna_-_Sober_.mp3', // Load media from the network
  title: 'Avaritia',
  artist: 'deadmau5',
  genre: 'Progressive House, Electro House',
  artwork: 'https://i.picsum.photos/id/100/200/200.jpg',
  date: '2014-05-20T07:00:00+00:00', // RFC 3339
};

const App = () => {
  useEffect(() => {
    startPlay();
  }, []);

  const startPlay = async () => {
    await TrackPlayer.setupPlayer();
    await TrackPlayer.add([track]);
    TrackPlayer.play();
  };

  return (
    <>
      <StatusBar barStyle="dark-content" />
      <Button title='Play' onPress={() => TrackPlayer.play()} />
      <Button title='Stop' onPress={() => TrackPlayer.pause()} />
    </>
  );
};

export default App;

index.js

import {AppRegistry} from 'react-native';
import {name as appName} from './app.json';
import TrackPlayer from 'react-native-track-player';

import App from './App';

AppRegistry.registerComponent(appName, () => App);
TrackPlayer.registerPlaybackService(() => require('./service'));

service.js

import TrackPlayer from 'react-native-track-player';

module.exports = async function() {

  TrackPlayer.addEventListener('remote-play', () => {
    TrackPlayer.play()
  })

  TrackPlayer.addEventListener('remote-pause', () => {
    TrackPlayer.pause()
  });

  TrackPlayer.addEventListener('remote-next', () => {
    TrackPlayer.skipToNext()
  });

  TrackPlayer.addEventListener('remote-previous', () => {
    TrackPlayer.skipToPrevious()
  });

  TrackPlayer.addEventListener('remote-stop', () => {
    TrackPlayer.destroy()
  });
};

You need to call updateOptions, and include the capabilities you want to appear in the notification.
https://react-native-track-player.js.org/documentation/#updateoptionsoptions

Hi. I'm using react-native-track-player version 2.1.2 and I'm unable to see the icons on the notification panel.

react-native info

System:
    OS: Windows 10 10.0.19044
    CPU: (8) x64 Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz
    Memory: 840.68 MB / 15.63 GB
  Binaries:
    Node: 16.14.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.15 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 8.5.2 - C:\Program Files\nodejs\npm.CMD
    Watchman: Not Found
  SDKs:
    Android SDK: Not Found
    Windows SDK:
      AllowDevelopmentWithoutDevLicense: Enabled
      Versions: 10.0.17763.0, 10.0.19041.0
  IDEs:
    Android Studio: Version     2<IP_ADDRESS> AI-211.7628.21.2111.8139111
    Visual Studio: 17.1.32319.34 (Visual Studio Community 2022)
  Languages:
    Java: 11.0.10
  npmPackages:
    @react-native-community/cli: ^7.0.3 => 7.0.3
    react: 17.0.2 => 17.0.2
    react-native: 0.67.3 => 0.67.3
    react-native-windows: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

playNotificationService.js

import TrackPlayer, { Event } from "react-native-track-player";

module.exports = async () => {

    TrackPlayer.addEventListener(Event.RemotePlay, () => TrackPlayer.play());

    TrackPlayer.addEventListener(Event.RemotePause, () => TrackPlayer.pause());

    TrackPlayer.addEventListener(Event.RemoteNext, () => TrackPlayer.skipToNext());

    TrackPlayer.addEventListener(Event.RemotePrevious, () => TrackPlayer.skipToPrevious());
};

I'm registering the player in index.js
index.js
import TrackPlayer from 'react-native-track-player';
import service from './playNotificationService';

AppRegistry.registerComponent(appName, () => App);
TrackPlayer.registerPlaybackService(() => service);

And setting it up in App.js
App.js
import TrackPlayer, { Capability } from 'react-native-track-player';

export default function App() {
  useEffect(() => {
    async () => {
      await TrackPlayer.setupPlayer({});

      await TrackPlayer.updateOptions({
        // Media controls capabilities
        capabilities: [
          Capability.SkipToPrevious,
          Capability.SkipToNext,
          Capability.Play,
          Capability.Pause,
        ],

      });
    }
  }, [])

return ( ... );
}

Please help me resolve it. I'm not sure where I went wrong.

check the latest doc,
notificationCapabilities: [
Capability.SkipToPrevious,
Capability.SkipToNext,
Capability.Play,
Capability.Pause,
],


E2E Tests: Add e2e tests for publisher logo file upload

Context
Publisher logos suffered a regression. This was fixed but we're adding tests for resiliency.
Summary
Adds an e2e test for publisher logos.
Relevant Technical Choices
n/a
To-do
n/a
User-facing changes
n/a
Testing Instructions
tests should pass
QA

[x] This is a non-user-facing change and requires no QA

Reviews
Does this PR have a security-related impact?
no
Does this PR change what data or activity we track or use?
no
Does this PR have a legal-related impact?
Does it matter what gifs we use?
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #7224

Thanks @spacedmonkey! Just fixed those 😄