[feat] Improve the implementation of Container Platforms

Allow disabling of pulling the container image.
Respect the commands defined inside the container image.

Fixes #1736

[x] Need to change also the documentation
[x] Change the tutorial example
[x] Add a tutorial example for making files available outside of the container
[x] Deprecate the workdir and command fields

@ChristianKniep please give your feedback

Sweat, just some minor comments. I reckon that the pull_command is a bandaid until a more sustainable way forward is found. Might be that skopeo is the ultimate swiss army knife for downloading and transforming image. @vrothberg
Valentin might also have an opinion about adding podman here... :)

Thanks for pulling me in.  Skopeo is super powerful and light weight. Podman, on the other hand, has Python bindings: https://github.com/containers/podman-py

Codecov Report

Merging #1775 (6ee1c94) into master (2f77b94) will increase coverage by 0.01%.
The diff coverage is 97.29%.

@@            Coverage Diff             @@
##           master    #1775      +/-   ##
==========================================
+ Coverage   87.35%   87.37%   +0.01%
==========================================
  Files          47       47
  Lines        7792     7770      -22
==========================================
- Hits         6807     6789      -18
+ Misses        985      981       -4

Impacted Files
Coverage Δ

reframe/core/pipeline.py
92.16% <ø> (ø)

reframe/frontend/executors/__init__.py
98.19% <0.00%> (ø)

reframe/utility/osext.py
84.71% <75.00%> (ø)

reframe/__init__.py
81.81% <100.00%> (ø)

reframe/core/containers.py
100.00% <100.00%> (ø)

reframe/core/decorators.py
97.08% <100.00%> (ø)

reframe/core/warnings.py
100.00% <100.00%> (ø)

reframe/frontend/runreport.py
92.55% <100.00%> (ø)

reframe/utility/versioning.py
96.77% <100.00%> (+3.08%)
: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 2f77b94...aae66cc. Read the comment docs.

Hello @teojgo, Thank you for updating!

In the file reframe/core/containers.py:

Line 51:10: E131 continuation line unaligned for hanging indent
Line 97:10: E131 continuation line unaligned for hanging indent

Do see the ReFrame Coding Style Guide

@kniec the PR is more or less ready. I have made various changed taking into account your comments. I have also expanded the tutorial for containers to include Sarus.

Thanks for the effort - I was busy this week, but I'll carve out some time in the next couple of days to go through it.


marble: Add version 2.2.0

Closes #10909

[x] I have read the Contributing Guide.

/verify

Although it may cause the same error on 32bit download url, priority is given to 64bit


[Infineon] Not to increment RebootCount if wakes from sleep state

Problem
RebootCount would increment if wakes from sleep state.
Change overview
Not to increment RebootCount if wakes from sleep state.
Testing
Tested manually with chip-tool.

If it's a low-power enough state that it's lost volatile memory, does that count as "wakes up from a sleep state", is the question?  Assuming that's the situation here, of course.

PR not updated for a long time, has CI failures and unresolved comments. Assuming stale and closing.


Add assert_exactly_one_delivered_email/1 (or similar) to Bamboo.Test

This is potentially a special use-case, but sometimes I need to test that an E-Mail was sent exactly once (to avoid sending duplicate E-Mails to customers for example).
As of now, I created myself a workaround like this:
def assert_email_delivered_exactly_once(email) do
    assert_delivered_email(email)
    refute_delivered_email(email)
end

If an E-Mail was sent twice, the refute_delivered_email/1 assertion will receive that second E-Mail and fail.
My question is: Do you see a need for this workaround to be included in the Bamboo.Test-module?
If yes, do you like this implementation or can you recommend a more generic solution? I'd be happy to implement it in a Pull-Request then.
If not, no problem as well since this might not be a common use-case I believe.

@PJUllrich thanks so much for opening this issue. Apologies for the delay. I can see how it might be valuable to have a function that could ensure an email is delivered exactly once. But I'm not sure about the implementation.
Looking at how assert_delivered_email(email) and refute_delivered_email(email) are implemented, I understand why the function you suggested works, but the semantics seem odd to me because it looks as though we're asserting two exact opposites and yet the test could pass. I understand it works because we're taking messages out of the process mailbox, so the process state is changing.
What I'd like to do is give all the testing helpers a thorough look (I haven't had a chance since starting to maintain Bamboo), and make sure they're consistent. So I'll try to loop back here when I do that and figure out if introducing one with the implementation you suggested makes sense or if we should do it differently.

Hey there, thank you for taking a look at this. I agree that calling these 2 opposite functions feels a bit weird and I only threw this code together for our own purposes. If you find this helper to be worthy enough, I'm happy to iterate on it and propose a less weird solution :)
Let me know once you've looked through the helpers as you have proposed.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/54031

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/54732

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/97916


Ignore VCR Cassettes in YAML Lint

VCR cassettes are auto-generated and thus shouldn't be linted

Isn't this already in the provider plugin generator?  Not sure why core needs it too.

@Fryguy https://github.com/ManageIQ/manageiq-providers-nsxt/pull/16#issuecomment-688968753

Right I was suggesting it be in the plugins yamllint, not core's.  In core I'd only expect it in the provider generator.

Haha alright will update later

Updated to just remove the openstack tools ignore


docs: update docker and podman anchor on CUDA section

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@SomeoneSerge I thought I had more things pending on the documentation side but I think this is the only comment that was pending from your side. Please let me know if I overlooked something.
As promised I'll check how to improve the implementation with a proper closure support.

As promised I'll check how to improve the implementation with a proper closure support.

I forget if I mentioned but it's a similar problem in nix-required-mounts (it's using exportReferencesGraph but it could be still more general than that) and we should probably write a more abstract tool solving both tasks so we can test it better...


Getting started link does not work

Description
Getting started link https://swiftpackageindex.com/apple/swift-testing/main/documentation/testing/temporarygettingstarted does not work
Expected behavior
No response
Actual behavior
No response
Steps to reproduce
No response
swift-testing version/commit hash
No response
Swift & OS version (output of swift --version ; uname -a)
No response

Note: SPI now needs to re-add swift-testing, so this change will need a bit of time to percolate. Thanks for reporting!

Resolution on SPI side tracked by:
https://github.com/SwiftPackageIndex/PackageList/pull/6347
https://github.com/SwiftPackageIndex/PackageList/pull/6348
Things should be working now.


Learning paths: Add setting to include all exercises relevant to the course score

🚨🚨 Contains Database Migration 🚨🚨
Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] I strictly followed the server coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[x] I strictly followed the client coding and design guidelines.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context

#8758
Description

Instructors can now specify in the learning path management that all non-bonus and non-optional exercises (i.e. those that are relevant for the course score) have to be included in the learning paths. If this is enabled, these exercises are preferred when selecting the exercises for each difficulty. Additionally these exercises are also selected even if the mastery can be achieved with less exercises.
To save this setting a new configuration is stored in the course that saves all settings related to learning paths, since more settings might come in the future.
Steps for Testing

Prerequisites:

1 Instructor
1 Student

Log in to Artemis
Create a few exercises with different difficulties and some are included in the course score and other are optional.
Link the exercises to a competency and set the mastery threshold of the competency lower (e.g. 50%)
Look at the learning path as a student and see that not all exercises are included
Enable as instructor in the learning path management view that all exercises that are relevant to the course score have to be included
Look at the learning path as a student and see that all exercises relevant to the course score are included

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.

Review Progress

Performance Review

[x] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[x] Test 1
[ ] Test 2

Test Coverage

Client

Class/File
Line Coverage
Confirmation (assert/expect)

learning-path-management.component.ts
94.25%
✅

learning-path.service.ts
76.92%
✅

Server

Class/File
Line Coverage
Confirmation (assert/expect)

LearningPathsConfiguration.java
100%
✅

LearningPathNavigationService.java
92%
✅

LearningPathRecommendationService.java
97%
✅

LearningPathService.java
97%
✅

LearningPathResource.java
97%
✅

Screenshots

Temporary setting until @JohannesWt refactors them in a future PR:

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Introduced Learning Paths Configuration for courses, allowing enhanced customization of learning paths.
Added user interface options to include all graded exercises in learning paths.
Enhanced course creation and update processes to support learning paths configuration.
New API endpoints for retrieving and updating learning paths configurations.

Bug Fixes

Improved query methods for fetching courses with specific learning paths configurations.

Refactor

Updated various services and repositories to integrate Learning Paths Configuration.
Enhanced learning path recommendation and navigation logic to utilize the new configuration settings.

Tests

Expanded integration tests to validate new functionalities related to Learning Paths Configuration.

Should learning paths display exercises that weren't released yet?

This is fixed in #9105, feel free to review as well

And there is a failing server test

Oh, fixed it!


Dual licensing

I'm looking into using the TIPLOC data compiled at https://github.com/oweno-tfwm/YA_Tiploc_List, which uses some data from woodpecker.
While most of its data sources use OGL3 or CC-BY(-SA) licenses, woodpecker uses Apache 2.0.
I was wondering whether you'd be open to dual licensing the data in the repo under CC-BY 4.0 or CC-BY-SA 4.0 to allow the STANOX data to be combined with these other data sources more easily.
My plan for the data is to use the combined data from YATIPLOCL and snap those locations back to the latest Network Rail network GIS data and republish this as (yet another) copyleft dataset.
Thanks in advance, and please let me know if you have any questions.

Good morning David, apologies for the delay in getting to this. I'll update the license now and make the data output CC-BY-SA.
If that works, I'll close the issue.

I have added a link in the README.md that indicates data is covered by the CC-BY-SA 4.0. I can't find a text copy of the corresponding CC-BY license so this is a link, but if there is a copy I'll update the LICENSE file or somesuch.

I think I found a plaintext deed here: https://creativecommons.org/licenses/by-sa/4.0/legalcode.txt

With your agreement as the data attribution license is now CC-BY-SA 4.0, I would propose closing the this. Unlike the Apache license, I can't see where I would put or update the CC-BY-SA license in this case.

Yeah, perfectly happy to close this now. Thanks for this. :)


Support for private repositories

Made it simpler to handle private repositories by adding the following properties to each subchart's image:

Property
Default
Descripton

registry
docker.io
The domain of the registry to pull the image from

name

The name of the image to pull

tag
1.1.0
The tag of the image to pull

usePullSecret
false
Indicates if the image should be pulled using authentication from a secret

pullSecretName

The name of the pull secret to attach to the respective serviceaccount used to pull the image

Example of custom property usage:
splunk-kubernetes-logging:
  image:
    registry: artifactory.foo.com
    name: splunk/fluentd-hec
    tag: 1.1.0

splunk-kubernetes-objects:
  image:
    registry: artifactory.foo.com
    name: splunk/kube-objects
    tag: 1.1.0
    usePullSecret: true
    pullSecretName: artifactory-registry

splunk-kubernetes-metrics:
  image:
    registry: artifactory.foo.com
    name: splunk/k8s-metrics
    tag: 1.1.0
    usePullSecret: true
    pullSecretName: artifactory-registry
  imageAgg:
    registry: artifactory.foo.com
    name: splunk/k8s-metrics-aggr
    tag: 1.1.0
    usePullSecret: true
    pullSecretName: artifactory-registry

LGTM, will test it and merge it.

Ok, great. Let mek now if there is anything else I can do.

@McShauno can you rebase this PR to the latest develop branch. Thanks.

@chaitanyaphalak It has now been rebased.  Thank you.

Hey @McShauno are you still working on this one? I might be able to do it for the community is that's ok with you.

@chaitanyaphalak Yes, please feel free to do it.  I have gotten busy with some other things at the moment.

@McShauno I did a minor refactor and rebase to the latest develop branch. https://github.com/splunk/splunk-connect-for-kubernetes/pull/222/files
Thanks for all the hard work @McShauno.

@chaitanyaphalak Thank you! Looks great. Appreciate it.


Improve copy_tutorial step

Description
Reported by several first time users. When following the tutorial on https://docs.atoti.io/0.3.1/tutorial.html:

It's not clear what it does and users do not know that a new folder is created. What would be great is:

add an additional sentence in the documentation mentioning that this will create a new folder called "tutorial" at the same location you are running the notebook from.
add some message feedback from the notebook like : "Tutorial created at ${full_path}. Check out ${full_path}/01. Getting Started.ipynb"

Also, should it be copy_tutorial or create_tutorial?

We have realized that running the command in a notebook is confusing as the user wants to keep this notebook and keep copy-pasting. In the next release we have added this command that can be run from command line:
python -m atoti.copy_tutorial tutorial

It then logs where the tutorial has been copied
We have also clarified a bit this part of the documentation


CodeSandbox react-scripts build issues

Repro: https://codesandbox.io/s/competent-thompson-tyrqc6?file=/src/App.js
Cannot use 'import.meta' outside a module

When downloading the codesandbox and running it locally, or using a CodeSandbox cloud sandbox, there's no issue.
Cloud sandbox: https://codesandbox.io/p/sandbox/musing-tu-yc8ll2

https://github.com/codesandbox/codesandbox-client/issues/8086


logger: use WarnLvl instead of prepending logs with "WARNING"

Hello @hyu, @maiamcc,
Please review the following commits I made in branch nicks/warnings:
2f5bf10e9670255ea8a2b02429e8fca5ca381293 (2020-01-07 13:02:59 -0500)
logger: use WarnLvl instead of prepending logs with "WARNING"

wooooo i think this is going to be life-changing


Auto-indent for Lua5.3 Is Driving Me Mad

Issue Type: Bug
Try using "if" statement inside a function, the "end" on a new line will automatically go to the head of the line ( without any indents!).
The result is the same even after I uninstall every Lua-related extention.
Never fussed about the detail settings for VSCode myself,  it took me some time before I could find the "editor.autoIndent" setting and disable the feature entirely.
VS Code version: Code 1.31.1 (1b8e8302e405050205e69b59abb3559592bb9e60, 2019-02-12T02:20:54.427Z)
OS version: Windows_NT x64 10.0.17134

I'm seeing this too, and it only started happening in a recent version of VS Code. I'll see if I can figure out which one.

This issue is occurring for me as well with Lua 5.1 on macOS (10.14.1) with both versions 1.31.0 and 1.31.1. Reverting my installation to 1.30.2 causes the issue to go away.
Additionally it seems as though the issue is not limited to if-then-end blocks but affects auto indentation in general.

I found this issue too, I reverted my release version to 1.30.2, and wait for the solusion...

Duplicate of #68876. I can reproduce this too.


Create Architecture of the APP

Discuss About DB structure
Discuss and come up with final Plan of the App
Come Up with basic structure of the design and features we want to implement

Come up with a user schema


Doc: Updates to prevent dangling references.

Set up ancillary file to hold system types to move toward a clean doc build.

I'm not sure about having some of the types in separate files (e.g. TSReturnCode) but at least we shouldn't have duplicates. We can consolidate those in the future if desired.


add code to handle submissions with v2 scheme

I really don't love that we need to check the version number so many times and then perform what amounts to exactly the same action.
I would prefer if we could fetch the sections we need in a smarter way. Perhaps by using question IDs since those are likely to never change even if the structure of the sections changes.
I also know we are short on time though so if we need to just get this done so we can move on to spotlight then I understand. Thoughts @iaincooper-tco ?

I really don't love that we need to check the version number so many times and then perform what amounts to exactly the same action.
I would prefer if we could fetch the sections we need in a smarter way. Perhaps by using question IDs since those are likely to never change even if the structure of the sections changes.
I also know we are short on time though so if we need to just get this done so we can move on to spotlight then I understand. Thoughts @iaincooper-tco ?

I think we could have done this in a slightly smarter way as those section IDs are really just strings.
We only really needed to do that version check once, and setup the correct section name to reference based on that rather than checking multiple times.


Improve documentation of Transactions and PDUs

Try to make these lie a little less.

@richvdh thank you for documenting this, it was something I was about to deal with, so awesome timing!


Build failure: manual-combined.drv

Steps To Reproduce
Steps to reproduce the behavior:

build manual-combined

Build log
error: builder for '/nix/store/fmgpd7jcrycmxw1qbf2i2g4mbqgrz2mx-manual-combined.drv' failed with exit code 3;
       last 10 log lines:
       > /nix/store/98dz83rz2iz3k7ffqs9rlhmlxbga1bwp-manual-combined/manual-combined.xml:1402: element listitem: Relax-NG validity error : Did not expect element listitem there
       >   1398        not bound to an attribute.</simpara>
       >   1399    <simpara>Some arguments are already passed by default, of which the
       >   1400     following <emphasis>cannot</emphasis> be changed with this option:</simpara>
       >   1401    <itemizedlist>
       >   1402  <listitem><para><simpara><varname>lib</varname>: The nixpkgs library.</simpara>
       >   1403 </para></listitem>
       >   1404      <listitem><para><simpara><varname>config</varname>: The results of all options after merging the values from all modules together.</simpara>
       >
       > /nix/store/98dz83rz2iz3k7ffqs9rlhmlxbga1bwp-manual-combined/manual-combined.xml fails to validate
       For full logs, run 'nix log /nix/store/fmgpd7jcrycmxw1qbf2i2g4mbqgrz2mx-manual-combined.drv'.
error: 1 dependencies of derivation '/nix/store/gyyxw20qp6j3371dzrp431pwwa02gq5f-man-pages.drv' failed to build
error: 1 dependencies of derivation '/nix/store/qbr76xld04kc57s8s72s1dn2jqkmzvdm-home-manager-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/bzgk63asds3vi4f16v8xnj8r3l0y3hq6-man-pages-fish-completions.drv' failed to build
error: 1 dependencies of derivation '/nix/store/9p89688bh58kwcbynijpn1h7ivnsiasq-man-paths.drv' failed to build
error: 1 dependencies of derivation '/nix/store/n4mppz9x7a0b5vpsf3zppb262p296g5v-home-manager-generation.drv' failed to build
error: 1 dependencies of derivation '/nix/store/sm2smwql389bmqi78s9hsjzbqgrlna6v-user-environment.drv' failed to build
error: 1 dependencies of derivation '/nix/store/1vcq3g5xb6vs8b2snp6g8hd2g6x9mg8g-etc.drv' failed to build
error: 1 dependencies of derivation '/nix/store/m8vqav97hs76ajd1rix2n76z23gm6a8r-nixos-system-Compootah-23.05.20221130.e76c78d.drv' failed to build

Additional context
I have no idea what package is causing this but here are all i have installed, it is through home manager it fails as without the home module it builds https://github.com/Eken-beep/Nix-Dotfiles/blob/25d1c4af391ff3e3f343a0d948f94ecbeded6b80/nix-user/home.nix
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.00 MiB download, 0.00 MiB unpacked):
  /nix/store/pf0xc9gryi9dyp4c6zmynnp3blc547z9-nix-info
copying path '/nix/store/pf0xc9gryi9dyp4c6zmynnp3blc547z9-nix-info' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.0.10, NixOS, 23.05 (Stoat), 23.05.20221130.e76c78d`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.0`
 - channels(edvin): `"nixpkgs"`
 - channels(root): `"home-manager, nixos"`
 - nixpkgs: `/home/edvin/.nix-defexpr/channels/nixpkgs`

I upgraded with sudo nix-channel --update and still the same error message, doing all that as non root yields the same results

I upgraded home-manager, nixpkgs and nix. It is currently building at 600smth of 817. Seems to work correctly now but dont know what fixed it, thank you.


Cannot find node.dll

I have installed node, After  that I have followed the instructions to run this app.
In step 3 it is mentioned that a copy of node.exe and node.dll should be copied in the root directory of project. So, I searched for Node.exe and Node.dll in the folder where I installed node.
I found Node.exe but I am unable to find node.dll.
The problem here is when I run the app, I can see a window. But it has a round progressbar. I don't get any content in short.

I'll try to make it more clear in the README where and why you need these. Get the node.exe and node.lib from here:
64bit - https://nodejs.org/dist/v6.3.1/win-x64/
32bit - https://nodejs.org/dist/v6.3.1/win-x86/

Thank you. The issue is solved. But after that I got another issue.  I can't see the text below welcome. The size of my window is 800 x 600 or full-screen, in either case the size of  object#browser-plugin-1 is 300 x 150. Also, the heading "Express" is cut-down. Why is it so?

I'm not sure I understand what issue you are experiencing now

I'm having the same issue, any pointer to get this running OSX? The express-app runs fine stand alone but not when I start Electron. Just getting the spinning loader also.
events.js:160 Uncaught Error: spawn node.exe ENOENT

I don't have a Mac so I cannot test this

No worries. Obviously node.exe being called on a mac won't work. I'll see if I can find a workaround and let you know. Cheers

That was easier than I thought - for OSX/Linux just change node.exe to node in index.html line 49
node = spawn("node.exe", ["./express-app/bin/www"], { cwd: process.cwd() }),

@jimwild you rock!

@frankhale I am running windows. Can you please provide a working sample without clipping of body as shown in above image?

@Vishal1419 looks like the CSS to make the webview the size of the window isn't working in your case. I'll test today to see if I can reproduce this in the latest version of Electron.

@Vishal1419 I just tested on Windows 10 and I cannot reproduce the behavior that you are seeing with respect to the size of the webview.
The following CSS should be kicking in for you:
.fullSize {
        position: absolute;
        width: 100%;
        height: 100%;
        display: none;
        overflow: auto;
 }

The webview should be using the class which makes it the full size of the window:
<webview id="expressApp" class="fullSize"></webview>

@Vishal1419 actually I can reproduce this. It looks like the webview does have the correct size, eg. it's being set to 100% width and height but the object that is inside the webview which contains the content isn't sized the same so the content is clipped like you see. I'll see what I can do to force the object inside the webview to be the full size of the window.

Needed to use flex layout. I guess this is a new thing added in recent versions of Electron.
http://electron.atom.io/docs/api/web-view-tag/#css-styling-notes
This is fixed in commit 861d81d13012bdf5bc806723071a344f075df86c

I have tested it further. And I can see that initially webview is sized to 100% of the available size. And then when I change the size of the window, the size of webview (or components inside it) does not have a change in width. So, a responsive approach is needed here. I don't know how

The resizing of the webview should be fixed in commit 861d81d. I just checked and when resizing the webview and it's contents are correctly resized based on the size of the window.

Here is an animated gif showing resizing content is working as expected.

@frankhale Thank you very much


CI: Integration tests - github.com/cilium/cilium/pkg/azure/ipam: FAIL: Test/IPAMSuite/TestIpamManyNodes

CI failure
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃   FAIL  package: github.com/cilium/cilium/pkg/azure/ipam   ┃
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛

--- FAIL: Test (0.16s)
--- FAIL: Test/IPAMSuite (0.16s)
--- FAIL: Test/IPAMSuite/TestIpamManyNodes (0.11s)

level=info msg="Synchronized Azure IPAM information" numInstances=100 numSubnets=3 numVirtualNetworks=1 subsys=azure
level=info msg="Discovered new CiliumNode custom resource" name=node0 subsys=ipam
level=info msg="Discovered new CiliumNode custom resource" name=node1 subsys=ipam
level=info msg="Discovered new CiliumNode custom resource" name=node2 subsys=ipam
... [skip]
level=info msg="Synchronized Azure IPAM information" numInstances=100 numSubnets=3 numVirtualNetworks=1 subsys=azure
level=info msg="Synchronized Azure IPAM information" numInstances=100 numSubnets=3 numVirtualNetworks=1 subsys=azure
level=info msg="Synchronized Azure IPAM information" numInstances=100 numSubnets=3 numVirtualNetworks=1 subsys=azure
    ipam_test.go:346:
        ... obtained int = 980
        ... expected int = 1000

make: *** [Makefile:130: integration-tests] Error 1

logs_1773835.zip
https://github.com/cilium/cilium/actions/runs/5899931931/job/16003248099

Maybe this is the reason:
level=warning msg="Unable to compute pending pods, will not surge-allocate" error="pod store uninitialized" instanceID=vm28 name=node28 subsys=ipam

(Doesn't look like a repetition of https://github.com/cilium/cilium/issues/27080)

Hit here as well: https://github.com/cilium/cilium/actions/runs/5939740701/job/16107642145


[BUG] Fix the File Validator Statistics chart in App Insights Prod Dashboard

Describe the bug
The File Validator Statistics chart in the App Insights Dashboard is not working.
Impact on ReportStream
RS team members have no visibility into File Validator usage.
Steps to reproduce
Steps to reproduce the behavior:

Go to App Insights Prod Dashboard
Go to File Validator Statistics chart
It displays an error

Expected behavior
It should not display an error and should display the File Validator Statistics
Resources

Device info
Desktop (please complete the following information)

OS: Mac
Browser Chrome
Version

Issue fixed

Thank you!


how to debug lite jni so in android

after building the so of lite, some how i need to debug in the android mobile; how can i debug the native c++ code using android studio?  appriciated

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Here is the guideline to debug code in android studio: https://developer.android.com/studio/debug/.
You can use LLDB to debug your native code. If you want to debug the whole flow: java --> jni --> native, I would recommend you to use logs for debugging.