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

Consolidate mounting code #1624

Closed
wants to merge 8 commits into from
Closed

Consolidate mounting code #1624

wants to merge 8 commits into from

Conversation

DrDaveD
Copy link
Collaborator

@DrDaveD DrDaveD commented Jun 6, 2018

Description of the Pull Request (PR):

The motivation for this PR is to get ready for another PR to address issue #1207, although I think it has quite a bit of merit on its own. It eliminates a lot of duplicated code dealing with mounting and overlay mountpoint creation and moves it into a common function. Mounts are put onto a queue and then one function handles them in the order they were requested. Functionality should be almost identical as before, with only minor changes that I don't expect anyone to notice.

This fixes or addresses the following GitHub issues:

Checkoff for all PRs:

  • I have read the Guidelines for Contributing, and this PR conforms to the stated requirements.
  • I have added changes to the CHANGELOG and and documentation updates to the singularityware documentation base.
  • I have tested this PR locally with a make test
  • This PR is NOT against the project's master branch
  • I have added myself as a contributor to the contributors's file
  • This PR is ready for review and/or merge

Attn: @singularityware-admin

@DrDaveD DrDaveD mentioned this pull request Jun 11, 2018
6 tasks
@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Jun 19, 2018

Rebased against current development-2.x

@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Jun 25, 2018

Rebased again on current development-2.x

@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Jul 4, 2018

Rebased to development-2.x

@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Jul 11, 2018

Rebased again on current development-2.x

@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Jul 11, 2018

Debian travis builds failed at least as of this commit, not due to this PR.

@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Jul 15, 2018

Rebased again on development-2.x

@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Jul 24, 2018

Rebased on development-2.x at tag of 2.6.0-rc2

@GodloveD GodloveD closed this Aug 4, 2018
@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Aug 4, 2018

@GodloveD I can't reopen, please reopen it. I rebased against release-2.6.

@GodloveD
Copy link
Collaborator

GodloveD commented Aug 5, 2018

@DrDaveD we've had several conversations about this through PM and email. Seems pretty clear that there is no appetite on our end to merge this into the 2.x branch right? And the 2.x branch is now officially closed to new features. That being the case, I'm sorry but I don't see a point in re-opening this PR.

Thanks very much for your contributions. I encourage you to switch any development efforts you wish to make to the master branch (formerly development-3.0).

@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Aug 5, 2018

@GodloveD It is understood that this won't get integrated into a release but it is integrated into EPEL and an open PR helps to keep the patch up to date when I need to rebase, and it also runs the helpful travis checks. Can it be reopened or do I need to make a new PR?

@GodloveD GodloveD changed the base branch from development-2.x to release-2.6 August 6, 2018 12:02
@GodloveD
Copy link
Collaborator

GodloveD commented Aug 6, 2018

Hi @DrDaveD. If it is convenient for you, I can re-open.

EDIT: Actually, it looks like I cannot reopen this PR either. It's complaining that the PR was force-pushed or recreated? Don't know.

screen shot 2018-08-06 at 8 14 07 am

Do you want to copy the changes and submit a new PR? Alternately, I think that you can actually rebase through the git cli and you can set travis up to work on your own fork too so that your branches go through the same tests as those on the main repo.

@DrDaveD
Copy link
Collaborator Author

DrDaveD commented Aug 6, 2018

I will submit a new PR

@DrDaveD DrDaveD mentioned this pull request Aug 6, 2018
6 tasks
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