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

Make blobstore default nginx configurable #26

Merged
merged 1 commit into from
Aug 2, 2016
Merged

Make blobstore default nginx configurable #26

merged 1 commit into from
Aug 2, 2016

Conversation

aemengo
Copy link

@aemengo aemengo commented Jul 25, 2016

Thanks for contributing to the capi_release. To speed up the process of reviewing your pull request please provide us with:

  • A short explanation of the proposed change:
    Sets the port of the default blobstore nginx server to blobstore.port
  • An explanation of the use cases your change solves
    The blobstore conflicts with gorouter when colocated. This allows us to run them on separate ports.
  • I have viewed signed and have submitted the Contributor License Agreement
  • I have made this pull request to the master branch
  • I have run CF Acceptance Tests on bosh lite
  • So that it doesn't conflict with gorouter

Signed-off-by: Anthony Emengo aemengo@pivotal.io

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
* So that it doesn't conflict with gorouter

Signed-off-by: Anthony Emengo <aemengo@pivotal.io>
@cfdreddbot
Copy link

Hey aemengo!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/127012923

The labels on this github issue will be updated when the story is started.

@thausler786 thausler786 merged commit f335ddb into cloudfoundry:master Aug 2, 2016
@thausler786
Copy link
Contributor

Thanks for the pull request, @aemengo! This port is also used by the public endpoint of the blobstore, but because of how nginx resolves where to send requests it should not interfere with traffic to that endpoint. Also, it passes CATS. Merged!

capi-bot added a commit that referenced this pull request Jun 22, 2023
Bump src/code.cloudfoundry.org/tps
  dependabot[bot]:
     Bump github.com/onsi/ginkgo/v2 from 2.9.2 to 2.11.0 (#26)
capi-bot added a commit that referenced this pull request Nov 9, 2023
…/tps

Bump src/code.cloudfoundry.org/cc-uploader
  dependabot[bot]:
     Bump github.com/onsi/gomega from 1.29.0 to 1.30.0 (#26)
Bump src/code.cloudfoundry.org/tps
  dependabot[bot]:
     Bump github.com/onsi/gomega from 1.29.0 to 1.30.0 (#44)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants