MaaS: Added check to monitor container storage

This check will alert when the storage within a container has
reached >= 85%.
Closes #1895
Connects: https://github.com/rcbops/u-suk-dev/issues/1255
Signed-off-by: Kevin Carter<EMAIL_ADDRESS>

This was added as part of the work being done in https://github.com/rcbops/u-suk-dev/issues/948

More debuging:
https://gist.github.com/antonym/d7d6fb3a471a25a1fe2e8311cd2916b2

Thanks @antonym for the review. I pushed an update.

Looks like the configs are getting the right names now, but the checks themselves are failing when running.  The plugin returns ok, but the alarms show:
metric "container_rpc-openstack_cinder_api_container-2780cb9d_storage_percent_used" is not available

I wonder if this is due to the check not always firing? the plugin is setup to not render a metric unless the value is >= 75%. I did this because the mertic system has a limitation of 50 metrics  and being that a container could be part of multiple storage backends I wanted to ensure we didn't run up against this limitation which seems to be imposed upon us by upstream RAX MaaS. I'll pull that conditional for now and see how it goes.

@antonym updated

@antonym Thanks for the review. i've further updated the PR.

20:05:26 fatal: [jramxp-264-32255]: FAILED! => {"changed": false, "cmd": " . /openstack/venvs/maas-r13.1.0rc1/bin/activate\n  /opt/rpc-openstack/scripts/rpc-maas-tool.py verify-local", "delta": "0:00:23.152837", "end": "2017-02-27 20:05:26.033108", "failed": true, "failed_when_result": true, "rc": 1, "start": "2017-02-27 20:05:02.880271", "stderr": "", "stdout": "The following metrics are required by alarms but not produced by any checks: set(['container_storage_percent_used_critical'])\nThe following checks failed to execute or didn't return 'okay' as their status: [('container_storage_check--jramxp-264-32255', \"status error [Errno 2] No such file or directory: '/etc/mtab'\\n\")]", "stdout_lines": ["The following metrics are required by alarms but not produced by any checks: set(['container_storage_percent_used_critical'])", "The following checks failed to execute or didn't return 'okay' as their status: [('container_storage_check--jramxp-264-32255', \"status error [Errno 2] No such file or directory: '/etc/mtab'\\n\")]"], "warnings": []}

recheck_xenial

recheck_xenial

recheck_xenial

recheck_xenial

recheck_xenial

recheck_xenial

recheck_xenial

recheck_xenial

recheck_xenial

recheck_xenial

recheck_xenial

Hi @antonym ,
I had a quick look at that instance we built today.  I've run some of the steps manually to see if I can replicate:
(maas-10.0.0rc5) root@ramxp-27-21d5:/usr/lib/rackspace-monitoring-agent# python
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import os
>>> import psutil
>>> os.chroot("/proc/%d/root/" % int(7166L))
>>> psutil.disk_partitions()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/openstack/venvs/maas-10.0.0rc5/local/lib/python2.7/site-packages/psutil/__init__.py", line 1452, in disk_partitions
  File "/openstack/venvs/maas-10.0.0rc5/local/lib/python2.7/site-packages/psutil/_pslinux.py", line 322, in disk_partitions
OSError: [Errno 2] No such file or directory: '/etc/mtab'
>>>

Looking on a random trusty instance, I get basically the same thing:
(.mattt-venv) root@infra1:~# python
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import os
>>> import psutil
>>> os.chroot("/proc/22605/root/")
>>> psutil.disk_partitions()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/root/.mattt-venv/local/lib/python2.7/site-packages/psutil/__init__.py", line 2029, in disk_partitions
  File "/root/.mattt-venv/local/lib/python2.7/site-packages/psutil/_pslinux.py", line 1067, in disk_partitions
OSError: [Errno 2] No such file or directory: '/etc/mtab'
>>>

Am I doing something wrong or are the Trusty builds incorrectly reporting success?
--Matt

@mattt416 @antonym I just wanted to confirmed on xenial via chroot, the file "/proc/self/mounts" does not exist but it does when I lxc-attach.
Manual run using a container on xenial via chroot
root@test:~# cat /etc/os-release
NAME="Ubuntu"
VERSION="16.04.1 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.1 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
UBUNTU_CODENAME=xenial

root@test:~# lxc-info -pn aio1_utility_container-dfead909
PID:            17814

root@test:~# chroot /proc/17814/root

root@test:/# stat /etc/mtab
  File: '/etc/mtab' -> '../proc/self/mounts'
  Size: 19              Blocks: 0          IO Block: 4096   symbolic link
Device: fd12h/64786d    Inode: 4718608     Links: 1
Access: (0777/lrwxrwxrwx)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2017-03-07 21:42:13.826015391 +0000
Modify: 2017-03-07 21:41:16.190217825 +0000
Change: 2017-03-07 21:41:16.190217825 +0000
 Birth: -

root@test:/# stat /proc/self/mounts
stat: cannot stat '/proc/self/mounts': No such file or directory

Manual run using a container on xenail via lxc-attach
root@aio1-utility-container-dfead909:~# cat /etc/os-release
NAME="Ubuntu"
VERSION="16.04.2 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.2 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

root@aio1-utility-container-dfead909:~# stat /etc/mtab
  File: '/etc/mtab' -> '../proc/self/mounts'
  Size: 19              Blocks: 0          IO Block: 4096   symbolic link
Device: fd12h/64786d    Inode: 4718608     Links: 1
Access: (0777/lrwxrwxrwx)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2017-03-23 13:16:06.114164924 +0000
Modify: 2017-03-07 21:41:16.190217825 +0000
Change: 2017-03-07 21:41:16.190217825 +0000
 Birth: -

root@aio1-utility-container-dfead909:~# stat /proc/self/mounts
  File: '/proc/self/mounts'
  Size: 0               Blocks: 0          IO Block: 1024   regular empty file
Device: e1h/225d        Inode: 154961043   Links: 1
Access: (0444/-r--r--r--)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2017-03-23 13:39:22.730051603 +0000
Modify: 2017-03-23 13:39:22.730051603 +0000
Change: 2017-03-23 13:39:22.730051603 +0000
 Birth: -

So it would see we're going to have to attach to the container and screen-scrape in order to retrieve these metrics due to differences in xenial. Unless we're able to come up with a better way?

recheck_ceph

According to @cloudnull, @major and @ant may lime and coconut this now

Mainly just wondering if it will run everywhere we need it to as it has a dependency on lxc-python2.  Other than that, LGTM.

@alextricity25 should we get this back-ported to 14.0 or wait for 14.1?

@antonym Negative. This will won't be included until 14.1. The newton-14.0 is currently frozen and r14.0.0 is due to release soon. We are not adding any more new features to that branch.


Stuck at "Syncing Mint and Robinhood... This window will automatically close when the sync is complete"

I've been stuck on this page on Robinhood for over 10 minutes. I logged into just Mint initially and was told that the sync was starting. However, I was stuck on that page for a long time so I logged into both Mint and Robinhood to try to debug and I'm still stuck on that page. Please help

Same issue today for me

Same issue today for me

getting the following for me:
Robinhood Mint Sync for Chrome
Syncing Mint with Robinhood.
×
Robinhood Mint Sync for Chrome
Sync Complete! Reload to see the change.
Robinhood Mint Sync for Chrome
Sync performed in the last hour. Not syncing.

getting the following for me:
Robinhood Mint Sync for Chrome
Syncing Mint with Robinhood.
×
Robinhood Mint Sync for Chrome
Sync Complete! Reload to see the change.
Robinhood Mint Sync for Chrome
Sync performed in the last hour. Not syncing.

Try signing into Robinhood.com manually before initiating the sync. I will look into why the login page is not being detected

Try signing into Robinhood.com manually before initiating the sync. I will look into why the login page is not being detected

