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

[Feature][Engine UI] partial pages of the engine #7602

Merged
merged 45 commits into from
Oct 25, 2024
Merged

Conversation

QiaoJ-Chen
Copy link
Contributor

This project is used to display engine information and has now completed the overview page, job page, and manager page.

@github-actions github-actions bot added the Zeta label Sep 8, 2024
@QiaoJ-Chen
Copy link
Contributor Author

once again

@QiaoJ-Chen QiaoJ-Chen closed this Sep 8, 2024
@QiaoJ-Chen QiaoJ-Chen reopened this Sep 8, 2024
@QiaoJ-Chen
Copy link
Contributor Author

20240908210512
20240908210520
20240908210524

Copy link
Member

@wuchunfu wuchunfu left a comment

Choose a reason for hiding this comment

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

@QiaoJ-Chen These packaged dist files should not be submitted

@Hisoka-X
Copy link
Member

Hisoka-X commented Sep 9, 2024

Shall we need add test case to build ui when creating PR?

@liugddx
Copy link
Member

liugddx commented Sep 10, 2024

Shall we need add test case to build ui when creating PR?

Is there any good way to test UI?

@Hisoka-X
Copy link
Member

Hisoka-X commented Sep 10, 2024

Is there any good way to test UI?

  1. Make sure it can be build normally. Just like we did on seatunnel-website. https://github.com/apache/seatunnel/blob/dev/.github/workflows/documents.yml#L50
  2. Add first test case with https://vitest.dev/guide/snapshot.html

@@ -0,0 +1 @@
# Apache SeaTunnel Engine UI
Copy link
Member

Choose a reason for hiding this comment

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

Please update the doc. How to use it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The README.md has been updated

@QiaoJ-Chen QiaoJ-Chen closed this Sep 16, 2024
@QiaoJ-Chen QiaoJ-Chen reopened this Sep 22, 2024
@dailai
Copy link
Contributor

dailai commented Oct 12, 2024

Can this feature integrate with this api #7818 to make searching logs easier.

@@ -0,0 +1,33 @@
# Apache SeaTunnel Web UI Documentation
Copy link
Member

Choose a reason for hiding this comment

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

Please update

"seatunnel-engine/telemetry"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, it has been updated just now

The Web UI of Apache SeaTunnel offers a user-friendly interface for monitoring and managing SeaTunnel jobs. Through the Web UI, users can view real-time information on currently running jobs, finished jobs, and the status of worker and master nodes within the cluster. The main functional modules include Jobs, Workers, and Master, each providing detailed status information and operational options to help users efficiently manage and optimize their data processing workflows.
![workflow.png](../../images/ui/overview.png)

## Jobs
Copy link
Member

Choose a reason for hiding this comment

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

Quick question: How can I access it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, images was ignored, I've just resolved it

Comment on lines 7 to 9
http.
enable-http: true
port: 8080
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
http.
enable-http: true
port: 8080
seatunnel:
engine:
http:
enable-http: true
port: 8080

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it had been updated


```
Then visit http://ip:8080/#/overview
```
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
```

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry,I just took care of it

## Access

Before accessing the web ui we need to enable the http rest api. first you need to configure it in the `seatunnel.yaml` configuration file
``
Copy link
Member

Choose a reason for hiding this comment

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

change `` to ```

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just took care of it

Copy link
Member

@Hisoka-X Hisoka-X 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 @QiaoJ-Chen and @liugddx!

@liugddx liugddx merged commit ba7fa23 into apache:dev Oct 25, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants