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

[stable8.2] make swift url type configurable #21118

Merged
merged 2 commits into from Jan 8, 2016

Conversation

butonic
Copy link
Member

@butonic butonic commented Dec 10, 2015

Allows configuring the Interface, typically 'public, 'internal' or 'admin'

cc @felixboehm

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @icewind1991, @MorrisJobke and @carlaschroder to be potential reviewers

@butonic
Copy link
Member Author

butonic commented Dec 10, 2015

Needs port to master

@butonic butonic self-assigned this Dec 10, 2015
@@ -77,7 +77,11 @@ protected function init() {
$serviceName = $this->params['serviceName'];
}

$this->objectStoreService = $this->client->objectStoreService($serviceName, $this->params['region']);
$urlType = null;
if ($this->params['urlType']) {
Copy link
Member

Choose a reason for hiding this comment

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

please use isset()

@MorrisJobke
Copy link
Contributor

Needs port to master

first of all we bring stuff to master and then backport stuff 🙈 ;)

@DeepDiver1975
Copy link
Member

first of all we bring stuff to master and then backport stuff 🙈 ;)

indeed

@MorrisJobke
Copy link
Contributor

@karlitschek Please approve this backport

@karlitschek
Copy link
Contributor

please backport

@felixboehm
Copy link
Contributor

This works nice.
Background here: Openstack provides 3 Swift endpoints and ownCloud used the service name "public" by default, which is ssl secured. We had some certificate issues then and want to use the "internal" service without ssl on an internal network.

@DeepDiver1975
Copy link
Member

@butonic please submit PR against master first - I'm uncertain about the state of the CI system on stable8.2 regarding swift - failing test look ugly

@MorrisJobke MorrisJobke added this to the 8.2.2-current-maintenance milestone Dec 11, 2015
@DeepDiver1975
Copy link
Member

not 8.2.2 from my perspective because not yet merged to master and both (master and this) pr fail on unit testing 👎

@butonic
Copy link
Member Author

butonic commented Dec 14, 2015

looking into master pr

@DeepDiver1975
Copy link
Member

moving to 8.2.3 - we are closing 8.2.2 tomorrow

@DeepDiver1975 DeepDiver1975 modified the milestones: 8.2.3-next-maintenance, 8.2.2-current-maintenance Dec 14, 2015
@PVince81 PVince81 changed the title make swift url type configurable [stable8.2] make swift url type configurable Dec 24, 2015
@MorrisJobke
Copy link
Contributor

Ping - the master PR is merged.

@DeepDiver1975
Copy link
Member

👍

DeepDiver1975 added a commit that referenced this pull request Jan 8, 2016
[stable8.2] make swift url type configurable
@DeepDiver1975 DeepDiver1975 merged commit f4a7991 into stable8.2 Jan 8, 2016
@DeepDiver1975 DeepDiver1975 deleted the makeurltypeconfigurable branch January 8, 2016 12:46
@carlaschroder
Copy link

What are the exact options, 'public, 'internal' or 'admin'? Are there any others?

@carlaschroder
Copy link

And which oC versions does this apply to?

@MorrisJobke
Copy link
Contributor

And which oC versions does this apply to?

bildschirmfoto 2016-01-08 um 18 26 38

8.2.3 ;) (see on the right sidebar)

@carlaschroder
Copy link

@MorrisJobke, bitter experience has taught me to not rely on the milestone :)

@MorrisJobke
Copy link
Contributor

@MorrisJobke, bitter experience has taught me to not rely on the milestone :)

Okay :)

@lock
Copy link

lock bot commented Aug 6, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants