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

Bugfix/compare profile page #7885

Merged
merged 3 commits into from Dec 19, 2023

Conversation

rimashah25
Copy link
Contributor

Compare Profile Page isn't working in TP. This PR fixes the same.


Which Traffic Control components are affected by this PR?

  • Traffic Ops
  • Traffic Portal

What is the best way to verify this PR?

Login to TP, Click on Configure -> Profiles -> More -> Compare Profiles.
Select any two profiles and hit comapare.
Without this fix, the below page will not appear and the user is redirected to profile page (list of profiles in Table)
Wit this fix, the user will see the below page
image

If this is a bugfix, which Traffic Control versions contained the bug?

7.1.0

PR submission checklist

Fixed get profiles to return list of asscoaited parameters to a profile.
Copy link

codecov bot commented Dec 18, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (36e3ae1) 28.87% compared to head (19e4b85) 31.81%.
Report is 1 commits behind head on master.

Files Patch % Lines
traffic_ops/traffic_ops_golang/profile/profiles.go 0.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #7885      +/-   ##
============================================
+ Coverage     28.87%   31.81%   +2.93%     
  Complexity       98       98              
============================================
  Files           602      719     +117     
  Lines         77355    82729    +5374     
  Branches         90      970     +880     
============================================
+ Hits          22339    26319    +3980     
- Misses        52925    54247    +1322     
- Partials       2091     2163      +72     
Flag Coverage Δ
golib_unit 53.85% <ø> (ø)
grove_unit 12.02% <ø> (ø)
t3c_unit 5.88% <ø> (+0.11%) ⬆️
traffic_monitor_unit 25.47% <ø> (ø)
traffic_ops_integration 69.42% <ø> (ø)
traffic_ops_unit 21.63% <0.00%> (-0.05%) ⬇️
traffic_portal_v2 74.37% <ø> (?)
traffic_stats_unit 10.78% <ø> (ø)
unit_tests 29.13% <0.00%> (+3.43%) ⬆️
v3 57.79% <ø> (ø)
v4 79.18% <ø> (ø)
v5 78.58% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@srijeet0406 srijeet0406 left a comment

Choose a reason for hiding this comment

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

LGTM, just waiting for the tp integration tests to pass.

@srijeet0406 srijeet0406 merged commit 83556e8 into apache:master Dec 19, 2023
44 checks passed
@rimashah25 rimashah25 deleted the bugfix/compare_profile_page branch December 20, 2023 15:47
rimashah25 added a commit that referenced this pull request Jan 3, 2024
* Fixed Compare Profiles Page Issue
Fixed get profiles to return list of asscoaited parameters to a profile.

* Updated changelog

* using a defined constant instead of hard coded value

(cherry picked from commit 83556e8)
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

3 participants