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

[SRE-26639] Update all Tank UI References to V2 API and update Admin Logs #239

Merged
merged 4 commits into from
May 3, 2023

Conversation

Zakaria-Kofiro
Copy link
Collaborator

@Zakaria-Kofiro Zakaria-Kofiro commented Apr 24, 2023

Update all Tank UI References to V2 API and update Admin Logs

Changes include:

  • Pointing all UI references (i.e download buttons) to new V2 endpoints
    • Data Files: CSV Data download button
    • Agent Tracker: Project and Job Harness File download button
    • Projects: Project Harness File download under Scripts, Job Harness File download under Job Queue
    • Scripts: Script Harness and Tank XML downloads (index page and script edit view)
  • Added get and download job harness XML routes to the Job service
  • Creating a new non-user-facing Logging endpoint to replace the V1 endpoint pointing to Admin Logs (via LogViewer)

This PR also includes a fix for the Project endpoint where creating a project via API also adds a default Test Plan, mirroring the default project creation process.

Screenshots:

Admin Logs
Screenshot 2023-04-24 at 6 44 29 PM

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

…Logs

updating err msg

moving backend changes to another branch, only keep front end + admin logs

moving over final backend change to other branch

Update SSL test to working URL

[SRE-26639] removing deprecated report page + disabling report service UI buttons

removing instead of commenting out

Update SSL test to working URL

small fix for project creation - add default Test Plan

updating util file

Update SSL test to working URL

[SRE-26639] removing deprecated report page + disabling report service UI buttons

removing instead of commenting out

Update SSL test to working URL
@shawn-h-park shawn-h-park self-requested a review May 3, 2023 21:15
Copy link
Collaborator

@shawn-h-park shawn-h-park left a comment

Choose a reason for hiding this comment

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

Lgtm

@Zakaria-Kofiro Zakaria-Kofiro merged commit b876c7d into master May 3, 2023
@Zakaria-Kofiro Zakaria-Kofiro deleted the zkofiro/move-remaining-V1 branch May 3, 2023 21:29
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.

None yet

2 participants