Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: migrate ui to c8y version 1020 #62

Merged
merged 3 commits into from
Jan 26, 2025

Conversation

ck-c8y
Copy link
Collaborator

@ck-c8y ck-c8y commented Oct 31, 2024

This PR contains the upgrade of the UI for the container plugin from webSDK 1018/angular 15 -> webSDK 1020/angular 17

@reubenmiller
Copy link
Collaborator

@ck-c8y There are some conflicts in the PR (I'm guessing you just forgot to sync your fork with the upstream). Can you sync your fork, then rebase this PR?

Copy link

github-actions bot commented Nov 1, 2024

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass % ⏱️ Duration
30 0 0 30 100 7m21.274574s

Passed Tests

Name ⏱️ Duration Suite
Get Container Logs 5.383 s Container-Logs
Get Container Logs with only last N lines 0.515 s Container-Logs
Remove Container 6.542 s Container-Remove
Remove Container Non Existent Container Should Not Through An Error 0.061 s Container-Remove
Update to tedge-container-plugin-ng 24.967 s Installation
Check for Update 10.746 s Operations-Clone
Clone Existing Container 25.375 s Operations-Clone
Clone Existing Container by Timeout Whilst Waiting For Exit 17.909 s Operations-Clone
Clone Existing Container but Waiting For Exit 25.380 s Operations-Clone
Ignore Containers With Given Label 20.975 s Operations-Clone
Install/uninstall container package from private repository - credentials file 15.488 s Operations-Private-Registries
Install/uninstall container package from private repository - credentials script 20.272 s Operations-Private-Registries
Install/uninstall container package from private repository - credentials script with cache 20.636 s Operations-Private-Registries
Install/uninstall container package from private repository - engine credentials 15.174 s Operations-Private-Registries
Install/uninstall container package from private repository - docker from docker 34.075 s Operations-Private-Registries
Get Configuration 4.965 s Operations
Install/uninstall container-group package 22.626 s Operations
Install invalid container-group 2.760 s Operations
Install container-group with multiple files - app1 15.850 s Operations
Install container-group with multiple files - app2 11.630 s Operations
Install/uninstall container package 12.034 s Operations
Install/uninstall container package from file 7.033 s Operations
Manual container creation/deletion 14.742 s Operations
Manual container creation/deletion with error on run 7.376 s Operations
Manual container created and then killed 9.780 s Operations
Self Update Is Present Using Self Type 0.149 s Self
Self Update Is Present Using Container Type 0.149 s Self
Self Update Is Not Present 0.146 s Self
Service status 17.678 s Telemetry-Main
Sends measurements 70.858 s Telemetry-Main

Copy link
Collaborator

@reubenmiller reubenmiller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for maintaining the UI, much appreciated

@ck-c8y
Copy link
Collaborator Author

ck-c8y commented Dec 2, 2024

@reubenmiller: Could you please merge my changes?

@reubenmiller reubenmiller changed the title Feat migration 1020 feat: migrate ui to c8y version 1020 Jan 26, 2025
@reubenmiller reubenmiller changed the base branch from next to main January 26, 2025 17:13
@reubenmiller reubenmiller added this pull request to the merge queue Jan 26, 2025
Merged via the queue into thin-edge:main with commit 7e8a7e9 Jan 26, 2025
3 checks passed
@ck-c8y ck-c8y deleted the feat-migration-1020 branch January 27, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants