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

duplicate apt install command #16

Closed
saisona opened this issue Feb 13, 2024 · 1 comment · Fixed by #24
Closed

duplicate apt install command #16

saisona opened this issue Feb 13, 2024 · 1 comment · Fixed by #24
Assignees

Comments

@saisona
Copy link

saisona commented Feb 13, 2024

apt-transport-https \

Hi, I've looked at your new docker file for v2.0.0 and noticed that you've installed apt-transport-https' and software-properties-common' twice.

Was this on purpose? If so, I'd like to better understand the reason, and if not, is it just a simple duplicate?

saisona added a commit to inarix/azure-devops-agent-on-kubernetes that referenced this issue Feb 13, 2024
@btungut btungut self-assigned this Dec 6, 2024
@btungut
Copy link
Owner

btungut commented Dec 6, 2024

Changes have been merged into release/v2.0.1 branch.
Please visit : b8175ee

@btungut btungut linked a pull request Dec 6, 2024 that will close this issue
btungut added a commit that referenced this issue Dec 7, 2024
* drop duplicate install lines

Closes (#16)

* Refactor Dockerfile to use ARG for base image tag and improve readability; update local-build.sh for clearer build output

* Update VSTS_AGENT_VERSION to 3.248.0 in Dockerfile

* Bump version to 2.0.1; update app version to 3.248.0 and optimize release notes

* Add compatibility matrix to README files for Helm chart and Docker tags

* Refactor Dockerfile to create a non-root user with docker group; update deployment.yaml to use trimmed image path; modify values.yaml to adjust user IDs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants