Skip to content

Add multiple etcd support.#1283

Merged
membphis merged 5 commits intoapache:masterfrom
Akayeshmantha:patch11
Mar 19, 2020
Merged

Add multiple etcd support.#1283
membphis merged 5 commits intoapache:masterfrom
Akayeshmantha:patch11

Conversation

@Akayeshmantha
Copy link
Copy Markdown
Member

Add multiple etcd address support

Issues resolved

Fix #1231

@Akayeshmantha
Copy link
Copy Markdown
Member Author

Akayeshmantha commented Mar 16, 2020

Hi @membphis

An issue in travis again

An error occurred while generating the build script.

Any idea ?

@membphis
Copy link
Copy Markdown
Member

membphis commented Mar 16, 2020

An error occurred while generating the build script.

I have restarted the Travis job. It seems fine

@Akayeshmantha
Copy link
Copy Markdown
Member Author

@membphis am so sorry for the PR rebase problem but the rebase was not done correctly for the first time it created a little mess there so I closed that PR.

@Akayeshmantha Akayeshmantha force-pushed the patch11 branch 2 times, most recently from 3e1726a to b536aa7 Compare March 17, 2020 15:02
@Akayeshmantha
Copy link
Copy Markdown
Member Author

@membphis done

Copy link
Copy Markdown
Member

@membphis membphis left a comment

Choose a reason for hiding this comment

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

We can merge it after running the test case. nice job @Akayeshmantha

@Akayeshmantha
Copy link
Copy Markdown
Member Author

thanks @membphis . It was a great learning experience

@Akayeshmantha
Copy link
Copy Markdown
Member Author

@membphis any documents to be updated ?

@moonming
Copy link
Copy Markdown
Member

@membphis any documents to be updated ?

I think you can add comments and a example in config.yaml

@Akayeshmantha
Copy link
Copy Markdown
Member Author

Akayeshmantha commented Mar 18, 2020

@membphis @moonming added more comments

@membphis membphis merged commit cee0322 into apache:master Mar 19, 2020
@membphis
Copy link
Copy Markdown
Member

@Akayeshmantha merged, many thx

SaberMaster pushed a commit to SaberMaster/incubator-apisix that referenced this pull request Jun 30, 2020
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.

[discuss] feature: support etcd cluster in Apache APISIX.

3 participants