Skip to content
This repository was archived by the owner on Sep 21, 2022. It is now read-only.

Conversation

@joachimvaldez
Copy link

If we're deploying single node w/o a proxy, jobs.proxy_z1.networks.mysql1.static_ips.[0] will not resolve.

If we're deploying single node w/o a proxy, `jobs.proxy_z1.networks.mysql1.static_ips.[0]` will not resolve.
@cfdreddbot
Copy link

Hey joachimvaldez!

Thanks for submitting this pull request!

All pull request submitters and commit authors must have a Contributor License Agreement (CLA) on-file with us. Please sign the appropriate CLA (individual or corporate).

When sending signed CLA please provide your github username in case of individual CLA or the list of github usernames that can make pull requests on behalf of your organization.

If you are confident that you're covered under a Corporate CLA, please make sure you've publicized your membership in the appropriate Github Org, per these instructions.

Once you've publicized your membership, one of the owners of this repository can close and reopen this pull request, and dreddbot will take another look.

@cf-gitbot
Copy link
Collaborator

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

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

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

cppforlife pushed a commit to cppforlife/cf-mysql-release that referenced this pull request Dec 21, 2016
…thub.com/onsi/gomega

Bump cloudfoundry-incubator/galera-healthcheck:
  Lyle Franklin:
     Added mysqld binary location to config object instead of hardcoding it in sequence_number.go ( Travis Unknown )
  Ben Calegari:
     Alias /galera_status to / to maintain backwards compatability ( Travis Unknown )
     Update /sequence_number to function even if DB is down ( Travis Unknown )
     Change sequence_number Check() to private ( Travis Unknown )
     Test galera binary ( Travis Unknown )
     Move http response responsibility to endpoint objects ( Travis Unknown )
     Push flag parsing logic into config package ( Travis Unknown )
     Remove integration test suite ( Travis Unknown )
Bump :
  Onsi Fakhouri:
     Merge pull request cloudfoundry#112 from robdimsdale/master
     Merge pull request cloudfoundry#113 from cf-guardian/master
     update changelog: mention WithTransform and boolean matchers
     pull oracle matcher into its own package
     Merge pull request cloudfoundry#108 from jim-slattery-rs/composition
     Merge pull request cloudfoundry#111 from sykesm/verify-proto
     Merge pull request cloudfoundry#105 from sykesm/verify-form
     add 1.5 to travis
     Merge pull request cloudfoundry#103 from rosenhouse/occurred
     ghttp server can take an io.Writer
     ghttp handles failures and panics in handlers
     Merge pull request cloudfoundry#98 from matt-royal/respond-with-json-sets-content-type
     tell travis to use the latest golang
     explain why gomega refuses to compare <nil> to <nil>
     Merge pull request cloudfoundry#96 from craigfurman/HaveOccurredMatcher-nil-check-consistency
     Merge pull request cloudfoundry#95 from jfmyers9/fix-respond-with-ptr
     Merge pull request cloudfoundry#93 from james-lawrence/directory-matcher
     Merge pull request cloudfoundry#91 from james-lawrence/file-based-matchers
     Merge pull request cloudfoundry#89 from alext/improve_succeed_output
     Merge pull request cloudfoundry#83 from hobeone/patch-1
     ContainElement no longer bails if a passed-in matcher errors
     session.Wait now emits the correct line number when failures occur
  Zhou Yu:
     Reformat
  Rob Dimsdale:
     Add VerifyBody method.
  Matthew Sykes:
     Add VerifyProtoRepresenting and RespondWithProto
     Add matchers to verify form data
  jim-slattery-rs:
     make Or() support MatchMayChangeInTheFuture() - simplify tests by using empty And() or Or() instead of Receive() with a closed channel
     make And() propagate MatchMayChangeInTheFuture() - considers both Match() cases and considers the appropriate matcher(s) in making decision - also added a comment to WithTransform.MatchMayChangeInTheFuture(), as it will not have correct behavior with non-deterministic transformer
     make WithTransform() propagate MatchMayChangeInTheFuture() from wrapped matcher
     make Not() propagate MatchMayChangeInTheFuture() from wrapped matcher
     add SatisfyAll() and SatisfyAny() as aliases for And() and Or(), respectively
     WithTransform() supports transform function with explicit types
     Added Add, Or, Not, WithTransform matchers, for composability. - Allows matchers to be composed into complex expressions that work even with the Eventually() assertion. - Also makes it easy to create new matchers -- can often write a function that composes a new matcher out of existing ones.
  Brian Cunnie:
     Standardize spelling: occured --> occurred
  Luan Santos and Matt Royal:
     Allow user to explicitly set a Content-Type header on ghttp JSON handlers
     Set Content-Type to application/json in RespondWithJSON* methods
  Craig Furman:
     Negative assertions on nil pointers to types that conform to error interface should not panic
  James Myers:
     Fix RespondWithJSONEncodedPtr
  James Lawrence:
     directory matcher string fixes
     file based matchers
  Alex Tomlins:
     Improve Succeed() failure message.
  Alex Suraci:
     query params match irregardless of order
  Daniel Hobe:
     Return prettyprinted JSON strings for errors.

Signed-off-by: Aditya Anchuri <aanchuri@pivotal.io>

[#106993650]
@cfdreddbot
Copy link

Hey joachimvaldez!

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.

@robdimsdale robdimsdale reopened this Jan 27, 2017
@cf-gitbot
Copy link
Collaborator

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

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

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

@robdimsdale
Copy link
Member

Closed and reopened to trigger the CLA bot as per its instructions above. We see that @joachimvaldez has made the membership of IBM public.

... Except we can't merge this due to the conflicts, so we will just re-implement this.

robdimsdale added a commit that referenced this pull request Jan 27, 2017
- a reimplementation of #108

[#121247455]
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.

4 participants