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.

Sign up for GitHub

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

Jump to bottom

Remove postgresql13-devel package dependency from Traffic Ops rpm #6266

Closed
rawlinp opened this issue Oct 11, 2021 · 0 comments · Fixed by #6449
Closed

Remove postgresql13-devel package dependency from Traffic Ops rpm #6266

rawlinp opened this issue Oct 11, 2021 · 0 comments · Fixed by #6449
Labels
database relating to setup/installation/structure of the Traffic Ops database dependencies Pull requests that update a dependency file improvement The functionality exists but it could be improved in some way. low difficulty the estimated level of effort to resolve this issue is low medium impact impacts a significant portion of a CDN, or has the potential to do so tech debt rework due to choosing easy/limited solution Traffic Ops related to Traffic Ops
Milestone
6.1.0

Comments

@rawlinp
Copy link
Contributor

rawlinp commented Oct 11, 2021

This Improvement request (usability, performance, tech debt, etc.) affects these Traffic Control components:

  • Traffic Ops

Current behavior:

The Traffic Ops rpm requires the postgresql13-devel package to be installed as a dependency.

New behavior:

The Traffic Ops rpm should not require the postgresql13-devel package to be installed as a dependency. The package description contains the following:

The postgresql-devel package contains the header files and libraries
needed to compile C or C++ applications which will directly interact
with a PostgreSQL database management server.

Traffic Ops is not a C or C++ application -- it is a Go application now. This package might've been necessary for TO-Perl, but it should no longer be necessary for TO-Go. Removing this as a package dependency makes the install more easy and lightweight.
@rawlinp rawlinp added Traffic Ops related to Traffic Ops tech debt rework due to choosing easy/limited solution improvement The functionality exists but it could be improved in some way. labels Oct 11, 2021
@zrhoffman zrhoffman added this to the 6.0.1 milestone Oct 11, 2021
@zrhoffman zrhoffman modified the milestones: 6.0.1, 6.0.2 Nov 4, 2021
@mitchell852 mitchell852 added the medium impact impacts a significant portion of a CDN, or has the potential to do so label Nov 16, 2021
@rawlinp rawlinp added the low difficulty the estimated level of effort to resolve this issue is low label Dec 7, 2021
@mitchell852 mitchell852 modified the milestones: 6.0.2, 6.1.0 Jan 4, 2022
@ragen14 ragen14 mentioned this issue Jan 6, 2022
Merged
4 tasks
@zrhoffman zrhoffman added dependencies Pull requests that update a dependency file database relating to setup/installation/structure of the Traffic Ops database labels Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
database relating to setup/installation/structure of the Traffic Ops database dependencies Pull requests that update a dependency file improvement The functionality exists but it could be improved in some way. low difficulty the estimated level of effort to resolve this issue is low medium impact impacts a significant portion of a CDN, or has the potential to do so tech debt rework due to choosing easy/limited solution Traffic Ops related to Traffic Ops
Projects
None yet
Milestone
6.1.0
Development

Successfully merging a pull request may close this issue.

removing postgresql13-devel package requirement ragen14/trafficcontrol
3 participants
@mitchell852 @rawlinp @zrhoffman