@TimGodfrey @ryanwnzl @hannahvu Just to confirm, did signing in to Robinhood.com manually before starting the sync work? I have a fix for the login page detection in an upcoming update (#20), but I wanted to confirm that was your issue.

@TimGodfrey @ryanwnzl @hannahvu Just to confirm, did signing in to Robinhood.com manually before starting the sync work? I have a fix for the login page detection in an upcoming update (#20), but I wanted to confirm that was your issue.

Hi,
I am still having this issue. I am logged into robinhood and mint. From mint overview, using 3.1.2 version of the app, I see nothing has been edittted in my mint account.
I can confirm there is not "property" called robinhood.
What can i do here? Thanks in advance.

Hi,
I am still having this issue. I am logged into robinhood and mint. From mint overview, using 3.1.2 version of the app, I see nothing has been edittted in my mint account.
I can confirm there is not "property" called robinhood.
What can i do here? Thanks in advance.

@pluong94 Do any of the extension prompts appear in the top right of the screen? (there should either be a prompt asking you to set up, or a prompt about it attempting to sync)
If so: Does it try actually syncing anything? (E.g. Do any other tabs open in chrome? When syncing, one tab should open for robinhood, and then another tab should open for Mint Settings)
If so: Can you turn on "Enable Debug" in the extension settings and then post what you see in the inspector console on the new tabs?

@pluong94 Do any of the extension prompts appear in the top right of the screen? (there should either be a prompt asking you to set up, or a prompt about it attempting to sync)
If so: Does it try actually syncing anything? (E.g. Do any other tabs open in chrome? When syncing, one tab should open for robinhood, and then another tab should open for Mint Settings)
If so: Can you turn on "Enable Debug" in the extension settings and then post what you see in the inspector console on the new tabs?

Hey Andrew,
Outside of the initial extension prompt asking me to set up, no additional
prompts appear. Once I click the set up from the extension prompt, it
brings me to this black screen as others have seen.
Nothing actually tries syncing.
I’ll do the debug when I get to my computer!
On Sun, Jan 17, 2021 at 6:57 PM Andrew Herder<EMAIL_ADDRESS>wrote:

@pluong94 https://github.com/pluong94 Do any of the extension prompts
appear in the top right of the screen? (there should either be a prompt
asking you to set up, or a prompt about it attempting to sync)
If so: Does it try actually syncing anything? (E.g. Do any other tabs open
in chrome? When syncing, one tab should open for robinhood, and then
another tab should open for Mint Settings)
If so: Can you turn on "Enable Debug" in the extension settings and then
post what you see in the inspector console on the new tabs?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pkmnct/robinhood-mint-sync-chrome/issues/17#issuecomment-761916719,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASP74ML3HJQZVL7CGZHJKG3S2OBOFANCNFSM4V45PSPQ
.

Hey Andrew,
Outside of the initial extension prompt asking me to set up, no additional
prompts appear. Once I click the set up from the extension prompt, it
brings me to this black screen as others have seen.
Nothing actually tries syncing.
I’ll do the debug when I get to my computer!
On Sun, Jan 17, 2021 at 6:57 PM Andrew Herder<EMAIL_ADDRESS>wrote:

@pluong94 https://github.com/pluong94 Do any of the extension prompts
appear in the top right of the screen? (there should either be a prompt
asking you to set up, or a prompt about it attempting to sync)
If so: Does it try actually syncing anything? (E.g. Do any other tabs open
in chrome? When syncing, one tab should open for robinhood, and then
another tab should open for Mint Settings)
If so: Can you turn on "Enable Debug" in the extension settings and then
post what you see in the inspector console on the new tabs?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pkmnct/robinhood-mint-sync-chrome/issues/17#issuecomment-761916719,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASP74ML3HJQZVL7CGZHJKG3S2OBOFANCNFSM4V45PSPQ
.

@superpowered
What am i suppose to be posting? I have turned on "enable debug" but am not sure what I am looking for from inspecting the background page.

@superpowered
What am i suppose to be posting? I have turned on "enable debug" but am not sure what I am looking for from inspecting the background page.

@superpowered
Actually, I think i found it?
Robinhood Mint Sync - [Content](Mint - Properties - Check) initialized
debug.ts:33 Robinhood Mint Sync - [Content](Mint - Triangle Fix) initialized
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Using default sandbox factory...
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 All core plugins started...
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Plugins registered...
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Layout activated
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Layout activated
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Application before-ready completed...
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Application ready...
app-284d2e17557964650d51748b0a1ee211.js:11016 LD: [error] Ignoring unknown config option "baseTimeout"
r @ app-284d2e17557964650d51748b0a1ee211.js:11016
inpage.js:1 MetaMask: MetaMask will soon stop reloading pages on network change.
For more information, see: https://docs.metamask.io/guide/ethereum-provider.html#ethereum-autorefreshonnetworkchange
Set 'ethereum.autoRefreshOnNetworkChange' to 'false' to silence this warning.
(anonymous) @ inpage.js:1
inpage.js:1 MetaMask: MetaMask will soon stop reloading pages on network change.
For more information, see: https://docs.metamask.io/guide/ethereum-provider.html#ethereum-autorefreshonnetworkchange
Set 'ethereum.autoRefreshOnNetworkChange' to 'false' to silence this warning.
(anonymous) @ inpage.js:1
inpage.js:1 MetaMask: MetaMask will soon stop reloading pages on network change.

Robinhood Mint Sync - [Content](Mint - Properties - Check) initialized
debug.ts:33 Robinhood Mint Sync - [Content](Mint - Triangle Fix) initialized
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Using default sandbox factory...
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 All core plugins started...
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Plugins registered...
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Layout activated
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Layout activated
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Application before-ready completed...
libs-f0625f303abccdbf22c19e5c5570eeb9.js:4 Application ready...
app-284d2e17557964650d51748b0a1ee211.js:11016 LD: [error] Ignoring unknown config option "baseTimeout"
r @ app-284d2e17557964650d51748b0a1ee211.js:11016
inpage.js:1 MetaMask: MetaMask will soon stop reloading pages on network change.
For more information, see: https://docs.metamask.io/guide/ethereum-provider.html#ethereum-autorefreshonnetworkchange
Set 'ethereum.autoRefreshOnNetworkChange' to 'false' to silence this warning.
(anonymous) @ inpage.js:1
inpage.js:1 MetaMask: MetaMask will soon stop reloading pages on network change.
For more information, see: https://docs.metamask.io/guide/ethereum-provider.html#ethereum-autorefreshonnetworkchange
Set 'ethereum.autoRefreshOnNetworkChange' to 'false' to silence this warning.
(anonymous) @ inpage.js:1
inpage.js:1 MetaMask: MetaMask will soon stop reloading pages on network change.

Okay, so was able to recreate after making a "Fresh" account with no properties....
https://github.com/pkmnct/robinhood-mint-sync-chrome/pull/27 This should address it.
@pluong94 I'm assuming you had no other properties in the account?

Hi,
@superpowered  Thanks, yeah it just started working for me. and No, i had no other properties in the account.

Hi,
@superpowered  Thanks, yeah it just started working for me. and No, i had no other properties in the account.

@pluong94 @superpowered I created a new issue for the account with no properties (#31).

@pluong94 @superpowered I created a new issue for the account with no properties (#31).


Bug in version 7.0.0 - EAS Build => cannot find symbol new String[] {Manifest.permission.POST_NOTIFICATIONS}

Trying to build with using expo eas build after updating from version 5.7.0 to 7.0.0 i am getting the below error :
I haven't change anything else from my code just updated the Notifee and then run expo prebuild --clean and then eas build
I had no problem at all building with version 5.7.0 just today i updated to 7.0.0 out of curiosity .
cannot find symbol
[stderr]         new String[] {Manifest.permission.POST_NOTIFICATIONS},

The above happens during run gradlew

Hey @goxr3plus - please search then post:
https://github.com/invertase/notifee/issues?q=is%3Aissue+cannot+find+symbol+new+String[]+{Manifest.permission.POST_NOTIFICATIONS
--> compileSdkVersion must be 33 or higher, it's a brand new permission symbol we need present during compilation


updated Machine-Learning

fixed formatting
renamed book section
removed dead links
added missing authors
added some libraries

Awesome work @bomholtm !! Have some 🍨 🍰 🎂 from AlgoWiki team !
Keep contributing!!! 👍

Thank you @Byte7


[DO NOT MERGE] Increase Sparsity Threshold to 50% for sparse compression

Background
We have observed instances where accidental sparsity induced by W4-style quantization exceeds 40%. These modules are being identified during the population of sparsity_config.
Changes
As discussed in our offline conversations, this PR increases the sparsity threshold to 50%.
Summary

Increased the sparsity threshold from 40% to 50% to better handle cases of accidental sparsity induced by W4-style quantization.

Merge pending verification from research team!

Shared the resulting "accidental" sparsity from W4A16 with @anmarques - doesnt seem like anything we should be concerned with


ADR for Text Embeddings

ADR goes here: https://github.com/defenseunicorns/leapfrogai/tree/main/docs/adr
Is Instructor XL the right choice for text embeddings?
Leaderboard: https://huggingface.co/spaces/mteb/leaderboard
Candidates:

[ ] GritLM (Embeddings + LLM in one model)
[ ] UAE Large
[ ] GIST Large Embeddings V0
[ ] ...

Also look at reranking performance. Do we need to ADR to have different models based on performance on specific use cases?

How do we handle reloading data into RAG when models change?


Enhance status popover

What is the motivation for adding/enhancing this feature?

New sequence and icons for status popover:

Improved the sequence, but didn't added icons as select option is not supporting icons.

Enhancement changes verified and working fine in release v1.1.0. Hence, closing it.
https://jam.dev/c/02221820-11fc-4a0b-96b8-f0a6798221fe


Cannot train FastQA without pre-trained embeddings

$ cat conf/fastqa-s.yaml
description: >
  A configuration inheriting from the default jack.yaml

parent_config: './conf/jack.yaml'

# Reader model to use, see jack/readers/implementations.py for options
model: 'fastqa_reader'

# fixed experiment seed
seed: 1337

# loader to use in experiment
loader: 'squad'

# where to store the reader
model_dir: './fastqa_reader'

# How large the support should be. Can be used for cutting or filtering QA examples
max_support_length: -1

# Use also character based embeddings in readers which support it
## with_char_embeddings: False

train: 'data/SQuAD/train-v1.1.json'
dev: 'data/SQuAD/dev-v1.1.json'
test: null

# [word2vec], [glove] or [memory_map_dir] format of embeddings to be loaded
## embedding_format: 'memory_map_dir'

# embeddings to be loaded
## embedding_file: 'data/GloVe/glove.840B.300d.memory_map_dir'

# Use fixed vocab of pretrained embeddings
vocab_from_embeddings: False

repr_dim: 300

epochs: 20

dropout: 0.5

batch_size: 64

lowercase: True

learning_rate_decay: 0.5
$ python3 bin/jack-train.py with config='./conf/fastqa-s.yaml'

[..]

  vocab_maxsize =<PHONE_NUMBER>000
  vocab_minfreq = 2
  vocab_sep = True
  with_char_embeddings = True
  write_metrics_to = None
INFO - jack - Completed after 0:00:00
2017-11-06 18:47:35.897644: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX
INFO - jack-train.py - Time since last checkpoint : 0.005min
INFO - jack.core.reader - Setting up data and model...
ERROR - jack - Failed after 0:00:08!
Traceback (most recent calls WITHOUT Sacred internals):
  File "bin/jack-train.py", line 154, in main
    jtrain(reader, train_data, test_data, dev_data, configuration, debug=debug)
  File "/home/pasquale/.pyenv/versions/3.6.3/lib/python3.6/site-packages/jack-0.1.0-py3.6.egg/jack/train_reader.py", line 94, in train
    l2=l2, clip=clip_value, clip_op=tf.clip_by_value)
  File "/home/pasquale/.pyenv/versions/3.6.3/lib/python3.6/site-packages/jack-0.1.0-py3.6.egg/jack/core/reader.py", line 235, in train
    batch_size, clip, optimizer, training_set, summary_writer, l2, clip_op, **kwargs)
  File "/home/pasquale/.pyenv/versions/3.6.3/lib/python3.6/site-packages/jack-0.1.0-py3.6.egg/jack/core/reader.py", line 244, in _setup_training
    self.setup_from_data(training_set, is_training=True)
  File "/home/pasquale/.pyenv/versions/3.6.3/lib/python3.6/site-packages/jack-0.1.0-py3.6.egg/jack/core/reader.py", line 140, in setup_from_data
    self.input_module.setup()
  File "/home/pasquale/.pyenv/versions/3.6.3/lib/python3.6/site-packages/jack-0.1.0-py3.6.egg/jack/readers/extractive_qa/shared.py", line 104, in setup
    self.emb_matrix = self.vocab.emb.lookup
AttributeError: 'NoneType' object has no attribute 'lookup'

@pminervini yes that is correct, because it is not supposed to be trainable without embeddings. It is also not easy to adapt the code to allow it.

Let's make it fail gracefully at least! :) I'll be on this later this evening


Error UX Improvements

When hitting an error, I noticed myself struggling to parse what was going on. (enough so that I had to switch to VS to figure it out)
First up, a slight issue is I have to scroll sideways to view the full thing,

but even when I view it, it seems to lack information compared to VS (doesn't tell me the location of the access violation)

compared to

The next issue is, at a glance it seems obvious what has happened in VS (memory couldn't be read)

But in raddbg I couldn't figure out what was happening. At first glance I thought the memory was just zeroed, only to realize later that it couldn't be read (hovering over it made things click for me)

This isn't an issue now that I've learnt what's going on, but I figured I'd share the experience anyway since it might be helpful for improving the UX for new users coming from VS.

It also might've been the hover eval that threw me off initially since there's no indication of it being unreadable (again, just thought it was zeroed)


fixes #8: keeps the filter[*] parameters in the original order

So, they are sorting the parameters, except array parameters, like filter['limit']. The OAuth.sorted_params() works in this way. We have to use OrderedDict() in other places too, to keep the original order of parameters.

Thanks :)


Improve disconnected errors (on mobile)

This ticket is about two things

We're going to be disconnected when the app is backgrounded on mobile. Right now, our error is a little too aggressive / jarring. We should make it say something like "Reconnecting" and be a warn instead of a red error.

See this comment from Chris for more context

The flash error above is too wide on mobile, we should make it narrower. It's from the Phoenix default templates, and they may have changed the styling since I last looked...

How's this as an option?

adds a"warn" @kind
uses @close since it blocks the menu
moves everything onto one line.

https://user-images.githubusercontent.com/12987958/206532130-3139016f-0c43-47b6-a6d6-16cf0360587f.mp4
Haven't tested this with other flash messages across the app yet to see how they look, tho.


Custom tools in the new project system

I'm not sure if this issue should go in msbuild or this repo, but is there/will there be support for "custom tools" in the new build system? By custom tool I mean the property you can set on a file in vs to run design time processing logic on a file, such as code generation.
I've seen that you can set a property called "custom tool" on some files but not all, this creates a Generator tag in the project file, but i'm not quite sure what to put in there, or how to craft a nuget file so that my generator is run.
if there are any docs (or tracking issues) on that feature i'd greatly appreciate a link :) but perhaps that feature is not implemented yet

Yes single file generators are supported in the new project system. There is some documentation here - https://github.com/Microsoft/VSSDK-Extensibility-Samples/tree/master/Single_File_Generator

Interesting, @Srivatsn, that sample seems to be for vs2015 though, is the same syntax supported in vs2017? also, how are custom tools located in vs2017? in earlier versions you had to put them in the registry if i'm not mistaken, but I imagine now that things are a bit more cross plat, that is not longer the only option? ideally i'd want to put mine in a nuget package
Also, are custom tools only run by vs or also by msbuild? (so that they' would work cross plat/outside VS as well)

These single file generators are only run in VS. To run tools x-plat and in msbuild, you probably want to build a CLI tool. @piotrpMSFT do we have any docs on that?


Add parameters to the title

like this

This would be adding too much complexity for what might be a niche feature.
You can already do something like this with:
- asdf
	set a = 1
	=> asdf

~ asdf
	do get_hope_money(a)


ZXingBarcodeImageView not working in iOS??

I'm currently working on an crossplatform app using Xamarin Forms (shared). I need to generate a EAN-13 barcode, and my code works fine on Android, but nothing happens on iOS. I'm using ZXingBarcodeImageView. This is my code.
public class CardPage : ContentPage
{
  ZXingBarcodeImageView barcode = new ZXingBarcodeImageView
  {
    HorizontalOptions = LayoutOptions.Center,
    VerticalOptions = LayoutOptions.Center,
  };

  barcode.BarcodeFormat = ZXing.BarcodeFormat.EAN_13;
  barcode.BarcodeOptions.Height = 25;
  barcode.BarcodeOptions.Width = 75;
  barcode.BarcodeValue = "2800100028014";

  Content = barcode;
}

Did you do the init?

Forgot the global::ZXing.Net.Mobile.Forms.iOS.Platform.Init(); in the AppDelegate.cs.


[Chore] Standardize auth responses and remove redundant error

What
Standardize auth responses and remove redundant error
Known limitations
[TODO or N/A]
Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).
[x] This PR's title starts with the name of the package, area, or subject affected by the change.

Thoroughness

[x] This PR adds tests for the new functionality or fixes.
[x] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[x] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[x] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[x] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[x] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[x] This is not a breaking change.
[x] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr15.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr15.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr15.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr15.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr15.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr15.previews.kube001.services.stellar-ops.com


Dependency mindplay/composer-locator doesn't work with composer 2.x

Running composer update I got this error:
./composer.json has been updated
Running composer update league/event
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - mindplay/composer-locator 2.1.3 requires composer-plugin-api ^1 -> found composer-plugin-api[2.0.0] but it does not match the constraint.
    - middlewares/referrer-spam v2.0.0 requires mindplay/composer-locator ^2.1 -> satisfiable by mindplay/composer-locator[2.1.3].
    - middlewares/referrer-spam is locked to version v2.0.0 and an update of this package was not requested.

You are using Composer 2, which some of your plugins seem to be incompatible with. Make sure you update your plugins or report a plugin-issue to ask them to support Composer 2.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

It seems its problem with one of referrer-spam dependencies.
Any idea how to fix it?

I think this should be fixed in composer-locator package: https://github.com/mindplay-dk/composer-locator
@mindplay-dk What do you think? Could you update composer-locator to work with composer 2?

I made mindplay-dk/composer-locator#8 a pull request for this dependency. Maybe this helps this a bit faster to solve this problem :|

Finally I released a new version without this dependency https://github.com/middlewares/referrer-spam/blob/master/src/ReferrerSpam.php#L114 (and PHP 8 support).


feat: added border radius for redesign

What type of PR is this? (check all applicable)

[ ] Refactor
[X] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
Added border radius to theme for redesign
Issue ticket number and link
#1844
Checklist:

[X] I have performed a self-review of my code.
[X] I have tested the change on desktop and mobile.
[X] I have added thorough tests where applicable.
[X] I've added analytics and error logging where applicable.

Screenshots (if appropriate):

The first two (at least in the preview) don't have the border-radius applied

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


How do you insert the point information to the database?

Hi,
I already have a point column in my database and found this package whilst trying to work out how to save to it.
I have 'lat, 'lng', 'geography' in my db/forms and have been trying to use a beforesave method to convert the lat/lng values into GeomFronText point values, but whatever I do it won't insert into the database.
How do you actually get the information in after creating the point column?

Ignore this.
I used the lat/lng form fields to populate those columns, removed geometry from the form and then used an afterSave class in my model to update the geometry field from the lat/lng values.


Extend operator Status for improving user experience

This is the follow-up of improving Operator Status topic.
For the time being the only information we can obtain from the Status is where the operator succeeded with creation runtime objects (Deployments, Services, default configurations etc)
Ideally, the Status should NOT be overloaded but contain enough information for User and dependent code to make it clear:

What's going on with runtime and where to see more detailed information if something going wrong.
How to reach Backstage application (best effort of exposing direct URL or some hints how to obtain it)
What configuration objects (ConfigMaps, Secrets) take part in Backstage Application config
As part of the Level 2 Capability Level, the operator should convey inability to manage an unsupported version of the Operand in the status section of the CR
... (not limited to)

As part of the migration from janus-idp to redhat-developer in https://issues.redhat.com/browse/RHIDP-1021, this will be tracked in https://issues.redhat.com/browse/RHIDP-1450


add 5 seconds timeout to notifications

See:
http://www.htmlxprs.com/post/30/getting-started-with-html5-web-notification-api
We can implement timeout.
I think it would be good to have an option to not show notifications at all in the settings too.


Update Composer dependencies (2023-10-11-00-10)

Visual regression test passed!
View the visual regression test report


Site to Site routing

Can NetBird be used in the scenario below?
Install a NB Linux peer that provides routing at Site A. Do the same at Site B. Nodes on both sites can reach each other without the need to install NB clients on individual nodes?
Unidirectional is fine as well as mostly I just need to remote into work and not the other way around. I have multiple devices at home that need to access work so installing NB on all of them is not feasible. I just don't know how I could instruct my home devices to go through the NB router if IP is <IP_ADDRESS>/22. I'm sure there's a way for pfSense to do this.

Currently using Tailscale + pfSense to achieve this but wanting to switch to NB. Tailscale when using their self hosted relay solution seems limited to 50Mbps whereas NB is 100Mbps.
Thank you.

Already possible. You can do it by installing netbird on, say, a router or a Linux device attached to the network, and then configure a network route for it on the dashboard. Do note to also make sure your firewall settings are properly configured to allow for this.

Thanks but how does another device at site A know to route through the NB device at site A to NB device at site B then to a final destination at site B?
E.g. if site A device (no NB <IP_ADDRESS>) wants to ping site B device (no NB <IP_ADDRESS>) it has to go through site A's NB device (<IP_ADDRESS>) to site B's NB device (<IP_ADDRESS>) to finally reach <IP_ADDRESS>.
Any idea how that can be achieved? Essentially I want to install NB on one device at each site and have every other device communicate to each other without the need to install NB.
Thanks.

Achieved this via iptables NAT on the Netbird client machine then setting static routing on non-Netbird clients to route the subnets via the Netbird client.
Annoying to do this individually and obviously will not work for devices like IoT.
Would be nice if there was a freebsd version or service for pfsense/opnsense so this can be done on the router level.
Anyway, thanks all.


Update cats-effect to 3.0.2

Updates org.typelevel:cats-effect from 2.4.1 to 3.0.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Migrations

github:typelevel/cats-effect/v3_0_0?sha=series/3.x

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-effect" } ]

labels: library-update, semver-major, scalafix-migrations

Built into CE3.


included user input for missing vars, disabling passwordAuth and enab…

Hi there,
I changed the script to my liking and thought you might be interested. I removed the necessity to use commandline options and inserted user input demands for the not provided (and neccessary) options.
Additionally added the option to modify the sshd_config file on the host to disallow passwordAuthentication and allow PubkeyAuthentication if desired. It also checks if those config changes would be able to be made due to permission restrictions.
It furthermore tries to get the currently used sshd_config file (ps aux | grep ...)
Some documentation changed and the information about a daemon reload was added additionally with a final login check.
Hope that was all and understandable. ;) Thanks for your work and sorry for the hasty PR, I'm in a hurry!

Thanks for the proposed changes. I will merge at least some of it when I find some time to take a closer look!


chatcommunicate.py: Ignore anything > 3 words

Avoid triggering on discussions where people link to SD and say "why"
or "true" as part of a longer sentence

Not a viable approach after all -- the test-a command and friends needs to operate on messages of arbitrary length.


Parameters section pushes the plot down

BUG DESCRIPTION
Problem
On some browsers/screen sizes, the column with the parameters is too wide and pushes the plot below instead of letting it on the same line.
Steps to reproduce

Numbered
Steps

Current behavior
A clear and concise description of what happens when you perform the steps above. Include screenshots if applicable.
Expected behavior
A clear and concise description of what you expected to happen.
Suggested Change
Could  add this line to the css:
#div-1 {width: auto !important;}

Comment
Any other relevant context or links to what you were working on when you
discovered the problem.

ISSUE OWNER SECTION
Assumptions

List of assumptions made WRT the code
E.g. We will assume input is correct (explaining why there is no validation)

Requirements

List of conditions to be met for the feature
E.g. Every column/row must display a value, i.e. cannot be empty

Limitations

A list of things this work will not fix
E.g. Getting edge case X to work requires too much effort and will not be
fixed in this effort.

Affected/Changed Components

Files
Environment variables
External executables
Database tables
Cron job settings
Etc.

DESIGN
GUI Change description
Describe changes the user will see.
Code Change Description (Pseudocode optional)
Describe code changes planned for the fix.
Tests
A test should be planned for each requirement (above), where possible.

Resolved in #39


non-controversial parts of #812

Thanks :)

Bah, this broke the mobile view, there's now a gap to the left of the menu

Easy fix, https://github.com/D-Programming-Language/dlang.org/pull/818


[Project Badging Table]: Fix Search Feature and Table cells sizing

Fix search feature and filter on project badging page
Resize table cells to align with the prototype
Fix table responsiveness.

@Teek-tech is this the issue you have been looking for?

Yea, where did it go to? I will jump on it right away.

hello @Teek-tech Are you done working on this issue?

Yes. @adeyinkaoresanya

Okay! Thank you very much!


Importing as Atlas Texture has no effect whatsoever

Godot version: 3.2.1. stable
OS/device including version: Win64
Issue description:
Following this procedure: https://godotengine.org/article/atlas-support-returns-godot-3-2 results in a forced restart, but upon restart of the editor, all pngs are exactly as before.
Steps to reproduce:
Follow this:
https://godotengine.org/article/atlas-support-returns-godot-3-2

I am also having this same issue. I'm running on Pop!_OS, 64 bit. I hope this gets a fix soon.

Can anyone confirm that the article https://godotengine.org/article/atlas-support-returns-godot-3-2 works for them, and what operating system/Godot version they're using? I assume this feature was tested and working at some point?

@Giwayume I'll try older versions in the weekend because it's not working for me on 3.2.1

@ Giwayume

Can anyone confirm that the article https://godotengine.org/article/atlas-support-returns-godot-3-2 works for them, and what operating system/Godot version they're using? I assume this feature was tested and working at some point?

I'm using Windows 64-bit. It looks like the feature works perfectly fine in 3.2, but not in 3.2.1. I REALLY hope this gets fixed, because it's unbelievable how easy it is to use.

I'm also having this problem 3.2.1 doesn't work. Using Windows 64-bit.

Could someone try to bisect the regression to see when it broke exactly? Also, try 3.2.1rc2 and 3.2.1rc1 to narrow the range of commits that need to be built to bisect the regression.

I think this was fixed in 3.2.2 beta already. Please try https://godotengine.org/article/dev-snapshot-godot-3-2-2-beta-3

Duplicate of #36976 (please comment if your tests show otherwise).

I can confirm this is fixed on 3.2.2 beta 3

Will this fix texture array creation option from the same place as this failed too?

@chucklepie I didn't try that and I am not sure what you mean, but I guess you can download the beta from the website and make sure.

Import as atlas works in beta. What I meant by the texture array is I presumed the import option 'texture array' would create a sprite sheet out of selected graphics, but I guess not.
Doesn't help when you read the manual there is absolutely nothing on the 'import as...' options...
https://docs.godotengine.org/en/stable/getting_started/workflow/assets/import_process.html?highlight=import
or here
https://docs.godotengine.org/en/stable/getting_started/workflow/assets/importing_images.html
It completely skips what any of the import

Importing Atlas textures still feels incredibly unreliably and broken due to various reasons:
On Win 64, the AtlasTexture reimport process fails with the exact same symptoms in Godot 3.2.3 stable and Godot 3.2.4 beta2, the way it is described here under "How to use atlases in Godot": https://godotengine.org/article/atlas-support-returns-godot-3-2
Currently this is the only documented guide for using Atlas Textures as using search will only bring up API docs.
Here is how I read those instructions:

Select a bunch of png files in the FileBrower tab,
go to the import tab,
select Import as: "TextureAtlas"
make sure that no AtlasTexture exists or to select an already existing AtlasTexture (if you do this the first time you obviously won't have an existing AtlasTexture, so you would think there is nothing to do, except to hit "reimport")
hit "reimport"

popup saying changing the type requires restart,

hit "Save Scenes, Reimport, and restart" button

Godot saves, (presumeably) reimports, shut down and restarts

Navigate to the folder with the individual png files:
no atlas file file
Try to double click one of the individual png files:
just no reaction whatsoever
Dragging individual png file into viewport:
showing the "forbidden" symbol
Switching windows in the OS to another program than back to Godot will cause an automatic reimport every time.
If you then go to the guide again to figure out what went wrong you might notice an "res://atlas2", so you think it might be required to write something there even though the guide said having no file is fine too.
So select all your png files again, and in the Import tab under AtlasTexture you write "res://atlas2.png" this time, just like in the screenshot, hit "reimport" .... but nothing happens. You double click the png file, nothing happens.
If you do all that instead of a file path (like "res://atlas2.png") you would type just a name in that field like "atlas2", after the reimport/restart, no whenever you double click a png file you get a "loading error" popup, asking you to fix dependencies (which you can't because the "atlas2" file was not created, or at least does not show up anywhere. When you try to drag one of the png files into the viewport a sprite gets added to the Scene Tree, but the texture does not show up (despite being loaded).
The only way to actually make AtlasFiles work is to not to do any of the above and right away

Select a bunch of png files in the FileBrower tab,
go to the import tab,
select Import as: "TextureAtlas"
type out a full filepath to where you want the Atlas file generated to, like (like "res://inages/atlas.png")
hit "reimport"

popup saying changing the type requires restart,

hit "Save Scenes, Reimport, and restart" button

Godot saves,  reimports, shut down and restarts

This is incredibly frustrating (and alienating to newcomers) because the UI is anything but clear about what the correct steps are and even the tiniest mistake leads to your files not working anymore without any feedback to the user as to what the issue is or where they did wrong, or how to reverse the steps so they can try again.

Hmmm I haven't had the same problem. My process:

Click the folder icon

On the dialog that pops up, navigate to a folder, type the filename and click "save".

Click the "Reimport" button.

And it works.

Lucky you.

Lucky you.

The snark isn't necessary. Documenting where it works and where it doesn't is important for fixing these kinds of issues. I'm also trying to clarify your step 4 as you've made no clear indication whether or not you've tried the file selection dialog.

I was not trying to be snarky. I mean it in the literal sense. There are so many ways the user can fall off the right path here and the engine is not communicating what is going wrong or how to fix it, so much that I really think you are literally lucky, lucky your intuition matched the intended "right" path, and did not lead you off that path into a ditch, causing you hours wasted for no good reason. The UI does not make any of this obvious. There are cliffs and pitfalls left and right. Any survivor that goes through this unharmed is imho for the lack of a better word: lucky.
Dokumentation is necessary of course, but this is not only documentation issue. There should not be any cliffs and pitfalls left and right. The right path should be clear and brightly lit, easy to see. And if anyone should stumble despite all that, there should be cushions left and right that softly bounce the user back into the right direction and keep them straight on the right path.

I'm also trying to clarify your step 4 as you've made no clear indication whether or not you've tried the file selection dialog.

You did not post a question in your previous comment, otherwise I would of course have answered: Yes I've tried clicking the file dialog. Unfortunately my 30+ years experience with file dialogs taught me that this was for opening files. What was I suppose to save? I have not done anything yet. I have not made an Atals Texture at this point. If this is indeed for saving I thought, it must be at a later point.
In my 30+ years of working with all kinds of software I have never experiences this kind of UX flow. I don't know where you got your experience so your intuition led you to the right path immediately, but I can tell you I would have never got there without meticulously and methodically testing each and every possible solution one by one for hours in painful trail and error.


[DO NOT MERGE][SPIKE] Update the selected references design and content for UR testing

Context
This is  not production ready code. It should not be merged into master. If we end up going with this design this will need to be refactored
This PR implements the UI and content changes for new selected references flow outlined in this document
https://docs.google.com/document/d/161YGuKP9rKGI_fSEDeHajKopWRMjQMkPNYNVGoAa5x8/edit
Changes proposed in this pull request

Update the content in the emails
Make the proposed content and UI changes in the document

Guidance to review
I think it's going to be easiest to run this locally or on the review app and work through the changes outlined in the document.
This whole references/selected references section needs to refactored into a component. I'll card something up, but it does not need to block testing this new flow.
Link to Trello card
https://trello.com/c/fh7Z2c6B/3763-dev-design-and-content-updates-for-requesting-and-selecting-references-spike
https://trello.com/c/S7ylRphB/3725-dev-design-and-content-updates-for-requesting-and-selecting-references-spike
Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] Required environment variables have been updated added to the Azure KeyVault

On More than 2 references received (but none selected), does this need to also include 'more than'?

On More than 2 references received (and 2 selected) this is still rendering:
You’ve selected 2 references to submit with your application already, but you can change your selection if you want.

Have addressed these two. This still needs a refactor though since it's now going to go live into prod


Add test_index from CPython 3.8.7

I think the issue with the other indexing was that _operator isn't in a #[pymodule] mod; could you try putting the fn index without the #[pyfunction] and just adding it to the py_module! macro at the bottom of the file?


Improvement suggestion: unify opened files' tabs

When you split screen you have a repetition of the opened files.
This won't bring any major benefit but it subtract precious space on the screen and make it difficult to navigate between the opened files.
My suggestion is to unify the tabs and let them common, see picture below:

Sometimes I do use gvim (though I prefer Spyder for a number of reasons, like for example the variable explorer, the outline tab, etc. other than it works amazingly just out of the box without getting mad in tewaking) and it has the desired layout, see below:

A good key binding for navigating through the opened files could be Ctrl-PgUp and Ctrl-PgDn.

Hi @Ubaldo79 thank you for the feedback! I'm not totally sure about this but what do you think @spyder-ide/core-developers ?
Also, just in case, I think currently you can use Ctrl + PgUp and Ctr + PgDn to move around the files. You can also use the File Switcher (Ctrl + P) to move around files:

Also, currently you can hide the tabs in the Editor if you want by unselecting the option Show tab bar at Preferences > Editor > Show tab bar:

Maybe a mix of this elements could be a way to achieve a better workflow for you @Ubaldo79 ? Let us know!

I understand @Ubaldo79's concern about redundancy. However, when the Editor has a split window, the user must be able to navigate independently in each split window and have each split window indicate which file is currently active, even when not focused. Having the tab bar visible in each split window accomplishes this. Unifying the tab bar would only indicate the currently active document in the currently focused split window; the user would have to give focus to a split window to know which document is active in that split window.

@mrclary you got the point, but in all honesty I’d rather have unified tabs thus losing the awareness of what is in the non-active split rather than having a very tiny small space that fills up extremely quickly but knowing what is in the other split just by looking at the tabs.
The former is certainly preferred, every day. :) Especially if you are doing a major work  that requires  many files to be opened at the same time :)
Note that also other editors like vim or vscode has unified tabs. :)

@dalthviz great for the C-PgUp/Dn! Hiding the tabs is not a solution, I need to jump from one file to another very often (especially if you are working in test-driven development) :)

Best solution would be to have the two open windows highlighted in different colors, so that on can see which is the active open file and the other opened file in the tabs.
But then it would be hard/impossible to open the same file next to each other to look a different portions of the same file (because it is so large)
Also if there are more than 2 windows open one would need a lot of different shades..
Somehow the current solution does not seem that bad and is quite generic.. Even though the current usability also has its flaws..

@maurerle the shades are not super important as the user can see the cursor on the active window, but having a shade only on the active window would be actually handy.
I think it would be enough ONE shade only for signaling the active window only, leaving all the others unshaded, and that’d be all. :)

To note, while probably not impossible, this would require a substantial amount of UX redesign and UI work given it the tab bar is currently intrinsic to the split panel.

Hiding the tabs is not a solution, I need to jump from one file to another very often (especially if you are working in test-driven development) :)

Instead of the tab bar, you can use the Outline pane with Show all files turned on (and Follow cursor turned off), which becomes a single unified vertical file switcher that can show far more files than even a full-width horizontal tab bar. This is what I do (and matches the UI of other editors, such as Notepad++), and I find it very effective at switching between the large number of files I tend to have open at once.

