Reorder warm migration flow for vSphere VMs

will update with description shortly

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 67 lines in your changes missing coverage. Please review.

Project coverage is 15.66%. Comparing base (d22c8b3) to head (0bd661a).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

pkg/controller/plan/migration.go
0.00%
38 Missing :warning:

pkg/controller/plan/adapter/vsphere/client.go
0.00%
18 Missing :warning:

pkg/controller/plan/adapter/openstack/client.go
0.00%
4 Missing :warning:

pkg/controller/plan/adapter/ovirt/client.go
0.00%
4 Missing :warning:

pkg/controller/plan/adapter/ova/client.go
0.00%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1184   +/-   ##
=======================================
  Coverage   15.66%   15.66%
=======================================
  Files         112      112
  Lines       23052    23052
=======================================
  Hits         3612     3612
  Misses      19155    19155
  Partials      285      285

Flag
Coverage Δ

unittests
15.66% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


chore: Added trufflehog secret scanning tool

Description
This PR introduces TruffleHog as a new open source tool for secret scanning to be used alongside native Github Secret scanning. This is being enforced as a replacement to the existing GitGuardian (commercial) tool.
Update trg-8-03.md

Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

I would like know if there are any blockers to review and merge this PR. It has been left open for quite some time.


Nt/ios notification fix

What does this PR?
Added a workaround to read initial and paused notifications on iOS, retained android side of things to what it was perviously working on.
 Still not able to resolve the source but used a third party package notifee to fix things for us on iOS.
Issue number
https://github.com/orgs/ecency/projects/2#card-86675629
Screenshots/Video
https://user-images.githubusercontent.com/6298342/201910280-f264b0eb-b91f-4e76-a8a2-f62eaa8587b8.mov

@noumantahir there is conflict, could you check?


Support for charting time series data

Does this currently support using dates as values in the dataset?
This does not work
<chart data="[	{'date':'2015-11-01','value':12},
       			{'date':'2015-11-02','value':18},
       			{'date':'2015-11-03','value':22},
       		 ]"
   		options="{	title: 'Downloads',
				    description: 'This graphic shows a time-series of downloads.',
				    x_accessor: date,
					y_accessor: 'value',
		}">
</chart>

nor does using a data element like this one that is on the Metrics Graphics main page
           [{'date':new Date('2014-11-01'),'value':12},
           {'date':new Date('2014-11-02'),'value':18}],

I've been able to get numeric values to make charts, but I need to plot dates.

I've forked this to work on this.  I'm adding a new directive chart-time-series.
Should I just be changing the files in /dist or should I change the /src and rebuild it? If its the latter, I need some guidance on how to do that...

hi Beau;
The problem with your example above is that the JSON representation of dates has to be converted into a JavaScript Date object before MetricsGraphics can use it.
I've added a new attribute convert-date-field="fieldname" to the library to enable you to do the data conversion.  Please pull the latest version, then either look at src/index.html or do grunt serve to compile and show the example page. I've added an example for this specific case. Let me know if you have any other issues with it.
Davis

Davis  - thanks! This looks like it will do what I was looking for. It is a slightly different approach than I was taking in my fork but this will work. I'm relatively new to angular, so it was cool to see your method for doing this.

While I was checking this out, I noticed that the changes from issue #2 got lost with this commit.
      // create the chart
      scope.$watch('data', function(){
        options.data = scope.data || [];
        MG.data_graphic(options);
      });

However, I've noticed two things with this. I don't know if these should be new issues or not.

It only watches the data not any of the components of data (say for instance data.value). This can be addressed by adding another argument to the scope.$watch() function, eg.

      // create the chart
      scope.$watch('data', function(){
        options.data = scope.data || [];
        MG.data_graphic(options);
      },true);

By re-firing MG.data_graphics() everytime that data, it is attempting to convert the JS data object again which throws an error from the MG.convert.date().  In my branch, you'll see that I only do the conversion if the date field is a string (not if it's an object).

I was going down the route of a different directive rather than build it into the same directive. I was also handling multiple lines differently (each line in it's own array) but I like your method better (single array with multiple y_acessor values).
I'd like to help with this, but I am traveling this week and I am not nearly as fast as you are with this. If you  spawn separate issues for these, I'll definitely try to work on them.

Davis - can you check out what I did in this branch It's not 100% yet. There are some constraints on how the data much be shaped and it is limited to only one y-accessor. I'm sure that I'm missing something.  Any suggestions would be welcomed.  I added a few more examples to the examples page - some of which show what is not working right.

I've opened issue #4 to restore riosc's patch.  Thanks for catching that.

I opened issue #5 to incorporate your patch and updated the example.


ignite-cli doesn't create ./ignite/Exemple folder and files

What's going on?
Ignite doesn't fail to compile but fail to load the project on ios or android. It is missing all the files in ./ignite/Exemple/
I follow the tutorial on youtube with the Textable App few week ago and everything was OK. I wanted to show to a friend this amazing app on his computer. Even node wasn't installed yet. So we started from zero.
If I build an appwith the CLI without any other modification and just run-ios at the end the app fail to load the module complaining about missing ../Examples/Components/animatableExample. After a lot of time I compare with my own code and see that the /ignite/Exemple/ was missing.
I just cp -r the file from my own app to his and everything is working now.
Can you fix that please ?

ignite doctor results:
System
  platform           darwin
  arch               x64
  cpu                8 cores                                        Intel(R) Core(TM) i7-4750HQ CPU @ 2.00GHz
  directory          /Users/gil/Code/igniteNew/galinetteChampetre

JavaScript
  node               8.1.4        /usr/local/bin/node
  npm                5.0.3        /usr/local/bin/npm
  yarn               0.27.5       /usr/local/bin/yarn

React Native
  react-native-cli   2.0.1
  app rn version     0.42.0

Ignite
  ignite             2.0.0-beta.9   /usr/local/bin/ignite

Android
  java               1.8.0_121    /usr/bin/java
  android home       -            /Users/gil/Library/Android/sdk

iOS
  xcode              9.0

Hey @gfelot , can you upgrade to the released version of Ignite?
npm uninstall -g ignite-cli
npm install -g ignite-cli

... and then try again?

Yep it's working. I don't know why because on my friend's computer we had to install everything brand new and had the same trouble.
But between this version of ignite and the one used in the Textable video, there is some change, right ?


Allow for containers to mount files or paths from sibling containers

If I'm running a container which has the following file/path of interest:
/resources/someservice.config
And I want to mount it to a path inside another container:
/etc/service/config
I haven't been able to find a clear indication if Docker is capable of doing something like this.  While there is the ability to mount all volumes from another container, there doesn't appear to be any way to map them individually.  It's all-or-nothing.
I'd love to see the ability to simply have two containers running on the same host be able to map files from each other.

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Does this work for your use case?
docker run -v sharedvol:/resources --name containera foobar

docker run -v sharedvol:/somewhere --name containerb someimage

However, this is a support question, not a bug or feature request, and GitHub is not intended as a support forum; for these kind of questions, please use;

forums.docker.com
the docker-user Google group
the #docker IRC channel on freenode
StackOverflow

I'm going to close this issue, but feel free to continue the discussion

Possibly related to this at this point: https://github.com/docker/compose/issues/424
I'm able to create named volumes for directories, but not files.

Also, @thaJeztah, after trying your solution, it doesn't seem to work.  What ends up happening is the filesystem gets blanked out on the source (containera).  Not shared as a named volume.
I don't think this is a "support question" anymore.

I'm able to create named volumes for directories, but not files.

This may be something that we can add to the documentation

What ends up happening is the filesystem gets blanked out on the source (container)

The files that are present inside the container on that location should be copied to the volume, the first time the container is started; what version of docker are you running, and on what platform?

In the most general of terms, I'd like one container to contain files that can be used by another.  But I have to be able to bind them on a file-by-file basis.  Not entire directories at a time.

You need to update to update to Docker 1.10 in order to have named volumes be propagated with the containers' content;
Build this dockerfile:
FROM ubuntu
RUN mkdir -p /config/ && echo "HELLO WORLD" > /config/config-file

$ docker build -t foo .
Sending build context to Docker daemon 3.072 kB
Step 1 : FROM ubuntu
 ---> 06ab2de020f4
Step 2 : RUN mkdir -p /config/ && echo "HELLO WORLD" > /config/config-file
 ---> Running in 156262dab99d
 ---> 1cd0db55d0d2
Removing intermediate container 156262dab99d
Successfully built 1cd0db55d0d2

To be sure, remove any existing sharedconfig volume
$ docker volume rm sharedconfig
sharedconfig

Start the first container (which contains the config-file), and create
a named volume, called sharedconfig
$ docker run --rm -it -v sharedconfig:/config/ foo cat /config/config-file
HELLO WORLD

Start another container (a standard ubuntu container), and use the sharedconfig
named volume:
$ docker run --rm -it -v sharedconfig:/config/ ubuntu cat /config/config-file
HELLO WORLD

Having named volumes for single files is not supported, and not actually sure we would be able to do that reliably; plugins may not support this, and there are various things that make this really tricky;
For example, if the file you want to make into a volume does not exist in the container, but created at runtime, a directory will be made, instead of a file.
For individual files, I think your best option is to  bind-mount that file, however, the file should be present on the host before doing so

Really, having the option would be nice as I'd be able to rely on my tools (docker-compose, rancher-compose) to ensure that all comes together nicely.
I still say it would be useful so that containers can ship preferred configurations for supporting containers (php-fpm, nginx).

Feel free to open a feature request (e.g. "add support for single-file volumes"). But i'm not sure it will be accepted and/or actually possible.
Mounting individual files (also bind-mounted) can produce really strange issues. For example, some tools/software never update a file, but write to a temp-file, then remove the original file, and put the updated file in its place. Doing so with mounted files doesn't work, so will lead to a lot of headaches.
I understand your use-case, but not sure we can help. Re-structuring the directory-structure of your application, so that files to be shared (between containers, or as a volume) are in separate directories, is probably the best solution.

Yeah, that makes sense.  I guess what makes that tricky is that I'm trying to do this so that I don't have to maintain my own customization of the public nginx container.  I just need to be able to configure it (not customize).
I'll open that ticket on the offchance that it does make sense to at least make available.

Thanks! It's not that we're not willing to, we just have to keep in mind having to maintain each feature as well, which is often the hard part :+1:

guys, here:
https://hub.docker.com/_/nginx/
they use mounting files.
is the documentation wrong?

@ninja-pirate bind-mounting files is supported (i.e. -v /some/file.cf:/in/container.cf), but not for "named" volumes (I.e. -v letsnameitfoo:/in/container.cf). It can be tricky though, because if the mounted file doesn't exist, docker doesn't know you meant a file, so currently makes an empty directory. This will change in future, see https://github.com/docker/docker/pull/19953

@thaJeztah, oh i see now, thank you!

Hi
@thaJeztah If I understand properly, starting from docker 1.11 according to https://github.com/docker/docker/issues/19304#issuecomment-171386745 files cannot be mounted with -v. Did I understand correctly?
Actually, we hare using -v /etc/localtime:/etc/localtime:ro to get our containers synced with host time.
Is this that the only way to achieve it?
Am I wrong or this forces to deploy using an entrypoint script or custom images built from dockerfiles?
Couldn't a --sync parameter be added to docker?
Regards

@mostolog bind-mounting a file is supported, but (as https://github.com/docker/docker/issues/19304#issuecomment-171386745) mentions; if the file you're trying to bind-mount doesn't exist, docker assumes you want it to create a directory and bind-mount that. There's no change in that respect (compared to older versions of docker)

@thaJeztah Thanks. Clear as water now. Shouldn't this issue be closed?


Assigning 'Cognitive Services OpenAI Contributor' role to managed identity.. brings an error

Deployment name: graphrag-deploy-2024-08-08_16-47-58
WARNING: The configuration value of bicep.use_binary_from_path has been set to 'false'.
ERROR: {"status":"Failed","error":{"code":"DeploymentFailed","target":"/subscriptions/xxxx/resourceGroups/xxxx/providers/Microsoft.Resources/deployments/graphrag-deploy-2024-08-08_16-47-58","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":[{"code":"ResourceDeploymentFailure","target":"/subscriptions/xxxxx/resourceGroups/xxxxx/providers/Microsoft.Resources/deployments/aks","message":"The resource write operation failed to complete successfully, because it reached terminal provisioning state 'Failed'.","details":[{"code":"DeploymentFailed","target":"/subscriptions/xxxxxx/resourceGroups/xxxxx/providers/Microsoft.Resources/deployments/aks","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":[{"code":"ResourceDeploymentFailure","target":"/subscriptions/xxxxx/resourceGroups/xxxxx/providers/Microsoft.ContainerService/managedClusters/aks-emt-grapghrag-dev-15/agentPools/graphrag","message":"The resource write operation failed to complete successfully, because it reached terminal provisioning state 'Failed'.","details":[{"code":"OverconstrainedAllocationRequest","message":"Code="OverconstrainedAllocationRequest" Message="Allocation failed. VM(s) with the following constraints cannot be allocated, because the condition is too restrictive. Please remove some constraints and try again. Constraints applied are:\n  - Networking Constraints (such as Accelerated Networking or IPv6)\n  - VM Size\n" Target="0"","details":[{"code":"OverconstrainedAllocationRequest","target":"0","message":"Allocation failed. VM(s) with the following constraints cannot be allocated, because the condition is too restrictive. Please remove some constraints and try again. Constraints applied are:\n  - Networking Constraints (such as Accelerated Networking or IPv6)\n  - VM Size\n"}]}]}]}]},{"code":"ResourceDeploymentFailure","target":"/subscriptions/xxxxx/resourceGroups/xxxxx/providers/Microsoft.Resources/deployments/apim","message":"The resource write operation failed to complete successfully, because it reached terminal provisioning state 'Failed'.","details":[{"code":"DeploymentFailed","target":"/subscriptions/xxxxxx/resourceGroups/xxxxx/providers/Microsoft.Resources/deployments/apim","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":[{"code":"Timeout","message":"Call to Management API apim-emt-grapghrag-dev-15.management.azure-api.net:3443 timed out for the Developer SKU service which will have inherent capacity issues due to it's scale. Please refer to SLA at https://aka.ms/apimsla and considering upgrading to a SKU Tier with higher SLA."}]}]}]}}
Assigning 'Cognitive Services OpenAI Contributor' role to managed identity...  ________________________________
/  Uh oh, an error has occurred.
\  Please see message below.     /
‾‾‾‾‾‾‾‾‾‾/‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾‾
/
__ /
/
~    ~
/ \  /_
\o/  \o/
|    |
||   |/
||   ||
||   ||
| _/ |
\     /
___/
Unable to parse service principal id from azure outputs, exiting...
This happens when the deployment runs locally.

This error appears to be related to a lack of resource capacity in your subscription quota and/or the region you're deploying to. I'd suggest you go to the Azure Portal -> {your resource group} -> Deployments and look for failed deployments. Try digging into the error messages to see which resource deployment is failing and why.

this is solved. thanks

this is solved. thanks

how did you solve it? I am facing same problem

@jamil77944 Could you explain how this issue was resolved? I'm encountering the same problem. All resources have been deployed successfully, but it seems the problem arises when attempting to add role assignments. appreciated.


Folds only argument list, not the entire function

VSCode Version: 1.51.1
OS Version: Big Sur 11.0.1 (Darwin x64 20.1.0)
ms-python: v2020.11.371526539

VSCode folds only the argument list when closing parentheses have smaller indentation than the argument list.
Expected behavior: folds the entire function body.

We're treating this as an enhancement, so we have marked this issue as "needs decision" to make sure we have a conversation about your idea. We plan to leave this feature request open for at least a month to see how many 👍 votes the opening comment gets to help us make our decision.

If it doesn't get enough attention and you decided not to proceed with it, I would be grateful if someone could provide instructions/ideas about the best way to achieve this functionality, and I will be happy to create a PR.


Cannot monitor creation of a file when parent directory does not pre-exist

Thank you for this library.
Say, I want to wait for a creation of file /a/b/c and /a exists, but /a/b does not yet exist. I can't get an event to fire here, even if checking recursive = true (I guess it is meant for this case?)
https://github.com/Sciss/schwatcher/blob/11a4e19f933a5ca27f8c611f9dddf5ad4c6d0f15/src/test/scala/de/sciss/foo/SomeTests.scala

Wow, it's been a while since I've looked at this library :)
Can you give persistent = true a try? Recursive is for controlling whether or not to watch all sub directories.

Thank you; persistent did the job.


Pls, help? Where rep?


Create mock-up webpage

This mock-up will contain placeholder images and projects, that will be refined later on and replaced with real projects.

Done.


Copy Row from other Excel

I have a requirement to consolidated several excel file to one. Which is the best practice to copy each row to a new worksheet and keep all the style or formula? I currently use row.values to add row, but it seems many errors and it only keeps the values if it succeed. Any help?
Here is the log:
(node:5420) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): Error: Shared Formula maste
r must exist above and or left of clone
(node:5420) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are
not handled will terminate the Node.js process with a non-zero exit code.

I really need to copy row(s) from other sheet. I try this:
const Excel = require('exceljs');

// target file
const wb = new Excel.Workbook();
const ws = wb.addWorksheet();

// read from a file
const workbook = new Excel.Workbook();
workbook.xlsx.readFile('33.xlsx')
    .then(function() {
        worksheet = workbook.getWorksheet();
        const lastRowNum = worksheet.rowCount;
        worksheet.eachRow((row, rowNum) => {
          if (rowNum !== lastRowNum) {
            const r = ws.addRow();
            Object.assign(r, row);
          }
        });

        wb.xlsx.writeFile('33-copy.xlsx').then(() => {
          console.log('done!');
        });
    });

But, it works not very well. There are some problems for rows which contain merged cells.

@blackmatch it is generating a new sheet but that is malfunction. data loss is happening.

@blackmatch your comment saved the day for me. I am trying something similar #1566 where data was getting copied but not styles. After i used
const r = ws.addRow();
Object.assign(r, row);

I could see that the data + styles are getting copied, the only issue i am having now is some rows which are supposed to be merged cells does not seem to be copying correctly.
For example, see the image below, the string should appear to be merged between all the rows but that does not seems to be happening.

Any help on how to fix will be really great, Thanks


[12.1.x] Backport of ISPN-13430 ISPN-13482 ISPN-13473 ISPN-13488

Backport for patching and testing purposes.
https://issues.redhat.com/browse/ISPN-13430
https://issues.redhat.com/browse/ISPN-13473
https://issues.redhat.com/browse/ISPN-13482
https://issues.redhat.com/browse/ISPN-13488
edit, added:
https://issues.redhat.com/browse/ISPN-13562

Fixed checkstyle error.

Merged, thanks


Rethink how interface tests are specified

          I know this is out of scope of this PR but the framework makes it non-trivial to check what is exactly validated. We're not asserting anything explicitly.

When I'm reviewing this test for example, I have a hard time figuring out, what exactly are we validating. The same goes for the next one where the devil's in the "schema=SchemaConfig.empty" detail.
Originally posted by @gruyaume in https://github.com/canonical/charm-relation-interfaces/pull/57#discussion_r1174022340

One possible API I have in mind at the moment is:
from interface_tester import register_test_case, Role, SchemaConfig

# minimal test case
register_test_case(
name='foo',
role=Role.requirer)

# schema
register_test_case(
name='bar',
role=Role.requirer,
schema=SchemaConfig.empty)

# validator, instead of the decorated function
register_test_case(
name='baz',
role=Role.provider,
validator=lambda output_state: output_state.status.unit.name == 'active')


runtime:compiler: python linters/checkers: add mypy and ruff; update pylint

Both new files origin from 0 and the former was added by @pbnj-dragon ; I have a hunch they're okay with this

thanks


Fix failing test: TestGeoPackageHydrofabric.test_uid_1_a

See #468 for context and discussion
dmod.modeldata - 0.9.3
Additions

pyogrio dependency
Use pyogrio as GeoPandas engine.

Changes

Fix geopackage hydrofabric uid regression (closes #468).

python/lib/modeldata : passed. Its complaining about 'ABCMeta' object is not subscriptable.

I'll look into this a bit later or first thing in the morning.


Fix IgorTimofeev/MineOS#120 : Add argument check

This pull request should fix IgorTimifeev/MineOS#120 for now.

Problem got fixed already.


Allow empty database name in OpenDBI method

Allow to the name value OpenDBI method as an empty string.
From lmdb documentation doc name can be empty  "If only a single database is needed in the environment, this value may be NULL."

The author provides opening the unnamed db as txn.OpenRoot function, but I agree the naming is not quite intuitive while we are reading the official LMDB documentation as main reference.


Grey out right-click->Copy if a user doesn't have the appropriate Create permission anywhere in the folder tree

There are a few issues here.
The create new item menu shows a selection of types that can be created in the selected folder. However the create item dialog then shows you all of the places in the tree that you can create the item and allows you to select one. Given that this is the functionality the create menu should just show all types that you could possibly create anywhere in the tree regardless of the current selection (I think).
The create, copy and move dialogs only show folders that you have read permission on so currently you would need both read permission plus one or more create permissions on that folder. Should a user need read permission on a folder to create items in it? Perhaps create permissions infer read so that these folders are visible.
And then copy... Yes, perhaps you shouldn't be able to select copy if you have no rights to create any of the selected items anywhere in the tree, but what if you are able to create some items in some places in cases where you have selected multiple different types? Perhaps it should only be enabled if there are places in the tree where you can create all of the selected item types and then only allow those as destinations.


Deprecation warning

Summary:

The configuration option config.serve_static_assets has been renamed to config.serve_static_files to clarify its role (it merely enables serving everything in the public folder and is unrelated to the asset pipeline). The serve_static_assets alias will be removed in Rails 5.0

@epintos :wink:

:+1:


« sed -i (edit in place) » used as root doesn't preserve uid nor gid

Hi!
Samsung Galaxy S4 Mini Plus I9195I (CM 13 unofficial)
toybox version : 0.7.0-892c602b10e7-android
# ls -l
-rw-rw---- 1 u0_a201 u0_a201 4 2016-09-30 11:35 bb_test.txt
-rw-rw---- 1 u0_a201 u0_a201 4 2016-09-30 11:35 tb_test.txt
# sed -i s/foo/bar/ tb_test.txt                                       (toybox)
# /system/xbin/sed -i s/foo/bar/ bb_test.txt        (busybox)
# ls -l
-rw-rw---- 1 u0_a201 u0_a201 4 2016-09-30 11:39 bb_test.txt (busybox)
-rw-rw---- 1 root    root    4 2016-09-30 11:39 tb_test.txt              (toybox)

On 09/30/2016 05:02 AM, ed38220 wrote:

Hi!
Samsung Galaxy S4 Mini Plus I9195I (CM 13 unofficial)
toybox version : 0.7.0-892c602b10e7-android
|# ls -l|
|-rw-rw---- 1 u0_a201 u0_a201 4 2016-09-30 11:35 bb_test.txt|
|-rw-rw---- 1 u0_a201 u0_a201 4 2016-09-30 11:35 tb_test.txt|
|# sed -i s/foo/bar/ tb_test.txt (toybox)|
|# /system/xbin/sed -i s/foo/bar/ bb_test.txt (busybox)|
|# ls -l|
|-rw-rw---- 1 u0_a201 u0_a201 4 2016-09-30 11:39 bb_test.txt (busybox)|
|-rw-rw---- 1 root root 4 2016-09-30 11:39 tb_test.txt (toybox)|

Ah! We were doing fchmod() (in lib/lib.c copy_tempfile()) but not
fchown(). So preserving permissions but not ownership.
And the gcc developers went even crazier, so now typecasting to (void)
won't suppress the "unused result" warning. But an if() with no body does.
Yes this call fails if we're not root, and we expect and ignore that. If
it fails when we are root what exactly are we supposed to do about
it? We're already doing the fd operation instead of the directory
operation to eliminate the create/delete/recreate race, if the fchown()
fails on an already opened filehandle it's either because our filesystem
got unmounted under us (in which case we can't write the file contents)
or else SELinux did something crazy (again) and you get to keep the pieces.
There's an old saying about applied computer science involving finding
the right wrench to pound in the correct screw. Any "are you sure you
want to do that" warning that does not take "yes" for an answer will be
fixed with hot irons, then publicly mocked.
Rob

close? fixed by https://github.com/landley/toybox/commit/3366af7aa8212dfa9838fdfd17bae7909eb3797d

rob: this can be closed.


Introduce nix tooling for reproducable dev envirionments

Added a nix-shell to the project.
This allows developers to ephemrally install pinned packages to easily get a working dev env.

feat: added nix tooling

This only has an effect if developers use nix, it changes nothing about the project for anyone else.

Thank you for your interest in the project.
The PR is merged, but I'll ping you to update that file in case new dependencies are added, ok? ;)
If you don't mind me asking, why did you include gnome.libsoup? Is it something you commonly use in developing Rust apps?

Great!

I'll ping you to update that file in case new dependencies are added, ok? ;)

No problem at all! Feel free to ping me if something about the nix code needs to be changed or dependencies added/removed.

If you don't mind me asking, why did you include gnome.libsoup? Is it something you commonly use in developing Rust apps?

I figured out the required depencies through trial and error.
Since you asked, I looked into it a bit more and figured out that libsoup is a depency of the project (c.f. cargo tree | grep soup). However it is also added by the webkitgtk dependency, so we can forego adding gnome.libsoup specifically.
I also noticed that I forgot to add yarn as a depency. It seems that I still was in another devshell providing yarn when creating this flake...
I'll add a follow-up PR removing gnome.libsoup and adding yarn.


can I set this up for free????

can I set this up for free? if were and how.
(try doing it but get [error]: LibreLink Up - Non-zero status code: {"status":2,"error":{"message":"notAuthenticated"}}
[error]: LibreLink Up - No AuthTicket received. Please check your credentials.)

Yes, did you get it sorted out?

Yes, did you get it sorted out?

no

How far did you get?
On Thu, May 2, 2024, 10:42 dumm_dogg @.***> wrote:

Yes, did you get it sorted out?
no
—
Reply to this email directly, view it on GitHub
https://github.com/timoschlueter/nightscout-librelink-up/issues/123#issuecomment-2090032933,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD3BD4K6BPK6GFTQ3FNHWLZAIDBXAVCNFSM6AAAAABCZDL4KOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJQGAZTEOJTGM
.
You are receiving this because you commented.Message ID:
@.***>

well i kinda gave up bc i am getting a new sensor in 2 weeks. but thanks for asking


Updates to deep/nested reactive props does not trigger a DOM update

Version
3.0.0-beta.15
Reproduction link
https://codepen.io/conrad-sollitt/pen/yLeaoGB
Steps to reproduce
The CodePen contains 4 main options depending on how code is commented:

Vue 2
Vue 3 - Composition API with Vue.reactive
Vue 3 - Composition API with Vue.ref
Vue 3 - Options API

The HTML allows Vue 2/Vue 3 to be toggled by commenting/un-commenting the correct version.
Based on whether Vue 2 or Vue 3 is used the JS will run code the correct version. When using Vue 3 two constants at the top of the JS file can be changed to affect what API is used:
const useCompositionAPI = true
const useReactive = false

Once mounted/onMounted is called a window.setTimeout()  runs after 500 milliseconds and makes updates on the nested list.
What is expected?
When using Vue 2 (both Prod and Dev) the updates always show in the DOM.
What is actually happening?
When using Vue 3 (regardless of the API used) the updates will only show if one of the top-level properties is set and not one of the deep/nested properties.
Code comments in the window.setTimeout() describe how to make it work with Vue 3.

This may be related to issue: https://github.com/vuejs/vue-next/pull/262
If it is then this feature will not be supported in Vue 3. If it can't be supported I think it would be a good idea to document deep/nested object updates somewhere in the final documentation.
I'm opening the issue because based on the DOM it's not clear to me if it should update or not. Additionally when using Dev Tools you can see that all nested objects use a Proxy object so this could be a Browser/JavaScript issue as well.
From the API Docs:
https://composition-api.vuejs.org/api.html#reactive

The reactive conversion is "deep": it affects all nested properties. In the ES2015 Proxy based implementation, the returned proxy is not equal to the original object.

Last Friday I opened the following issue related to the global window object: https://github.com/vuejs/vue-next/issues/1353
I'm happy to follow up my report that I feel that the new Vue 3 API is easy to work with for migration and keeping Vue 2 Options API while supporting the new Composition API works great and is well thought out! I've been waiting a while to migrate as I wanted to wait till near release rather than making many changes while the new API was in early development.
I was able to migrate 7 mini web apps to Vue 3 while keeping support for Vue 2 (actually running Vue 2 in production still until the final release) but each web app has the option for both Vue 2 or Vue 3 simply by swapping out the CDN link. This was the main issue that I found but it only required a few lines of code extra in one app in order to work.

I'm seeing something similar. But it's an update behind. When the nested object changes, the UI doesn't change until a second render cycle is called.

The issue is that you create a plain, non-reactive object here:
const obj = {
  name: 'test (2 child list items should show)',
  items: [],
}

and then continue to mutate this plain, nonreactive object:
obj.items.push({ label:'item-1', score:1 })

In Vue 3, this change will not be observed, even though before that mutation, you added the object to an reactive array.

In Vue 2, this makes the object itself reactive as Vue 2 will manipulate the original object's getters and setters.
In Vue 3, this will not do anything to the original object. the original object will continue to be non-reactive, so any change you apply to this object directly will not be registered.

So yes, this is a caveat of the new, proxy-based reactivity.
The solution is: never work with plain objects that are meant to become reactive. Always make them reactive proactively:
const obj = reactive({
  name: 'test (2 child list items should show)',
  items: [],
})

now, obj no longer references the original object, it references the reactive proxy. Any change you apply to it will be picked up.

This challenge will be covered in the documentation and, where possible, also supported by eslint rules in eslint-plugin-vue.

The issue I found was in my code. Once I tried to reproduce it in Beta.15, I couldn't. So I am happy to have it closed.

Thanks for the detailed feedback @LinusBorg


Compatible with good 6.4

Removed good-reporter which is now deprecated
Follows the new Reporter interface

This is a breaking change, as the instantiation options are not the same anymore:
new GoodSlack(events, {
  url: "webhook url",
  slack: {
    channel: "#general",
    // etc.
  }
});

This is a very rough PR: tests do not pass yet, I'd love if anyone could help me update them.

I'm gonna check on the new spec you linked to. I'd like definitely to help you out on this as soon as I get some time free. I'll try to take a look on it today later on.

Follow up on #6


Module not found: Can't resolve 'cluster' in...

I installed phantom-html-to-pdf on windows. I am using it in the client side. it throw this error
./node_modules/net-cluster/lib/utils.js
Module not found: Can't resolve 'cluster' in
any suggestions?
thanks

What you mean by "using it client side" ?
cluster is nodejs native module for very long time so that error doesn't make sense to me.


Support more meta for ServerNode

What problem does this PR solve?
Issue Number: resolve #850
Problem Summary:
What is changed and the side effects?
Changed:
支持设置更多meta信息（地区信息、灰度信息等），满足多样的路由规则。
Side effects:

Performance effects(性能影响):

Breaking backward compatibility(向后兼容性):

Check List:

Please make sure your changes are compilable(请确保你的更改可以通过编译).
When providing us with a new feature, it is best to add related tests(如果你向我们增加一个新的功能, 请添加相关测试).
Please follow Contributor Covenant Code of Conduct.(请遵循贡献者准则).

LGTM


[gulp 4] How to batch changes with gulp 4

Hello,
sorry if this question has been answered before, I haven't been able to find it.
When I watch a repository, it triggers the watch once per file changed. I though that it wasn't supposed to do that, but maybe I did something wrong with my config:

var PATHS = {
    ts: './src/app/**/*.ts',
    sass: './src/scss/**/*.scss'
};

gulp.task(ts2js);
gulp.task(sass2css);

function ts2js() {
 // does something
}

function sass2css() {
    return gulp.src("src/scss/*.scss")
        .pipe( .....
}

function watch(server) {
    gulp.watch(PATHS.ts, ts2js);
    gulp.watch(PATHS.sass, sass2css);
}

If I save multiple ts or scss files, it will trigger the watcher multiple times and recompile the files as many times...
With gulp 3 I could use gulp-batch to solve this problem. I tried to use it with gulp 4 like this:
gulp.watch(PATHS.ts, batch(function(events, done) {
    gulp.series(ts2js)(done);
}));
gulp.watch(PATHS.sass, batch(function(events, done) {
    gulp.series(sass2css)(done);
}));

It works, but then when I end the task (with CMD + C), I get the message:
[09:30:37] The following tasks did not complete: <anonymous>, <anonymous>
[09:30:37] Did you forget to signal async completion?

With one  per execution of the watch.
So, is there a better way than using batch ? If not, what would be the correct syntax ?
Thank you for any help that you could provide.

Support questions should be directed to stack overflow.  I would recommend you migrate away from gulp-batch

So, I come here with a detailed issue, after searching thoroughly if there ever was an answer for it.
I explain the problem in details which might be related to a bug or a missing feature in a beta version of this library (which I took time to test even though it is not released yet).
I even take the time to give you an example of code that leads to this problem.
And you just slam the door in my face without even telling me if this is a legitimate question.
The next time someone will have a similar issue he might find this and not even find the answer he was looking for, and maybe he will just give up in trying to make this work.
Not very kind of you.

@ocombe Man, gulp maintainers are not responsible for plugins. Ask on stack overflow or on gulp-batch. And somebody will find your question on stack overflow. Issue is not support, they are for product developing process.

Sorry if you feel like we were rude, but we don't provide support for other projects (plugins, modules, etc.) on this repository - we are happy to help on IRC, gitter, or StackOverflow though.

Ok, sorry about my reaction then. But I think that you guys misunderstood the root problem here. I don't want to use gulp batch, I just tried it to see if that would solve the underlying problem.
Is it normal to have multiple watch events (one per file) when you watch using glob regex?
Isn't there a way to trigger only one watch event per folder?
I saw something about a possible debounce option (not yet developed), would it be the answer here?

@ocombe That seems like an issue with the file watcher, use gulp 4.0 - we have a new file watcher which fixes all of these watcher issues

I am using gulp 4 :-/

@ocombe Ah hold up, I think I get you. Try using a debounce module around the functions you pass to watch - https://www.npmjs.com/package/debounce
I think chokidar (our file watcher) has some debounce settings you can toy with as well

Thanks, I will try that.

For what it's worth, gulp-batch seems to just be a very simple debounce function anyway: https://github.com/floatdrop/gulp-batch/blob/master/index.js

@ocombe Sorry for misunderstanding.  This is actually a duplicate of https://github.com/gulpjs/gulp/issues/1304 and you just reminded me of a thought I had to solve this.  Going to post it over there to keep the discussion focused.

im use like this $.gp.watch($.config.paths.styles.watch.app, $.gp.batch(function(events, cb) {
events.on('data', $.gulp.series('sass:app')).on('end', cb);
}));`


Add JinaBert model

I want to share my work on the JinaBert model.
Not sure if you want to include it at all, since it's not officially part of transformers, you must specify trust_remote_code=True when running it with transformers, and there is still an open issue.
This PR would enable bumblebee users to run the jina embeddings v2 models.
The implementation of JinaBert is here.
Another issue with this being a custom implementation is that there is another variant that I started to work on: jinaai/jina-embeddings-v2-base-code.
Both, jinaai/jina-embeddings-v2-base-en and jinaai/jina-embeddings-v2-base-code, specify JinaBertForMaskedLM as architecture but point to different implementations.
 "_name_or_path": "jinaai/jina-bert-implementation",
  "architectures": [
    "JinaBertForMaskedLM"
  ],
  "auto_map": {
    "AutoConfig": "jinaai/jina-bert-implementation--configuration_bert.JinaBertConfig",
    "AutoModelForMaskedLM": "jinaai/jina-bert-implementation--modeling_bert.JinaBertForMaskedLM",
    "AutoModel": "jinaai/jina-bert-implementation--modeling_bert.JinaBertModel",
    "AutoModelForSequenceClassification": "jinaai/jina-bert-implementation--modeling_bert.JinaBertForSequenceClassification"
  },

vs.
  "_name_or_path": "jinaai/jina-bert-v2-qk-post-norm",
  "architectures": [
    "JinaBertForMaskedLM"
  ],
  "auto_map": {
    "AutoConfig": "jinaai/jina-bert-v2-qk-post-norm--configuration_bert.JinaBertConfig",
    "AutoModel": "jinaai/jina-bert-v2-qk-post-norm--modeling_bert.JinaBertModel",
    "AutoModelForMaskedLM": "jinaai/jina-bert-v2-qk-post-norm--modeling_bert.JinaBertForMaskedLM",
    "AutoModelForSequenceClassification": "jinaai/jina-bert-v2-qk-post-norm--modeling_bert.JinaBertForSequenceClassification"
  },

Is there a mechanism in bumblebee to distinguish these?
There are still some issues in this PR, I will add comments and can work on them over the next days/weeks.

Hey @joelpaulkoch, thanks for the PR and a great article!
To be honest, I am hesitant to support implementations from the Hub because (a) theoretically they are less stable, because they may be still subject to tweaks; (b) model proliferation is more likely, the jina-embeddings-v2-base-en vs jina-embeddings-v2-base-code is a good example.
We generally wait until models make it to hf/transformers, though from https://github.com/huggingface/transformers/issues/27035 it's not clear if that's ever going to happen.
At the moment, I would defer the decision and see how the status quo evolves. People can still use the model by installing bumblebee as {:bumblebee, github: "joelpaulkoch/jina-embeddings-v2"}.


Disable encryption on port 587

Subject

[ ] I would like to contribute to the project (code, documentation, advocacy, integration, ...)
[ ] I would like some help for integrating this container with another service (rainloop, PostFWD, ...)
[ ] I would like to configure a not documented mail server use case
[ ] I would like some feedback concerning a use case
[x] I have questions about TLS/SSL/STARTTLS/OpenSSL
[ ] Other

Description
Hi!
I would like to disable encryption on port 587 completely as it will only be used from within the container's network and I don't want to go through the whole encryption setup for no reason. How would I go about doing this?
Best Regards
Dominik

Use port 25 instead of 587? The README has got you covered.
The Postfix configuration is provided by Google I guess.

Use port 25 instead of 587? The README has got you covered.
The Postfix configuration is provided by Google I guess.

Okay, I thought 25 was a port responsible for something else. Still, thanks!

Okay, I thought 25 was a port responsible for something else. Still, thanks!


Override deployed tokens on mainnet

We need to override the deployed tokens on main net to ensure we can perform a compass handover again. They can just be temp dummy tokens, as long as we maintain control and can move them along.

@byte-bandit Okay so the third option? And this is not in the current release?

@byte-bandit this one is complete. yes?

Once the votes are out, yes.

The vote is complete. Closing.


Added Static typings

The only non-static typing in this PR is the handling of _record_input_props as its just 1 line and solves #250
As for the version bump for each PR, I suggest implementing git hooks which automatically upgrade those values after each PR, because that is the only conflict here (I resolved it, but this has version 1.8.4 just like #276 but I guess if both are merged its good, I will resolve merge conflicts if any)

Hey, thank you for the contribution, but unfortunately I cannot accept this PR in its current form.
First off, it not only contains static typings, but variable renames, doc adjustments, value changes and some minor refactors even. One PR should ideally do only one thing, as outlined in the contribution section of the README.
Second, it's not very pleasant how you just decided to not care about the existing coding convention of the project and just straight up rewrite it without even asking about it :slightly_smiling_face: Since it's a PR it has no negative effects on the project, I'm just sad to see contributor effort go to waste like that.
Instead, I recommend creating a discussion about code conventions and static typing either here or on Discord. That way we can decide whether we want a change like this and in what shape or form.
Also, if there's parts of this PR that fix other things, please submit them as separate PRs so I can review them.
Thanks!


Persistent storage fails

When app relaunches, the UUID is always regenerated instead of loaded from storages.
Either persisting the UUID to phone memory fails or the loading of the UUID fails.

I'll take a look.


Add example config file in README

Add example config file in README to allow simple copy/paste if someone doesn't want to rely on the config file auto-creation feature.

Maybe a animated GIF or asciinema recorded session can be useful.
We can also improve the aestethic of teh README adding a banner image on top of the page.

Sure, but I don't think a video on how to configure the file is the best approach to solve this. Certainly, we can use asciinema recorded session to better explain the CLI usage.


Support for more variables and variable expansion in app definition

Posted this in the google group and would love to see it as an enhancement: https://groups.google.com/forum/#!topic/marathon-framework/lVNyLTCAipU
I would love to see:

Support for more variables such as $ip
Ability to have these variables expanded in the app's definition.

This would allow people to satisfy a lot of requirements of docker containers in the wild and reduce the need to build their own fork to get it to work with marathon.

Thanks for the feature request, I'm adding it to our backlog.

+1

Hi @F21, thanks for your input! Please try to be as specific as possible, look for duplicates and try to break up potentially unrelated things into multiple issues.
"Support for more variables such as $ip" -- You mean along the lines of https://mesosphere.github.io/marathon/docs/task-environment-vars.html? So probably better call them "$MARATHON_IP" or something similar? How should these variables be defined? Please note that Marathon has to be able to calculate these variables when passing the TaskInfo to Mesos. If the IP is assigned randomly by docker, Marathon cannot know it in advance.
"Ability to have these variables expanded in the app's definition." -- Do you mean that Marathon should expand them? Marathon can only expand them if it knows the values of the referenced variables when it launches the task. Thus it can only possibly substitute other variables passed in the "env" configuration or variables that it determines automatically when launching (see https://mesosphere.github.io/marathon/docs/task-environment-vars.html). If we do this, we also have to be careful to come up with a good syntax for quoting values (to indicate that we do not want substitution) and dealing with reference cycles.
For backward compatibility, we also might want to only substitute variables if another configuration option is set or a global configuration option of Marathon is set. Otherwise old AppDefinitions might start to behave differently.
(This seems to be related to https://github.com/mesosphere/marathon/issues/1401)

I can think of at least one i really wish marathon had:  counter variables! :)
I.e. Pass in something like "Foo" : "bar_idx" to env.
And marathon would track current value per task started/stopped/restarted.
It could help with naming containers or for stateful services with data replication mechanisms.
Though it kind of breaks the dynamic nature of mesos/marathon there are tools around which still rely on unique naming of cluster members and require some script or mechanism to habdle it outside the mesos/marathon framework

@sepiroth887 that's very similar to https://github.com/mesosphere/marathon/issues/1242.
Maybe we should reopen this.

That'd be nice :)
I also thought of another thing though:
What about a build in variable store?
E.g. A REST api to set/update/delete kv pairs.
Those could be prefixed to avoid clashes and then be referenced in app payloads.
It would allow other tools to keep them up to date by whatever logic they require and can be audited/viewed quite easily that way too

@sepiroth887 Even though it is not trivial to implement, I like this related idea: https://github.com/mesosphere/marathon/issues/1863

That'd do just fine :)

@kolloch Thanks for taking a look at this!
By $IP, I meant the ip of the node the task executes on. $HOST is useful, but sometimes things like ceph requires an ip address rather than the hostname.
Being expanding variables Marathon knows in places such as env and perhaps other parts of the app config would be extremely useful and simplify a lot of things.

follow

Hi @F21, as far as I know, Marathon cannot know the IP of the host your task is launched on via the Scheduler API of Mesos.
For variable substitution, it would be helpful if you came up with specific examples of syntax & use cases.

@kolloch:
One use-case would be to get Marathon variables expanded in the env definition:
{
   "id":"ceph",
   "cpus":0.3,
   "mem":256,
   "instances":1,
   "maxLaunchDelaySeconds":36000,
   "container":{
      "docker":{
         "image":"ceph/demo:latest",
         "network":"HOST"
      }
   },
   "env":{
	  "CEPH_NETWORK":"<IP_ADDRESS>/24",
	  "MON_IP": "$HOST"
   }
}

I'd like to expand variables in the volumes portion of app definitions.
{
  "id": "app",
  "cpus": 0.25,
  "mem": 256,
  "instances": 1,
  "container": {
    "type": "DOCKER",
    "docker": {
      "image": "private-registry.example.com/ns/app:version"
    },
    "volumes": [
      {
        "hostPath": "/mnt/blobs/$MESOS_TASK_ID",
        "containerPath": "/var/log/app",
        "mode": "RW"
      }
    ]
  }
}

The use case for me would be to pass environment variables to the docker image without having to fork it and adapt it to work. For example, I need to specify where some configuration folder is (which is in my $MESOS_SANDBOX) via an environment variable and so would like to be able to add the following to my marathon config:
{
  ...
  "env": {
    "CONFIG_FOLDER": "$MESOS_SANDBOX/config"
  },
  ...
}

Throwing out another example where the expansion in env is desirable.
This doesn't work:
{
  "id": "/apache/zeppelin",
  "cmd": "./zeppelin-0.6.0-incubating-SNAPSHOT/bin/zeppelin.sh",
  "env": {
    "ZEPPELIN_PORT": "$PORT0",
    "ZEPPELIN_JAVA_OPTS": "-Dspark.executor.uri=hdfs://bin/spark-1.4.1-bin-hadoop2.4.tgz -Dspark.driver.port=$PORT1 -Dspark.fileserver.port=$PORT2 -Dspark.executor.memory=8g -Dspark.cores.max=8 -Dspark.mesos.coarse=true"
  },
  "ports": [0, 0, 0],
  ...
}

Notice I need all three ports in different places. This workaround in JSON :disappointed:
{
  "id": "/apache/zeppelin",
  "cmd": "ZEPPELIN_PORT=$PORT0 ZEPPELIN_JAVA_OPTS='-Dspark.executor.uri=hdfs://bin/spark-1.4.1-bin-hadoop2.4.tgz -Dspark.driver.port='$PORT1' -Dspark.fileserver.port='$PORT2' -Dspark.executor.memory=8g -Dspark.cores.max=8 -Dspark.mesos.coarse=true' ./zeppelin-0.6.0-incubating-SNAPSHOT/bin/zeppelin.sh",
  "ports": [0, 0, 0],
  ...
}

This is a very very challenging thing for Marathon to even attempt to implement.
In the example provided by @flungo it is quite literally impossible for Marathon to resolved the value of CONFIG_FOLDER.  The environment variable MESOS_SANDBOX is something that is only available in the environment when the mesos docker containerizer is used.  This information is never made available to Marathon in any way.
The real solution to this problem probably involves an upstream change to Mesos.  One approach could be to create a file containing all of the environment variables that are declared on the ExecutorInfo and the CommandInfo and have Mesos source that file before running the specified command.  This approach would make sure that it's the shell doing the variable expansion in the same environment that the command is going to be executed in and keeps the responsibility of variable resolution/expansion from having to be implemented by Mesos or any framework.

@BenWhitehead I have found that MESOS_SANDBOX is always /mnt/mesos/sandbox so I have been explicitly declaring this, for the time being.
I think the best way to do this, may be to have it expanded by the shell (possibly through a generated file that is given to the container and sourced before the entrypoint/cmd). This way it is guaranteed that every environment variable that is available at runtime for the container can be expanded into the given environment variables.
Not sure if this would be useful or risky but this could also lead to being able to expand all shell syntax such as $(command) to use the output of a command on the container to set an environment variable.

@flungo I think we're mostly in agreement.
I feel the shell is the only appropriate location for this expansion to take place.
Part of my point is that MESOS_SANDBOX env variable only exists when running your task in a docker image. If the mesos containerizer is used that variable doesn't even exist.  I suspect there are a number of subtleties that will creep up if/when the work to do variable expansion is started (what happens if the shell tries to expand a variable that doesn't have a value? or doesn't yet have a value?).

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3002. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


eosiocpp output error

GuangdeMacBook-Pro:bin tg$ ./eosiocpp -o output.wast /Users/tg/Downloads/eos/contracts/hello/hello.cpp
/Users/tg/Downloads/eos/contracts/hello/hello.cpp:1:10: fatal error:
'eosiolib/eosio.hpp' file not found
#include <eosiolib/eosio.hpp>
^~~~~~~~~~~~~~~

run sudo ./eosio_install.sh first

@lazaridiscom Tried

@isTg Great

Also update to the 1.2.1 release.

Is this issue solved?

Tried other ways


Facets should not disappear after selection

Search result:

After selecting Uberon:

Should it be possible to search across multiple ontologies, i.e. the facets being multiselect?

See also #170


Fix Havoc Analysis

Currently, the havoc analysis is not even working properly.  In particular, on the WETH contract it only spots 2 out of 4 loops.  I have attached the manually corrected version which does terminate.  As one example, this is a loop which it failed to correctly analyse:
_0x00f1:
	jumpdest
        havoc 0
	dup4
	dup2
	lt
	iszero
	push 0x010c
	jumpi
	dup1
	dup3
	add
	mload
	dup2
	dup5
	add
	mstore
	push 0x20
	dup2
	add
	swap1
	pop
	push 0x00f1
	jump

And here is the other:
_0x031a:
	jumpdest
        havoc 0
	dup4
	dup2
	lt
	iszero
	push 0x0335
	jumpi
	dup1
	dup3
	add
	mload
	dup2
	dup5
	add
	mstore
	push 0x20
	dup2
	add
	swap1
	pop
	push 0x031a
	jump

In contrast, this loop was (I believe at least) correctly analysed:
_0x0556:
	jumpdest
        havoc 0
	dup2
	sload
	dup2
	mstore
	swap1
	push 0x01
	add
	swap1
	push 0x20
	add
	dup1
	dup4
	gt
	push 0x0556
	jumpi

I believe the issue is related to instructions which have implicit rather than explicit operands.  This information, unfortunately, is not actually made available within the generated dependency info.    The dup instructions need to have a dependency.

That got it :)


Phone not connecting

Computer: Apple M1 MAX, macOS Ventura 13.0.1
Phone: Samsung Galaxy S22, Android 13.
Software version: OpenMTP v.3.2.0 Mac ARM64.
Had no problem prior to upgrading to macOS Ventura and OpenMTP up from 3.1.15. Going back to OpenMTP 3.1.15 does not solve the issue.
Refreshing OpenMTP per the instructions only results in error "an error occured while setting up the Phone".
Combinations of switching between Kalam / Legacy modes and enabling/disabling auto-detection has no effect on issue.
Thank you,
Alexander.

Computer: Apple M1 Pro, macOS Ventura 13.0.1
Phone: Pixel 7 Pro, Android 13 (fully updated)
Software version: OpenMTP v.<IP_ADDRESS> Mac ARM64.
Don't know what version I was on before v.<IP_ADDRESS> but when I updated just now it stopped working.

Thank you for the report. Momentarily what you could do is to download the version 3.1.5 from GitHub releases and install the app.
Later disable the internet and open the application.
Turn off the auto updater from settings of OpenMTP.
This should prevent auto update.

Thank you for the feedback @ganeshrvel. Followed the steps but even the older 3.1.5 version is not working now. it seems something has permanently changed as the phone filesystem never shows up on OpenMTP any longer.
Let me clarify that
-- "Legacy" is the only connection method; if I select "Kalam" then OpenMTP keeps producing the error "an error occured while setting up the Phone".
-- As soon as the phone is connected by cable to the computer, the phone presents me fine with the prompt to allow data transfer. However, no matter how many times I try refreshing the OpenMTP phone panel, it appears that OpenMTP is non-responsive to the connected phone.
I am guessing that either macOS Vantura and/or Android 13 entirely block OpenMTP from viewing the phone's filesystem. Happy to further assist with testing if you need.

Same issue here but with a Galaxy Fold 4, and same scenario too

I'm waiting for my cicd partner's response to make a new release which patches this issue

Grateful for the efforts!

Update: for me installing the Samsung smart switch app for the phone fixed the issue, not my phone connects with no trouble to openMTP and Android file transfer app.

I can confirm that the latest version 3.2.10 resolves the issue; thank you all for the feedback and Ganesh for the release.

Samsung A71 5G is not recognized after Android 13, One UI 5.1 update.
Tried rebooting, uninstalling smart switch, setting default USB configuration to "transferring files"

Hi.
Kindly follow the steps on this page (https://github.com/ganeshrvel/openmtp/issues/276) and check if it resolves your issue.

Besides disabling the disturbing apps on your Mac – If nothing helps you can try to start your phone in recovery mode:

switch off your phone
connect via USB to your computer (just connecting)
press Volume-up + Power button for 8–10 seconds (at least for Samsung phones)
in the recovery mode menu select 'wipe cache partition' (not dangerous, will be re-generated) and confirm
last not least select 'repair apps'
the phone will restart

This helped twice in my case (last time after the update to One UI 6.0)

@newdev4ios

Update: for me installing the Samsung smart switch app for the phone fixed the issue, not my phone connects with no trouble to openMTP and Android file transfer app.

did you mean installing helped or uninstalling helped you?

I think it was installing Samsung smart switch app on my MacBook helped.
I think I meant now my phone connects with no trouble...
I can't remember 100% as its been a while because my fold 4 completely broke in two (it is currently in the form of two paper weights)


Snapping to roads

Hi! I just watched your FOSS4G talk and wow, releasing this library couldn't have come at a better time for me. I've been working on a tool to quickly sketch line-strings that snap to some kind of network. Right now my approach hacks on top of mapbox-gl-draw, and every time I peek into its codebase to figure out how a proper integration would work, I'm daunted by the complexity (and not inspired by the backlog of PRs the repo is accumulating). I'd love to explore building on top of terra-draw instead (and reap the benefits of Leaflet support too!)
You can try the tool so far here. After it loads, click the button on the right. If you're curious about the implementation, see https://github.com/acteng/atip/tree/main/route-snapper. I'm new to modern web dev (and sticking to vanilla JS so far). Most of the logic is happening in a Rust layer right now, compiled to WASM. That's mostly due to convenience for me (it's a very familiar language) -- the code is simple enough to write in JS or TS for sure, and the speed is probably fine. The routing itself works off a static network loaded once from a file, so it's bound to a fixed area. When that area gets large enough, contraction hierarchies or a similar graph preprocessing technique might be worth trying instead of A*. And in that case, I'd lean heavily towards just building something like https://github.com/easbar/fast_paths/ to WASM.
I have more ideas about snapping based on arbitrary data preprocessed from OSM, like drawing areas that can snap to the edge of roads or waterways. There's a whole space of techniques for getting the idea to work outside of a fixed area too... I'll probably be exploring tiling and dynamically generating the networks soon. (The latter idea would be something like an Overpass query to extract OSM data wherever the viewport is, then piping through a Rust library to turn into the right kind of network).
Anyway, I'll keep a close eye on this repo as it evolves. From a quick skim, I'd be way more comfortable making a proper integration with this than mapbox-gl-draw. Let me know if you have any plans around snapping that might be relevant!

Hey @dabreegster thanks for your interest and feedback! It seems like a great use case for terra-draw. The LineString mode does have some builtin basic snapping functionality which can be turned on by setting snapping: true in the mode constructor.
To snap to some other layer, you may need a custom mode or two but I don't think it's an unreasonable amount of work. Terra Draw has the concept of 'render modes' which are essentially layers that can be rendered to the map, but don't explicitly have a way of adding new geometries to them via drawing. In theory you could create a render mode with no styling if you wanted an invisible layer to snap to via another custom linestring layer. There may be some complexities/performance problems with the amount of data but if it's not being rendered this may not be a limiting problem.
I am looking to implement some documentation around writing custom modes that would probably make this process easier - hopefully I can get round to that soon!

I'm tracking this in https://github.com/dabreegster/route_snapper/issues/17. Haven't forgotten it, but not time yet to try it out. I am separately considering moving one of my active projects, https://github.com/acteng/atip, to terra-draw, and if that happens, I'll definitely prioritize this

Hi @JamesLMilner I was wondering if you have any updates on whether you intend on incorporating a route snap mode into terra-draw any time soon. Was thinking it's a great use case for terra-draw and I'll love to hear whether you have any plans on doing so!

Hey @howllian27 - At the moment there are no plans to support this directly in Terra Draw, but I may look and see if the route snapping mode created for https://github.com/JamesLMilner/terra-draw-route-snap-mode could be turned into it's own package for others to use. Hope this helps!

Just mentioning that the prototype mentioned in the first comments here is a proper NPM package now, https://github.com/dabreegster/route_snapper/. The "new route" tool at https://acteng.github.io/atip/pipeline/scheme.html?authority=TA_North+Lincolnshire is a better demo than the one linked from NPM. Gluing to Terra Draw as a frontend could still be a nice idea.

@dabreegster @howllian27 I'm going to close out this as it's been open for quite a while without much movement. I think there is a nice example in the route snapping mode repo and I hope that provides enough guidance on how to implement a custom mode that could accommodate the behaviour required here!
Always happy to reopen or let someone create a new issue if it needs discussing in more detail.

@JamesLMilner Thank you for sharing the route snapping mode. Based off that and the standard terradraw linestring mod i was able to put together a tool that allows drawing polygons with geometry snapped to topography, something i wanted to have in my app for some time.

@risusinf that's amazing to hear. What are you working on exactly? 🙂

@JamesLMilner Just an osm based map. It's getting on the advanced side though. I built js functions that handle user input to terradraw so i can imitate things like undo actions, geometry merging/dissecting when polygons overlap, topography tracing, some other features. Should work on that more, but currently stuck with osm tags processing and styling. Ready templates i could find don't fit my needs and i find them quite horribly made.


Update animated gif in repo entry page

We need to update the animated gif in the entry page of the GitHub repo: https://github.com/Microsoft/vscode/blob/master/README.md

This looks like the code.visualstudio.com home page image which @bgashler1 is in the process of updating.

Looks like it was updated.


Update sbt-ci-release to 1.5.11

Updates com.geirsson:sbt-ci-release from 1.5.7 to 1.5.11.
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!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.geirsson", artifactId = "sbt-ci-release" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.geirsson", artifactId = "sbt-ci-release" }
}]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #370.


Remove desktop for now

Code Coverage

File
Coverage [0.00%]
:x:

composeApp/src/commonMain/kotlin/shared/state/TextFieldState.kt
0.00%
:x:

Total Project Coverage
0.32%
:x:


Build issue tenserflow 2.11.0 for tensorflow quantum

Issue type
Build/Install
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
tf 2.11.0
Custom code
Yes
OS platform and distribution
Mac OS M1
Mobile device
No response
Python version
3.9
Bazel version
bazel 5.3.0
GCC/compiler version
Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1) Target: arm64-apple-darwin22.5.0 Thread model: posix
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Trying to build package using bazel compiler, as given here.  Fails to build the file. Gives the following output mentioned in log output, after running the given command.
Output:

Standalone code to reproduce the issue
bazel build -c opt --cxxopt="-O3" --cxxopt="-march=native" --cxxopt="-std=c++17" --cxxopt="-D_GLIBCXX_USE_CXX11_ABI=1" //tensorflow/tools/pip_package:build_pip_package

Relevant log output
ERROR: /private/var/tmp/_bazel_gb/4de1e45218bdc87c870302861e9b2675/external/boringssl/BUILD:161:11: Compiling src/crypto/x509/t_x509.c [for host] failed: (Exit 1): cc_wrapper.sh failed: error executing command external/local_config_cc/cc_wrapper.sh -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -Wunused-but-set-parameter -Wno-free-nonheap-object -fcolor-diagnostics ... (remaining 44 arguments skipped)
external/boringssl/src/crypto/x509/t_x509.c:321:18: error: variable 'l' set but not used [-Werror,-Wunused-but-set-variable]
    int ret = 0, l, i;
                 ^
1 error generated.
Target //tensorflow/tools/pip_package:build_pip_package failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 709.728s, Critical Path: 52.92s
INFO: 4419 processes: 1508 internal, 2911 local.
FAILED: Build did NOT complete successfully

Hi @Gaurang-Belekar ,
This issue observed in Tf2.12v also. Its due to issue in compilation problem in boringssl and it was fixed in nightly at that time. Please refer to similar issue #60191 which resolved subsequently in nightly version.
Could you please check the build with Tf2.13v as the changes might not be cherry picked in Tf2.12 and earlier.
Thank you!


Not listed yet: www.usenix.org

The website at this url is unlisted: https://www.usenix.org/conference/srecon17americas/speaker-or-organizer/tameika-reed-women-linux-0

Thanks for your help, the website is in the latest pull request.


Metadata description in the docs

Please confirm that this PR has done the following:

[ ] Add log message when nan values are being dropped on initialisation
[ ] Tests Added
[x] Documentation Added
[ ] Description in RELEASE_NOTES.md Added

Adding to RELEASE_NOTES.md (remove section after adding to RELEASE_NOTES.md)
Please add a single line in the release notes similar to the following:
- (#XX)[http://link-to-pr.com] Added feature which does something

Description of PR
Improves the documentation about the metadata table.

Coverage decreased (-0.3%) to 85.067% when pulling 722322947f812cfe44de4645e3b0a21bbffbb44e on znicholls:meta-docs into a4e266c93f531f94285311fd354aaa881aa403c4 on IAMconsortium:master.

Thanks @znicholls for starting the discussion.
You are correct, meta is (so far) only intended for categorization and quantitative indicators at the model/scenario level - primarily due to consideration that at this level, it's fairly obvious we need something. Otherwise, you end up with a table with as many rows as data (after all, why have meta-tags only for variables but not years or region or units) and with potentially a lot of duplicate rows.
Can you describe a use case where you would need tagging at some intermediate level, which would not be covered by a proper workflow or provenance tracking (see #287)?

Otherwise, you end up with a table with as many rows as data (after all, why have meta-tags only for variables but not years or region or units) and with potentially a lot of duplicate rows.

That's true, but the duplicats rows are only a problem because of the long form data structure of pyam. Given it's timeseries based, storing things internally in wide format (like what comes out of .timeseries()) would save a lot of that duplication would make this meta much more feasible. You can still cast to long when you need to for whatever operation. That's the approach we've taken in scmdata and it's way quicker for filtering and means you can have a timeseries by timeseries metadata table without it becoming stupidly big.

Can you describe a use case where you would need tagging at some intermediate level, which would not be covered by a proper workflow or provenance tracking (see #287)?

Adding an extra column, climate_model for determining where e.g. temperature output came from. That would solve the problem of the mangled variable names like in the SR1.5 database.

Given it's timeseries based, storing things internally in wide format would save a lot of that duplication would make this meta much more feasible.

But what would happen if you want to add meta-tags for particular years? For example, keeping track that year 2025 was interpolated during the postprocessing (with a particular method), whereas 2020 and 2030 were reported by the modeling team. I'm hesitant to implement a feature that will support some additional dimensions but not all.

Adding an extra column, climate_model for determining where e.g. temperature output came from. That would solve the problem of the mangled variable names like in the SR1.5 database.

It's already possible to add extra-columns in data, e.g., to distinguish between multiple Temperature|Global Mean timeseries for each scenario. What additional climate-model-specific information would need to be stored in meta?

I think I finally get the data model now. I've re-written what do you think?

I'm hesitant to implement a feature that will support some additional dimensions but not all.

Yep good call (I had misunderstood how pyam works).

What additional climate-model-specific information would need to be stored in meta?

Yep also good call (again I had misunderstood how pyam works).

Hi @znicholls and @danielhuppmann great discussion! As a new user, I was already wondering about the designed purpose/use of the meta column.
I took the liberty to restructure the paragraphs a bit, but left all information the same. To me, all this info seems very relevant and should be in the docs.
I'm only a bit torn as to whether the bit about data point-by-data point metadata should be described under the meta column, as it actually is used under data .
So what about splitting up this docs section into two? A "meta column: scenario-model specific metadata" section and a "Data point specific metadata" section? (feel free to change these headers into something you deem more clear.)

thanks @jkikstra, good call to separate some of this into an own data section. If you all don't mind, I'd like to do another round of editing/rewriting later today or tomorrow.

If you all don't mind, I'd like to do another round of editing/rewriting later today or tomorrow.

Works for me I won't touch it until that's done then

@znicholls, I think that this is good to be merged after you looked https://github.com/znicholls/pyam/pull/4


[V1] Cherry pick dock updates to v1

From https://github.com/microsoft/pxt-arcade/pull/6377
@kiki-lee do you need any others?
Steps I ran to make this (using cli)
cd pxt-arcade
git fetch origin
git checkout origin/v1 -b some-branch-name
git cherry-pick 3fac852
git push origin some-branch-name

Then, you open a PR with the new branch
You can get the hash of the commit at the bottom of your PR after you merge it:

Also, when you open the PR, you'll need to change the base branch to v1:

Oh, so it looks like when I'm making the PR for my merge, I can select the v1 option from the dropdown and then when I squash things should go where they need to be?

@kiki-lee correct! It's exactly the same as making a PR against master, just the result ends up in a different branch


[Feature] Server side Infinite scrolling List Widget

Summary:
As a dev, I should be able to load a list data in an async manner with infinite scrolling pagination.
Requirements:

Title
Description
Comment

Pagination type
As a dev, I should be able to choose if the list should be paginated with number controls or load with infinite scrolling
This is still under debate, It might not make sense to add another widget for this and also the list component commonly operates with infinite scrolling

Items per page
How many items to render per api call. The size of the view port could automatically decide this
-

Loading threshold
Defines the number of items pending in the viewport from the un-scrolled area to identify the time to call the api again.
-

 Front conversations

@Nikhil-Nandagopal @areyabhishek we think this should be a separate widget to avoid complications and maintenance issue. Do you think it makes sense from the UX standpoint?

@somangshu I don't think that makes sense because this widget will be the most widely used form of the list widget. I also don't see why implementation should differ between these 2. In fact I am ok killing this way of pagination in favor of only scroll method because this method of pagination was a hack we introduced but list widgets generally don't work like this

@riodeuno @jsartisan can you explain why we choose to create a separate widget for this technically?
Also I think I would agree to this being the default and only pagination for the list widget! Does this solve out problem better? Although this will required some additional feedback / poll.

We will go with implementing the virtualized list in the same widget. Thinking to toggle the virtualization with a toggle control.

@somangshu is there another issue for the list widget to scroll instead of pagination for all data on the client side? This small fix would actually improve the UX for a lot of list implementations.
Infinite scrolling can be done after we implement it for the table widget.

@dilippitchika I did not catch what small fix you are talking about!
I am also not sure if actually the table widget implementation for infinite scrolling will fix it for the list widget as well.
I think once we iron out the initial big problems, we will be able to get to these

Agreed I am just using this to kick off a discussion, what i was referring to was list ONLY having pagination today and no scroll. If i don't want pagination, there's no way out of it.
Most list implementations have scroll and not pagination. However, we don't offer that choice to the user since we auto-paginate. We should allow a way to switch off pagination, because all items will simply show up in a scrollbar.
Infinite scrolling design i believe will be similar to what we do in the table widget, not from an implementation POV. This is what i was referring to.

I agree with @dilippitchika. Pagination should be a choice, not a forced option. I just got stuck because of this while creating a comment feed. How can I increase the number of records per page on a list? @areyabhishek @Nikhil-Nandagopal

@divyadeepsawhney thanks for the opinion :)
As Dilip pointed out the pagination is automatic today, So to have more records per page you just need to increase the height of the list widget. The list widget tries to accommodate as many items as possible on the viewport based on its height, Post that it will paginate the rest of the items. Hope this answers your questions!

My thought is not only about asynchronous reloading. But also with data that can no longer be displayed in the list or table. With the list:  If there are more data than are displayed we can automatically display pagination. There is no possibility here to make a list scrollable which I would prefer.

The same applies to the tables. Here I would also like to see a scroll functionality. Although you can deactivate the pagination here. But that doesn't necessarily make the table scrollable. Whereby here in general would have to be looked over again. Here is an example for 10 records. Here should be just when pagination is active no longer scrollable or?

@lublak we are tackling it for the table widget here - https://github.com/appsmithorg/appsmith/issues/13180
Regarding the list widget yes we plan to support scrolling as well here

@lublak by the way if you're not using server side pagination, you can still achieve this in the table by turning the server side pagination option on. It will make the table scrollable but there will be performance issues if you're loading a very large data set

@SatishGandham @ashit-rath internal doc for PRD, we will pick up design this week. https://www.notion.so/appsmith/List-widget-infinite-scrolling-403e612eb40a4db6a1a2c1b64037e7ca

[Infinite scrolling designs ]
figma
loom & reasoning wip

Is there approximate date when this feature might be done?

@ogbofjnr we are trying to pick this up early next year to deliver.

how did this end?

I would like to have scrollable List Widget too

Hello Devs! I need help. How to create a vertical scrolling similar to a chat box for a live support system.

Hello! Scrollable List Widget would be a nice feature. Pagination is just unusable when loading many records.

@laioncenet vertical scroll is not yet supported.
@wawa79 you can paginate lots of records following this guide https://docs.appsmith.com/build-apps/how-to-guides/Setup-Server-side-Pagination-on-List

When will the inverted vertical scrolling feature for the table widget be launched?

@laioncenet I'm not clear on what you mean by inverted vertical scrolling feature.

Okay, I'll give you an example: in the Retool tool, you have the option to create applications, and there is a template called AI Chat. Editing this model, the agent finds a chat message project, thus, when several messages are sent, a vertical scroll bar is created and it always descends automatically as messages are sent or the AI agent sends them to us.
So I think that in appsmith, would there be an option for the scroll bar instead of going up and down automatically? Do you have an estimate of whether this feature will be released?
I managed to create a chat app integrated with the WhatsApp Business API. However, the scroll bar keeps going up, so all new messages keep coming up.
The correct thing is for the messages to keep coming down.
What is the solution?

Any news about this feature ? I would be a very nice feature... Waiting for!

you can expose an event that reaches the bottom of the list element?
onBottomReached, and maybe allow an offset from bottom.
This can be achieved with Observation API.

I agree with the previous comments about pagination, it's really clunky when you need to scan through a lot of items in a list. Infinite scroll is much more natural.


Not working against a fresh OpenShift cluster

I'm running OpenShift Origin on four bare metal CentOS 7 servers. I've been trying to get this to work for days now.
Always this happens:
Using OpenShift CLI.
NAME        STATUS    AGE
glusterfs   Active    3m
Using namespace "glusterfs".
Waiting for deploy-heketi pod to start ... OK
Found node bart on cluster 7eaa67c5087f92b7bd64b349d44ead8c
Found device /dev/loop0
Creating node lisa ... Unable to create node: Unable to execute command on glusterfs-6ddhz:
Creating node marge ... Unable to create node: Unable to execute command on glusterfs-6ddhz:
Creating node homer ... Unable to create node: Unable to execute command on glusterfs-6ddhz:
Error loading the cluster topology.
Please check the failed node or device and rerun this script using the --load option.

I verified that the names of the "manage" nodes are exactly what they are in my oc nodes output. I've added the privileged SCC to the default and router service accounts in the namespace. At this point I'm not sure what else to try.

I traced the issue to:

Heketi runs gluster peer probe ip_address_here
The glusterd daemon segfaults (!!)

Logs here:
glusterd[51]: pending frames:
systemd[1]: glusterd.service: main process exited, code=killed, status=11/SEGV

@paralin can you specify the gluster image name and tag?


[Feature][UI Next][V1.0.0-Beta] Change the css style usage of the blue number flopper in the alarm center to css variables.

Purpose of the pull request
close #9510

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #9511 (75a09b7) into dev (8a2fbd2) will increase coverage by 0.00%.
The diff coverage is 7.14%.

:exclamation: Current head 75a09b7 differs from pull request most recent head 2bd3ba1. Consider uploading reports for the commit 2bd3ba1 to get more accurate results

@@            Coverage Diff            @@
##                dev    #9511   +/-   ##
=========================================
  Coverage     39.90%   39.90%
+ Complexity     4418     4416    -2
=========================================
  Files           826      826
  Lines         33258    33252    -6
  Branches       3683     3683
=========================================
- Hits          13272    13270    -2
+ Misses        18754    18752    -2
+ Partials       1232     1230    -2

Impacted Files
Coverage Δ

.../server/worker/processor/TaskExecuteProcessor.java
0.00% <0.00%> (ø)

...ler/server/worker/processor/TaskKillProcessor.java
0.00% <0.00%> (ø)

...uler/server/worker/runner/WorkerManagerThread.java
0.00% <0.00%> (ø)

...er/server/builder/TaskExecutionContextBuilder.java
72.09% <100.00%> (+0.66%)
:arrow_up:

...org/apache/dolphinscheduler/remote/utils/Host.java
37.77% <0.00%> (-2.23%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.11% <0.00%> (-1.41%)
:arrow_down:

Continue to review full report at Codecov.

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


getting error on 'make build'

# github.com/atahani/golang-rest-api-sample/controller/user
controller/user/user_controller.go:337: invalid operation: token.Claims["exp"] (type jwt.Claims does not support indexing)
controller/user/user_controller.go:338: invalid operation: token.Claims["uid"] (type jwt.Claims does not support indexing)
controller/user/user_controller.go:339: invalid operation: token.Claims["roles"] (type jwt.Claims does not support indexing)
controller/user/user_controller.go:340: invalid operation: token.Claims["name"] (type jwt.Claims does not support indexing)
controller/user/user_controller.go:341: invalid operation: token.Claims["img"] (type jwt.Claims does not support indexing)
controller/user/user_controller.go:342: invalid operation: token.Claims["aid"] (type jwt.Claims does not support indexing)
controller/user/user_controller.go:343: invalid operation: token.Claims["tid"] (type jwt.Claims does not support indexing)
Makefile:29: recipe for target 'build' failed
make: *** [build] Error 2

Golang version : 1.6

solved by using jwt-go version under v3.0.0

hi @Misaghlb
i used glide as package management, when you import package to project as dependency glide store HEAD hash of package repository as version in glide.lock so when you run glide install command, glide get all of the dependencies in compatible version and locate inside ./vendor/ directory.
i think you have different version of github.com/dgrijalva/jwt-go package in your $GOPATH and compile with incompatible version.

i will update whole sample project at this weekend 😊

yes i solved with by jwt-go version 2.7 and build it
but i think its not working correctlly so im waiting for your updates :wink:

by replacing this for claims the build problem will gone (jwt-go latest ver) :
	token.Claims = jwt.MapClaims{ 		"exp": accessToken.ExpireAt.Unix(), 		"uid": u.Id.Hex(), 		"iat": time.Now().Unix(), 		"roles": u.Roles, 		"name": u.DisplayName, 		"img": u.ImageFileName, 		"aid": trustedAppId.Hex(), 		"tid": accessToken.Id.Hex(), 	}


Backport "Don't leave underspecified SAM types in the code" to LTS

Backports #19461 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-19520.
Reference


Cannot connect to any kafka brokers

Hi,
Using the example without zookeeper.
$hostList = '<IP_ADDRESS>:2181';

$produce = \Kafka\Produce::getInstance(null, null, $hostList);
// get available partitions
$partitions = $produce->getAvailablePartitions('MyFirstTopic1');
var_dump($partitions);
// send message
$produce->setRequireAck(-1);
$produce->setMessages('test', 0, array('test11111110099090'));
$produce->setMessages('test', 0, array('test22222220099090'));
$result = $produce->send();
var_dump($result);

The PHP Error:
An uncaught Exception was encountered

Type: Kafka\Exception

Message: Could not connect to any kafka brokers

Filename: ../vendor/nmred/kafka-php/src/Kafka/MetaDataFromKafka.php

Line Number: 202

The zookeeper log file:
[2017-03-27 15:59:25,305] INFO Accepted socket connection from /<IP_ADDRESS>:56502 (org.apache.zookeeper.server.NIOServerCnxnFactory)
[2017-03-27 15:59:25,309] WARN Exception causing close of session 0x0 due to java.io.IOException: Unreasonable length = 223181126 (org.apache.zookeeper.server.NIOServerCnxn)
[2017-03-27 15:59:25,309] INFO Closed socket connection for client /<IP_ADDRESS>:56502 (no session established for client) (org.apache.zookeeper.server.NIOServerCnxn)

Using Kafka version: <IP_ADDRESS>
Thanks

I downgraded to <IP_ADDRESS>.
[root@localhost kafka_2.11-<IP_ADDRESS>]# [2017-03-27 06:49:59,685] INFO Accepted socket connection from /<IP_ADDRESS>:56506 (org.apache.zookeeper.server.NIOServerCnxnFactory)
[2017-03-27 06:49:59,693] WARN Exception causing close of session 0x0 due to java.io.IOException: Unreasonable length = 74737011 (org.apache.zookeeper.server.NIOServerCnxn)
[2017-03-27 06:49:59,693] INFO Closed socket connection for client /<IP_ADDRESS>:56506 (no session established for client) (org.apache.zookeeper.server.NIOServerCnxn)

Looks like same problem.

Following the instructions from here:
http://stackoverflow.com/a/27194583

and instead of using <IP_ADDRESS>, I am using the IP of the server.
I'm now getting:
array(1) { [0]=> int(0) } array(1) { ["test"]=> array(1) { [0]=> array(2) { ["errCode"]=> int(0) ["offset"]=> int(0) } } }

Hi, @no1youknowz
Please refer https://github.com/weiboad/kafka-php/blob/master/example/ProduceWithoutZooKeeper.php
// use kafka host
// $hostList = 'xxxx:9092'; that is kafka broker host:port
$produce = \Kafka\Produce::getInstance(null, null, $hostList);

// use zookeeper host connect kafka broker
// one param: zookeeper host:port
// two param is zk connect session timeout
$produce = \Kafka\Produce::getInstance('<IP_ADDRESS>:2181', 3000);

I'm doing exactly that!
I'm now going to compile the ZooKeeper php extension and then test with <IP_ADDRESS>.
Before I close the ticket.

So with 0.9.x seems to work fine.
With <IP_ADDRESS> I am getting:
Type: Kafka\Exception\SocketEOF

Message: Unexpected EOF while reading 4 bytes from stream (no data)

Filename: /mnt/hgfs/Share/development/ecomzilladev/application/vendor/nmred/kafka-php/src/Kafka/Socket.php

Line Number: 286

There are 3 outstanding tickets the same issue!
https://github.com/weiboad/kafka-php/issues/6
https://github.com/weiboad/kafka-php/issues/69
https://github.com/weiboad/kafka-php/issues/81
How to fix?
Thanks


SDK parameters strategy

Description:
We probably have to rethink were to put all the parameters from each of the SDK modules since new ones are being added to the existing ones (stake/gov/auth/slashing/distribution).
Do we want to display them all ? In separate tabs ? all in the same tab ? where ? my guess is maybe within Governance since we'll have the parameter change proposals in the future.

we should display parameters that are relevant for delegators (our users) in the params tab for staking / gov ... why does this need to change?

Because there are other parameters as well that can be changed by delegators

they can all go into the same parameters view. if they are special we can mark them somehow.


[Autoloader] include_once is not needed on Autoloader::loadClass() with no namespace

include_once is not needed on get class with no namespace for Autoloader::loadClass() call as it try to get app\Config that autoloaded.
Checklist:

[x] Securely signed commits

The test for it is already in :
https://github.com/codeigniter4/CodeIgniter4/blob/5fc58f91c306c27c51728d74c2f92b236788a0fc/tests/system/Autoloader/AutoloaderTest.php#L175-L189

Actually, it should just return false as it somehow handled by Fabricator, iirc, it previously was needed as cause class redeclare, @paulbalandan what do you think of non - namespace return false?

@paulbalandan looking at the previous change on this, this was introduced for faster load due to loadLegacy() at https://github.com/codeigniter4/CodeIgniter4/pull/3413 .
As the loadLegacy() no longer exists, it is safe to just return false early. On looking on Config class, eg: 'Modules' config, there is config('Modules') usage for that.
I updated at https://github.com/codeigniter4/CodeIgniter4/pull/4756/commits/81c54543b106d83276eab2dcd761b04100beaf88

@MGatner @paulbalandan thank you for the review.