Mitigate UseIIS conflict

Today CreateDefaultBuilder calls UseKestrel and then it calls UseIIS for conditional in-proc support. Each of these register IServer and the last one wins.
The conflict comes when you call UseKestrel again after CreateDefaultBuilder to set some options. This works fine when you run self-host (e.g. on your Mac), but when switch to IIS in-proc (like publishing to Azure) Kestrel replaces IIS in-proc and the server fails to start.
There's a workaround today where instead of calling UseKestrel you can call
                builder.ConfigureServices(services =>
                {
                    services.Configure<KestrelServerOptions>(options =>
                    {
                        // ...
                    });
                })

Proposal: Promote the workaround to a top level API
                builder.ConfigureKestrel(options =>
                {
                    // ...
                })

This becomes a drop in replacement for anyone already calling UseKestrel. Discovery would be a problem still, but we could update our samples to help with that.
@shirhatti @halter73 @pakrym @jkotalik

Would we deprecate the UseKestrel overload with an options configuration callback?

That's pretty agressive. Maybe in 3.0 if we saw a lot of people run into this conflict?

I'm with Chris on this. No need to deprecate it unless it proves to be a pit of failure


Download Button unresponsive

When clicked, download button does not prompt user to save .ics file.

Thanks for bringing this to our attention! There's a PR out for this now and we will have it deployed shortly.

The fix for this has been deployed 🚢


Web Components:  [Lead space] Structure and variant overhaul Visual QA

User Story

As a [user role below]:
DDS Designer and Developer

I need to:
test the Lead space

so that I can:
be sure the developer's code correctly works as designed

Additional Information

See the Epic for the Design and Functional specs information
Designer will be added as a reviewer to the Pull Request.  Visual QA reviewer will be notified by Github when review is needed.
Creating a QA bug
Dev Issue (#5796 )
Storybook link provided by engineer in comments below

Tasks
Ensure the Lead space component has been tested and reviewed by design + dev.
Acceptance criteria

[ ] Pattern is reviewed by designer and design defects are identified
[ ] New issues are opened to log individual defects
[ ] Screenshots of as-is and to-be design should be added to the issue when needed
[ ] Developer associated with building the pattern is assigned to the new issues
[ ] Appropriate labels are added to the issue  eg.  bug, dev

Closing as the visual QA will be handled during the PR approval process.


Accreg

updated register and login. Got links to work for other pages.

made more updates and will make a new PR


MQTT -> Connect EV3 to Broker as a Subscriber (HiveMQ)

Im trying to connect my EV3 to the Broker on my Laptop.
I am using the gradle-template so what I basically do is deploying my Subscriber Class as a gradle command and then remoteBrickRun. If I execute the class normally on the IDE, the subscription on the Broker is successfull. I am using the Broker from HiveMQ.
Here is my Code Snippit:
public class SubscriberEV3 extends Thread{
    static Mqtt5BlockingClient client;
    static Mqtt5BlockingClient clientError;

    public static void main(String... args) throws IOException {
        client = Mqtt5Client.builder()
                .identifier("Subscriber")
                .serverHost(InetAddress.getByName("<IP_ADDRESS>"))
                .serverPort(1883)
                .buildBlocking();
        client.connect();

Hi @Kub-sn, what is the problem? do you have any issue in the console?
Juan Antonio

Here are the error messages:
ev3dev#1|Exception in thread "main" com.hivemq.client.mqtt.exceptions.ConnectionFailedException: io.netty.channel.ConnectTimeoutException: connection timed out: /<IP_ADDRESS>:1883
ev3dev#1|	at com.hivemq.client.internal.mqtt.MqttBlockingClient.connect(MqttBlockingClient.java:91)
ev3dev#1|	at com.hivemq.client.mqtt.mqtt5.Mqtt5BlockingClient.connect(Mqtt5BlockingClient.java:64)
ev3dev#1|	at example.SubOnly.main(SubOnly.java:21)
ev3dev#1|Caused by: io.netty.channel.ConnectTimeoutException: connection timed out: /<IP_ADDRESS>:1883
ev3dev#1|	at io.netty.channel.nio.AbstractNioChannel$AbstractNioUnsafe$1.run(AbstractNioChannel.java:267)
ev3dev#1|	at io.netty.util.concurrent.PromiseTask$RunnableAdapter.call(PromiseTask.java:38)
ev3dev#1|	at io.netty.util.concurrent.ScheduledFutureTask.run(ScheduledFutureTask.java:127)
ev3dev#1|	at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:163)
ev3dev#1|	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:404)
ev3dev#1|	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:474)
ev3dev#1|	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:909)
ev3dev#1|	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
ev3dev#1|	at java.base/java.lang.Thread.run(Thread.java:834)
ev3dev#1|
ev3dev#1|real	1m42.075s
ev3dev#1|user	0m0.317s
ev3dev#1|sys	0m0.295s

Task :remoteBrickRun
Failed command ev3dev#1 with status 1: time brickrun -- java -jar /home/robot/template_project_gradle-2.6.0.jar

Task :remoteBrickRun FAILED

Here are the logs of the deployAndBrickRun command:
2020-08-10 22:52:01,942 [ 436715]   INFO - s.plugins.gradle.GradleManager - Instructing gradle to use java from C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.1\jbr
2020-08-10 22:52:01,955 [ 436728]   INFO - s.plugins.gradle.GradleManager - Instructing gradle to use java from C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.1\jbr
2020-08-10 22:52:02,072 [ 436845]   INFO - xecution.GradleExecutionHelper - Passing command-line args to Gradle Tooling API: --init-script C:\Users\kubiy\AppData\Local\Temp\ijresolvers.gradle
2020-08-10 22:52:05,383 [ 440156]   INFO - rationStore.ComponentStoreImpl - Saving appFileTypeManager took 11 ms
2020-08-10 22:52:05,467 [ 440240]   INFO - rationStore.ComponentStoreImpl - Saving Project (name=template_project_gradle, containerState=ACTIVE, componentStore=C:\Users\kubiy\Desktop\EV3-Maven\template-project-gradle) RunManager took 15 ms
2020-08-10 22:56:27,969 [ 702742]   INFO - rationStore.ComponentStoreImpl - Saving appFileTypeManager took 12 ms, FindSettings took 66 ms, GeneralSettings took 14 ms, XDebuggerSettings took 56 ms
2020-08-10 22:56:28,023 [ 702796]   INFO - rationStore.ComponentStoreImpl - Saving Module: 'template_project_gradle'ExternalFacetManager took 13 ms
2020-08-10 22:56:28,214 [ 702987]   INFO - rationStore.ComponentStoreImpl - Saving Project (name=template_project_gradle, containerState=ACTIVE, componentStore=C:\Users\kubiy\Desktop\EV3-Maven\template-project-gradle) RunManager took 36 ms, com.intellij.coverage.CoverageDataManagerImpl took 11 ms, libraryTable took 31 ms
2020-08-10 22:58:38,972 [ 833745]   INFO - rationStore.ComponentStoreImpl - Saving appDaemonCodeAnalyzerSettings took 12 ms, FindSettings took 57 ms, JavaCodeFoldingSettings took 13 ms
2020-08-10 22:58:39,121 [ 833894]   INFO - rationStore.ComponentStoreImpl - Saving Project (name=template_project_gradle, containerState=ACTIVE, componentStore=C:\Users\kubiy\Desktop\EV3-Maven\template-project-gradle) Kotlin2JvmCompilerArguments took 18 ms
2020-08-10 22:59:22,375 [ 877148]   WARN - ConfigurableExtensionPointUtil - ignore deprecated groupId: language for id: preferences.language.Kotlin.scripting
2020-08-10 22:59:23,755 [ 878528]   INFO - ateSettings.impl.UpdateChecker - failed to load plugin descriptions from : Malformed URL:
2020-08-10 22:59:54,870 [ 909643]   INFO - rationStore.ComponentStoreImpl - Saving appKeymapManager took 16 ms
2020-08-10 22:59:54,933 [ 909706]   INFO - rationStore.ComponentStoreImpl - Saving Project (name=template_project_gradle, containerState=ACTIVE, componentStore=C:\Users\kubiy\Desktop\EV3-Maven\template-project-gradle) RunManager took 15 ms
2020-08-10 23:01:35,520 [1010293]   INFO - j.ide.actions.RevealFileAction - Exit code 1
2020-08-10 23:01:42,614 [1017387]   INFO - rationStore.ComponentStoreImpl - Saving Project (name=template_project_gradle, containerState=ACTIVE, componentStore=C:\Users\kubiy\Desktop\EV3-Maven\template-project-gradle) RunManager took 14 ms

Hi @Kub-sn,
did you Google a bit about this exception:
ev3dev#1|Exception in thread "main" com.hivemq.client.mqtt.exceptions.ConnectionFailedException: io.netty.channel.ConnectTimeoutException: connection timed out: /<IP_ADDRESS>:1883
ev3dev#1|	at com.hivemq.client.internal.mqtt.MqttBlockingClient.connect(MqttBlockingClient.java:91)
ev3dev#1|	at com.hivemq.client.mqtt.mqtt5.Mqtt5BlockingClient.connect(Mqtt5BlockingClient.java:64)
ev3dev#1|	at example.SubOnly.main(SubOnly.java:21)
ev3dev#1|Caused by: io.netty.channel.ConnectTimeoutException: connection timed out: /<IP_ADDRESS>:1883

ConnectTimeoutException
Juan Antonio

Hi @jabrena
I did google the exception. Basically it means, as you already said, that the connection is not happening fast enough. I tried to increase the timeout on serverside, but I still get the same problem.

Remember that CPU/Memory capabilities at EV3 side is reduced, this is the main issue in the connection.

Well I actually have a PiStorms v2 too, so I hooked it up and tried it with that. I had half of a success:
My PiStorms could connect to the online Broker of HiveMQ easily. But I still get the same error if I try to connect to my broker on my notebook.
Do you have an idea what the problem could be?

I found the problem. Windows Firewall was blocking the necessary ports. I can now connect the Pistorms to the broker on my notebook. I don't know why but it does not work with the Lego EV3 Brick but I don't really care about that now. The Pistorms is just fine for what I need. Still thank you for your help

The point with EV3 is the computational capability. EV3 is nice but it is slow.
Enjoy with your project. If you build something interesting, share it!
cheers


Minor fixes

some small issues I saw while doing other things. See commit messages for details

seems this breaks test, will repoen after fixing...

I had created the kind of bug the additional javadoc warns against. Fixed now.


Version bump and Documentation Updates

Tidied up the docs
Updated MNIST example to match the version in the paper
Bumped Project.toml to v0.2.3

@JuliaRegistrator register

Comments on pull requests will not trigger Registrator, as it is disabled. Please try commenting on a commit or issue.


intel-compute-runtime: 20.33.17675 -> 20.34.17727

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/intel/compute-runtime/releases.
meta.description for intel-compute-runtime is: "Intel Graphics Compute Runtime for OpenCL. Replaces Beignet for Gen8 (Broadwell) and beyond."
meta.homepage for intel-compute-runtime is: "https://github.com/intel/compute-runtime"
meta.changelog for intel-compute-runtime is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 20.34.17727 with grep in /nix/store/a87p6ycicvf0cxh9zq3gh7s77n7s61wz-intel-compute-runtime-20.34.17727

directory tree listing: https://gist.github.com/99a98f2e2cab08bc20469e7a59ad1e76

du listing: https://gist.github.com/a716ebb9adb0b4e7ce692936c7057b91

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
intel-compute-runtime

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/a87p6ycicvf0cxh9zq3gh7s77n7s61wz-intel-compute-runtime-20.34.17727 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A intel-compute-runtime https://github.com/r-ryantm/nixpkgs/archive/c695c6d3a545232789a6fae8a81c1faf4d57e387.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/a87p6ycicvf0cxh9zq3gh7s77n7s61wz-intel-compute-runtime-20.34.17727
ls -la /nix/store/a87p6ycicvf0cxh9zq3gh7s77n7s61wz-intel-compute-runtime-20.34.17727/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:

intel-compute-runtime

Maintainer pings
cc @gloaming for testing.

@ofbrog eval

@GrahamcOfBorg eval

@ofborg eval

@ofborg eval


container PID namespace isolation with NVML

We'd like to deploy NVML-based monitoring tools to each task container, providing GPU information for ML engineers to take performance analysis.
However, if the PID namespace of the task container is isolated from the host machine's, we found that, even deployed within the container, the NVML (func nvmlDeviceGetComputeRunningProcesses) gives the PID(s) on the host machine. That makes the following info processing difficult because only the container PID namespace is visible to users (ML engineers).
Is there any solution to overcome this pid namespace isolation? Or does NVML has any plan to extend nvmlDeviceGetComputeRunningProcesses so that it can return pid in the container PID namespace?

How are you using the NVML api inside the container? Give more information about your system abd what what you are doing?

OS: CentOS Linux release 7.7.1908
Kernel: Linux kube-node-zw 3.10.0-1062.9.1.el7.x86_64 #1 SMP Fri Dec 6 15:49:49 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Docker Info:
Engine:
Version:          19.03.5
API version:      1.40 (minimum version 1.12)
Go version:       go1.12.12
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.2.10
nvidia:
Version:          1.0.0-rc8+dev
docker-init:
Version:          0.18.0
Nvidia GPU:
Model: 1080 Ti
Driver: 440.44
We are simply try to call nvmlDeviceGetComputeRunningProcesses and nvmlDeviceGetGraphicsRunningProcesses inside the container. The result shows:

Both functions, while running inside the container, can get all GPU processes on the host
The returned pids in the nvmlProcessInfo_t array are host PIDs instead of container PIDs.

Another instance here, where we are using Gitlab CI/CD to launch Docker containers (which have access to the host's NVML library). These fail to inspect their own process info because of PID namespace mismatches, and there's no easy way to launch those containers with --pid=host.


Validator not accepting single quotes around selector

Challenge http://freecodecamp.com/challenges/waypoint-bring-your-javascript-slot-machine-to-life has an issue.
Using $($('.slot')[0]).html(slotOne); with single quotes rather than double quotes is not accepted by the validator.

Hey @jangeroo  thanks for posting this.
We have a few duplicate postings of this so see #2522 for updates on this issue.
Please close this as a duplicate, and take a look at our Bug Report Guidelines.


feat(#855): data sources link added to the footer

Description
Data sources button added to the footer.

I had to move the state one level up but we can combine the 2 components in order to solve this, or we can leave it like that, I don't think it's a big issue for now.

discord username: İlker Sarıca#7138
closes #issue #855
Please describe your changes. Also describe your aim and content. Do not forget to list the dependencies required caused by those changes.
## Things to check before creating a PR

[ ] I have inspected my topic, checked.
[ ] If it is a core feature, executed detailed tests.

Creating PR rules

[ ] PR must be created for an issue with approved tag. Otherwise PR will be rejected.
[ ] Relevant issue number: The issue number related to PR must be attached to head of PR header, after prefix after prefix # must be attached in parenthesis. A header like this could be used "prefix(#issue_number): PR header" .
[ ] A descriptive and understandable title: The PR title should clearly describe the nature and purpose of the changes. The PR title should be the first thing displayed when the PR is opened. And it should follow the semantic commit rules. For example, a title like "docs(#issueId): Add README.md" can be used.
[ ] Related file selection: Only relevant files should be touched and no other files should be affected.
[ ] Format and Lint Suitability : The code should be made in accordance with a certain format standard and examined according to the lint rules.
[ ] Clean commit history: The commits where the changes are made should be clear and organized.
[ ] Opening PR when job is completed: PR should be opened when job is completed and sent for review by other team members.

Changes

[ ] A new feature (a change that adds a new feature, not a breaking change)
[ ] A new refactor (a change that is not a breaking change, that improves the readability or performance of the code)
[ ] A breaking change
[ ] Documentation change

How were these changes tested?
Please describe the tests you did to test the changes you made. Please also specify your test configuration.
Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

this task has already been made. please check this


⚠ Bofrost.at has degraded performance

In 0c4526f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2608 ms

Resolved: Bofrost.at performance has improved in ffff6f5 after 6 minutes.


Display two same file shares after pasting one file share

Storage Explorer Version: 1.11.0
Build: 20191106.4
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one storage account -> File Shares.
Select one file share and copy it.
Select another storage account -> Expand the storage account node -> Refresh the selected storage account.
Right click the File Shares node  -> Click 'Paste File Share'.
Check the result.

Expect Experience:

Paste successfully.
The File Share node is expanded.
Display the copied file share in the tree view.

Actual Experience:

Paste successfully.
The File Share node is expanded.
Display two same copied file share in the tree view.

More Info:

The copied file share shows well after manually refresh.
This issue doesn't reproduce for one blob container/ table.

Won't fix. Refreshing fixes the issue.


I am using the docker desktop but I am not able to access the device webcam.

Tell us about your request
A clear and concise description of what you want to happen or the change you would like to see
Which service(s) is this request for?
Let us know which product(s) you want this for?
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Are you currently working around the issue?
A clear and concise description of any alternative solutions or features you've considered or are using today.
Additional context
Add any other context or screenshots about the feature request here.

In windows I am using it so, I don't know whether I can do it or not. But when tried from a container I am not able to do it.

probably related to;

https://github.com/docker/roadmap/issues/511


List Perl as an environment

I'd like to improve several reports related to the Perl language and ecosystem, I cannot submit the form because the improvement form has the ecosystem as a required field, and there is not entry for "Other" or some such.
I suggest some combination of these:

Do not require an environment value to improve a report. The allows someone to improve a report even without a listed ecosystem.
Use an "Unknown" or "Other" value for the ecosystem, which could either be a virtual ecosystem or simply uses no value (the current state) but still allows the form field to be required but have a null value.
Add Perl as an ecosystem. There are plenty of people who would help categorize the backlog of over 200,000 uncategorized reports.
Even without a "Perl" ecosystem, I'd still like to improve the values in the ecosystem section of the improvement form to note the versions affected, package name, and so on. In general, any improvement, even if incomplete, is valuable.

This looks like the same request for C/C++ in #2963.

Fully support this

I support this!

Hi all, thanks for opening this issue! And wow that is a lot of 👍 interest!
We have opened an issue internally to look into this and see what we would need to do to support it.

@KateCatlin - I didn't see another way to get in touch with you, but as one of the people who maintains some of the Perl tools that do security audits for Perl projects, I'd be happy to talk to you about how the Perl community could help the GitHub Advisory Database. I'm happy to help as a volunteer in any way that I can be useful. If you want to take it offline, my email is on https://briandfoy.github.io .
For example, I maintain the CPAN Security Advisory, which is a secondary source of information that collates a bunch of different sources for our tools. Currently I'm adding the GitHub Advisory ID to anything we are tracking. As part of that, I've collected a bunch of information on affected versions, fixed versions, and a few other things for Perl advisories. It's something I've been doing for awhile. There are a lot of people that help, so we have a lot of information that can improve the GitHub reports.

Thanks for offering, Brian! We'd love to have this conversation!
I'm actually going to pass this over to @taladrane who is the leader of our Advisory Database Curation team, the team that would be most involved in taking on a new ecosystem to support. I'll let you two follow up and connect from here!

@KateCatlin @taladrane Hi! I'm one of the members of the CPAN Security Group (@CPAN-Security), and I'd like to support the effort by @briandfoy to add Perl as an environment in your advisory database.
Some of our goals are to help triage vulnerabilities with the Perl and CPAN community, secure the CPAN supply chain and help with the development of security related tooling. You can find more information about our efforts on https://security.metacpan.org/ or contact us on<EMAIL_ADDRESS>

We had a good meeting with @taladrane and part of her team today. I have some homework to pull together various things about how Perl modules work and so on so GitHub can see how that would fit into their workflow. This is progressing satisfactorily, and neither side is making any promises about anything. We're a long way from actual support, but I'm very happy that I even got the meeting and that they had lots of good questions. :)


[Doc] Modules Update

This pull request has automatically created by PinpointAutoBot
Updated Item : [elasticsearch-rest-high-level-client] has been updated
*Updated item list may differ if this PR is updated*

Codecov Report

Merging #7489 (60f8ea8) into master (f9eb3fe) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7489      +/-   ##
==========================================
- Coverage   38.63%   38.61%   -0.02%
==========================================
  Files        3443     3443
  Lines       98652    98652
  Branches    12129    12129
==========================================
- Hits        38110    38097      -13
- Misses      57596    57608      +12
- Partials     2946     2947       +1

Impacted Files
Coverage Δ

...rp/pinpoint/rpc/packet/ControlHandshakePacket.java
51.51% <0.00%> (-24.25%)
:arrow_down:

...vercorp/pinpoint/rpc/packet/ClientClosePacket.java
55.55% <0.00%> (-11.12%)
:arrow_down:

...tor/metric/datasource/DefaultDataSourceMetric.java
75.00% <0.00%> (-3.13%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
78.04% <0.00%> (-2.44%)
:arrow_down:

...pinpoint/profiler/monitor/DeadlockMonitorTask.java
18.60% <0.00%> (-1.17%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
78.39% <0.00%> (-1.05%)
:arrow_down:

...vercorp/pinpoint/web/vo/ApplicationAgentsList.java
69.23% <0.00%> (+2.56%)
:arrow_up:

...pc/server/LoggingServerMessageListenerFactory.java
100.00% <0.00%> (+10.00%)
:arrow_up:

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 f9eb3fe...60f8ea8. Read the comment docs.


[ObjectLiteral] Fails on DTO's implementing interfaces with properties

Expected
To compile with no errors
Actual
Compile error: ObjectLiteral doesn't support virtual methods
Steps To Reproduce
Deck

ObjectLiteral is simple object reflecting plane JS object, not 100% sure but i does not support fields like in your case, and hence conflict, since interface forces to use properties instead of fields.
Maybe that would be nice feature to request for Bridge team as long as there is no implementation provided for such properties.
Also as workaround you can use other ObjectLiteral class as base class to have those fields in place.

I remember seeing this error when making a clone version of AjaxOptions - it had ObjectLiteral and virtual fields, It compiled on bridge 1.1 then it threw exception on bridge 1.5, i just took it out.

The reason why ObjectLiteral is not supported interface members is that you can define two interfaces with same member name
interface IContract
{
    int Value {get;set;}
}

interface IContract2
{
    int Value {get;set;}
}

[ObjectLiteral]
class Contract : IContract, IContract2
{
	public int Value { get; set; }
    int IContract2.Value { get; set; }
}

and if you try to access to member via interface cariable then Bridge cannot resolve member correctly
IContract2 c2 = new Contract();
c2.Value;

May be can allow properties but forbid methods and explicit interface implementation. We will discuss it. Thanks for pointing us to it

Alright, thanks for the clarification.
Best regards.

Hi,
In Bridge 15.5, your original test case will be supported but interface must be marked as ObjectLiteral also

Thats awesome! thanks @vladsch for the reply


Blank screen on launch [w/ solution]

PoB Version: 1.4.69
OS: Windows 10 x64
Video: Nvidia GTX 1060 Mobile / Intel HD Graphics 630
When launching application, the screen appears completely blank. This issue appears to only occur when the Intel HD graphics are used (which is default after installing the program).
Using the Nvidia Control panel, you can change the preferred graphics processor to the Nvidia processor:

Once this is done the applications seems to work fine.
Noting this bug in hopes of including a way to default to high-performance graphics processor for dual graphics systems, and as a means of posting this solution for others with this issue. I could not find a troubleshooting forum for this tool, but if such a thing exists, let me know and I can post this fix there as well.

I've already made a fix for this, which will be out in the next update (1.4.70).

This has come up a few times; #672 describes the same general issue, so I will close this as a duplicate.


Fix: Newline at the end of text tag in code block is rendered as markup

Fixes issue https://github.com/aspnet/Razor/issues/485#issuecomment-214654609
@NTaylorMullen

:shipit:


queue.length always returns double the actual number of messages

Because messages get inserted in pairs of [id, contents], simply calling llen on the messages list returns twice as many messages as actually exist.

I'm assuming you're referring to queue.count and not queue.length


linker cannot access definitions in interrupts.asm

The linker cannot find the definitions of the isrs from the assembly file, inturrupts.asm.

C:\Users\Administrator\Documents\os\src>i686-elf-gcc -ffreestanding -c kernel/kernel.c  -o bin/kernel.o
i686-elf-gcc: error: drivers/ports.c: No such file or directory
i686-elf-gcc: error: drivers/screen.c: No such file or directory
C:\Users\Administrator\Documents\os\src>nasm boot/kernel_entry.asm -f elf -o bin/kernel_entry.o
C:\Users\Administrator\Documents\os\src>i686-elf-ld -o bin/kernel.bin -Ttext 0x1000 bin/kernel_entry.o bin/kernel.o --oformat binary
i686-elf-ld: warning: cannot find entry symbol _start; defaulting to 00001000
bin/kernel.o: In function isr_install': kernel.c:(.text+0x598): undefined reference to isr0'
kernel.c:(.text+0x5a8): undefined reference to isr1' kernel.c:(.text+0x5b8): undefined reference to isr2'
kernel.c:(.text+0x5c8): undefined reference to isr3' kernel.c:(.text+0x5d8): undefined reference to isr4'
kernel.c:(.text+0x5e8): undefined reference to isr5' kernel.c:(.text+0x5f8): undefined reference to isr6'
kernel.c:(.text+0x608): undefined reference to isr7' kernel.c:(.text+0x618): undefined reference to isr8'
kernel.c:(.text+0x628): undefined reference to isr9' kernel.c:(.text+0x638): undefined reference to isr10'
kernel.c:(.text+0x648): undefined reference to isr11' kernel.c:(.text+0x658): undefined reference to isr12'
kernel.c:(.text+0x668): undefined reference to isr13' kernel.c:(.text+0x678): undefined reference to isr14'
kernel.c:(.text+0x688): undefined reference to isr15' kernel.c:(.text+0x698): undefined reference to isr16'
kernel.c:(.text+0x6a8): undefined reference to isr17' kernel.c:(.text+0x6b8): undefined reference to isr18'
kernel.c:(.text+0x6c8): undefined reference to isr19' kernel.c:(.text+0x6d8): undefined reference to isr20'
kernel.c:(.text+0x6e8): undefined reference to isr21' kernel.c:(.text+0x6f8): undefined reference to isr22'
kernel.c:(.text+0x708): undefined reference to isr23' kernel.c:(.text+0x718): undefined reference to isr24'
kernel.c:(.text+0x728): undefined reference to isr25' kernel.c:(.text+0x738): undefined reference to isr26'
kernel.c:(.text+0x748): undefined reference to isr27' kernel.c:(.text+0x758): undefined reference to isr28'
kernel.c:(.text+0x768): undefined reference to isr29' kernel.c:(.text+0x778): undefined reference to isr30'
kernel.c:(.text+0x788): undefined reference to `isr31'
C:\Users\Administrator\Documents\os\src>nasm boot/bootsect.asm -f bin -o bin/bootsect.bin
C:\Users\Administrator\Documents\os\src>cd bin
C:\Users\Administrator\Documents\os\src\bin>copy /b bootsect.bin + kernel.bin os-image.bin
bootsect.bin
1 file(s) copied.
C:\Users\Administrator\Documents\os\src\bin>pause
Press any key to continue . . .

How are you compiling and adding interrupt.asm into the main os-image.bin?

Just don't worry about it and proceed on to lesson 20 and it should go away (It did for me)

yeah i am  getting the same problem. Once i try to link kernel.o with kernel_entry.o the ld linker shows the above message as if  it can't define the isr's or irq's. How can i define them? Also i'm not compiling the interrupt.asm at all, i just left it as is. I have reached lesson 23 with no luck and the same  message above.

Nevermind Got it figured out. It was a simple fix.

What was the fix? :)

@somerandomdev49 Did u update ur makfile?

I think fixed it.


render aeroway=heliport

Style change requested
aeroway=heliport does not currently seem to render.
i have just added the three Disneyland heliports to the map and they do not appear.
Where this is visible
the third heliport should show up on the west side of Disneyland in 1966
the first should appear in Tomorrowland in 1956-57, and the second should
appear slightly south of the first from 1957 to 1960.
https://www.openhistoricalmap.org/#map=17/33.81122/-117.91892&layers=O&date=1966-08-01&daterange=1824-01-01,2024-12-31

👋 these are now displayed, see demo style (will be incorporated into OHM in the next update)

https://openhistoricalmap.github.io/mbgl-timeslider/demo/#15.898/33.81069/-117.91742/1956,-4000-2023

now in the latest: https://www.openhistoricalmap.org/#map=17/33.81120/-117.91892&layers=O&date=1957-02-02&daterange=1824-01-01,2024-12-31


Win32-network package

Currently it includes more than we need:

asyncronous interruptible named pipes / file api - System.Win32.Asyn is a standalone module for Async api
syncronous interruptible named pipes / file api
events api, though we decided to use async api, instead of events, we might want to re-try it later.
completion port api

The last two are not used and we could remove it.

Oh and lets squash all these changes together. It's just adding new code so we don't need the development history on master.

bors r+


RequirementMachine: Random fixes from protocol minimization testing

With these changes, more validation tests pass with -requirement-machine-protocol-signatures=verify.
There were a couple of highly-invalid cases where the requirement machine's result was more "correct", so I hacked the GSB just to make a few tests pass. If these GSB changes cause some unforseen breakage, they can be reverted separately without issue.
Also, a couple of optimizations to the homotopy reduction algorithm. It's still too slow.

@swift-ci Please smoke test

Running source compat for the GSB changes.

@swift-ci Please test source compatibility


Add scan metadata to systems details table

Feel free to reject this one if you don't think its worth it ... but I think its nice to have :smiley_cat:

Codecov Report

Merging #14 (e868d27) into master (43242f5) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master     #14   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          33      33
  Lines         490     491    +1
  Branches      163     164    +1
======================================
- Misses        443     444    +1
  Partials       47      47

Impacted Files
Coverage Δ

src/Components/SysDetailsTable/SysDetailsTable.js
0.00% <0.00%> (ø)

src/operations/queries.js
0.00% <ø> (ø)

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 43242f5...e868d27. Read the comment docs.


Config for CSRF

The CSRF check blocks client-side POSTs. Maybe this is a good thing, maybe not. The point is that it is currently blocking me from uploading a file using the browser's fetch api.

I've just hit this issue again. It's sometimes useful to specify that CSRF is not a concern on certain URIs. It makes complete sense to block POSTs by default without a valid token, because this removes a lot of attack vectors, but certain URIs are simply required to be used with a single, naked POST request, without having to request a valid token first.

Another thing to configure is whether or not to use a shared token across all forms vs. or one token per form.
And the length of the token...
I'll add to this issue with other configuration requirements to deal with.

[ ] Non-protected URIs with wildcards
[ ] Token length
[ ] One token per Form/Page


Remove axios

Hey folks, took a look at completing the work to remove axios. Thought this a good way to explore the library a bit.

Adds a wrapper around the native fetch to type response data and handle errors thrown from bad responses.
setupAxios was replaced with setGlobalRequestOptions, which will allow custom settings on fetch.
Update to the tests to replace the axios mock. I used nock here to mock the HTTP requests themselves. Nock does not support Node's fetch yet and requires node-fetch at the moment.

I needed to add application/json headers to the requests, otherwise the lib wouldn't work with the validator in @cashu/cashu mint

Ah, that makes since. Axios adds those common headers.
Made a change to leave off the content-type headers when there is no request body.

Pls wait before merging this PR .
Will review this PR tomorrow

😥

sorry, should I revert?

Nop😀


Deploy Patroni Database

As a developer
I want to have deploy a highly available database
so that my service will be available as much as possible
DoD:

[x] Patroni helm charts are brought in to the codebase
[x] Postgres helm charts are brought into the database
[x] Test deploy to OpenShift

PR merged - CLOSED


Support built-in PHP classes static-constant-access

If the constant does not exist at compile time what would be the behaviour? I think it should throw an exception

@andresgutierrez Yes, If the constant does not exist will still throw an exception.
if (defined($name)) {
}


Update Dashboard

For the "homepage" dashboard (i.e. https://keas.azurewebsites.net/caesdo), let's clean it up to remove testing pages.
Dashboard Should have:
Assets (if they can see their team assets)
My Stuff (ideally we can put a badge up if they have assets waiting to confirm)
Team Administration (if they are an admin)
Header Bar should have:
Assets (if they have access)
My Stuff
Reports (if they have access)
Admin (if they are admin)


json validation expects extension to be in the right place...

The following resources fails validation with an 'Unexpected error' (and no others):
If the extension is moved above the subject, then the other errors appear (missing code etc. )
{
"resourceType": "Observation",
"meta": {
"profile": [
"http://hl7.org/fhir/StructureDefinition/genetics"
]
},
"subject": {
"reference": "Patient/1843164",
"display": "John Doe"
},
"extension": [
{
"url": "http://hl7.org/fhir/StructureDefinition/geneticsGenomicStart",
"valueInteger": "12"
}
]
}

Sorry - wrong server!


gromacs: 2018.1 -> 2018.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/gromacs/versions.
These checks were done:

built on NixOS
Warning: no invocation of /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/GMXRC had a zero exit code or showed the expected version
/nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/GMXRC.bash passed the binary check.
/nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/GMXRC.zsh passed the binary check.
Warning: no invocation of /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/GMXRC.csh had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/demux.pl had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/xplor2gmx.pl had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/gmx-completion.bash had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/gmx-completion-gmx.bash had a zero exit code or showed the expected version
/nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/gmx passed the binary check.
3 of 9 passed binary check by having a zero exit code.
0 of 9 passed binary check by having the new version present in output.
found 2018.2 with grep in /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2
directory tree listing: https://gist.github.com/1679f0362dfcc7299a6d5b8d9e76c14e
du listing: https://gist.github.com/2f848bb0fe181676c17b61a9d5a94953

meta.description for gromacs is: '"Molecular dynamics software package"'.
Experimental: this build is cached with Cachix. To use the cache follow these experimental instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 1815db24b616974faed9d9d4d349215e37629585 && nix-shell --pure -I nixpkgs=. -p gromacs

Success on x86_64-linux (full log)
Attempted: gromacs
Partial log (click to expand)
-- Installing: /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/gmx-completion-gmx.bash
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2
shrinking /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin/gmx
shrinking /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/lib/libgromacs.so.3.2.0
gzipping man pages under /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/lib  /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2/bin
patching script interpreter paths in /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2
checking for references to /build in /nix/store/illh3rp28ipz35z0wipsfc18z4m03sah-gromacs-2018.2...

Success on aarch64-linux (full log)
Attempted: gromacs
Partial log (click to expand)
-- Installing: /nix/store/3v5yxl3r4q99q1j9szpx09z5lkfbdj4q-gromacs-2018.2/bin/gmx-completion-gmx.bash
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/3v5yxl3r4q99q1j9szpx09z5lkfbdj4q-gromacs-2018.2
shrinking /nix/store/3v5yxl3r4q99q1j9szpx09z5lkfbdj4q-gromacs-2018.2/lib/libgromacs.so.3.2.0
shrinking /nix/store/3v5yxl3r4q99q1j9szpx09z5lkfbdj4q-gromacs-2018.2/bin/gmx
gzipping man pages under /nix/store/3v5yxl3r4q99q1j9szpx09z5lkfbdj4q-gromacs-2018.2/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/3v5yxl3r4q99q1j9szpx09z5lkfbdj4q-gromacs-2018.2/lib  /nix/store/3v5yxl3r4q99q1j9szpx09z5lkfbdj4q-gromacs-2018.2/bin
patching script interpreter paths in /nix/store/3v5yxl3r4q99q1j9szpx09z5lkfbdj4q-gromacs-2018.2
checking for references to /build in /nix/store/3v5yxl3r4q99q1j9szpx09z5lkfbdj4q-gromacs-2018.2...

Success on x86_64-darwin (full log)
Attempted: gromacs
Partial log (click to expand)
-- Installing: /nix/store/m5x025lpwhwq7dcbwms65fn3gfwmnhkl-gromacs-2018.2/include/gromacs/trajectoryanalysis/cmdlinerunner.h
-- Installing: /nix/store/m5x025lpwhwq7dcbwms65fn3gfwmnhkl-gromacs-2018.2/bin/gmx
-- Up-to-date: /nix/store/m5x025lpwhwq7dcbwms65fn3gfwmnhkl-gromacs-2018.2/bin
-- Installing: /nix/store/m5x025lpwhwq7dcbwms65fn3gfwmnhkl-gromacs-2018.2/bin/gmx-completion.bash
-- Installing: /nix/store/m5x025lpwhwq7dcbwms65fn3gfwmnhkl-gromacs-2018.2/bin/gmx-completion-gmx.bash
post-installation fixup
gzipping man pages under /nix/store/m5x025lpwhwq7dcbwms65fn3gfwmnhkl-gromacs-2018.2/share/man/
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/m5x025lpwhwq7dcbwms65fn3gfwmnhkl-gromacs-2018.2/lib  /nix/store/m5x025lpwhwq7dcbwms65fn3gfwmnhkl-gromacs-2018.2/bin
patching script interpreter paths in /nix/store/m5x025lpwhwq7dcbwms65fn3gfwmnhkl-gromacs-2018.2


Clean up

Clean up existing codebase to simplifying reviewing new changes. Almost all previously merged code has been moved into cpg-utils with https://github.com/populationgenomics/cpg-utils/pull/63, so it's better to start from clean slate.

I can do that all day long! Just point me to a repo


fix Make file issue

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #161 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #161   +/-   ##
=======================================
  Coverage   12.02%   12.02%
=======================================
  Files          44       44
  Lines        2088     2088
=======================================
  Hits          251      251
  Misses       1800     1800
  Partials       37       37

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 46a5859...a725c1f. Read the comment docs.

lgtm


Allow use of multiple areas

Allow the use of multiple areas by upgrading tabula's lib and adding some changes on cmd.js.
Now it is possible to use an array of areas as an option for Tabula.
Example:
var options = {
      area: ["191.85,11.03,245.46,835.21","171.879375,65.703125,184.494375,98.29"]
    };

Thanks for this. Really appreciated.
Its a simple fix so I'm happy to merge it.


WSL2 is running but terminal hangs when connet WSL

Environment
Windows build number: 19041.264
Your Distribution version: Centos7
Whether the issue is on WSL 2 and/or WSL 1: WSL2

Steps to reproduce

Before run the wsl，thewsl -l -v shows:
  NAME       STATE          VERSION
* CentOS7   Stopped         2

After run the wsl on PowerShell or use the terminal to launch the Centos7, It show nothing but just a cursor. Also, the wsl -l -v shows:
  NAME       STATE          VERSION
* CentOS7    Running         2

It seems that the Centos has been activated，but the terminal show nothing. No matter what I type, it's only printed on the screen and without any response.

WSL logs:  I cant get any logs, if there is something I missed, plz tell me and I will upload it.
Expected behavior
Connet the Centos7 successfully and Linux commands can be used properly.

Actual behavior
Nothing reponse :(!!

Does Ubuntu WSL2 distro work? If it does you can do sudo dmesg -w there, let the window open and try running again Centos7. You should see some error appear in the dmesg window.

please check out https://github.com/microsoft/WSL/issues/5973

please check out #5973

Please stop. Not one of these issues cites a 0x80070005 error.

Does Ubuntu WSL2 distro work? If it does you can do sudo dmesg -w there, let the window open and try running again Centos7. You should see some error appear in the dmesg window.

Sadly not,  neither Centos nor Ubuntu can be launched successful，I guess the problem is caused by loading configuration files, eg: ~/.bashrc.  Is there any way to view the configuration file of the WSL system from the Windows?

@FavonianKong You can try launching with the root user with wsl.exe -d CentOS7 -u root

@FavonianKong You can try launching with the root user with wsl.exe -d CentOS7 -u root
If it's a configuration problem then you should be able to access files via windows explorer. Open Run (win+R) and type \\wsl$\ to see the CentOS7 files.

Thanks for your advice，I failed to slove the problem in the end.  I find it so strange why the terminal never responds, even though the Centos7 is already running.

@FavonianKong If it happens again please collect some WSL Logs following these steps:
https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md#8-collect-wsl-logs


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In ddfab03, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 22228 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in f6f47bc after 15 minutes.


[BUG] Warn localization not working if menu is disabled

txAdmin/FXServer versions:
4.13.3/5429
Describe the bug
Localization is not working for Warns when menu is disabled in TxAdmin config
To Reproduce
Steps to reproduce the behavior:

Set Langage to French in Settings->Global
Enable the Menu in  Settings->Menu
Restart Server
Warn a player => NUI Warn menu is in French
Disable the Menu in  Settings->Menu
Restart Server
Warn a player => NUI Warn menu is in English

Expected behavior
Warns to be in the correct langage with Menu Disabled

Confirmed.
Related to #608.

The bug can most likely be attributed to improper ServerCtx event listener logic for clients when the menu is disabled. . During local testing with the menu disabled, clients never receive the "setServerCtx" event upon update of web settings. ServerCtx is updated within GlobalState but the event is never properly dispatched to clients, as we never populate the TX_ADMINS table when the menu is disabled.
Here  is where its blowing up
In fact, after glancing over some of the code, does localization even work in NUI for servers that don't enable OneSync?

Confirmed that this is still a thing in v5, even though #608 was fixed some time ago :(

Issue finally fixed.
Will be available on the next update.


Optimize eos_config_deploy_cvp

Enhancement summary
create a v2 role for eos_config_deploy_cvp_v2 that runs tasks in parallel using delegate_to functionality instead of running things in series locally
Which component of AVD is impacted
others
Use case example
today it takes quite long to create/add/update configlets because it's done in series using the local machine,
at high scale this can cause 15+ minute playbook runtime or more
Describe the solution you would like
Claus proposed using delegate_to: cv_server which looks promising
Describe alternatives you have considered
No response
Additional context
No response
Contributing Guide

[X] I agree to follow this project's Code of Conduct

We have optimized a lot in arista.cvp collection which speeds up the overall deployment time. We will close this issue.


Changes to the homepage do not stage as expected

When staging changes to the homepage, the content publishes all the way up to production. This is hampering our ability to have stakeholders preview a change before it's published.
See Slack thread when this last happened: https://cadotgov.slack.com/archives/C02F10G3L94/p1632507531002300

Carter has been working on a preview function and Chach has been working on consolidating the functionality into the publishing engineering pipeline. I believe she's working on UI elements for this.

Jim to test if the staging tag is working as expected.

I believe the problem is the use of the staging tag.  The correct tag (at the moment) is staging-only.  "staging" is ignored, and treated as a production update.

It's possible today's fixes alleviated this.  Testing...

Staging publishing of homepage is working in my tests.

Removing p0 tag since I believe this is performing better now.

@jbum tagging you on this. How is it better? Can you summarize here? :)
I think we've never been able to stage anything on the homepage.
cc @thepegisin you can expand here.


Deploy: Archicad build is not being found by the new Manager Feed CLI

Something has changed in the Archicad build that is preventing Manager.Feed from finding the file.
We should look into what's going on there before 2.9 release.
Side note: Manager.Feed should return a non-zero exit code to tell CircleCI when something went wrong.

This was fixed in #1665


Got error when trying to delete image from local registry

hi all,
I deploy  a local registry as bellow
https://docs.docker.com/registry/deploying/
After push an image to that local registry, I delete this image from local registry by Docker Registry HTTP API V2
https://docs.docker.com/registry/spec/api/
And then run garbage collect
docker exec registry bin/registry garbage-collect /etc/docker/registry/config.yml
After that I push the same image, that I push before, to local host and then try to pull it again from the local registry,
I got error
"Error response from daemon: manifest for localhost:5000/my-ubuntu:latest not found: manifest unknown: manifest unknown".
And when I try to delete this image again I got status " HTTP/1.1 404 Not Found"
Any one know what wrong and how to fix it?

Thank you for your feedback. However, it looks like your issue is not about Docker documentation, but that you may be having trouble with Docker itself. Please file a Docker support ticket or check out https://forums.docker.com/ to see if you can find a solution.
If this really is a documentation issue and I have misunderstood, please reopen it with more information.


Secure by default?

What is the policy for Lotus regarding security? Should Lotus be (reasonably) secure by default or should this be a responsibility of the developer? (I'm thinking about rack-protection for example, I' not so sure about a mechanism similar to SafeBuffer)

@thegcat @rkh @lasseebert @activestylus @emptyflask @PragTob @janko-m Folks, I've carefully listened and considered your advices: Lotus will adopt a secure by default policy.
Security is an important aspect of web apps, but it's often underestimated. In my career I've seen companies only allocate budget for features, without a serious investment in security.
Having a framework that educates and protects against the top vulnerabilities would limit the attack surface for those companies.
For instance CSRF is still in the top 10 of OWASP, and that is a low hanging fruit to fix for frameworks.
However, I'm not sure if we'll use rack-protection. The first reason is that, security is a complex topic and we can't claim that Lotus is serious about it, by only blindly using it. We want to evaluate case by case, and where a single component from it is useful we'll eventually use it.
My second concern about rack-protection is still about perf. Because it's a generic tool, each middleware is designed to run in isolation. For instance, in different middleware, it instantiate several times Rack::Request to check params. Because Lotus::Controller, already has a in instantiated set of params, we can directly check them.
I'm closing this as it was a question and not a specific technical issue. I hope my explanation addressed to all your concerns. As usual, I'm open to any further discussion.

+1 I'm totally fine with people not using rack-protection as long as there
is some means of protection. I'm also open to refactoring rack-protection
(which I have on my Sinatra 2 roadmap, anyway) to address your issues. I do
believe that there is value in sharing such critical code between projects.
On Mon, Jan 5, 2015 at 11:31 AM, Luca Guidi<EMAIL_ADDRESS>wrote:

Closed #64 https://github.com/lotus/lotus/issues/64.
—
Reply to this email directly or view it on GitHub
https://github.com/lotus/lotus/issues/64#event-213996390.

Here are the security mechanisms that we want to ship with Lotus v0.3.0: Automatic markup escape, Content-Security-Policy and X-Frame-Options headers, HttpOnly cookies https://discuss.lotusrb.org/t/security-features-for-v0-3-0/34


Update repo for deliver and gym to be fastlane-old

Temporary fix to change repo name of deliver and gym to be fastlane-old so that we can still push github releases there.

Approved

nevermind

OK


write_to_prefix (and similar ones) take ownership of the buffer

Please correct me if I'm wrong, but by taking ownership of the buffer, it seems to release the buffer as soon as the function exits, and subsequent code cannot use the buffer.
The method is defined as follows:
    /// Writes a copy of `self` to the prefix of `bytes`.
    ///
    /// `write_to_prefix` writes `self` to the first `size_of_val(self)` bytes
    /// of `bytes`. If `bytes.len() < size_of_val(self)`, it returns `None`.
    fn write_to_prefix<B: ByteSliceMut>(&self, mut bytes: B) -> Option<()> {
        let size = mem::size_of_val(self);
        if bytes.len() < size {
            return None;
        }

        bytes[..size].copy_from_slice(self.as_bytes());
        Some(())
    }

I think the method should only borrow the buffer.
Again, correct me if I'm wrong please!

If your B type is &mut [u8], you can reborrow like this. That said, ByteSliceMut is also implemented for RefMut, so your suggestion is still a good one! I'll consider changing the signature of this and other methods/functions.

This was fixed in https://github.com/google/zerocopy/pull/204.


Improve error message when CfCli returns StdOut containing duplicated "Instances starting..." lines

df4ed51 removes all error msg lines that contain that particular string ("Instances starting...\n")
Example result:


Support custom credential provider

Added new options:

Removed auto-refresh for role-based credentials
I discovered that this is already implemented in
AssumeRoleProvider.
Implement the CustomCredentialsProvider trait
This trait adds the ability to implement providing the credentials
outside of the ssstar, the user of this crate can implement this trait
and pass it into to use a custom way to get the credentials

This is a part of the fix of the issue https://github.com/elastio/elastio/issues/6796

There's no link to an issue, or any description of what this is or why it's useful.  Please elaborate on what this is for.

@anelson I fixed CI checks error, it failed because of a new version of cargo +nightly, to fix it I hardcoded specific version of nightly to +nightly-2023-06-15, with this version the CI checks pass, with the old version of nightly, udeps failed to like that:

I found that somebody commented in an github's issue that it reproduces on the latest version of nightly https://github.com/dtolnay/proc-macro2/issues/356#issuecomment-1614449061
and someone else suggests downgrading the version of nightly https://github.com/dtolnay/proc-macro2/issues/356#issuecomment-1614467505
I just want to make sure that it is okay


Error with regex

I get an error with the script:
Traceback (most recent call last):
  File "./test.py", line 940, in <module>
    main()
  File "./test.py", line 923, in main
    cacherdata = cacher(rawLog.readlines(), targetDate, friendlyNames, site_name)
  File "./test.py", line 324, in cacher
    x).group(1).replace('OS X ', 'macOS/').split('/')[0]
AttributeError: 'NoneType' object has no attribute 'group'

Attached is the debug log that causes this error (the target date is 2017-02-18
Debug.zip
Running the script on a different target date of 2017-02-28 works fine however.
Attached is the debug log for that target date as well.
Debug_28-02.log.zip

If this is on an old caching server, I have no intention of fixing it.

From: Calum<EMAIL_ADDRESS>Sent: Wednesday, March 1, 2017 6:43:39 PM
To: erikng/Cacher
Cc: Subscribed
Subject: [erikng/Cacher] Error with regex (#34)
I get an error with the script:
Traceback (most recent call last):
File "./test.py", line 940, in
main()
File "./test.py", line 923, in main
cacherdata = cacher(rawLog.readlines(), targetDate, friendlyNames, site_name)
File "./test.py", line 324, in cacher
x).group(1).replace('OS X ', 'macOS/').split('/')[0]
AttributeError: 'NoneType' object has no attribute 'group'
Attached is the debug log that causes this error (the target date is 2017-02-18
Debug.ziphttps://github.com/erikng/Cacher/files/812618/Debug.zip
Running the script on a different target date of 2017-02-28 works fine however.
Attached is the debug log for that target date as well.
Debug_28-02.log.ziphttps://github.com/erikng/Cacher/files/812620/Debug_28-02.log.zip
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/erikng/Cacher/issues/34, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFa_GDrTzREC2KUaFNaqpAMmeAUaTpTJks5rhizbgaJpZM4MQeow.

Turns out it trips up on this
2017-02-18 09:58:06.322 #d92df43NRCrA Received GET request from <IP_ADDRESS>:56062 [Configurator/2.0] for /iOS9.3.5/031-74004-20160825-995D69FE-6715-11E6-A974-933E34D2D062/iPad2,5_9.3.5_13G36_Restore.ipsw
2017-02-18 09:58:06.323 #d92df43NRCrA ECAssetHandler[0x7fe6d8415470]: Asset /iOS9.3.5/031-74004-20160825-995D69FE-6715-11E6-A974-933E34D2D062/iPad2,5_9.3.5_13G36_Restore.ipsw, path /Library/Server/Caching/Data/60A9474E-71B1-4401-9F63-9178B6F6C584, extents none
2017-02-18 09:58:06.323 #d92df43NRCrA ECAssetRequestor[0x7fe6d87ee370]: Issuing outgoing full request for URL http://appldnld.apple.com/iOS9.3.5/031-74004-20160825-995D69FE-6715-11E6-A974-933E34D2D062/iPad2,5_9.3.5_13G36_Restore.ipsw
2017-02-18 09:58:06.348 #d92df43NRCrA ECCacheManager: space requested 1.54 GB, current cache size 976.0 MB (limit unlimited) for asset /iOS9.3.5/031-74004-20160825-995D69FE-6715-11E6-A974-933E34D2D062/iPad2,5_9.3.5_13G36_Restore.ipsw
2017-02-18 09:58:06.348 #d92df43NRCrA ECResponse[0x7fe6d8528710]: Info loaded: response code = 200, file length =<PHONE_NUMBER>, reader = 0x7fe6d841be30
2017-02-18 09:58:06.348 #d92df43NRCrA ECAssetHandler[0x7fe6d8415470]: Opened extent [0,0r-] for writing at offset 0 (offset into file = 0)
2017-02-18 09:58:06.348 #d92df43NRCrA ECAssetHandler[0x7fe6d8415470]: Opened extent [0,0r-] for reading at offset 0 (offset into file = 0)

Figured it would be configurator. That was the one value no one could give me so I guessed. Try and figure it out. I won't have time to fix it for a bit.

From: Calum<EMAIL_ADDRESS>Sent: Wednesday, March 1, 2017 6:59:51 PM
To: erikng/Cacher
Cc: Erik Gomez; Comment
Subject: Re: [erikng/Cacher] Error with regex (#34)
Turns out it trips up on this
2017-02-18 09:58:06.322 #d92df43NRCrA Received GET request from <IP_ADDRESS>:56062 [Configurator/2.0] for /iOS9.3.5/031-74004-20160825-995D69FE-6715-11E6-A974-933E34D2D062/iPad2,5_9.3.5_13G36_Restore.ipsw
2017-02-18 09:58:06.323 #d92df43NRCrA ECAssetHandler[0x7fe6d8415470]: Asset /iOS9.3.5/031-74004-20160825-995D69FE-6715-11E6-A974-933E34D2D062/iPad2,5_9.3.5_13G36_Restore.ipsw, path /Library/Server/Caching/Data/60A9474E-71B1-4401-9F63-9178B6F6C584, extents none
2017-02-18 09:58:06.323 #d92df43NRCrA ECAssetRequestor[0x7fe6d87ee370]: Issuing outgoing full request for URL http://appldnld.apple.com/iOS9.3.5/031-74004-20160825-995D69FE-6715-11E6-A974-933E34D2D062/iPad2,5_9.3.5_13G36_Restore.ipsw
2017-02-18 09:58:06.348 #d92df43NRCrA ECCacheManager: space requested 1.54 GB, current cache size 976.0 MB (limit unlimited) for asset /iOS9.3.5/031-74004-20160825-995D69FE-6715-11E6-A974-933E34D2D062/iPad2,5_9.3.5_13G36_Restore.ipsw
2017-02-18 09:58:06.348 #d92df43NRCrA ECResponse[0x7fe6d8528710]: Info loaded: response code = 200, file length =<PHONE_NUMBER>, reader = 0x7fe6d841be30
2017-02-18 09:58:06.348 #d92df43NRCrA ECAssetHandler[0x7fe6d8415470]: Opened extent [0,0r-] for writing at offset 0 (offset into file = 0)
2017-02-18 09:58:06.348 #d92df43NRCrA ECAssetHandler[0x7fe6d8415470]: Opened extent [0,0r-] for reading at offset 0 (offset into file = 0)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/erikng/Cacher/issues/34#issuecomment-283541873, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFa_GLhYraKiSPl7WHVT7zQlqMbFUFMNks5rhjCngaJpZM4MQeow.

Cool no probs i'll try to fix and send a PR :)

Fixed as of https://github.com/erikng/Cacher/commit/812c51f698dfe318fd4dafbdc28c2d5af8199734


Please address pull request

https://github.com/Urigo/meteor-ionic/pull/47

Addressed here:
https://github.com/Urigo/meteor-ionic/pull/47#issuecomment-113951360
thanks


#1197 fixed QG-standalone installation on Openshift

Looks good, I`d like to try the installer with and without the quality-gates option on GKE to ensure that it still works.


Feature/phpmyadmin

Hello.
I've added configuration for the PhpMyAdmin container (https://hub.docker.com/r/phpmyadmin/phpmyadmin/), as it may be a valuable addition for developers who don't have access to user-friendly GUI-based mysql clients.
What do you think ?
Tom.

Thanks for your contribution :tada:
Would you please set the target branch to develop and add VIRTUAL_HOST env variable (only for phpmyadmin) to phpmyadmin.docker? Thanks :)

OK, will do :-)


Support user-defined constants

Tasks

Resolves #116

Status
This PR is still unfinished (WIP). See "notes" section. Also implementation is a bit naive.
Changes

[x] Introduced constant annotations with const keyword. First I thought that it would be better to use define, however it would cause much alterations, including annotation filtering and distincting functions from consts. In the end I figured that we can use other keyword instead.
[x] Constants are inserted to Scribble utils contract and are scoped by SourceUnit.
[x] Simplified TypeEnv and dedicated separate classes UserConstantsScope and UserFunctionsScope to handle user-defined funcs and consts.
[x] Added simple test samples.

Notes

Consts are order-dependent for now. E.g. developer can not use Y := X + 1 and then X := 1. For that we would either require a dependency graph or hoisting. I see both of them as a complication (as developers may be consfused by not paying much attention to maintaining proper order of constants), so did not invested much time into this.
Consts can not yet be imported to other SourceUnits. This is to simplify MVP feature implementation. We can introduce imports later.
Consts are allowing some constructs, that should not be available. I didn't yet found a way to restrict usable expression to allow only literals and other consts.
I guess we need more tests, considering shadowing and other possible edge-cases.
There are several TODOs, that worth discussing first (that may happen during PR review).

Regards.

Codecov Report

Merging #180 (c5e0537) into develop (82dbc1b) will increase coverage by 0.00%.
The diff coverage is 89.62%.

@@           Coverage Diff            @@
##           develop     #180   +/-   ##
========================================
  Coverage    87.09%   87.10%
========================================
  Files           72       73    +1
  Lines         5224     5303   +79
  Branches      1178     1194   +16
========================================
+ Hits          4550     4619   +69
- Misses         410      415    +5
- Partials       264      269    +5

Impacted Files
Coverage Δ

src/instrumenter/transpiling_context.ts
89.24% <50.00%> (+0.11%)
:arrow_up:

src/spec-lang/tc/typecheck.ts
83.39% <71.42%> (-0.47%)
:arrow_down:

...-lang/ast/declarations/user_constant_definition.ts
75.00% <75.00%> (ø)

src/spec-lang/tc/typeenv.ts
97.29% <96.15%> (-2.71%)
:arrow_down:

src/instrumenter/instrument.ts
94.76% <96.42%> (+0.09%)
:arrow_up:

src/bin/scribble.ts
80.81% <100.00%> (+0.09%)
:arrow_up:

src/instrumenter/annotations.ts
86.69% <100.00%> (+0.18%)
:arrow_up:

src/instrumenter/instrumentation_context.ts
99.45% <100.00%> (+<0.01%)
:arrow_up:

src/instrumenter/transpile.ts
91.80% <100.00%> (+0.18%)
:arrow_up:

src/spec-lang/ast/declarations/annotation.ts
78.57% <100.00%> (+0.79%)
:arrow_up:

... and 4 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

There is currently some issue with CI we are still investigating. Tests pass locally so force-merging.


Restore disabled check for #3070 (except on MSVC)

Restore the previously disabled check for regression #3070 on all compilers but MSVC.
To summarize the issue:
Given namespace fs = std::filesystem.
On MSVC attempting to construct an fs::path from json results in an ambiguous overload resolution because fs::path can be constructed from both a std::string as well as another fs::path. To the best of my knowledge there is no way to fix an ambiguous overload situation involving a type we do not control and with json implicitly converting to both std::string and fs::path.
Re-enabling the check where it compiles and keeping it disabled for MSVC is the best we can do.
Closes #3377 and #3382.

Looking at #3229 we may want to disable it for ICPC as well. Should allow the PR to pass CI and be merged.

Coverage remained the same at 100.0% when pulling db1196f41b617d25929f8e7afd32b9a62ee5e8d1 on falbrechtskirchinger:resolve-3377 into ab5cecb34feb18f5a030517547b0ead1148d05fc on nlohmann:develop.


NetTopologySuite plugin

Work in progress, DO NOT MERGE.
This PR introduces a new plugin API, which allows users to register external Npgsql plugins which can add type mappings and method translators (for now).
The first plugin, for NetTopologySuite, is in progress.

@ajcvickers @divega I'd like to get your opinion on this...
In https://github.com/aspnet/EntityFrameworkCore/issues/11236 we discussed how to add provider-wide features, since I wanted to add plugins to Npgsql, in order to modify type mappings etc. After thinking about this again, this seems much simpler and elegant than I thought it would be - I implemented plugins as a simple list on NpgsqlOptions, so that a plugin can be set up as follows:
builder.UseNpgsql("<connection string>", b => b.UseNetTopologySuite());

Then, Npgsql services which support plugins, simply accept INpgsqlOptions, pull out the plugin list and call the appropriate methods on the plugins. For example, NpgsqlTypeMapperSource iterates over the plugins and calls .AddMappings() and passing itself. The plugin can then modify the mappings, introducing new ones etc. The same is implemented for NpgsqlCompositeMethodCallTranslator, which allows the plugin to register translators for its types.
This has allows me to start work on the NetTopologySuite... It looks quite promising - spatial types are mapped to PostgreSQL spatial types (PostGIS), and I even have some very rudimentary SQL translation. Although this will take a lot of work, I hope to be able to ship some substantial spatial support with 2.1.
Can you please take a look at the plugin model and tell me what you think? I'm especially unsure about dependency injection and how it all works across context instances etc. The commit in question is Introduced new plugin API (currently cb896fd but that may change). You can also look at the latest commit (WIP on NTS plugin, currently 60373bd), to see how a plugin actually looks like.
Note that the EF Core plugin's UseNetTopologySuite() adds an ADO.NET-level plugin of its own (NpgsqlConnection.GlobalTypeMapper.UseNetTopologySuite()), so the plugins are layered. This is currently done at a totally global level (so plugins could in theory interfere).

Accidentally closed due to rename of branch, new branch is https://github.com/roji/Npgsql.EntityFrameworkCore.PostgreSQL/tree/plugins.


docs: add replay guide & concept

Added documentation on how to use replay feature and the concept of replay. Will add more doc on this part after added backup in replay.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 69.749%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4284

Relevant Lines:
6142

💛  - Coveralls

Hi @kushsharma, I have fixed the incorrect grammar as suggested.

@arinda-arif docs look good, you also need to add a new heading in sidebar.js to make this page visible in docs.

added to sidebar @kushsharma, please check 😃


Name obtained by Get-AzureNetworkSecurityGroup

I realize this is an ASM command but we have an advisory case in CSS asking for clarification.
Get-AzureNetworkSecurityGroup apparently requires that you specify the name of the resource as "Group  ", which doesn't seem to line up with anything else.
Is this by design? Azure module 5.3.0.
Examples
Fails when using the resource name alone
PS C:\WINDOWS\system32> Get-AzureNetworkSecurityGroup -Name "classic_jptest"
Get-AzureNetworkSecurityGroup : ResourceNotFound: The Network Security Group classic_jptest does not exist.
At line:1 char:1
+ Get-AzureNetworkSecurityGroup -Name "classic_jptest"
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : CloseError: (:) [Get-AzureNetworkSecurityGroup], CloudException
    + FullyQualifiedErrorId : Microsoft.WindowsAzure.Commands.ServiceManagement.Network.NetworkSecurityGroup.GetAzureNetworkSecurityGroup

Succeeds when including the RG name
PS C:\WINDOWS\system32> Get-AzureNetworkSecurityGroup -Name "Group jptest classic_jptest"

Name                        Location Label
----                        -------- -----
Group jptest classic_jptest West US

@dcbrown16 Thanks for your question. Please provide us with the link to the documentation you are referring.

https://docs.microsoft.com/en-us/powershell/module/servicemanagement/azure/get-azurenetworksecuritygroup?view=azuresmps-4.0.0
would be the doc for this command. If there's a repo somewhere for the Azure module itself please let me know.

but if this is by design it would be good to update the document, since this is a nonstandard use of "name".

@dcbrown16 Thanks for bringing this to our attention.
I understand the issue and have reached out to our internal Team to get more details. Will keep you posted on my findings.

Looping in @barbkess.

This is an obscure problem that I was only able to reproduce after numerous attempts.

When a classic network security group is created with PowerShell, it works as designed in both PowerShell and in the portal.
When a classic network security group is created in the portal using a resource group other than the one named Default-Networking, it only shows up in PowerShell using the following syntax Get-AzureNetworkSecurityGroup -Name 'Group <ResouceGroupName> <NetworkSecurityGroupName>.


Notice of future breaking changes in osm-login-python package

Problem Definition Briefly (might be explained in steps)

I am helping to maintain the osm-login-python package.
We plan to make BREAKING changes to the API in the next release.
I noticed that you are using unpinned dependencies in your setup.py: https://github.com/mapilio/MapSyncer/blob/7d481ac9132aea834cbf699c877595205bb3b865/setup.py#L22-L33
This would break your next build of MapSyncer, so I thought I would make you aware!

Solutions
Option 1

Pin osm-login-python==1.0.3, as this will work in the same way as previously.

Option 2
Update this line:
https://github.com/mapilio/MapSyncer/blob/7d481ac9132aea834cbf699c877595205bb3b865/MapSyncer/app.py#L103
to token = parsed_data['user_data']
then also update:
https://github.com/mapilio/MapSyncer/blob/7d481ac9132aea834cbf699c877595205bb3b865/MapSyncer/app.py#L110-L112
to
 @app.route('/get_my_data/<user_data>', methods=['GET'])
 def get_my_data(user_data):
     user_data = osm_auth.deserialize_data(user_data)

Hope this helps! 😄

Hi @spwoodcock, your contribution was marvelous, thanks for letting us know about this problem, we've fixed it. We really appreciate it :tada:

Changes has been made according to given information by @spwoodcock :rocket:
Merge Link


.gitignore: Add .cache

I think these .cache directories are coming from newer versions of py.test, which cache info about test runs.
$ git status
On branch master
Your branch is up-to-date with 'upstream/master'.
Untracked files:
  (use "git add <file>..." to include in what will be committed)

	.cache/
	cli/.cache/

Can one of the admins verify this patch?

ok to test

👍

@mesosphere-ci retest this please


Axe parse error

Nice, that's significantly cleaner and it's also dropping a bunch of strings that I didn't like!


Update book for 0.4

A decent amount of the book needs updating for consistency with the shiny new API. Additionally, a page or so about how rkyv_dyn works would be a big help.

Fixed by d56d263cc5d6edea4c0c5bf56b3e5128b02bf2fb


Adding thresholds branch

I added a command to generate the threshold json files from the nightly summary json files. TO DO: combine these thresholds with the plotting and status functions so they can be used in generating the HTML files.

still in progress, need to get rid of some hard-coded things

I ran into a cosmics_rate issue when I tried to run it:

Seems to work fine, though some Bokeh warnings did pop up while running qqa-monitor.

Problems with old summary files, seems fine now on my end as well

@sbailey I added most of the things we talked about yesterday: making plotting thresholds optional, adding docstrings, generating the threshold jsons outside of qqa, filling in missing thresholds with default values, and making the glyphs on per-amp plots larger.
However, when I was looking into shifting from per-night to per-exposure ranges for generating the threshold files, I realized it would take some more significant restructuring- currently I read in the nightly summary jsons to calculate the thresholds, so I don't have a per-exposure granularity already introduced. It might be better to merge this branch first, and then I can open a new branch to add that capability?

@ana-lyons yes let's wrap this up and merge it even if it is still based upon per-night ranges rather than per-exposure ranges.  Ping me when you think it is ready for that.

@sbailey I retested the branch, looks ready to merge


DFP-34X-2C2 not reporting some stats (temperature, Tx Bias Current, etc) to Mikrotik RB4011iGS+

Any chances to get it working?

Mikrotik RB4011iGS+
DFP-34X-2C2 with M110_sfp_ODI_220923 firmware

Duplicate of https://github.com/Anime4000/RTL960x/issues/87

Maybe there are some options to get those stats via console?
I want to add it to Zabbix monitoring

Maybe there are some options to get those stats via console? I want to add it to Zabbix monitoring

You could by telnet then query by calling diag then close.
this got me thinking... if possible put basic status at login page then curl then result?

probably easier to remove logon check for the status page itself ?

that need to patch mini webserver bin file 😆
so, modify login.asp and put basic status there

Heh. No vi, nano, etc on this SFP.
How do you guys modify files?

just usual way, I have make some script to run firmware on local computer, that way you can modify the firmware then flash it
https://github.com/Anime4000/RTL960x/tree/main/Tools/emulator

Maybe there are some options to get those stats via console? I want to add it to Zabbix monitoring

Via Telnet:

diag pon get transceiver bias-current
Bias Current: 10.936000 mA

diag pon get transceiver part-number
Part Number: RTL8290

diag pon get transceiver rx-power
Rx Power: -24.559320 dBm

diag pon get transceiver temperature
Temperature: 85.000000 C

diag pon get transceiver tx-power
Tx Power: -inf dBm

diag pon get transceiver vendor-name
Vendor Name: REALTEK

diag pon get transceiver voltage
Voltage: 3.019400 V

omcicli get onuid
ONU ID: 65

omcicli get state
ONU state: 5

pon get transceiver temperature
Temperature: 58.625000 C

diag pon get transceiver temperature Temperature: 85.000000 C

Your Temps seem too high!!

This is just an example. Temperature such as of Dlink DPN-100 SFP.
This is a very hot stick.

Where do you guys get new firmwares? Manufacturer not answering emails and don't have firmwares on website :(

Fast and dirty solution to send data to Zabbix using zabbix_sender.

Mikrotik RB4011iGS+ DFP-34X-2C2 with M110_sfp_ODI_220923 firmware

What version of your MikroTik? If you are in version 6, you will see all information, but on version, you will not see anything.

What version of your MikroTik? If you are in version 6, you will see all information, but on version, you will not see anything.

7.6

What version of your MikroTik? If you are in version 6, you will see all information, but on version, you will not see anything.

7.6

Can you try to downgrade to version 6.48.6 and check again?

Understood your issue. I also own a RB4011iGS+RM. It is very difficult to recognize the SFP stick, especially SFP GPON stick. In some case, I must disable auto negotiation, and force speed at 1Gbps so that it can bring sfp-sfpplus1 up.

I did not change any speed setting at the ODI stick and have Auto Negotiation with Mikrotik. Works fine so far.

I'm not good at Mikrotik scripts, but made a dirty solution to read the status from webgui and send through telegram for every one hour.  It works well for me.
#/log info "gponhealth start"
:do {
/tool fetch url="http://<IP_ADDRESS>/boaform/admin/formLogin?username=admin&password=xxxxx&save=Login&submit-url=^%^2Fadmin^%^2Flogin.asp" keep-result=no
} on-error={ :put "login failed"};
:delay 2s
#/log info "gponhealth login"
/tool fetch url="http://<IP_ADDRESS>/status_pon.asp" keep-result=yes
#/log info "gponhealth ponstatus"
:delay 6s
:local statuscontent [/file get status_pon.asp contents];
:local statusf [:pick $statuscontent 1086 1097];
:local statusfinal ("GPON-Status\nTemperature = " .$statusf);
:set statusf [:pick $statuscontent 1218 1228];
:set statusfinal ($statusfinal . "\nVoltage = " . $statusf);
:set statusf [:pick $statuscontent 1350 1363];
:set statusfinal ($statusfinal . "\nTx Power = " . $statusf);
:set statusf [:pick $statuscontent 1485 1500];
:set statusfinal ($statusfinal . "\nRx Power = " . $statusf);
:set statusf [:pick $statuscontent 1626 1638];
:set statusfinal ($statusfinal . "\nBias Current = " . $statusf);
#/log info $statusfinal;
:local pload "chat_id=-xxxxxxxx&text=$statusfinal"
#/log info "before fetch"
/tool fetch mode=https http-method=post http-data=$pload url="https://api.telegram.org/botXXXXXXX/sendMessage" keep-result=no;
#/log info "gponhealth script `completed"

Create a scheduler for every one hour or so, it will send message to telegram.

wow this is very nice, I add this on the README.md about reading GPON ONU SFP status.

Thanks.
Just an additional note.  Please replace your onu password, telegram chatid and botid accordingly in the above script.

Please report this upstream so it may be fixed instead of here, no?

HSGQ/ODI still use Static EEPROM, best way is this for now
@smnrock

I wrote a Prometheus collector to grok the SFP SoC and optical signal stats via HTML and plot the data in Grafana since I use Telegram for alerting, not monitoring.
Put the collector somewhere in your $PATH and create a systemd user service (the collector runs as a simple web server on port 8000):
# /home/strykar/.config/systemd/user/odi.service
[Unit]
Description=Prometheus collector for the HSGQ / ODI GPON SFP
After=network-online.target

[Service]
Type=simple
ExecStart=/usr/bin/python3 /home/strykar/.bin/hsgq_prometheus_collector.py

[Install]
WantedBy=default.target

Add the collector to Prometheus under any existing scrape_configs::
  - job_name: "gpon_collector"
    scrape_interval: 5m  # Adjust this to how frequently you want Prometheus to scrape metrics
    static_configs:
      - targets: ["localhost:8000"]

Ensure Grafana can poll Prometheus and create a dashboard in Grafana:

Nice @Strykar !
I add this into README.md for GPON Stats via Grafana

HI @Strykar , how do we handle authentication to the GPON with your collector script?
The web interface does not allow unauthenticated access, I've tried basic authentication (adding user:pass) to the URL but that doesn't work

@jjeziorny  Web auth is convoluted with MD5 hashing, I have been unable to login via terminal.
If you find and post the md5.js file used during the login process, I can try to add authentication to the current collector.
I found that it does not log me out for hours, so just signing in once per day manually for now does the trick.
If login via script is not possible, I will move the script to auth over SSH and poll the data from the terminal.

@Strykar you can try using the url format with login details appended like the one i gave in the mikrotik script above.

@Strykar you can try using the url format with login details appended like the one i gave in the mikrotik script above.

@jjeziorny @smnrock  Try - https://gist.github.com/Strykar/48fa636e55de0a33ff6a137ed4a09538
Let me know if it works and I'll update the repo.

Seems to work fine
root@prometheus:/usr/local/etc/rc.d # /usr/local/bin/python3 /root/hsgq_prometheus_collector2.py Logged in successfully

@jjeziorny Cool, this needs to be refactored at some point to deal with the firmware quirks of single-login only and its related errors.
SSH may be easier to deal with than HTTP (state) in Python for this.

Yeah, or modify the firmware and get snmp agent on it

If only HSGQ released source code.

btw, at some point if you have the time you can add command line arguments to it so people can set IP, password and listener port without having to edit the script.
In my case I had to create two scripts as I have two GPONs in my network.

@jjeziorny  New prometheus collector that uses SSH instead of parsing HTML - https://gist.github.com/Strykar/584c6467ed023f90b13a059f511d4d1c and can deal with multiple SFPs.

@jjeziorny New prometheus collector that uses SSH instead of parsing HTML - https://gist.github.com/Strykar/584c6467ed023f90b13a059f511d4d1c and can deal with multiple SFPs.

OK, just switched to it and seems to be running fine.
# ps aux
USER         PID %CPU %MEM     VSZ    RSS TT  STAT STARTED    TIME COMMAND
root       14537  5.2  0.4   57352  32112  -  SJ   08:52   0:00.24 /usr/local/bin/python3 /root/hsgq_prometheus_collector.py --hostname <IP_ADDRESS> --port 22 --user admin --password xxxx --webserver-port 8001 (python3.9)
root       14530  2.2  0.4   57352  32108  -  SJ   08:52   0:00.24 /usr/local/bin/python3 /root/hsgq_prometheus_collector.py --hostname <IP_ADDRESS> --port 22 --user admin --password xxxx --webserver-port 8000 (python3.9)
root       14529  0.0  0.0   12816   2200  -  SsJ  08:52   0:00.00 daemon: /usr/local/bin/python3[14530] (daemon)
root       14536  0.0  0.0   12816   2200  -  SsJ  08:52   0:00.00 daemon: /usr/local/bin/python3[14537] (daemon)

Cool, it can work via just one process BTW:
/usr/bin/python hsgq_prometheus_collector.py \
    --hostname <IP_ADDRESS> --port 22 --user admin --password pass1 \
    --hostname <IP_ADDRESS> --port 22 --user admin --password pass2

Hi @Strykar ,
After not having much time over the weekend to tweak things I finally got a change this morning. But run into some issues.

CPU/MEM stats not available on the SSH version
GPON state was always comming 0
SFP only allows a single SSH connection, therefore if you are SSH'ed into the device the script would fail to connect.

So, based on point 3 I don't think it's a good ideal to collect these via SSH
I've reverted back to the HTTP version for now

There's no point in an HTTP version, here's a small example of stats that could be pulled by using SSH:
# diag gpon get ?
gpon get
active-timer                                     - activation timer configuration
aes-framecnt                                     - AES frame counter
alarm-status                                     - current alarm status
auto-boh                                         - auto update BOH configuration
auto-tcont                                       - auto add or delete tcont configuration
bwmap                                            - bwmap
dbru-block-size                                  - dbru block size
ds-bwmap                                         - downstream BWMAP configuration
ds-eth                                           - downstream ethernet configuration
ds-flow                                          - downstream flow configuration
ds-gem                                           - downstream GEM configuration
ds-laser                                         - downstream laser configuration
ds-omci                                          - downstream OMCI configuration
ds-phy                                           - downstream PHY configuration
ds-ploam                                         - downstream PLOAM configuration
eqd-offset                                       - EQD offset configuration
multicast-filter                                 - multicast filter configuration
multicast-filter-entry                           - multicast filter entry configuration
onu-state                                        - ONU state, O1-O7
password                                         - password configuration
password-hex                                     - password configuration
pps-cnt                                          - PPS cnt information
rdi                                              - RDI configuration
rogue-sd-cnt                                     - rogue ont SD cnt information
serial-number                                    - serial number configuration
serial-number-hex                                - serial number configuration
serialnumber                                     - serial number configuration
tcont                                            - TCONT configuration
tx                                               - transmit configuration
us-dbr                                           - upstream DBR configuration
us-flow                                          - upstream flow configuration
us-laser                                         - upstream laser configuration
us-phy                                           - upstream PHY configuration
us-ploam                                         - upstream PLOAM configuration

There are a bajillion more metrics available via the terminal:
# diag ?

acl                                              - acl configuration
auto-fallback                                    - Auto-Fallback configuration
bandwidth                                        - bandwidth configuration
classf                                           - classification configuration
cpu                                              - cpu configuration
debug                                            - debug configuration
dot1x                                            - dot1x configuration
epon                                             - epon configuration
exit                                             - exit diag shell
field-selector                                   - field selector configuration
flowctrl                                         - flowctrl configuration
gpon                                             - GPON configuration
i2c                                              - I2C configuration
igmp                                             - igmp configuration
interrupt                                        - interrupt configuration
iol                                              - iol configuration
l2-table                                         - l2 table configuration
l34                                              - L34 configuration
led                                              - led configuration
meter                                            - shared meter configuration
mib                                              - mib configuration
mirror                                           - mirror configuration
oam                                              - oam  configuration
pbo                                              - pbo configuration
pon                                              - pon configuration
port                                             - port configuration
ppstod                                           - ppstod configuration
qos                                              - qos configuration
register                                         - register configuration
rldp                                             - rldp configuration
rlpp                                             - RLPP configuration
rma                                              - rma configuration
security                                         - security configuration
storm-control                                    - storm-control configuration
stp                                              - stp configuration
svlan                                            - svlan configuration
switch                                           - switch configuration
time                                             - time configuration
trap                                             - trap configuration
trunk                                            - trunk configuration
vlan                                             - vlan configuration

Use what works for now, I will keep updating the collector and dashboard as I get time since I intend to track these metrics, especially the receive signal.

suddenly see my edgerouter shows details about my DFP-34X-2C2 today, not sure if it's correct or not.

that's correct

suddenly see my edgerouter shows details about my DFP-34X-2C2 today, not sure if it's correct or not.

What hardware revision is your ODI SFP?

no idea, I only know its fw version, V1.0-220923

no idea, I only know its fw version, V1.0-220923

Look at SFP Pin/Pad, it would write like:
STICK3V06

no idea, I only know its fw version, V1.0-220923

Look at SFP Pin/Pad, it would write like:
STICK3V06

It's stick V06.

welp, need V08 to have DDM

Yes We need to Have V8 For DDM
On Sun, Jun 2, 2024, 9:31 AM Anime4000 @.***> wrote:

welp, need V08 to have DDM
—
Reply to this email directly, view it on GitHub
https://github.com/Anime4000/RTL960x/issues/99#issuecomment-2143687390,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGXZFBWTFQSTIMCRM5MPKVTZFKKKHAVCNFSM6AAAAAASMGWCN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGY4DOMZZGA
.
You are receiving this because you commented.Message ID:
@.***>

welp, need V08 to have DDM

Hi , hope you're all well.
What are the differences / changes between the V06 and the V08 ? ( beside the DDM part)
Thank you in advance for the answer.

Nothing except the DDM just add 4 USD To the price for the V8
On Sun, Jun 2, 2024, 1:46 PM canope-25 @.***> wrote:

welp, need V08 to have DDM
Hi , hope you're all well.
What are the differences / changes between the V06 and the V08 ? ( beside
the DDM part)
Thank you in advance for the answer.
—
Reply to this email directly, view it on GitHub
https://github.com/Anime4000/RTL960x/issues/99#issuecomment-2143745782,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGXZFBSKWWBUMNZWUBMN4WDZFLIFJAVCNFSM6AAAAAASMGWCN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTG42DKNZYGI
.
You are receiving this because you commented.Message ID:
@.***>

I paid V6 26 $ VATIN ( 20% since in Europe )
So maybe more than 4 USD

Nothing except the DDM just add 4 USD To the price for the V8
On Sun, Jun 2, 2024, 1:46 PM canope-25 @.***> wrote:

welp, need V08 to have DDM

Hi , hope you're all well.

What are the differences / changes between the V06 and the V08 ? ( beside

the DDM part)

Thank you in advance for the answer.

—

Reply to this email directly, view it on GitHub

https://github.com/Anime4000/RTL960x/issues/99#issuecomment-2143745782,

or unsubscribe

https://github.com/notifications/unsubscribe-auth/BGXZFBSKWWBUMNZWUBMN4WDZFLIFJAVCNFSM6AAAAAASMGWCN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTG42DKNZYGI

.

You are receiving this because you commented.Message ID:

@.***>

So, did I get it correctly.
If I buy new stick (from fibershop?) it will report stats as it supposed to do?

Why don't you try to contact hsgq directly?
On Sun, Jun 2, 2024, 2:50 PM rndm2 @.***> wrote:

So, did I get it correctly.
If I buy new stick (from fibershop?) it will report stats as it supposed
to do?
—
Reply to this email directly, view it on GitHub
https://github.com/Anime4000/RTL960x/issues/99#issuecomment-2143768923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGXZFBXMCLOSJMNRGDRESF3ZFLPXFAVCNFSM6AAAAAASMGWCN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTG43DQOJSGM
.
You are receiving this because you commented.Message ID:
@.***>

Different prices for different clients I guess
On Sun, Jun 2, 2024, 2:42 PM canope-25 @.***> wrote:

I paid V6 26 $ VATIN ( 20% since in Europe )
So maybe more than 4 USD
Nothing except the DDM just add 4 USD To the price for the V8
On Sun, Jun 2, 2024, 1:46 PM canope-25 @.***> wrote:
welp, need V08 to have DDM
Hi , hope you're all well.
What are the differences / changes between the V06 and the V08 ? ( beside
the DDM part)
Thank you in advance for the answer.
—
Reply to this email directly, view it on GitHub
#99 (comment)
https://github.com/Anime4000/RTL960x/issues/99#issuecomment-2143745782,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGXZFBSKWWBUMNZWUBMN4WDZFLIFJAVCNFSM6AAAAAASMGWCN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTG42DKNZYGI
.
You are receiving this because you commented.Message ID:
@.***>
—
Reply to this email directly, view it on GitHub
https://github.com/Anime4000/RTL960x/issues/99#issuecomment-2143766318,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGXZFBU34PPMLQMDTEQIGATZFLOZNAVCNFSM6AAAAAASMGWCN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTG43DMMZRHA
.
You are receiving this because you commented.Message ID:
@.***>

I'm interested , what is the price of the V08 ?

Different prices for different clients I guess

I got it for 34 USD a month ago but if you add customs and shipping it goes
high to 45 USD.
On Sun, Jun 2, 2024, 4:47 PM canope-25 @.***> wrote:

I'm interested , what is the price of the V08 ?
Different prices for different clients I guess
—
Reply to this email directly, view it on GitHub
https://github.com/Anime4000/RTL960x/issues/99#issuecomment-2143807798,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGXZFBTKU45YTIDKAJNXFV3ZFL5K3AVCNFSM6AAAAAASMGWCN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTHAYDONZZHA
.
You are receiving this because you commented.Message ID:
@.***>

Thank you so , nothing beside the DDM part , no improvement regarding the laser / receive sensitivity ?
Still -28dbm receive ? +4dbm max transmit ?
Still 2.5 watt of consumption ?
I'm interested if you have a spec doc.
Thank you

I got it for 34 USD a month ago but if you add customs and shipping it goes
high to 45 USD.
On Sun, Jun 2, 2024, 4:47 PM canope-25 ***

I'm interested , what is the price of the V08 ?

Different prices for different clients I guess

I think with the release of the new stick this thread is basically resolved.

sorry for commenting on closed issue but just want to update a little bit, my sfp is V6 and I think the value on edgerouter is not true. It's nice to have feature but not mandatory as long as it functions correctly, stable.


lakectl - Add default endpoint_url

currently, lakectl uses the endpoint_url set in the lakectl config file(~/.lakectl.yaml)
In case it is not set we receive an error
For example:
user@MacBook-Pro ~ % lakectl repo list
Error executing command: Get "https:///api/v1/repositories?amount=-1": http: no Host in request URL

The default host should be http://<IP_ADDRESS>:8000

HI can i work on this issue?

All yours! Thanks for your help.

@abraham-john-paul Assigned to you! Let us know if you need any help.
Many thanks!

Hi, I know it's assigned to @abraham-john-paul but I have actually solved it a week ago, and just been on holydays so I couldn't fix the required changes in the PR. I have now reopened it so you are welcome to use it :)

Thanks @eylonronen - we are reviewing.
@abraham-john-paul apologies for our confusion here. Somehow we missed it. Happy to help find another good issue to work on.

Sure , no problem 🙂