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

Ensure apt::update is run before installing Gitlab #388

Merged
merged 1 commit into from
Sep 22, 2021

Conversation

baurmatt
Copy link
Contributor

Pull Request (PR) description

This PR ensures that apt::update is run before installing Gitlab. Otherwise the installation will fail with E: Unable to locate package gitlab-(ce|ee)

This Pull Request (PR) fixes the following issues

@baurmatt
Copy link
Contributor Author

CI errors seem unrelated to this change.

@baurmatt baurmatt added the bug Something isn't working label Jul 23, 2021
@smortex
Copy link
Member

smortex commented Aug 27, 2021

@baurmatt CI should be fixed with GitLab 14.

Can you please rebase your changes on top of the main branch?

From your working directory:

git fetch origin         # Download the latest code we have here
git rebase origin/master # Move your commits on top of the main branch
git push -f              # Send the changes (-f is required because we re-wrote history)

@baurmatt
Copy link
Contributor Author

Thanks @smortex! I'm not near a full shell for another 2-3 weeks. If this is urgent, feel free to do it yourself. I think the permission should be sufficient.

@smortex
Copy link
Member

smortex commented Aug 27, 2021

I think I can't edit existing commit, just add new ones on top:

romain@zappy puppet-gitlab % git remote -v
origin	git@github.com:voxpupuli/puppet-gitlab.git (fetch)
origin	git@github.com:voxpupuli/puppet-gitlab.git (push)
syseleven	git@github.com:syseleven/puppet-gitlab.git (fetch)
syseleven	git@github.com:syseleven/puppet-gitlab.git (push)
romain@zappy puppet-gitlab % git branch -vv               
* apt-dependency 576d3ee [syseleven/apt-dependency: ahead 6, behind 1] Ensure apt::update is run before installing Gitlab
  master         dec357e [origin/master] Merge pull request #391 from voxpupuli/ubuntu20
romain@zappy puppet-gitlab % git push -f
ERROR: Permission to syseleven/puppet-gitlab.git denied to smortex.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

In spite of what is said above, no isse for reading the repository :-)

But no problem for waiting a bit :-) Have a nice rest away from the shell!

@smortex smortex linked an issue Sep 4, 2021 that may be closed by this pull request
@baurmatt
Copy link
Contributor Author

@smortex Looks good now :)

Copy link
Member

@smortex smortex left a comment

Choose a reason for hiding this comment

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

LGTM

@baurmatt baurmatt merged commit e5447cb into voxpupuli:master Sep 22, 2021
@baurmatt baurmatt deleted the apt-dependency branch September 22, 2021 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add repository does not trigger apt-update
2 participants