Docker driver - volume /var/rund/docker.sock:/var/run/docker.sock is not mapped or is removed

Impossible to map volume (host to container) /var/run/docker.sock to /var/run/docker.sock using Docker driver.
Possible root cause for molecule:
docker_container module
Possible base root cause in docker.py usage of the module

Is it possible to fix ?
Yes, but Its pretty hacky.
How ?
Map docker.sock to some other location, symbolic link the sock to /var/run/docker.sock

Molecule and Ansible details
ansible 2.6.3
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/ilhicas/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python2.7/dist-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.15rc1 (default, Apr 15 2018, 21:51:34) [GCC 7.3.0]

ansible 2.6.3
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/ilhicas/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python2.7/dist-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.15rc1 (default, Apr 15 2018, 21:51:34) [GCC 7.3.0]

Molecule installation method (one of):

source

Ansible installation method (one of):

os

Driver used:
Docker
Desired Behavior
Volume /var/run/docker.sock is mapped to internal /var/run/docker.sock
using volumes
Actual Behaviour
/var/run/docker.sock is missing inside container
Please give some details of what is actually happening.
molecule.yml
dependency:
  name: galaxy
  enabled: true
  options:
    ignore-certs: True
    ignore-errors: True
    #Relative path search here walks from role root folder (this case ansible) all the way to the scenario (in this case it lives under ansible/requirements.yml, for specific set ansible/molecule/scenario-name/requirements.yml)
    role-file: molecule/scenario_name/requirements.yml
driver:
  name: docker
lint:
  name: yamllint
  enabled: false
platforms:
  - name: container_name
    image: fiercely/centos7:systemd
    privileged: True
    published_ports:
      - "80"
      - "443"
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:rw
      - /var/run/docker.sock:/var/run/docker.sock
    command: "/usr/sbin/init"
    groups:
      - some_group
provisioner:
  name: ansible
  roles_path: roles
  lint:
    name: ansible-lint
    enabled: false
  inventory:
    links:
      group_vars: ../../group_vars/
scenario:
  name: scenario_name
  test_sequence:
    - destroy
    - dependency
    - create
    - prepare
    - converge
    - side_effect
    - verify
    - destroy
verifier:
  name: testinfra
  lint:
    name: flake8

playbook.yml
---
- name: Converge
  hosts: some_group

How to reproduce:
molecule test -s scenario_name

docker exec -it container_name bash
#Inside container
ls -laF /var/run | grep docker.sock
#Mapped docker sock is not found

I suspect the cause is that a tmpfs is mounted on /var/run (or /run if it's symlink) after /var/run/docker.sock is mounted, so the socket file is no longer visible:
$ molecule login
# mount | grep run
tmpfs on /run/docker.sock type tmpfs (rw,nosuid,nodev,noexec,relatime,size=204704k,mode=755)
tmpfs on /run type tmpfs (rw,nosuid,nodev,mode=755)

As a work around for this issue you can

in your molecule.yml platforms/instance mount your volume like this: /var/run/docker.sock:/tmp/docker.sock
create prepare.yml with following steps

name: create docker.sock
raw: touch /var/run/docker.sock
become: true
changed_when: false
name: move docker.sock from tmp
raw: mount --move /tmp/docker.sock /var/run/docker.sock
become: true
changed_when: false

this will move the /tmp/docker.sock to /var/run/docker.sock where it should be :D

DoD is a known issue and you should don even try to use containers to test docker related playbooks. I will close this because that is not something molecule can do anything about.

I was able to start a container insdie the molecule instance. I installed docker with Dockerfile.j2. Then I followed @wildone instructions to move the docker sockets mount inside the prepare.yml, but ran into an error. I was able to start a container with a symbolic link from inside the molecule host.
- name: Prepare
  hosts: all
  tasks:
    - name: "link docker.sock to /var/run"
      raw: ln -s /tmp/docker.sock /var/run/docker.sock


Add definitions and references for constructors

Closes #249
This replicates VSCode's gotodef.
Test plan
Tested locally with a snapshot, still need to test edge cases + implement and test inherited constructors.

CI failing due to different TypeScript versions :/
Not sure why it's not respecting the ^ directive in the package.json and just getting the oldest compatible version instead. I guess I can downgrade and regenerate.

Undo changes in yarn.lock, I suspect this might fix the failing CI

The current package.json seems to be invalid due to a peer resolution issue (I ran yarn install --immutable --immutable-cache to not modify the yarn.lock) so my install won't go through. How do you install the deps for this repo without modifying the package.json/lockfile? :)

Figured out the issue - the yarn version used here is a few years behind it seems, so I installed v1.22.19 and everything worked out!

. How do you install the deps for this repo without modifying the package.json/lockfile? :)

I normally run yarn without a subcommand


refactor: add Node 12 to testing matrix

I am afraid this will not cut it. We need a bit more to run tests on both node 12.x and 14.x (and newer). Please see: https://github.com/trygve-lie/rollup-plugin-import-map/pull/10


Use semver in the dependencies

something like the following:
 <dependency>
      <groupId>junit</groupId>
      <artifactId>junit</artifactId>
      <version>[3.0, 3.8.2)</version>
    </dependency>

http://www.mojohaus.org/versions-maven-plugin/examples/resolve-ranges.html
https://docs.gradle.org/current/userguide/dependency_management.html#sub:dynamic_versions_and_changing_modules

Could you provide justification on why we need this?
I think it's a bad idea because it introduces non determinism into the build process and makes builds unreproducible. Personally, I would be particularly annoyed by a build that works locally, but not on CI, or a build that worked in CI, but doesn't work after a retrigger, or a local build that worked yesterday but is broken today.
This makes an assumption that the authors of the dependencies use semver correctly, and that the cost of manually updating dependencies is higher than the cost of figuring out potential issues arising from dependencies being bumped up arbitrarily.


Japanese IME input mode switched back to Direct Input after pressing Enter key

Windows Terminal version
1.14.1451.0
Windows build number
10.0.19044.1766
Other Software
No response
Steps to reproduce

Turn on Hiragana input mode
Type something then press Enter to convert to japanese

Expected Behavior
The input mode should not switch back to Direct Input
Actual Behavior
The input mode switched to Direct Input automatically after conversion

Might be related: https://github.com/microsoft/terminal/issues/12731

If we can't have a smarter way to detect when should we use compose mode and when should we use direct input, I would like to have an option to disable this behavior. Since I already installed a separate English (US) language along side Chinese (Simplified, MS Pinyin) so that I can turn off IME for good if I don't need it.

At least MS Pinyin handles it this way with conhost:
On first start of conhost, uses direct input by default.
If you enable compose mode with Shift, do not automatically switch back to direct mode.

I have encountered this issue, and it never occurred in previous versions.

Best regards.

Might be related: #12731

It's indeed related as the problem occurred after upgrading to v1.14.1432.
Hope that there is a way to disable this behavior, because I tend to press enter key to end IME composition.
Automatically switch back to direct mode each time make it hard to input Japanese text in the terminal.

I also have the same problem.
When I type Chinese, then I press enter it jumps to English IME.
But I am not done with my Chinese typing.

This problem does not occur with Google Japanese Input.
However, there is a bug that it automatically switches to "HankakuEisu" mode when starting Windows Terminal.

I'm having the same problem with MS-IME on Windows Terminal version 1.14.1963.0.
I hope it fixes.  🙏

When the Microsoft IME version was lowered, this event no longer occurred.

When the Microsoft IME version was lowered, this event no longer occurred.
It didn't work for me. Does someone know other work-arounds?

If finalize with Ctrl-m instead of Enter, the IME will not turn off. But it is incredibly annoying method 😅

I'm running into the same issue.
In my environment, I can avoid the issue by waiting 0.5 - 1.0 seconds before pushing enter. Hope this helps some debugging.

I also have the same problem.
In my case, the workaround mentioned by @YossyHal works fine.

When the Microsoft IME version was lowered, this event no longer occurred.

ty.

I wonder if there's a way to only request AlphanumericHalfWidth the very first time the IME opens for a pane, and then just continue using the previously used mode for subsequent invocations.
@YanceyChiew as an FYI. I'm not as familiar with the intricacies of the IME APIs, so you might have some better ideas here.

I wonder if there's a way to only request AlphanumericHalfWidth the very first time the IME opens for a pane, and then just continue using the previously used mode for subsequent invocations.

In short, yes, there is.
This is the default behavior for Chinese input, so I thought there was no need for additional processing, but I didn't expect it to cause trouble for Japanese input method users.

The root of the problem lies in these few lines of code:
    void TSFInputControl::ClearBuffer()
    {
        if (!_inputBuffer.empty())
        {
            TextBlock().Text(L"");
            const auto bufLen = ::base::ClampedNumeric<int32_t>(_inputBuffer.length());
            _inputBuffer.clear();
            _editContext.NotifyFocusLeave();
            _editContext.NotifyTextChanged({ 0, bufLen }, 0, { 0, 0 });
            _editContext.NotifyFocusEnter();
            _activeTextStart = 0;
            _inComposition = false;
        }
    }

It can be seen that this code will not perform any effective operation when _inputBuffer.empty(), which is consistent with the performance of the Japanese input method that does not exit compose mode when press Enter directly with empty command line.
Maybe it is because the Microsoft Japanese input method is in the English (direct) input mode by default, and it's handling of lost input focus is also different from the Chinese input method, after each call to _editContext.NotifyFocusLeave(), it will exit from the compose input mode to the direct input mode.
Incidentally, in another issue(#1304), someone wanted such a feature, but I found it difficult to implement cleanly.
(ps. When using the old Japanese input method, it will exit to direct input mode when pressing enter only when there are no characters in the input buffer of compose mode, so only newlines will cause compose mode to exit uncontrollably.)

The easiest solution I can think of is to modify the _editContext.InputScope back to CoreTextInputScope::Text as soon as the input method gets focus for the first time. By testing in another example project, I found that it works fine with both Chinese and Japanese input methods.
But in this project, I haven't found a suitable place to insert that line of code yet. It can also be inserted in the code block quoted above, but that looks a bit dirty doing pointless operations multiple times.

It seems you are right.
I removed the changes in PR #13028 from the current HEAD(7976e4819) source, compiled it, and confirmed that it works fine with the new Japanese MS-IME.
I changed the line at at src/cascadia/TerminalControl/TSFInputControl.cpp:43

    _editContext.InputScope(CoreTextInputScope::AlphanumericHalfWidth);

to

    _editContext.InputScope(CoreTextInputScope::Text);

However, there may be other causes as well.
It seems that this issue only occurs with the current version of Japanese MS-IME, not with Google-Input or MS-IME(previous ver). I also tried ATOK2022, a typical Japanese IME, but it didn't occur this issue.

This may be because InputScope is not implemented by the two non-Microsoft input methods mentioned above, its enum values are simply ignored. And in my test, the old version of Microsoft Japanese input method will also be affected when wrapping.

So, the question now is, use the simple solution to change the _editContext.InputScope back to CoreTextInputScope::Text as soon as the TSFInputControl is connected to the input method for the first time, or add a setting to the _editContext.InputScope as originally envisaged if it doesn't satisfy all users?
Both methods can solve the current problem, If we choose the former, since it cannot be assumed that the window is initialized with input focus, the code may have to be inserted before _editContext.NotifyFocusLeave(), making it itself or the code that determines whether it needs to be executed repeatedly. If we choose the latter, do we need to expose the option to the settings page in the gui, and have the default value determined by the user's system locale?

I think we should wait a while for other reports.
cf. @ironsand says that even using an older version of the IME did not work around this issue.
I'd like to vote for "add a setting". And "Set default values determined by the user's system locale" is a good idea. With this, most people will not need to change this setting, so it seems to me that there is no problem even if editing the json is the only way to change this setting. (of course, it would be nice to have a gui)

I believe the easiest fix is to remove the calls to NotifyFocusLeave() and NotifyFocusEnter() inside TSFInputControl::NotifyFocusEnter(). But I'm currently waiting to see whether I can fix the general behavior of the input first (#11479).

I believe the easiest fix is to remove the calls to NotifyFocusLeave() and NotifyFocusEnter() inside TSFInputControl::ClearBuffer()

I've tried it and confirmed that it works fine for this issue.
However, the commit #13677 seems to have excluded this fix.
It looks like this issue is still open and I hope it gets fixed.
ty

I'm sorry for bringing this up again.
Thanks to the commit https://github.com/microsoft/terminal/pull/13678, the original issue is fixed in the latest version.
But recently, I've found that the new behavior is not very desirable. At least for me.
For example, when I'm in the middle of editing something in Hiragara mode then switch to a different application or make the Terminal lose focus. Then when the Terminal get focus again, the IME's input mode will be switched to Half-width alphanumeric.
It's not a bug, but kind of inconvenience. So may I request a setting to disable the new behavior?
v1.13.10983.0

v1.15.2282.0

I don't have other CJK languages installed, so it might happen with the Japanese MS IME only.
Also, I believe that the new behavior is causing alt + shift to not retaining the last used input mode as mentioned in https://github.com/microsoft/terminal/issues/13681.
v1.13.10983.0

v1.15.2282.0


Increase default command timeout on operations against SQL Server master

Compensate for slowness in creating a database on SQL Server.

Looks good. :shipit:


futures_ordered does not preserve order of errors

I was surprised by the behavior of futures_ordered when faced with errors.
Here's what I was trying to do:

Issue stats gRPC calls to a list of hosts, timing out slow hosts.
Print the results in order. For hosts that timed out, print "timed out" instead (in order).

I created a vector of futures (one future per host), and wanted to get a vector of Results. I thought futures_ordered would be a nice way to do that. However, it doesn't appear to handle ordering of errors.
Here's an simple test case (against v0.1.20):
#[test]
fn futures_ordered_error() {
    use futures::future::ok;
    use futures::future::err;
    let stream = futures_ordered(vec![
        ok::<u32, String>(1),
        err::<u32, String>(String::from("second")),
        ok::<u32, String>(3),
    ]);
    let mut iter = stream.wait();
    assert_eq!(Some(Ok(1)), iter.next());
    assert_eq!(Some(Err(String::from("second"))), iter.next());
    assert_eq!(Some(Ok(3)), iter.next());
    assert_eq!(None, iter.next());
}

This fails with:
---- futures_ordered_error stdout ----
	thread 'futures_ordered_error' panicked at 'assertion failed: `(left == right)`
  left: `Some(Ok(1))`,
 right: `Some(Err("second"))`', tests/futures_ordered.rs:114:5
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Is this the intended behavior for futures_ordered?

This is definitely intended in the current implementation, but I think it's not an unreasonable ask to change queued_results to store a heap of Result<T::Item, T::Error> rather than just T::Item.

I see the Err option is going away in v0.3, so I'm closing this now.


Code Addition to Resolve SSL Error When Downloading CIFAR-10 Dataset

While downloading the CIFAR-10 dataset, an SSL error occurred.
To address this, I have wrapped the download method call within a try-except block.
If an SSL error is encountered, the code switches to an unverified SSL context to retry the dataset download.
This can serve as a temporary workaround in the face of SSL certificate verification issues.
After that run ufmt.

Thanks for the PR @dldldlfma . I guess this is a good workaround for users to have within their own code if they want to take the risk, but unfortunately this isn't something we should be having within torchvision: we don't want to disable SSL on behalf of users.


⚠ Jenkins Plugins Artifactory has degraded performance

In 5554f12, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1276 ms

Resolved: Jenkins Plugins Artifactory performance has improved in fbf710c after 19 minutes.


Compiler does not detect wrong variable type when ternary operator is used

Unfortunately the compiler does not detect if one tries to assign a wrong type so that this kind of error is only found during runtime. If you write
    List list = List();
    list = false ? List(): "";

It compiles fine but will throw an error during runtime when it tries to assign a String "" to the list "list" (due to the "false" it will use the last part of the ternary operator, which is a String). However just writing
    List list = List();
    list = "";

causes an error already at compile time.
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.17.5, on Mac OS X 10.15.7 19H2, locale en-DE)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
[✓] Android Studio (version 4.0)

Hi @dg76
From what I can see, the issue is related to Dart rather than to Flutter itself. Please open the issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself.
If you disagree, please write in the comments,
providing your flutter doctor -v, your flutter run -v,  your pubspec.yaml
a minimal reproducible code sample that does not use 3rd party plugins,
and I will reopen it.
Thank you


Validate Checkpoint in ChangePack for PushPull API requests

Description:
When the Client calls the PushPull API and sends the request ChangePack, Checkpoint in the request may be tampered with due to various reasons such as network delays causing duplicate requests, new SDK bugs, or intentional tampering by malicious clients.
Therefore, it is beneficial for the stability and security of the system to validate Checkpoint.
Consider the following validation checks:

Changes in the request ChangePack passed to PushPull API are created by a single Client, so Change.ID.Checkpoint.ClientSeq should increment sequentially by one.
Checkpoint.ServerSeq in the request ChangePack for PushPull API cannot be greater than the server's Checkpoint.ServerSeq since it is set when the server saves the Change to the database.

If the Checkpoint is invalid, consider the following exception handling:

If duplicate requests caused by network delays treat them as OK
For other reasons, return INVALID_ARGUMENT

Why:
This validation will help ensure the integrity and security of the system.

I'm interested in this issue. may I try this?

I'm interested in this issue. may I try this?

@binary-ho Sure. If you have any questions, feel free to ask.

Hello, I have some questions while trying to resolve the issue.

Issue: Validate Checkpoint in ChangePack for PushPull API requests
Reference: PushPull function in yorkie_server.go file

1. Where do we obtain the server's ClientSeq and ServerSeq?
To resolve this issue, I believe comparing the ClientSeq and ServerSeq included in the client request with the ClientSeq and ServerSeq stored by the server in the Document of the project would be appropriate.
I think we can use the ClientSeq and ServerSeq of the Checkpoint in the ClientInfo struct, which can be obtained through the ClientId of the Request Message. Is my understanding correct?

2. Where do we obtain the client's ClientSeq and ServerSeq?
The ChangePack of the Request Msg contains a Checkpoint and an array of Changes.

The Checkpoint has its own ClientSeq and ServerSeq.
Each Change in Changes also has its own ClientSeq and ServerSeq in its ChangeID.

According to the issue description, "so Change.ID.Checkpoint.ClientSeq should increment sequentially by one." Should the client's ClientSeq refer to the Seq values of each Change in this context? (Not the ClientSeq in the Checkpoint of the ChangePack in the Msg.)
Additionally, you mentioned "Checkpoint.ServerSeq in the request ChangePack for PushPull API cannot be greater than the server's Checkpoint.ServerSeq since it is set when the server saves the Change to the database."
For the client's ServerSeq, should we use the ServerSeq in the Checkpoint of the ChangePack in the Msg? (Not the ServerSeq values of each Change.)

3. How do we determine if the ClientSeq is invalid?
You mentioned "Change.ID.Checkpoint.ClientSeq should increment sequentially by one." Does this mean it is problematic if ClientSeq_sent_by_client ≤ ClientSeq_of_clientInfo?
(I am likely confused because I don't know exactly where and how the ClientSeq is generated.)

4. How do we determine if a request is a duplicate?
In the case where ClientSeq_sent_by_client == ClientSeq_of_clientInfo, can we consider this as a duplicate request due to network delays? I seem to be unclear about the criteria for determining a "duplicate request."

5. How do we determine if the ServerSeq is invalid?
If ServerSeq_sent_by_client > ClientSeq_of_clientInfo, is this an invalid situation?


setup/cri: add a section about cgroup drivers

A cgroup driver is an important setting for the CRI and the kubelet.
Add a section that explains why "systemd" is the suggested value
for a driver when installing a CRI.
In the case of Docker the kubelet will receive this value automatically.
For the other CRIs, the users should tackle this manually.
Also default the value to systemd on Debian installs for Docker.
this is not a new feature so i'm sending against master.
original text by @mauilion
cc @timothysc @kubernetes/sig-cluster-lifecycle-pr-reviews
/sig cluster-lifecycle
fixes kubernetes/kubeadm#1394

/priority important-soon

/assign @chenopis

/lgtm
nice!

/lgtm

/lgtm

fixed typo Pleas -> Please

/lgtm

@neolit123 did a quick revision for style/grammar. Please let me know if it's still technically correct.

@zparnold

did a quick revision for style/grammar. Please let me know if it's still technically correct.

thanks, the changes seem good.

/lgtm

/approve


Line indent doesn't handle leading whitespace properly

Indenting a line with leading whitespace seems to have some corners that need ironing out.  It appears to be attempting something smart about tab-stop alignment, but it should really just be dumb and indent all lines the same.
e.g.

Right-justified tabular data is affected by this (e.g. output of many Powershell commands, or the robocopy usage message)

Maybe we should have an option for "dumbing down" the indent/unindent actions. Our thinking was to optimize them for source code, where mostly I'd say "leading whitespace aligned" lines are highly desirable.

I would recommend the default behavior be "dumb" block indenting (same as Visual Studio, Notepad++, every other editor), with an option "Block indent performs auto-formatting" or the like.
Even in source code this can be destructive/annoying.  Unless every line is already aligned to a tab stop, stuff is liable to get jumbled.  Certainly I like that my editor can auto-format for me when I ask it to, but I don't want it to do so during a bulk indent action.

The indentation commands will now respect the editor.useTabStops setting. In other words, to get this behaviour, set editor.useTabStops: false.


Add Missing StatusBarStyle Values

Right now only default and light-content are supported but IOS has more.

Looks like other values are deprecated.


Documentation is for the "old experience" and should be updated for the "new and improved" eDiscovery experience.

This current documentation is over one year old, and it documents the "old experience" that is used for doing a Content Search.  The Security & Compliance Center now has a "new and improved" eDiscovery experience, and we do have documentation that goes into detail on this new experience, but it's not specific to Microsoft Teams.  Can we update this Teams specific documentation to describe the new and improved user experience?  This is the documentation for general eDiscovery with the new experience: https://docs.microsoft.com/en-us/Office365/SecurityCompliance/ediscovery-cases?redirectSourcePath=%252farticle%252f9a00b9ea-33fd-4772-8ea6-9d3c65e829e6.

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

ID: 5346de0f-d09a-86ce-8390-5a33d8139bfb
Version Independent ID: 5052193b-fd71-d1fe-e64c-f2c47218531c
Content: Use Content Search in Microsoft Teams
Content Source: Teams/content-search.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@dgorgone Thank you for pointing this out
I will get this infromation about the article needing updated over to the Teams writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Thanks for the heads up @dgorgone  . We’ve updated the article and removed the screen shots. We also moved the link to the existing Office 365 Content Search documentation to the top of the article so it’s easier to find full documentation for this feature

@dariomws306  Thank you very much for the contribution and sharing this explanation. @dgorgone
Hope this comment is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


"restDimensions" not working with Inverted Containment

When using a 'responsive' container or when just updating the dimensions of the container after initialisation, inverted contain then no longer works.
This can be seen on the demo page:
http://timmywil.github.io/jquery.panzoom/demo/#inverted-contain
If you start out at desktop , zoom in , shrink down the browser and then try to pan, you now see the white space around the inner element.
I have tried resting the zoom level on resize and also using rest dimensions to alert the parent of the change, but none of these calls seem to solve this issue.
Can anybody advise?
many thanks in advance,
Liam

You spelled it wrong a couple times, so I'll point it out: it's resetDimensions. And that works for me.


What push service is used?

According to Web Push Protocol there should be a push service.What push service is used here or am I misunderstanding something?
Thanks in advance.

Where are push messages maintained until endpoint user launches browser?

Hi, the Push service is provided by your browser. For example, for Mozilla it's autopush and for Google it's (kind of) FCM.
Encrypted push messages are stored on their servers until the user launches his browser.

Thank you for your answer I really appreciate it.


New week - Week #26

A new week has arrived. let's scan the upcoming week.

[x] Earnings
[x] Dividend
[x] Look ahead news

Past weeks Earnings
Week 25 - https://twitter.com/EarningsAhead/status/1670594890156089344
Week 24 - https://twitter.com/EarningsAhead/status/1667794732880220160
Week 23 - https://twitter.com/EarningsAhead/status/1665477434634821635
Week 22 - https://twitter.com/EarningsAhead/status/1662939045998018563

Week 26 - https://twitter.com/EarningsAhead/status/1673186940902834177


AdaptyError (code 0)

I see this error in Sentry issues while makePurchase running
adapty_flutter: ^2.2.4
device: iPhone 7
adapty_flutter.dart in Adapty._invokeMethodHandlingErrors at line 241 within adapty_flutter
Called from: <asynchronous suspension>
adapty_flutter.dart in Adapty.makePurchase at line 89 within adapty_flutter
Called from: <asynchronous suspension>
adapty_service.dart in AdaptyService.purchase at line 91 within uppy
Called from: <asynchronous suspension>
premium_viewmodel.dart in PremiumViewModel.purchaseWithAdapty at line 834 within uppy

This error means that StoreKit returned an Unknown error with code 0. How often this error occurs?


Setup uptime monitoring for A2P API

This is to catch API downtime (like we experienced on Monday) as soon as it happens.

This is set up to test auth + citation fetching. Let me know if you want to be one of the email recipients.

Currently the uptime test is run every 20 minutes. Maybe should lower that interval for prod.


sqlx can't build in workspace?

Firstly, a fresh git clone of sqlx will not compile if you try to build examples/sqlite/todos because of a missing tls feature flag? This shouldn't be needed for sqlite I would've thought, but the main issue:
After following the instructions for exporting the DATABASE_URL and running requisite sqlx commands the example will still not build with the following error:
error: error returned from database: unable to open database file
  --> examples/sqlite/todos/src/main.rs:80:16
   |
80 |       let recs = sqlx::query!(
   |  ________________^
81 | |         r#"
82 | | SELECT id, description, done
83 | | FROM todos
84 | | ORDER BY id
85 | |         "#
86 | |     )

Note that cargo build will fail both from the workspace root and from the sub-crate dir. I assume this worked at some point because otherwise the example would have never compiled?
moving todos.db and migrations to the workspace root and the example will build again.
I then attempted to use offline mode by adding the "offline" feature flag and running cargo sqlx prepare, the generated json file also needs to be moved into the root or else unable to open database file will be returned also, so compiling the sub-crate in offline mode does not work either afaict.
I have a small workspace project that can show this issue as a repro if you want, but it should be easily reproducible from the main sqlx repo

hi, in order to just run examples/sqlite/todos run:
export DATABASE_URL="sqlite:/sqlx/examples/sqlite/todos/todos.db"
sqlx db create
sqlx migrate run
previously to cargo run in /sqlx/examples/sqlite/todos directory.
i hope this helps, regards.

hi, in order to just run examples/sqlite/todos run:
export DATABASE_URL="sqlite:/sqlx/examples/sqlite/todos/todos.db"
sqlx db create
sqlx migrate run
previously to cargo run in /sqlx/examples/sqlite/todos directory.
i hope this helps, regards.

hi, in order to just execute /examples/sqlite/todos do:
export DATABASE_URL="sqlite:/YOUR ABSOLUTE PATH/sqlx/examples/sqlite/todos/todos.db"
sqlx db create
sqlx migrate run
previously to cargo run
hope this help, regards.

It turned out that my own problem described above was the result of an error on my side (see my edit above). The problem with relative paths therefore seems to really be only related to the macros.

This works for me and doesn't require any absolute path or use of CARGO_MANIFEST_DIR env var:
export DATABASE_URL=sqlite://./todos.db

Superceded by #3099


Line spacing in Hebrew text to be increased to avoid overlapping

Problem Statement
The Hebrew currently displays in such a way that the lines overlap:
Private Zenhub Image
**Proposed Solution **
the Line spacing between Hebrew text to be at 2.0em
Details
Current line spacing is 0.5em
Private Zenhub Image
If the line spacing is set to 2.0 em it should look like this:
Private Zenhub Image
DoD: Hebrew text should not overlap and have good line spacing(2.0em)

Looks good in v2.3.0 build 06539c8 QA. Line spacing for Hebrew text is 2.0.

Agree that the line-height is set to 2.0em for Hebrew text. Tested with v2.3.0 build 06539c8 QA.


Remove -o pipefail, not available everywhere...

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Schema registry compatibility check (why not ?)

Hey, it's me again.
When using avro with schema registry, we serialize the data by using an avro schema, what about the check compat using schema registry ?
In the official avro serializer KafkaAvroSerializer, it checks if the used schema is compatible to the remote current schema, and fails if there is a compat issue, depending on the schema's compat mode registered in the schema registry.
While in avro4k, it just serialize depending on the local schema whatever the schema registry.
To keep the same workflow, we have to first convert the data class instance to generic record using Avro.default.toRecord(kotlinSerializer, schema, value), and then use AbstractKafkaAvroSerializer#serializeImpl to convert the generic record to binary stuff.
 ByteArray? {
            if (value == null) {
                return null
            }
            val record = Avro.default.toRecord(kotlinSerializer, schema, value)
            // Then do the exact same as KafkaAvroSerializer
            return serializeImpl(
                getSubjectName(topic, isKey, value, avroSchema),
                record,
                avroSchema
            )
        }
    }

inline fun <reified T> KafkaAvroKotlinxSerializationDeserializer(isKey: Boolean, config: Map<String, *>): Deserializer<T> =
    object : AbstractKafkaAvroDeserializer(), Deserializer<T> {
        private val kotlinSerializer = serializer<T>()
        private val schema = Avro.default.schema(kotlinSerializer)

        init {
            this.isKey = isKey
            configure(KafkaAvroDeserializerConfig(config))
        }

        override fun deserialize(topic: String, record: ByteArray?): T? {
            if (record == null) {
                return null
            }
            // Do the exact same as KafkaAvroDeserializer
            val value = deserialize(topic, isKey, record, schema) as GenericRecord?
            return value?.let { Avro.default.fromRecord(kotlinSerializer, it) }
        }
    }

What we can do
Maybe this implementation is not perfect and can be done in another way, like putting a factory method inside Avro. But we will be happier if using this library removes boilerplate like this. By the way, it will not require any additional library, since it's using the org.apache.kafka.common.serialization package.
First "bad" idea
Normally, the KafkaAvroSerializer should serialize the instance as SpecificRecord since we use a specific schema. But to implement it, you have to get a field value using an index, and here is the tricky part.
I can make a PR, but I prefer to validate with you if it corresponds to the lib scope.

(for the moment, it's ok, but regarding performance, it should be really better to avoid the GenericRecord step by just having the schema registry call (cons: for me, too much code to copy/paste/maintain) OR having a way to easily implement SpecificRecord (maybe have a static register with a cache)

Did you have a look at https://github.com/thake/avro4k-kafka-serializer? It provides a connection between Confluent Schema-Registry and Avro4k. Have a look at the discussion in #1 for more information.

Thanks for the answer. I'm still surprised that it's not inside this repository since it's totally coherent with this library. Because it adds more dependencies to littlest libs that have not much stars, that can afraid for long term vision.
Maybe it was declined for your scope ?

not inside this repository since it's totally coherent with this library

IMO, not really. This library has no dependencies on Kafka or HTTP clients. Adding the Confluent library would add both... And exclude other implementations of the Registry API like Apicurio.

@OneCricketeer thanks for you comment. I'm with you on this. I'll close the issue as not planned.


attribute macro has no access to attributes defined before it

Hi! Thank you for syn and all your Rust work!
I ran into this issue while debugging this: https://github.com/roignpar/optfield/issues/4
It seems that attribute macros have no access to attrs defined before them.
#[before]
#[after]
struct Item;

If both before and after are attribute macros:

before will have after in Item's attrs field
after will not have before in Item's attrs field

The same happens with any kind of attribute in relation to attribute macros.
I haven't tried this with other proc macro types.
Most documentation I could find was here: https://doc.rust-lang.org/reference/procedural-macros.html#attribute-macros, but it says that "The second TokenStream is the rest of the item including other attributes on the item."
I couldn't find any mention of attribute order or anything like that.
Is this intended behavior?

This issue is not related to syn. Syn is not in control of what tokens rustc passes into an attribute macro.


Typo issue in Content Types I

There is a typo issue in Content Types I of the plone 6 training docs, the issue is in 10.4 Behaviours paragraph. Kindly find the attached screenshot, that describes the issue.

Kindly, assign me this issue, I would like to work upon this and get started with my first contribution in Plone Community.

@jsrjhbaryan959 please read and follow Contributing, First-time contributors, especially Things not to do, and Contributing to Plone.


Add Support Notice to README

Maybe this will help quench the hearts of some people, and put out our beliefs and status on piracy. @krzychu124 @VictorPhilipp I'd strongly suggest also adding this notice to the Steam Workshop pages.
Cc. @Xyrhenix @CosignCosine

After merge, I plan on skipping the waiting period due to importance of issue and urgency of need.


Create a React custom hook to make a stateful dropdown

The custom hook function should take in 3 parameters:-

label of type string
defaultState of type string
options of type string array

It should return an array with current state, the Dropdown component and the function to update state.
You can read more about custom hooks in react here - https://reactjs.org/docs/hooks-custom.html
For more help you can DM me your queries in the Uplift Project by GirlScript Foundation slack workspace.

can i take up this issue?

Hey @RheaAdh  are you still working on this issue ??

yes

ohkay great


Repo is marked as a template

This can lead to a misunderstanding of this repo purpose

Same thing on the frontend part

Well, I have to use it as a template as I cannot fork my own repos, lol :D


mendeley: 1.16.3 -> 1.17.7 - WIP

Motivation for this change
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Since 1.17 Mendeley uses Qt5. And its become massive, growing from 33 MB to 130 MB.

@ttuegel any suggestions on how to make it find the vendored Qt5 libraries?

If you want to use the vendored libraries, you should remove Qt 5 from the buildInputs and either use patchelf to set the RPATH or make a wrapper to set LD_LIBRARY_PATH.

@ttuegel okay, I will try to use the system libraries first. I'm surprised it links against so many xorg libraries.

Getting close I think
This application failed to start because it could not find or load the Qt platform plugin "xcb".

I thought wrapQtProgram would solve finding plugins, but it didn't.

Right now your expression is still mixing our Qt with the vendor's. If you want to use our Qt, you need to change the library RPATH or LD_LIBRARY_PATH to point there, or else continue to use the vendor's libraries (and point RPATH or LD_LIBRARY_PATH there).

I got a bit further using the vendored libraries. It doesn't work quite yet though
$ nix-shell -p mendeley -I nixpkgs=.  --run mendeleydesktop
xkbcommon: ERROR: failed to add default include path /usr/share/X11/xkb
Qt: Failed to create XKB context!
Use QT_XKB_CONFIG_ROOT environmental variable to provide an additional search path, add ':' as separator to provide several search paths and/or make sure that XKB configuration data directory contains recent enough contents, to update please see http://cgit.freedesktop.org/xkeyboard-config/ .
QXcbIntegration: Cannot create platform OpenGL context, neither GLX nor EGL are enabled
QSslSocket: cannot call unresolved function d2i_X509
QSslSocket: cannot call unresolved function d2i_X509
QSslSocket: cannot call unresolved function d2i_X509
QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
QSslSocket: cannot call unresolved function SSLv23_client_method
QSslSocket: cannot call unresolved function SSL_CTX_new
QSslSocket: cannot call unresolved function SSL_library_init
QSslSocket: cannot call unresolved function ERR_get_error
QSslSocket: cannot call unresolved function ERR_get_error
QSslSocket: cannot call unresolved function d2i_X509
QSslSocket: cannot call unresolved function d2i_X509
QSslSocket: cannot call unresolved function d2i_X509
Qt WebEngine Plugins directory not found. Trying fallback directory... Plugins as for example video codecs MAY NOT work.
Crash handler wrote minidump /home/freddy/.local/share/data/Mendeley Ltd./Mendeley Desktop/minidumps/03c3c4b4-51d5-5777-6e569cb4-74af8af5.dmp
/run/user/1000/nix-shell.G__yfF/rc: line 1: 14428 Segmentation fault      mendeleydesktop

@ttuegel any suggestions on how to get this working?

It looks like the vendored libraries have hardcoded paths, in particular
xkbcommon: ERROR: failed to add default include path /usr/share/X11/xkb

It looks like you will have to substitute a xkbcommon build from Nixpkgs, at least.

I see now I forgot to update the PR. I've added libxkbcommon and set a path for it as well.
$ $(nix-build -A mendeley)/bin/mendeleydesktop
QXcbIntegration: Cannot create platform OpenGL context, neither GLX nor EGL are enabled
QSslSocket: cannot call unresolved function d2i_X509
QSslSocket: cannot call unresolved function d2i_X509
QSslSocket: cannot call unresolved function d2i_X509
QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
QSslSocket: cannot call unresolved function SSLv23_client_method
QSslSocket: cannot call unresolved function SSL_CTX_new
QSslSocket: cannot call unresolved function SSL_library_init
QSslSocket: cannot call unresolved function ERR_get_error
QSslSocket: cannot call unresolved function ERR_get_error
QSslSocket: cannot call unresolved function d2i_X509
QSslSocket: cannot call unresolved function d2i_X509
QSslSocket: cannot call unresolved function d2i_X509
QLayout: Attempting to add QLayout "" to ToolBarActionWidget "", which already has a layout
QLayout: Attempting to add QLayout "" to ToolBarActionWidget "", which already has a layout
Qt WebEngine Plugins directory not found. Trying fallback directory... Plugins as for example video codecs MAY NOT work.
Crash handler wrote minidump /home/freddy/.local/share/data/Mendeley Ltd./Mendeley Desktop/minidumps/0506ac28-1a2e-2bd5-70e6449e-59e666cc.dmp
Segmentation fault

That looks to me like missing OpenSSL. If that's not it, try searching for the symbol names.

Merged https://github.com/NixOS/nixpkgs/issues/23563


STM32H7B3xx programming failure due to wrong sector size

Hello,
While programming the mcu there are several warnings and it fails finally.
pyocd load app.elf --pack Keil.STM32H7xx_DFP.3.1.1.pack --target stm32h7b3zitx
 0002763 W Page size (32768) is larger than sector size (8192) for flash region FLASH_Bank2; using sector size [flm_region_builder]

Issue is H7B0, which is used as buildin target, have different sector size etc.
Besides that 'Keil.STM32H7xx_DFP.3.1.1.pack' have also some issues.

I created a patch specifically for the STM32H3 series.
0001-Added-target-STM32H7B3xx-and-specific-target-stm32h7.patch


Underscores in newick tip names get removed

Hey, I'm using skbio to clean up some newick files since it is more flexible than some, but ran into this:
import io
import skbio
skbio.tree.TreeNode.read(io.StringIO(u"((a_2,b)c,(d,e)f)root;")).to_newick(escape_name=False)
#=> '((a 2,b)c,(d,e)f)root;'

I wasn't expecting skbio to remove the _ after the a, i.e. I was expecting
#=> '((a_2,b)c,(d,e)f)root;'

This is with 0.2.3, I attempted to install from master branch but was unable to - apologies if this is a duplicate.
Thanks,
ben

Hey @wwood, to_newick is deprecated, we recommend using write or str instead. So your command should look like this:
In [1]: import io

In [2]: import skbio

In [3]: str(skbio.tree.TreeNode.read(io.StringIO(u"((a_2,b)c,(d,e)f)root;")))
Out[3]: '((a_2,b)c,(d,e)f)root;\n'

write is called internally by str

Hey @wwood,
accordingly to the newick format, the "_" characters are interpreted as spaces, so that is another thing to take into account.

@josenavas The problem is that the _ characters are being replaced with spaces when encoded, which is exactly the opposite of what one would want to have happen.
The new reader and writer do not have this issue.

Hi, thanks for the quick response. Somehow I'd noticed from_newick was deprecated, but missed that to_newick was too.
However, my larger problem is not solved, because I specifically do not want single quotes around the names as FastTree doesn't recognise the quotes as far as I can tell. (hence escape_names=False). Is there some way to do this with str or otherwise? e.g. replicate
t = skbio.tree.TreeNode.read(io.StringIO(u"(('a_2','b')'c',('d','e')'f')root;"))

In [11]: t.to_newick(escape_name=False)
Out[11]: '((a_2,b)c,(d,e)f)root;'

whereas with str
In [10]: str(t)
Out[10]: "(('a_2',b)c,(d,e)f)root;\n"

@wwood I see what is happening. Your name starts out as 'a_2' because it is already escaped (with the quotes) it will treated as a literal string resulting in a node which has a_2 as its name. When that is written again, a_2 will be escaped as the node's name has an _ (it isn't converting with a simple replace it because it isn't a space). According to what little standards seem to exist, this behaviour is the most correct, and loses the least amount of information.
To solve your problem specifically, you can just iterate over each node and replace it's name like so:
In [1]: import io; import skbio

In [2]: t = skbio.tree.TreeNode.read(io.StringIO(u"(('a_2','b')'c',('d','e')'f')root;"))

In [3]: for n in t.traverse():
    n.name = n.name.replace('_', ' ')

In [4]: str(t)
Out[4]: '((a_2,b)c,(d,e)f)root;\n'

Essentially it will only quote something if one of the characters in the name is in set(",:_;()[]"). So if you have any of those you might want to replace them as well. Otherwise it will always choose the simplest escaping possible.

OK, makes sense - I agree that is the most correct behaviour.


Updated to use new method of spawning multiple sagas

Fixes the redux-saga depreciation warning mentioned in #11

That's true, I overlooked that. Do you want to update the peer dependency for redux-saga? It looks like all was added in 15.0, which blocks out a lot of versions currently supported by redux-form-submit-saga.

I've made a fork of your fork. In there I checked if alleffect is undefined before using it. And in the case it is not I use the old notation. I also updated the package.json to support redux-saga 0.15.x as well.
I've sent a PR to your fork so you can update this PR.

Thanks!

@colinbate can you merge this?

I'll merge this once my comments above are addressed. Sorry for my absence over the past couple months.

@colinbate, I guess it's fixed now!

Thanks, I'll look to merging it now.

Thanks @hugobessa for taking it all the way

Published at 1.2.1
Let me know if you have any issues with it.


base-images.md broken links

There are broken links that exist on https://github.com/balena-io/docs/blob/master/pages/reference/base-images/base-images.md
Check under the #image-tags section

Thanks for reporting it in @cgsmith We are currently working on automating these pages of docs are pulling in accurate information directly from our base images build pipelines and populating the entire base images docs using that. Till that happens in the future, I have updated the 404 links with better locations that folks can visit in the future and they ideally won't 404


After upgrade of local cluster to 1.22, UI is not reachable though all pods are healthy

Rancher Server Setup

Rancher version: 2.6-head Commit ID b97e3c2
Installation option (Docker install/Helm Chart): HA Deploy Jenkins Job

Information about the Cluster

Kubernetes version: 1.22
Cluster Type (Local/Downstream): Local cluster upgraded from 1.21 to 1.22

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information
Admin
Describe the bug
Created an HA rancher on 2.6-head with jenkins job. After upgrading the local cluster to k8s 1.22, the UI is unreachable. The cluster is up, all pods are good. Can deploy to cluster successfully from CLI.
To Reproduce

Create Rancher HA 2.6-head k8s v1.21.9-rancher1-1
Created downstream Ec2 clusters
Installed v1 and v2 charts for testing and k8s upgrade verification
Upgraded local cluster k8s version to 1.22

Result
Upgraded successfully. Able to login to cluster after upgrade and verify that the k8s version was upgraded. After ~ 30 minutes, there was a slack message that the UI was unreachable. This morning verified same. Able to verify that the cluster is actually up and running, able to deploy workload from CLI.
Expected Result
UI is reachable and able to login to rancher.
Screenshots

Additional context
Since the cluster was up and returning 200s from command line, debugged by replacing infrastructure (target groups, lb and route53) no improvement.

And looking at the rancher pod logs there are just a nearly endless amount of syncing errors like below (snippet)
2022/02/10 16:08:02 [ERROR] error syncing 'c-jsbwx/p-prf2v': handler system-image-upgrade-controller: upgrade cluster c-jsbwx system service pipeline failed: upgrade system service cattle-pipeline:registry-proxy failed, Put "https://REDACTED-IP:6443/apis/apps/v1/namespaces/cattle-pipeline/daemonsets/registry-proxy": cluster agent disconnected, requeuing
2022/02/10 16:08:06 [ERROR] error syncing 'c-jsbwx/cluster-scan-alert': handler cluster-alert-group-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing
2022/02/10 16:08:11 [ERROR] error syncing 'c-jsbwx/scheduler-system-service': handler cluster-alert-rule-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing
2022/02/10 16:08:11 [ERROR] error syncing 'p-prf2v/projectalert-workload-alert': handler project-alert-group-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing
2022/02/10 16:08:12 [ERROR] error syncing 'c-jsbwx/cluster-scan-manual-failure-only': handler cluster-alert-rule-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing
2022/02/10 16:08:22 [ERROR] error syncing 'c-jsbwx/deployment-event-alert': handler cluster-alert-rule-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing
2022/02/10 16:08:37 [ERROR] error syncing 'c-jsbwx/high-number-of-leader-changes': handler cluster-alert-rule-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing
2022/02/10 16:08:38 [ERROR] error syncing 'p-prf2v/memory-close-to-resource-limited': handler project-alert-rule-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing
2022/02/10 16:08:41 [ERROR] error syncing 'p-prf2v/less-than-half-workload-available': handler project-alert-rule-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing
W0210 16:08:45.483964      32 warnings.go:80] policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+
2022/02/10 16:08:46 [ERROR] error syncing 'c-jsbwx/etcd-system-service': handler cluster-alert-rule-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing
2022/02/10 16:08:49 [ERROR] error syncing 'c-jsbwx/cluster-scan-scheduled--failure-only': handler cluster-alert-rule-deployer: clean up alertmanager failed, Delete "https://REDACTED-IP:6443/api/v1/namespaces/cattle-prometheus/secrets/alertmanager-cluster-alerting": cluster agent disconnected, requeuing

