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

Fix Link to IF-MAP Spec #756

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix Link to IF-MAP Spec #756

wants to merge 1 commit into from

Conversation

tpo
Copy link

@tpo tpo commented Aug 7, 2017

maybe there's a canonical URL to the spec, however this is what I found that works

maybe there's a canonical URL to the spec, however this is what I found that works
@pcarver
Copy link
Contributor

pcarver commented Dec 10, 2018

Contrail/Tungsten Fabric does not accept GitHub pull requests. All changes should be submitted via the Gerrit code review tool. If you're not familiar with Gerrit, take a look at section 4 of https://github.com/tungstenfabric/docs/blob/master/Contributor/GettingStarted/getting-started-with-opencontrail-development.md for more info.

The key points are that you need to login to https://review.opencontrail.org with an Ubuntu One account in order to establish your SSH public key. You need to clone the repos from Gerrit rather than (or in addition to) GitHub (or manually add an additional remote to an existing repo rather than cloning if you prefer). And you need to install and use the "git review" extension to Git in order to push changes to Gerrit.

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.

2 participants