Unifying the tab bar would only indicate the currently active document in the currently focused split window; the user would have to give focus to a split window to know which document is active in that split window.

To note, this is what I thought too at first but the path bar in each pane still shows the full file path, albeit perhaps a bit harder to read at a glance.

Best solution would be to have the two open windows highlighted in different colors, so that on can see which is the active open file and the other opened file in the tabs.

This also wouldn't work if the tabs are relatively far apart from one another.

Thank you all for the feedback! Just in case, I will move this issue in to the spyder-ide/ux-improvements repo to continue the discussion there


⚠ Bofrost.be has degraded performance

In 6e799fb, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1278 ms

Resolved: Bofrost.be performance has improved in ce803ed.


ListTasks Function is not able to paginate the tasks

Thanks for stopping by to let us know something could be better!
Environment details

which product (packages/*): @google-cloud/tasks
OS: Mac M1
Node.js version: v20.10.0
npm version: 8.19.4
google-cloud-node version:

Issue Description
We want to list tasks & run them for a queue. We had around 6K tasks in our queue. We were trying to use paginate and run tasks manually using below code snippet. However it was taking too long on listTasks method. After long time, it returned all 6k tasks in the array.
Steps to reproduce

Mention project, queue and location.
Make sure that queue has more tasks then pageSize.
Run below code snippet.
It would print tasks.length which is actual number of tasks pending instead of pageSize.

import { CloudTasksClient } from '@google-cloud/tasks'

async function executeTasksInBulk() {
  try {
        const project = 'TEST_PROJECT_NAME'
        const location = 'us-central1'
        const queue = 'TEST_QUEUE_NAME'

        const client = new CloudTasksClient()
        console.log(`Starting the queue processing`)
        // List tasks in the queue
        const parent = client.queuePath(project, location, queue)
        const [tasks, nextPageToken] = await client.listTasks({ parent, pageSize: 100, pageToken: null })
        console.log(`Number of tasks found are ${tasks.length} & nextPageToken is ${nextPageToken}`)
  } catch (e) {
      console.error(e)
  }
}
executeTasksInBulk()

Hi @pragnesh-highlevel, could you try setting autopaginate: false?
const request = {request, pageSize: 50}
const [resultArray, nextPageRequest, rawResponse] =
  await client.samplePaginatedMethod(request, {autoPaginate: false});

From docs, if you want to specify pageSize, you need to do it with autopagination.

Closing for now, feel free to reopen if this doesn't work.


ninja error: build.ninja:441: unknown pool name

@Maratyszcza Hi, thanks for your code sharing.
But there is something wrong during the setup. How could I settle it?

Install the latest Ninja.


This repository: Some build job seems stuck

This has been running / pending for 20+ days, is it normal?
https://github.com/OfficeDev/Open-XML-SDK/runs/6361756954

Thanks for pointing this out. There are some other issues with our backend build (blocking the other PRs) that we're straightening out which is why I didn't check this build. It actually was pending permission on the signing phase. I gave it permission and will check again on it.

Looks like it succeeded. Thanks again @m-gallesio


Implement mongodb query syntax for task filters

This adopts code from https://github.com/kapouille/mongoquery to implement a mongodb-like syntax for task filtering in karton.
As mentioned in the code, there are some pitfalls since the old negative matching works in three-state logic and can't be translated to the new query syntax literally. For a short example:
[
  {"platform": "!win32"},
  {"platform": "!linux"}
]

will match all non-linux non-windows samples, as was probably intended, but:
[
  {"platform": {"$not": "win32"}},
  {"platform": {"$not": "linux"}}
]

is wrong - this means literally task.headers["platform"] != "win32" or task.headers["platform"] != "linux" and hence will match every task.
To get equivalent behaviour with mongo syntax, you should use:
[{
  "platform": {
    "$not": { "$or": ["win32", "linux"] }
  }
}]

See #246 for original reasoning behind the old-style behaviour.
Background compatibility considerations

"Old style" filters should keep working (any regression here is a bug). We don't plan to remove or obsolete that syntax in the forseeable future. No migration is necessary.
It's not recommended to mix mongodb-style filters and negative matches, to avoid unexpected behaviour. We may actually want to explicitly check it and forbid it?
"Nesting" old style wildcards and mongodb syntax doesn't work. "platform": {"$or": ["win*", "linux*"]} will only work for a win* literal, for example.

closes https://github.com/CERT-Polska/karton/issues/245


Add UI preference to use mempool.space as alternative Liquid explorer

It would be cool if there was a UI preference to change explorer from blockstream.info to mempool.space

Duplicate of this #105


Reindent all the exercises

There was some inconsistent spacing across the board. I have no idea if this is something we want, but I went ahead and ran all the files through my syntax/indentation settings.

Personally I'm not too fazed as long as there's no tabs, they're just evil. :-)

Ah, I think my current syntax file is all tabs. Let me fix that. What is the usual choice for erlang for indentation? 2 spaces? 4 spaces?

4 spaces is more common in my experience, Erlang/OTP uses 4 and so does Basho.

Cool, I'll do that then. Thanks :) I'm having fun playing with erlang, even though I'm not actually doing any real programming in it.

This'll need to be redone / rebased / re-something before merging. Think I'd be a good idea to keep the formatting somewhat consistent.

Sorry, didn't notice this pull request before making several changes. If nothing else I'd have at least fixed indentation to 4 spaces before submitting my PRs.

Oh, hah. I completely forgot about this PR. I'll redo it :)

It looks like the files had all been reformatted. Closing :)


Doesn't compile on non-uwp Windows

So I currently get this when trying to compile nanorand on my Windows box:
  = note: libnanorand-3df99cf1d4f057c6.rlib(nanorand-3df99cf1d4f057c6.nanorand.3usfjb36-cgu.13.rcgu.o) : error LNK2019: unresolved external symbol BCryptGenRandom referenced in function _ZN8nanorand7entropy7windows19entropy_from_system17heb014656c23127ddE

This appears to be due to this being a non-uwp platform. Note that this happens even if the getrandom feature is enabled, because the windows entryopy module is still included with #[cfg(windows)], causing it to be linked in.
This appears to be the cfg that should be checked for to make sure BCryptGenRandom is available.
#[cfg(all(windows, target_vendor = "uwp"))]

OTOH, all the necessary platform detection already appears to be implemented in getrandom, so maybe we should just be relying on it instead? There's also some error handling being done which does not seem to be reproduced in nanorand as well.

What version of Windows are you on? It should work on anything Vista/Server 2008 and up.
Also, the reason getrandom isn't default is because I want there to be an option to compile without any deps or derives.

Try commit 7076fbd7322f40730524fa99574b0b7d5de457ab.

What version of Windows are you on? It should work on anything Vista/Server 2008 and up.

While probably available on the system (I have the Windows SDK enabled), it requires bcrypt.lib to be linked. And it's probably just the case that the uwp target of rustc links it in by default. So filtering the availability by target_vendor = "uwp" might be appropriate.

Also, the reason getrandom isn't default is because I want there to be an option to compile without any deps or derives.
All right. But when getrandom is enabled, the windows module should probably not be included at all I'm guessing?

Try commit 7076fbd.

Yep, that should work. We raced past each other :)

Should be fixed in nanorand 0.4.3.


React Native Sample Application

Is there any possibility to provide a sample using React Native with or without the Expo platform?

@someyoungideas We will be rolling out an example soon.


Results number should continue to increase while SSMS is open

Right now, the number resets to 1 each time.  If it continued to increase, it would be much easier because it would avoid having to rename the temp table each time after the first use.

Fixed in 9ce470706d21a316cf29621dcda1f0b91b6fc09d.


Breakpoint 2.6.1 missing from Ruby Gems site

I'm in the process of testing out a new libsass workflow, but I was updating my Ruby workflow and noticed that Breakpoint 2.6.1 hasn't been pushed to ruby gems. https://rubygems.org/gems/breakpoint
Since the gemspec file is still in the repo, I assume this was accidental.

I suspect this is in part due to a bug in breakpoint.gemspec used for building the gem via gem build breakpoint.gemspec.
the file references README.markdown and CHANGELOG.markdown instead of README.md and CHANGELOG.md, which causes the build to fail.

I'd really like to get this fixed ASAP. I've got an open-source code base that uses breakpoint and I'd like it to work for Ruby users or for node-sass users.
Unfortunately, Breakpoint 2.6.1 complains about using $breakpoint-* variables every single time you use breakpoint(), and Breakpoint 2.5 doesn't have breakpoint-set(), so there's no way to provide a single codebase that works for both.

I'll roll a release after the holiday break

On Nov 27, 2015, at 12:42 AM, John Albin Wilkins<EMAIL_ADDRESS>wrote:
I'd really like to get this fixed ASAP. I've got an open-source code base that uses breakpoint and I'd like it to work for Ruby users or for node-sass users.
Unfortunately, Breakpoint 2.6.1 complains about using $breakpoint-* variables every single time you use breakpoint(), and Breakpoint 2.5 doesn't have breakpoint-set(), so there's no way to provide a single codebase that works for both.
—
Reply to this email directly or view it on GitHub.

Please release :)


HADOOP-16794 S3 Encryption key is not getting set properly during put operation.

Debugging details:
When a put operation is performed without -d parameter is used, it happens in two steps. First is copying to a temporary file and then copying the temp file to the right location. The is issue is durning the copy step as an extra header "x-amz-server-side-encryption: aws:kms" is set.
When -d parameter is used, the file is uploaded directly and in that api call this kms header is not set.
Solution: Removing the kms header from the ClonedObjectMetatData.
Testing: Tested by compiling the complete hadoop distribution and invoking the hadoop shell command locally.
Bucket used : https://mthakur-data.s3.ap-south-1.amazonaws.com/file2
Please suggest how to write testcases for this. Thanks

My solution is something like -
Just override the test method in the derived test class and put @ignore annotation.

yes, do that. it'll stop the test runs doing needless work
one thing to consider (actually, we may want to do it in the base), is verify that on directory rename the keys propagate properly too. I know, it's "obvious" it will, but with object stores, we have to not retain our filesystem preconceptions.
Maybe: in testEncryptionOverRename , rename the file into a directory. Then followup with a rename of the directory and verification of the file again

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
30m 35s
Docker mode activated.

_ Prechecks _

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

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

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

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
18m 14s
trunk passed

+1 :green_heart:
compile
0m 35s
trunk passed

+1 :green_heart:
checkstyle
0m 28s
trunk passed

+1 :green_heart:
mvnsite
0m 40s
trunk passed

+1 :green_heart:
shadedclient
13m 30s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 30s
trunk passed

+0 :ok:
spotbugs
0m 58s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 56s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 32s
the patch passed

+1 :green_heart:
compile
0m 26s
the patch passed

+1 :green_heart:
javac
0m 26s
the patch passed

-0 :warning:
checkstyle
0m 18s
hadoop-tools/hadoop-aws: The patch generated 1 new + 13 unchanged - 1 fixed = 14 total (was 14)

+1 :green_heart:
mvnsite
0m 32s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedclient
13m 20s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 25s
the patch passed

+1 :green_heart:
findbugs
1m  2s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 25s
hadoop-aws in the patch passed.

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

85m 28s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1823

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 69568f12de59 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 1e3a0b0

Default Java
1.8.0_242

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/3/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/3/testReport/

Max. process+thread count
411 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/3/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  5s
Docker mode activated.

_ Prechecks _

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

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

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

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
18m 13s
trunk passed

+1 :green_heart:
compile
0m 35s
trunk passed

+1 :green_heart:
checkstyle
0m 27s
trunk passed

+1 :green_heart:
mvnsite
0m 39s
trunk passed

+1 :green_heart:
shadedclient
13m 29s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 28s
trunk passed

+0 :ok:
spotbugs
0m 58s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 57s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 33s
the patch passed

+1 :green_heart:
compile
0m 27s
the patch passed

+1 :green_heart:
javac
0m 27s
the patch passed

+1 :green_heart:
checkstyle
0m 19s
hadoop-tools/hadoop-aws: The patch generated 0 new + 13 unchanged - 1 fixed = 13 total (was 14)

+1 :green_heart:
mvnsite
0m 31s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedclient
13m 20s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 27s
the patch passed

+1 :green_heart:
findbugs
1m  2s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 25s
hadoop-aws in the patch passed.

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

55m 56s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1823

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 48283c2c2842 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 1e3a0b0

Default Java
1.8.0_242

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/4/testReport/

Max. process+thread count
454 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/4/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

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

This message was automatically generated.

Hi @mukund-thakur and thank you for offering up this PR. I've run into this bug and while removing the SSE header when copying will work (assuming the bucket has that KMS set as the default encryption on the bucket), another option could be to copy both the SSE header x-amz-server-side-encryption as well as the x-amz-server-side-encryption-kms-key-id header that specifies the specific key to use. That would ensure that files don't revert to using the generic aws/s3 KMS key and might be less likely to cause issues in edge cases. Just my $0.02... :)

@davidarcher -so if we add an extra header, that will do it? OK.
That's an interesting thought. It means that on rename() whatever key an object had is preserved, but I don't see any problems with that...AFAIK people don't deliberately rename files to change the encryption options

@davidarcher -so if we add an extra header, that will do it? OK.

Yeah, it's mentioned in the S3 docs:

When you upload an object, you can specify the AWS KMS CMK using the x-amz-server-side-encryption-aws-kms-key-id header. If the header is not present in the request, Amazon S3 assumes the AWS managed CMK.

The "AWS managed CMK" here means the generic aws/s3 KMS key.
For context, my use case is Apache Spark jobs writing to S3 buckets. The client has no encryption settings specified and each bucket is configured with its own KMS key set as the default encryption. We noticed that files were still showing up using the generic aws/s3 key and determined it is because of this behavior of files being uploaded and then renamed to their final name (and switching to the generic aws/s3 KMS key due to this bug).

@davidarcher  Yes this could be the other solution which even I thought initially. But looks like S3's ObjectMeta doesn't have any setter method to set this x-amz-server-side-encryption-aws-kms-key-id directly. I think this is done to ensure the transparency of default S3 encryption.

@davidarcher Yes this could be the other solution which even I thought initially. But looks like S3's ObjectMeta doesn't have any setter method to set this x-amz-server-side-encryption-aws-kms-key-id directly. I think this is done to ensure the transparency of default S3 encryption.

Thanks for looking into it @mukund-thakur -- From the AWS example for using S3 KMS with Java, it looks like it needs to be set directly on the request using the withSSEAwsKeyManagementParams or setSSEAwsKeyManagementParams method.
We are already calling setSSEAwsKeyManagementParams here in the setOptionalCopyObjectRequestParameters method but we only consider the KMS key id configured on the client -- maybe we can fallback there to looking at the KMS key id from the source object?

So what is proposed is

If we have local SSE-KMS settings, use that
fall back to that on the source object

What if we have local SSE-S3 enabled? would we want to switch for a simple rule: "if you have encryption set, renamed files pick it up; otherwise files encrypted with SSE-KMS keep their encryption"
As @mukund-thakur's patch currently has
"files encrypted with SSE-KMS retain that encryption setting when renamed, irrespective of your client settings"
I think I prefer the one in mukund's patch as its simpler to explain. Encryption with SSE-KMS is "sticky"

So what is proposed is
If we have local SSE-KMS settings, use that
fall back to that on the source object
What if we have local SSE-S3 enabled? would we want to switch for a simple rule: "if you have encryption set, renamed files pick it up; otherwise files encrypted with SSE-KMS keep their encryption"

I'll defer to you for expected semantics here as I'm very new to this code base but my understanding is that this is the existing behavior for Copy operations for other SSE types.

As @mukund-thakur's patch currently has
"files encrypted with SSE-KMS retain that encryption setting when renamed, irrespective of your client settings"
I think I prefer the one in mukund's patch as its simpler to explain. Encryption with SSE-KMS is "sticky"

Isn't it "files encrypted with SSE-KMS reset to the bucket default encryption when renamed, irrespective of your client settings" though?
It seems a bit inconsistent that for other SSE types, renames don't change encryption settings (unless explicitly specified in the client settings) but for SSE-KMS, with this change, renames reset to whatever is specified as default encryption on the bucket (unless explicitly specified in the client settings).

see #1843 for a patch on top of this, to help propagate settings better; let's keep discussion in this patch & mukund can cherrypick that new commit while we get this right.
I think we should have a consistent policy here of

if the client has any encryption settings, including explicit AES256, KMS+default key, KMS+custom key, then they will set the encryption options on the copy.
else the encryption settings of the source file are retained.

This is nice and memorable. It needs to apply for all s3a encryption settings; this patch now only does it for SSE-KMS.

Not sure why two tests are failing. Btw they are failing in trunk branch as well.

[ERROR] Tests run: 12, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 20.515 s <<< FAILURE! - in org.apache.hadoop.fs.s3a.ITestS3AMiscOperations
[ERROR] testEmptyFileChecksums(org.apache.hadoop.fs.s3a.ITestS3AMiscOperations)  Time elapsed: 2.728 s  <<< FAILURE!
java.lang.AssertionError: checksums of empty files expected:<etag: "1463cbcf1dbae282559d1fec36f131f2"> but was:<etag: "a92916168376ec017a3c98c92102dd7e">
at org.apache.hadoop.fs.s3a.ITestS3AMiscOperations.testEmptyFileChecksums(ITestS3AMiscOperations.java:165)
[ERROR] testNonEmptyFileChecksumsUnencrypted(org.apache.hadoop.fs.s3a.ITestS3AMiscOperations)  Time elapsed: 2.031 s  <<< FAILURE!
java.lang.AssertionError: checksums expected:<etag: "e52b1ed8f37dfca9a3bc8954875d049a"> but was:<etag: "4aaae27ccdbc799df38d35eed11f4c52">
at org.apache.hadoop.fs.s3a.ITestS3AMiscOperations.testNonEmptyFileChecksumsUnencrypted(ITestS3AMiscOperations.java:219)

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
19m  0s
trunk passed

+1 :green_heart:
compile
0m 34s
trunk passed

+1 :green_heart:
checkstyle
0m 27s
trunk passed

+1 :green_heart:
mvnsite
0m 39s
trunk passed

+1 :green_heart:
shadedclient
15m  3s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 28s
trunk passed

+0 :ok:
spotbugs
0m 59s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 58s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 33s
the patch passed

+1 :green_heart:
compile
0m 28s
the patch passed

+1 :green_heart:
javac
0m 28s
the patch passed

-0 :warning:
checkstyle
0m 19s
hadoop-tools/hadoop-aws: The patch generated 8 new + 10 unchanged - 3 fixed = 18 total (was 13)

+1 :green_heart:
mvnsite
0m 31s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedclient
13m 57s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 25s
the patch passed

+1 :green_heart:
findbugs
1m  2s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 24s
hadoop-aws in the patch passed.

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

58m 16s

Subsystem
Report/Notes

Docker
Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/5/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1823

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 077010daea5d 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / a562942

Default Java
1.8.0_242

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/5/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/5/testReport/

Max. process+thread count
440 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/5/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
19m 22s
trunk passed

+1 :green_heart:
compile
0m 36s
trunk passed

+1 :green_heart:
checkstyle
0m 27s
trunk passed

+1 :green_heart:
mvnsite
0m 40s
trunk passed

+1 :green_heart:
shadedclient
15m  9s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 29s
trunk passed

+0 :ok:
spotbugs
1m  0s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 56s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 33s
the patch passed

+1 :green_heart:
compile
0m 28s
the patch passed

+1 :green_heart:
javac
0m 28s
the patch passed

-0 :warning:
checkstyle
0m 19s
hadoop-tools/hadoop-aws: The patch generated 15 new + 17 unchanged - 3 fixed = 32 total (was 20)

+1 :green_heart:
mvnsite
0m 32s
the patch passed

-1 :x:
whitespace
0m  0s
The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1 :green_heart:
shadedclient
13m 39s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 26s
the patch passed

+1 :green_heart:
findbugs
1m  2s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 19s
hadoop-aws in the patch passed.

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

58m 39s

Subsystem
Report/Notes

Docker
Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/6/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1823

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 5fb73514b598 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / a562942

Default Java
1.8.0_242

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/6/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt

whitespace
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/6/artifact/out/whitespace-eol.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/6/testReport/

Max. process+thread count
449 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/6/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 46s
Docker mode activated.

_ Prechecks _

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

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

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

_ trunk Compile Tests _

-1 :x:
mvninstall
7m  4s
root in trunk failed.

+1 :green_heart:
compile
0m 44s
trunk passed

+1 :green_heart:
checkstyle
0m 26s
trunk passed

+1 :green_heart:
mvnsite
0m 37s
trunk passed

+1 :green_heart:
shadedclient
15m 54s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 27s
trunk passed

+0 :ok:
spotbugs
0m 57s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 55s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 34s
the patch passed

+1 :green_heart:
compile
0m 29s
the patch passed

+1 :green_heart:
javac
0m 29s
the patch passed

-0 :warning:
checkstyle
0m 19s
hadoop-tools/hadoop-aws: The patch generated 2 new + 17 unchanged - 3 fixed = 19 total (was 20)

+1 :green_heart:
mvnsite
0m 32s
the patch passed

-1 :x:
whitespace
0m  0s
The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1 :green_heart:
shadedclient
14m 42s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 26s
the patch passed

+1 :green_heart:
findbugs
1m  2s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 25s
hadoop-aws in the patch passed.

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

49m 44s

Subsystem
Report/Notes

Docker
Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/7/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1823

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 773c98ea0241 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / ec75071

Default Java
1.8.0_242

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/7/artifact/out/branch-mvninstall-root.txt

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/7/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt

whitespace
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/7/artifact/out/whitespace-eol.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/7/testReport/

Max. process+thread count
455 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/7/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

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

This message was automatically generated.

Don't worry about the test failures, we have a patch for that which I should see about merging in
https://issues.apache.org/jira/browse/HADOOP-16319

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
19m  8s
trunk passed

+1 :green_heart:
compile
0m 36s
trunk passed

+1 :green_heart:
checkstyle
0m 28s
trunk passed

+1 :green_heart:
mvnsite
0m 40s
trunk passed

-1 :x:
shadedclient
16m 22s
branch has errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 29s
trunk passed

+0 :ok:
spotbugs
1m  5s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m  1s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 35s
the patch passed

+1 :green_heart:
compile
0m 29s
the patch passed

+1 :green_heart:
javac
0m 29s
the patch passed

-0 :warning:
checkstyle
0m 20s
hadoop-tools/hadoop-aws: The patch generated 18 new + 17 unchanged - 3 fixed = 35 total (was 20)

+1 :green_heart:
mvnsite
0m 32s
the patch passed

-1 :x:
whitespace
0m  0s
The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1 :green_heart:
shadedclient
15m 37s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 24s
the patch passed

+1 :green_heart:
findbugs
1m  5s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 19s
hadoop-aws in the patch passed.

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

61m 46s

Subsystem
Report/Notes

Docker
Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/8/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1823

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 0ed51e100dcb 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 10461e0

Default Java
1.8.0_242

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/8/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt

whitespace
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/8/artifact/out/whitespace-eol.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/8/testReport/

Max. process+thread count
455 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/8/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
19m 23s
trunk passed

+1 :green_heart:
compile
0m 35s
trunk passed

+1 :green_heart:
checkstyle
0m 27s
trunk passed

+1 :green_heart:
mvnsite
0m 39s
trunk passed

+1 :green_heart:
shadedclient
14m 55s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 29s
trunk passed

+0 :ok:
spotbugs
1m  0s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 56s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 32s
the patch passed

+1 :green_heart:
compile
0m 27s
the patch passed

+1 :green_heart:
javac
0m 27s
the patch passed

-0 :warning:
checkstyle
0m 19s
hadoop-tools/hadoop-aws: The patch generated 3 new + 17 unchanged - 3 fixed = 20 total (was 20)

+1 :green_heart:
mvnsite
0m 31s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedclient
13m 57s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 25s
the patch passed

+1 :green_heart:
findbugs
1m  3s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 25s
hadoop-aws in the patch passed.

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

58m 42s

Subsystem
Report/Notes

Docker
Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/9/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1823

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 2dd49a081a50 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / fccfb02

Default Java
1.8.0_242

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/9/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/9/testReport/

Max. process+thread count
420 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1823/9/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

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

This message was automatically generated.

All review comments addressed.

Thanks @steveloughran.


return errno instead of -1

return errno instead of -1, so that the test can tell why the start() fails

lets keep what it is


I suspect you didn't mean to use self.scalar...;

Instead, just the reference to the passed parameter: scalar,
which speaks to how rarely this particular method is getting used (in testing?)

Good Idea. Forgot about that, and yes, self.scalar is definitely not needed.

Thanks for reporting that bug.


Add basic git commands page link in footer

In the footer section under resources section add another link to this page.
https://clueless.vercel.app/whattodo/basic-github-commands.
You should use a Link tag here and the href will be /whattodo/basic-github-commands" .
Please add a screenshot after you make a PR.

Hey @Rajdip019 , can you assign this task to me?

Sure go ahead

Sure go ahead

Here's the PR for the issue!
https://github.com/Clueless-Community/first-contribution/pull/360

Closing this issue as it is done.


[Bug] 树形图 polyline 更新数据时连接线会消失

Version
5.4.3
Link to Minimal Reproduction
https://echarts.apache.org/examples/zh/editor.html?c=tree-polyline&version=5.4.3&code=M4UwLgkgdmIE4DcCGAbAFAMwK5QMZgEsB7KNASgAIBvAKAotxODAoBMkwkKBeau-ilCQBbEAC4KAciSSANP3q4AFgRSs4IKBIDaCgbQGHBI8RQCyHJQDo4SKKyLDy8oxQC-LgQF1-bgNz8RAAOhCQ8fIagcAQgwDp69AauFGAAnkGmkmAaIHIJAuycOoVIXp6uIKwA5iAAykpIGRIAREFEKKkoBFAgzeVGKEgARiAoEknJFENIuADWVXBEOKwAwu1EcBKSAMQYe3mT9G3ABKFaUiggGGAHhwjwhLioAIJdVeeSwgSsrJe3k6gCO8ttEqkobvl6B5IRRLkh7nEIodBiMxkjDhRjqdiB9QeD_ncHgQnihXkCPl8fn9-gC3h9LtdJDCoTDoa43HofFCAvRhKkVg04GArKAwAB5EI4tDBM5kAIeCgAJgADKq5UA
Steps to Reproduce
setInterval(function() {
const data = {
name: 'a',
children: [
{
name: Math.random(),
},
]
};
option = {
series: [
{
type: 'tree',
data: [data],
edgeShape: "polyline",
label: {
backgroundColor: '#fff',
position: 'left',
verticalAlign: 'middle',
align: 'right'
},
leaves: {
label: {
position: 'right',
verticalAlign: 'middle',
align: 'left'
}
},
}
]
};
myChart.setOption(option);
}, 2000);
Current Behavior
当数据更新时，连接线消失
Expected Behavior
当数据更新时，连接线不消失
Environment
- OS: Windows 10.8.4
- Browser: Chrome 115
- Framework: None

Any additional comments?
No response

This looks like a replace operation, not an update.
For a proper replace, use myChart.setOption(option, true);.
For an update do this.
Both work as intended.

This looks like a replace operation, not an update. For a proper replace, use myChart.setOption(option, true);. For an update do this. Both work as intended.

Thank you for reply, but the first solution myChart.setOption(option, true); does not work, and the second solution works.
And the third solution is:
	echarts.dispose(chart);
	var myChart = echarts.init(chart);

I don't know why the first solution does not work.

Because of edgeShape: "polyline", - remove it, sorry did not mention that.
NB: please close issue if problem solved.

I'm sorry but I need edgeShape: "polyline" because of the line style.
So the issue is not solved for update operation.
The only workaround is to dispose the instance and re-init.

It is all related to this bug #18448

Yes I agree


Improve bin deser error message

Messed up, it's the same as #24.

src/serde_bin.rs

What do you mean?


Button on Notifications Not Present

I have just moved over to TSMessage to RMessage and the buttons on the notifications are not visible. I took a look on the example project and yet they aren't present on those either.
Heres my code:
[RMessage showNotificationInViewController:self
                                             title:@"Welcome to Version 4.4"
                                          subtitle:@"Would you like to take a read of the release notes?"
                                         iconImage:[UIImage imageNamed:@"iconNotes"]
                                              type:RMessageTypeCustom
                                    customTypeName:@"sdTheme"
                                          duration:RMessageDurationEndless
                                          callback:^{
                                              NSLog(@"Tapped Notification");
                                          }
                                       buttonTitle:@"Read"
                                    buttonCallback:^{
                                        [self showReleaseNotes];
                                    }
                                        atPosition:RMessagePositionTop
                              canBeDismissedByUser:YES];

That produces this but no button is visible:

This is expected, as per the README, the button functionality is not enabled unless you are on RMessage 3.x. Yes you can pass a button in in the calls. No it will not work unless on 3.x.

@docash59 I added button functionality to the develop branch. Now you can pass in a button via the parameters. Much more customizeable buttons can be passed in via the customizeMessageView delegate function and the setButton call on RMessageView.

That great thank you @donileo!
I will give it a test later.


Fix CI Pytest

@tomerguttman and I encountered some errors regarding the Pytest section
can you please check it before the project submission on Tuesday?
@ShayAvraham @davidmdi

Make sure it's your highest priority

@ilanKushnir  I'm on it!


Transaction simulation failed: Blockhash not found

Very often when i try to transfer fungible token i receive this error:  Transaction simulation failed: Blockhash not found any idea how to handle it

Hi there, we don't use GitHub for support, please join us in Discord at #developer-support https://solana.com/discord


What is the purpose of this text field

I was wondering if this input field offer some hidden feature.
Thanks

When you click on the arrow on the far right of every kint container, it pops out in a new window, this can be used for keeping track, comparing dumps or whatever. You can use this field to jolt something about what you're seeing :)

I mean the tiny blue arrow -> in this picture

Thanks!


somelist.getSegment() fails when there is an Interests condition type.

java.lang.IllegalArgumentException: No enum constant com.github.alexanderwe.bananaj.model.list.segment.Operator.INTERESTCONTAINS
	at java.lang.Enum.valueOf(Unknown Source)
	at com.github.alexanderwe.bananaj.model.list.segment.Operator.valueOf(Operator.java:6)
	at com.github.alexanderwe.bananaj.model.list.MailChimpList.getSegment(MailChimpList.java:304)

JSON response from Mailchimp API
{
    "id": 40233,
    "name": "Math",
    "member_count": 1,
    "type": "saved",
    "created_at": "2018-02-28T22:41:39+00:00",
    "updated_at": "2018-02-28T22:41:39+00:00",
    "options": {
        "match": "any",
        "conditions": [{
            "condition_type": "Interests",
            "field": "interests-6dc9e2022a",
            "op": "interestcontains",
            "value": ["1a51eb7461"]
        }]
    },
    "list_id": "17a08b3a2d",
    "_links": [{
...
    }]
}

The schema for each condition varies depending on the condition_type. In this case it will also fail because value is an array type where banbanaj only accepts a string for value.


remove magic-secateurs. No longer needed

Remove magic-secateurs plugin as it is no longer needed after Poll 78 changes.

Instead of deleting the plugin, set disabled=true

You can just re-open this PR and push a new commit instead of opening a fully new PR