metoo

BTW, https://github.com/rancher/rke/releases lists "v1.22.6-rancher1-1" under "experimental". I didn't ask for the experimental version. Using it by default for a release version is hardly acceptable.

@ronhorton how much ram was on these nodes.  When you say you installed a v1 and v2 chart was it monitoring?

The upgrade succeeded, but takes 3 minutes to load local cluster when clicking explore
v2.6-head Commit ID 3f30288

Can be tested with 2.6.4-rc9

Pass verified in v2.6.4-rc10 with rke 1.3.8-rc8

created ha rancher with k8s 1.21
installed downstream cluster
installed apps on downstream
upgraded rancher to 1.22

pods restarted; k8s version upgraded successfully. ui functions as expected. did not lose connectivity between UI and backend


Request History on partner should show # of items instead of request id

Summary
The partners' Request History is currently showing the request id as the second column.   It should show the total # of items instead.
Why change
Forestall partner confusion.  The number doesn't mean anything to them, or the banks (the banks never see it).  We don't display it anywhere else (that @cielf is aware of)
Details
Replace the display of the request id with the total # of items
Recreation
(on local) Sign in as<EMAIL_ADDRESS> Choose Essentials Requests from the left hand menu

Criteria for completion

[ ] Request History is showing total number of items instead of request id
[ ] automated tests to demonstrate this

Happy to take a swing at this if you still need help.

You're the first one to come to bat for it, so do please take a swing.


feat: add link to highlight/filter my adopted trees on the map

add link to highlight/filter my adopted trees on the map on in the profile -> adopted trees section

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1207132261291428

I can have a look after staging is merged onto this branch @raphael-arce  :)

@Jaszkowic my bad, now it is ready to review 👍


Link to build-states

Hoping to disambiguate scheduled jobs vs scheduled_builds, I get them mixed in my mind.

@pbuckley updated some of the wording, let me know what you think!


Client Interface: to enable client mocking

Description
It would be nice to have an interface for the Client to be able to mock it for testing purposes where the client is in use within a larger context.

+1

https://github.com/ogen-go/ogen/pull/1038


Update the plugin's directory structure

Minor updates to the addon directory structure to match the latest updates to the documentation in https://github.com/godotengine/godot-docs/pull/7884

I'm a little confused why we moved from export to .export and are now moving back to export as a folder. I personally prefer it without the dot prefix so I'm happy with this change

Originally this was to prevent including the exported binaries in the generated apk twice, but I was able to change the Android export logic, so this is no longer needed.


gateway perf: restrict cds clusters to only those required by the gateway

@istio/wg-networking-maintainers-pilot
We have avoided this problem till date but fixed it for sidecars. Sidecars decide on the services to generate a cluster for, using the sidecar CRD. But for gateways, we have no such mechanism. In small-average clusters, this is not a problem. But in very large clusters (one of our customers has about 1000+ services!!), this quickly becomes a big problem in terms of memory usage at the gateway, as well as making it hard to debug things.
In the past (before 0.8), we used to return the list of clusters in almost every function (route generation, listener generation) and collect all these clusters and fix them up. This technique turned out to be very error prone and hard to maintain. So we should avoid going down this path again.
One option is to create some transient xds context in the Node struct, collect the set of all cluster names and perhaps the services [no subsets], ports associated with the cluster as well. And then in the CDS code for gateway, we could look at all the entries with a valid service+port and generate clusters only for those. This seems to be the least intrusive option that I can think of. This would require updating the context in httproute/route, tls.go, networkfilter.go, etc. We also need to be very careful in future to update this context in all places where a cluster is referred to. Otherwise we end up getting 503s or connection issues due to missing clusters.
Thoughts? We should be able to get this into 1.4 pretty easily

A more detailed form of https://github.com/istio/istio/issues/11936

Another option: infer the needed clusters from gateway and virtualservice. But donot forget to handle multiclusters gateway.

Didn't @mandarjog do this at some point but the PR was dropped?
On Thu, Oct 10, 2019 at 8:18 PM Zhonghu Xu<EMAIL_ADDRESS>wrote:

Another option: infer the needed clusters from gateway and virtualservice.
But donot forget to handle multiclusters gateway.
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/17753?email_source=notifications&email_token=AAEYGXPKK6ELT2SDVD476VLQN7WBXA5CNFSM4I7M5YF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA6TSAY#issuecomment-540883203,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXONFB36KYN3NQT2YVTQN7WBXANCNFSM4I7M5YFQ
.

There were 3 sides

Use gw and vs to figure out the clusters that are needed.
Ability to opt out for mcg
Process EnvoyFilter api for its dependencies.
A way to keep required clusters like mixer etc.

I had implemented and tested 1,2,4 .
At that time i needed equivalent to sidecar’s egress field on gateways.

@mandarjog Can you link the old pr?

A way to keep required clusters like mixer etc.
Isn't mixer cluster added via bootstrap? pilot, als, metrics, tracing clusters are added via bootstrap. So they will always be there. If this is the case, then users (without multicluster) can trim the clusters at the Gateway by setting the SNI-DNAT to false by default.

With that, we are now down to reducing the normal clusters at the gateway - one short cut is to look at the host spec in the gateway. If the hostspec is of the form namespace/host, then we can import that namespace. IOW, if we tell users to only use the namespace/host format, then we will get a list of namespaces to import and we could do some trimming. This will still be not perfect compared to the logic I originally outlined but its better than nothing.


update interface for query api

Update IDL for query API. Implementation will be in a separate CR.

Coverage decreased (-0.2%) to 67.027% when pulling 3a0f64fb2e5a45634acf68e9b4d4cb9861c5e80c on yiminc:query_idl into 19c948a6d5692f46d76968371adf624a353f3d60 on uber:master.

Changes Unknown when pulling 06b31d951a2d5ad8bb56bf46317375fdb65e8aa3 on yiminc:query_idl into ** on uber:master**.


rangeslider__fill / 2 differents color for 2 differents slider

Hello,
First, Thanks for you job @andreruffert, it's great to use.
I hope I don't ask for something easy, anyway I don't get trought it :+1:
I have multiple rangeSlider in a web page and I like to display 2 differents colors for the low and high value of the result of each slide.
For exemple if 34%, dark blue.
if 87%, light blue.
I've tried to duplicate a rangeslider2 but the rangeslider appears twice.
Finally, I don't find the way to select the ID or class of my input to make something like :+1:
 <input type="range" class="rangeslider2" value="35" step="1" min="1" max="100">
.rangeslider__fill .rangeslider2 { 	background: #EC0E11; 	position: absolute; }
Thanks a lot for help, if like !
Alfred

wrap each inputs with 2 different container class (known as contextual style). e.g:

  <input type="range">

  <input type="range">

.style-input-red .rangeslider {
 background: red;
}

.style-input-blue .rangeslider {
 background: blue;
}


Incorrect Downloadable Image Versions for Orange Pi Zero Plus 2 H3

On both servers there are images with RetroOrangePi 4.0 - not 4.1 as mentioned in download section!
P.S. Not sure where to post it here or on the forum (but I can't register there).

It's 4.1 indeed. EmulationStation menu will mention 4.0 (sorry about that) because the binary was only updated afterwards.
4.0 version only exists by manual update from 3.0.1. It has three partitions, while 4.1 has only two.

Ok, thank you for explanation. But, nevertheless, it would be good if somewhere on the site there were a notice that the user must update sytem first befor trying to instal in on eMMC for example.

eMMC install was fixed after v4.1 release, we are working on v4.2 update, so expect more info to go public soon, thanks


MFA enhancement

consider transitioning to NewportPython
display STATUS on front panel
recover method when coms lost

Closing as hardware is transitioning to yaq (newport already there, though not used within pycmds at this point) Also, cannot find NewportPython...


Strange behaviour on signin/signup modal

Describe the bug
On localhost, everything seems fine.
On QA, signin/signup modal has a double padding
To Reproduce
Steps to reproduce the behavior:

Go to qa
Click on "Create my first assistant"
Padding is incorrect (2*24 instead of 24px)

Expected behavior
The modal should be the same on localhost and qa
Screenshots
On QA :

On my localhost

Additionnal context
On QA, modal section is duplicated, but not on localhost

Seems to be fixed on latest QA version. Closing


Added 'mesos_configs' list to be able to set any mesos config

Can be used for example set log level:
mesos_config:
  - name: LOG_LEVEL
    value: WARNING

Also removed mesos_options because that configuration wasn't used in anywhere in the cookbook.

Will review late tonight estd.
-Jason

On Aug 21, 2015, at 3:07 AM, Erno Aapa<EMAIL_ADDRESS>wrote:
Can be used for example set log level:
mesos_config:

name: LOG_LEVEL
value: WARNING
You can view, comment on, or merge this pull request online at:

https://github.com/AnsibleShipyard/ansible-mesos/pull/28
Commit Summary
Added 'mesos_configs' list to be able to set any mesos config
File Changes
M defaults/main.yml (6)
M templates/conf-mesos.j2 (5)
Patch Links:
https://github.com/AnsibleShipyard/ansible-mesos/pull/28.patch
https://github.com/AnsibleShipyard/ansible-mesos/pull/28.diff
—
Reply to this email directly or view it on GitHub.

This is nice, makes things usable.

I'm a +1, @ninthnails you?

@JasonGiedymin Yeah, I'm good with the latest change. Let's go!


Mapping a response to Void

I was thinking perhaps there may be some benefit to mapping a promise result to Void.
For example, I do not care about the result of a task, only that it has completed.
I use the VIPER pattern quite heavily and a lot of the time I simply want my components to know another task has finished, without needing to give it context of models, or responses etc.
This could be a fairly simple addition -
extension Promise {
    func asVoid() -> Promise<Void> {
        return .resolve(())
    }
}

This would allow me to have chains within the call that had context of the response and made use of it's model, but the original trigger could simply move along to the next promise in it's async block.

@nodediggity This is a very good point 👌 . I actually have this extension in my own project.
Can you explain why you closed the issue ?
extension Promise {
    public func toVoid() -> Promise<Void> {
        return self.then { _ -> Void in }
    }
}


Policy - Official vs. Community - Assets

We will need to figure out a policy for the cooperative community channels. meaning how they can represent themselves and use the RChain assets.
Is this an agenda item for governance committee?

@drbloom just linked this too me, important.
.coop terms

There was a trademark session at the Governance Forum; I'd appreciate it if somebody would cite the records from this issue, highlighting any follow-up items.


Fix copy zeroing error

Fixes https://github.com/EnzymeAD/Enzyme.jl/issues/584

Codecov Report
Base: 73.61% // Head: 70.44% // Decreases project coverage by -3.17% :warning:

Coverage data is based on head (141abc1) compared to base (5fce502).
Patch coverage: 66.66% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #600      +/-   ##
==========================================
- Coverage   73.61%   70.44%   -3.17%
==========================================
  Files          17       17
  Lines        6116     5793     -323
==========================================
- Hits         4502     4081     -421
- Misses       1614     1712      +98

Impacted Files
Coverage Δ

src/compiler.jl
72.35% <66.66%> (-1.13%)
:arrow_down:

src/compiler/orcv2.jl
0.00% <0.00%> (-61.91%)
:arrow_down:

src/compiler/interpreter.jl
24.52% <0.00%> (-60.93%)
:arrow_down:

src/compiler/utils.jl
58.57% <0.00%> (-37.32%)
:arrow_down:

src/logic.jl
75.00% <0.00%> (-5.00%)
:arrow_down:

src/compiler/validation.jl
59.59% <0.00%> (-2.48%)
:arrow_down:

src/compiler/pmap.jl
75.27% <0.00%> (-2.27%)
:arrow_down:

src/api.jl
60.00% <0.00%> (-1.70%)
:arrow_down:

src/pmap.jl
83.33% <0.00%> (-1.29%)
:arrow_down:

... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


merge package template from Kronos-Tools/npm-package-template

Updated standard to latest version

Current coverage is 83.61% (diff: 100%)

Merging #76 into master will not change coverage

@@             master        #76   diff @@
==========================================
  Files             1          1
  Lines           177        177
  Methods          14         14
  Messages          0          0
  Branches         41         41
==========================================
  Hits            148        148
  Misses           29         29
  Partials          0          0

Powered by Codecov. Last update 147c989...bd11d85


r-click menu ideally shows only action available for this particular tree entry type

[ ] updating all the modules action from the r-click on a single module makes little sense
[ ] open in browser action on assignment weeks/rounds and exercises does nothing (no notification either)

We decided that refreshing all the modules in the menu makes sense.
The open action now does something for all items


Issue VXQUERY-225-fn:serialize

Just a few quick thoughts/questions:

The escaping of the XML entities in the tests looks ugly, but it might be the right thing to do - haven't thought carefully about this.
Where do those entities get introduced? During the evaluation of fn:serialize or in the serialization of the final output?
If it's the serialization of the final output, do we have a way to avoid that and get a "readable" output for some queries (e.g. by wrapping the strings in some container)?
Do the XQTS tests for fn:serialize pass?

It seems like escaping XML entities is nice for some scenarios, but I am not sure if its right for all.
It seems the XML entities are added when serialization of the final output. (At least that is my guess from looking at code. Christina can you confirm with your new trace function?)
Nothing currently exists.
The fn:serialize function is part of the XQuery 3.0 specification. Currently we are using XQuery 1.0 Test Suite.

I have checked with the trace function and these entities are indeed added in the final output.

While there seems to be some questions on how to print a serialized value, I think this change is a good step in the right direction. Should we merge it as is?


Feat: Add path type filter

Update README
The TOC of README was generated by idea.

this fails web build with
/usr/lib/flutter/pub-cache/hosted/pub.dev/photo_manager-2.7.0/lib/src/filter/path_filter.dart:243:16:
Error: The integer literal<PHONE_NUMBER>854775807 can't be represented exactly in JavaScript.
        return<PHONE_NUMBER>854775807;
               ^^^^^^^^^^^^^^^^^^^


Response returning zero status code

Issue Summary
Attempting to send mail has all of a sudden started returning zero response codes.
Steps to Reproduce
<?php

// get sendgrid
$sendgrid = new \SendGrid('MY_KEY');

// create sender and recipient
$from = new SendGrid\Email('App name', 'no-reply@app-name.com');
$to = new SendGrid\Email('Users name', 'user@user.com');

// create subject and content
$subject = 'Email title';
$content = new SendGrid\Content('text/plain', 'Test content');

// create email
$mail = new SendGrid\Mail($from, $subject, $to, $content);

// send email
$result = $sendgrid->client->mail()->send()->post($mail);

Technical details:
SendGrid\Response Object
(
    [statusCode:protected] => 0
    [body:protected] =>
    [headers:protected] => Array
        (
            [0] =>
        )
)

sendgrid-php Version: master (latest commit: aa09fb89e2be2624de9db4dde914d27cbdece95c)
PHP Version: 7.0.11

Hello @garyrutland,
Unfortunately, I am unable to reproduce. Here is what I used:
<?php
// If you are using Composer
require 'vendor/autoload.php';

// get sendgrid
$sendgrid = new \SendGrid(getenv('SENDGRID_API_KEY'));

// create sender and recipient
$from = new SendGrid\Email('App name', 'dx@sendgrid.com');
$to = new SendGrid\Email('Users name', 'elmer@sendgrid.com');

// create subject and content
$subject = 'Email title';
$content = new SendGrid\Content('text/plain', 'Test content');

// create email
$mail = new SendGrid\Mail($from, $subject, $to, $content);

// send email
$result = $sendgrid->client->mail()->send()->post($mail);

Which returned
SendGrid\Response Object
(
    [statusCode:protected] => 202
    [body:protected] =>
    [headers:protected] => Array
        (
            [0] => 1
        )
)

Does you email get delivered?

I just came across this issue as well on a freshly installed Windows development machine, and for me it turned out to be the cURL PHP module missing CA certificates and thus refusing to connect to the SendGrid API. The issue can be resolved by downloading the required cacerts.pem file and configuring it, see here.
It would probably be a good idea to add some error detection to sendgrid/php-http-client here so that cURL errors may be catched and an error may be thrown instead of failing silently.

Sorry, for the late response but haven't had a chance to carry one working on this.
I have just tried again with the same code as before and am now getting 202 responses and emails are being sent.

I found that zero may return with curl error
SSL certificate problem: unable to get local issuer certificate
so need setup certificate for php to use this service , no option to disable

@DvdGiessen it works like a charm! thanks!

I was facing the same problem and spent a couple of hours figuring out. I set the CURLOPT_SSL_VERIFYPEER  in Client.php of sendGrid API.
I have set CURLOPT_SSL_VERIFYPEER => to false instead of true

I was facing the same problem and spent a couple of hours figuring out. I set the CURLOPT_SSL_VERIFYPEER in Client.php of sendGrid API.
I have set CURLOPT_SSL_VERIFYPEER => to false instead of true

Works for me!! thanks

Since I'm still subscibed to this issue, I noticed the replies by @shahrpt and @apineda7 about how setting CURLOPT_SSL_VERIFYPEER to false also makes the error message go away. THIS IS NOT RECOMMENDED!

Instead, as mentioned in my previous comment, fix the actual problem and you won't have to disable your security. See https://snippets.webaware.com.au/howto/stop-turning-off-curlopt_ssl_verifypeer-and-fix-your-php-config/.
@thinkingserious Any chance we could prioritize getting https://github.com/sendgrid/php-http-client/pull/99 merged which would clear up the error message and next perhaps add a line about this specific issue in the documentation to prevent users from potentially shooting themselves in the foot by disabling CURLOPT_SSL_VERIFYPEER?

@DvdGiessen thanks, setting up the curl.cainfo option inside php.ini fixed the problem for me :+1:

@DvdGiessen I got curious. It would be good to know why this happens just in windows machines (if that's the case, as it seems to be).
As far as I know, the Client.php and the php.ini are the same (about curl CA stuff).


ONNX export

Thank you very much for your work, it is very important. I am trying to use your model but I would need to export it to onnx, would it be possible?

Thanks for your kind words. I assume you may encouter some issues because we use external libs for fps.


Feature[alert]: add interruption-level support in the aps payload

Based on apple docs
https://developer.apple.com/documentation/usernotifications/unmutablenotificationcontent/interruptionlevel

GOAL:
Adding interruption-level key nested in aps
Proposal Behavior:

Default value for interruption_level is None to not impact in current alert behavior.
There is a validation in PayloadAlert class to restrict interruption_level  argument only the values defined in Apple docs using by typping.Literal and the setter decorator for the mentioned argument.

Extra NOTES:
This  effort  is required to support other libraries such as django push notifications issue:->
https://github.com/jazzband/django-push-notifications/issues/708

Open issue related  -> https://github.com/Pr0Ger/PyAPNs2/issues/158


Removed for loops in StartImportingBasicBlock()

This was suggested by @jkotas in https://github.com/dotnet/corert/pull/3513.
Since we store the handle indexes on the basic block we can take advantage of them later in the StartImportingBasicBlock method and we can get rid of the for loops.
Changes:
-We also store the FilterIndex on the BasicBlock
-StartImportingBasicBlock method: No more for loop, we use the index values directly

Btw. if this change goes through we can also verify Endfilter easily.


Implement trades DB processor

While working on this please make sure to use BatchInsertBuilder. Without it processing will be too slow for large ledger (we experience this in the old ingestion).

implemented in https://github.com/stellar/go/pull/2100


is this dead ?

if yes then i'll make a kotlin fork because scene2d makes me want to kashoot myself

Pretty much dead, yeah :( Feel free to cannibalize it however you want!


Add marshaler for the structs with readonly properties

Fixes #438

Hi @jhendrixMSFT the CI has passed, please have a look, thanks!

And @jhendrixMSFT regarding the big files issue, I am wondering would this patch create more gigantic models.go files? Is it possible to split something into different files?

It will increase the size yes.
If you want, you could split all the stuff for enums into its own file.  We did this in track 2 for just this reason.

🤖 AutoRest automatic publish job 🤖
success (version: 2.1.149)


Does the docker image have a java environment?

Do I need to configure java in docker environment to run java microservice?
docker refers to tarscloud / frameword 2.4.0

You can use the docker of tarsnode, the framework document writes only recommends deploying framework services and not deploying business services.

你可以使用tarsnode的docker，framework文档写了只建议部署框架服务 不要部署业务服务。

If you want to deploy Java microservices, because Java depends on the environment, so this can't run, is there any solution?

如果要部署Java微服，因为Java就依赖环境，所以这个跑不起来，有什么处理办法吗？

The node has docker in java environment, just use that.

节点有java环境的docker，使用那个就可以了。


Bug..? app doesnt launch as its supposed to

Describe the bug
Alrighty so idek whats wrong but i followed the basic guide for installing and everything goes through except the app itself doesnt run for sum reason i tried 2 of the windows setup thingys both the msi and setup files but nothing works i did also try running as administrator but even that didnt work..
To Reproduce
Steps to reproduce the behavior:

Download as usual
install
Run app
nothing happens :/

Expected behavior
the app to work as normal lol wit the gui and everything and to jam ofc
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS:Windows
Browser: FireFox i do have chrome as well tho
Version i believe this is windows version? i have a custom version of windows but heres the build version 22h2 19045.3803

Additional context
Add any other context about the problem here.

Hi @Lolguy8321, thank you for submitting an issue.
My first assumption for something like this is that you may not have microsoft edge browser installed. The browser should come pre-installed with windows builds, but since you said you have a custom build, I am assuming that this may be the case. The app depends on this in order to serve you the UI: requirements
Here is a download link in case you don't have it: edge
However if that is not the issue, then I would like to ask for some more details:

Which version of the app did you download and attempt to run. You can normally see the version of the app as part of the name of the file you downloaded, eg muzik-offline_0.x.0_x64-setup.exe?
Which version of Microsoft Edge do you have. You can find this information in the Settings -> About Microsoft Edge of the browser?
Could you launch the app from the terminal and report any errors thrown:

Open the Windows terminal.
In the terminal, navigate to where you stored the app when you download, eg C:\Users\User\AppData\Local\muzik-offline
Once in the correct directory, you can run the app by typing muzik-offline.exe and pressing Enter.
If any errors occur during the launch process, they will be displayed in the terminal window. Copy whatever is printed out and paste it here as a comment.


Does pebble have a function to make at least one input to be filled , even if all the attributs are optional

i need to create a transformation request with peeble , and i have to send all my attributs as optional , but at least one of them have to filled in the request.
the first code is like :
{% if personne is not empty %}
{% if personne.nom is not empty %}
  <nom>{{ personne.nom }}</nom>
{% endif %}

{% if personne.prenom is not empty %}
   <prenom>{{ personne.prenom }}</prenom>
{% endif %}

{% endif %}
In my real code , i have at least a 100 attributs.

you can define a macro for that


pyautogui.hotkey('ctrl','c') - KeyboardInterrupt

Using pyautogui.hotkey('ctrl','c') to copy marked text trickers 'KeyboardInterupt'.
Which I guess is expected behavior, but not the intended copy marked text behavior?
Instead, at least for windows, pyautogui.hotkey('ctrl','insert') can be used to copy marked text.

Sorry for the delay in replying.
I can't repro this on my machine. I suspect you have the interactive shell window focused when the function call happens, and that's causing the KeyboardInterrupt.


How to use Create-React-App + TypeScript + Ant-Design without Eject

react-app-rewired + awesome-typescript-loader = babel-plugin-import
boilerplate project
happy coding!

I'll close this since people can find this if they search, but we're not advertising use of react-app-rewired.
Sorry!

(create react app + typescript + antd) without eject and without using Babel.
https://github.com/qinyang1980/create-react-antd-app.git


"Show bags" still showing items in show-bags layout even after it is toggled off

Addon Version
v0.1.11
World of Warcraft Version
Retail
Describe the bug
betterbags toggling of "show bags" does not always work as expected.

Wow retail on macbook pro.  installed the addon for first time today.
Steps to reproduce

Open bank full of items with betterbags enabled.
click on 'show bags' and the bags become viewable on upper left of window.  Items also show up separated by bags as expected.
Click "show bags" again to toggle it off and the bags on upper left disappear.  But all items still show up separated by bags.  It does not return to previous, non-show-bags layout.    It is still stuck this way after closing all bags and clicking on banker again.

I got it to return to normal by playing around and opening and closing windows, clicking the toggle on and off, etc.   Then it returned to normal.      But repeating the above steps broke it again.

i also have a buttload of reagents that won't show up in this view in their respective bags...  but I'm assuming you already have a ticket for that somewhere

Confirmed, will push .12 out today with a fix.

.12 pushed to all addon sites.

Correction, out with .13

still broken.    I installed bugsack hoping to catch an error but it's not getting any.

I can't reproduce this anymore -- what part is broken? Bags continue to stay broken out?

yes..  banker bags stay broken out.  bugsack has errors now too.  trying to figure out how to share these.

please do -- I'll fix them right away. you're on .13?

Also you can just copy/paste the errors in here

yes i'm no .13.    trying something first.

Ok that was strange.   I deleted betterbags and it's 2 dozen plugins, made sure it was gone from Interface/AddOns, then reinstalled it WITHOUT all the addons.   Now I get no bugsack errors but the banker bag started out broken out from the get go.    I never restarted wow, just did reload from the cli so maybe something was loaded into memory .    gonna try restarting.

that's actually a known issues, toggle the bag show button and it will fix itself. deleting an addon doesn't delete the data for it, that's in a different folder in the wow folder. it looks like it might have been a plugin that was broken -- you may want to reach out to that author if you figure out which one.

figures.   I logged in with a character i never used betterbags with and it still opens the bank bags broken out.       so you're saying i should toggle 'show bags' ON and then do what, log out?  resrtart the game?

turn it on then off, and it should go back to normal bags -- i have a patch for bags being broken out on boot w/o the window showing soon.

tried that.  hasn't worked for me.   where is data for it stored?  perhaps deleting that will fix it?

let me put out .14 with the proposed fix -- i would rather try to get you out of this broken state the "right" way in case this affects other people. give me like 2 min.

Okay, just pushed .14. If for whatever reason this doesn't work, _retail_\WTF\Account\<your account name>\SavedVariables\BetterBags.lua has the settings file you can delete.

cool.  the update did not fix it, but deleting the shared files for betterbags did.  thanks!
My other issue is that my reagents won't show up in 'show bags' view.  and breaking bags in the "reagent bank" doesn't do much.  it still shows everything in one big bucket.    I looked around and I think i see a ticket for that?  altho it states that depositing reagents doesn't work.  it works, just can't see it.

so this is by design -- this is because the way the Blizzard API works for banks is weird. you have to switch "tabs" under the covers, and that is what decides where right clicking an item in your backpack goes. I can't hijack either and make it magically deposit correctly, so I decided to make it explicit. there's no fix for that currently, and I expect it to get even more confusing with warband bags, unless blizzard redoes the API.

Ah I didn't know that.  maybe there's a way to hide reagent bags and show then in the "reagent bank" side but broken up.
at any rate.  thanks for the work you put into this addon.   as it is, I still have to switch between this one and bagnon.


Scripts with systemcommand does not work properly

Note: This pull request is not yet ready for merge.
Each time a systemcommand in a script is processed, a new QProcess is allocated. This allocated object is never deleted. Except from leaking memory this does also have the implication that, after a while, the "Max open files" limit is reached on Linux systems. Each executed process seems to open 5 files (pipes) used for communication between Qt and the executed task. The default limit in Ubuntu 20.04 seems to be 1024 open files (Could be check with cat /proc/`pidof qlcplus`/limits and changed with prlimit -n4096 -p `pidof qlcplus` ). This will result in about 200 systemcommand executions before it will start to fail.
This bug (and the fix) can be verified by checking the number of open file descriptors for the QLC+ process:
ls -1 /proc/`pidof qlcplus`/fd | wc -l
This bug could be patched in three different ways
1) The blocking way
This is what I have done for now. This will block the QLC+ engine until the script is finnished. This will also freeze the UI for up to two seconds. This is nothing more than a proof of concept.
QProcess *newProcess = new QProcess();
newProcess->start(programName, programArgs);
newProcess->waitForFinished();
delete newProcess;

2) The non blocking way - Qt event loop
In theory it should be possible to let the Qt event loop take care of destroying the QProcess as soon as it is finnished. However, this does not work for some reason. I would guess that the Qt event loop is not inherited correctly because somewhere a QObject is created without providing a parent. I have not yet been able to track down if this is the case.
I would assume that this is the prefered way of fixing this bug.
QProcess *newProcess = new QProcess(this);
newProcess->start(programName, programArgs);
newProcess->deleteLater();

3) The non blocking way - Workaround
If number 2 is not feasible, then another solution would be to make the engine to the cleanup, or make an object that destroys itself.
More things to consider

scriptrunner.cpp needs to be updated accordingly
Maybe it is a good idea to be able to choose if a systemcommand should be blocking or not?
The process is executed with QProcess::start(), maybe it is better to use QProcess::startDetached() ?

Another quick and dirty way to fix the "Max open files" problem is to use QProcess::startDetached() instead of QProcess::start(). This will solve the problem with the stuck pipes, but the memory leak will still be there.

Quick hint about draft-PRs when they're not yet ready to be merged. 😉

Quick hint about draft-PRs when they're not yet ready to be merged. wink

Didn't know about that feature! I have converted it to a draft now. Thanks!

Fixed by #1358


add spinner for page loads in web console

stats daily summary
stats summary
uid load
log load

can we just automatically add it to every page on the web side? need to avoid adding it to the api router.

see "spinner" branch for early attempt -- we can get a spinner to load, but it's still there when the user uses the "back" button to go to a previous page. Tried and failed to find a window event (popstate) that would fire when they go back using the back button, but it didn't work.
next thing to try: if we rebuild the console in react+express, would that work better?
Other things we looked at:

this would be easier to manage if the ejs page were to do the content loading on the client side instead of express doing it on server side, but that would mean exposing our API credentials in the client side and that's no good.
could try making the spinner go away after some number of seconds but the page load times vary too much to make that a good option

@Meng-V found the answer: listen for the 'pageshow' event as described here:
https://stackoverflow.com/questions/43043113/how-to-force-reloading-a-page-when-using-browser-back-button

closed by #120


Cannot import backend - Backend file was unable to load due to user configuration error

I'm using mySQL as my backend and trying to import my file backend into mySQL but getting this error everytime I try /pex import file http://pastebin.com/6T1DbwjW

Not much I can do to help you because you're using a broken server panel that strips out log lines with important details. Make sure you permissions file is valid yaml, and you may find more info in your full server log file.


Intent to package libzmq

I'd like to provide https://github.com/zeromq/libzmq and https://github.com/zeromq/pyzmq
As always, could you create the respective repos (maybe named libzmq and python-zmq)?
edit: just saw that we have some dedicated python/python3 naming scheme. I slightly prefer the simple name without 3 but would package for python3 only...

Sure, I will add repos.

Done! Good luck!
Closing over here

@rinigus please create repos for python-packaging and python-pyparsing as well

Done