-
Notifications
You must be signed in to change notification settings - Fork 24
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
Website directs users to the old gerrit #34
Comments
I believe I wrote that and it is still accurate, but you have to keep reading the entire sentence. Immediately after the part you quoted it continues to say :
and install the git-review extension<https://docs.openstack.org/infra/git-review/> which will allow you to submit specs as well as code changes.
and if you click on the hyperlink it takes you to this documentation https://docs.openstack.org/infra/git-review/ although doing a google or other web search would get you this somewhat more detailed documentation https://www.mediawiki.org/wiki/Gerrit/git-review
Your attempts to do a “git push” instead of a “git review” are why you are unsuccessful.
Although https://gerrit.tungsten.io/r/admin/repos/q/filter:specs should eventually replace https://review.opencontrail.org/#/admin/projects/?filter=specs the current status is that the current status is that the tungsten.io one is empty and all specs are in the opencontrail.org one. After the repo is copied from the later to the former, then process will still be “git review”, not “git push”, but the remote will be the tungsten.io one.
…--
Paul Carver
VoIP: 732-545-7377
Cell: 908-803-1656
E: [email protected]<mailto:[email protected]>
h=6.626 070 15 × 10-34
e=1.602176634 × 10-19
k= 1.380649 × 10-23
NA=6.02214076 x 1023
From: Matthew Davis <[email protected]>
Sent: Wednesday, May 15, 2019 22:46
To: tungstenfabric/website <[email protected]>
Cc: Subscribed <[email protected]>
Subject: [tungstenfabric/website] Website directs users to the old gerrit (#34)
This page<https://urldefense.proofpoint.com/v2/url?u=https-3A__tungstenfabric.github.io_website_&d=DwMFaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=HBNonG828PGilNRNwXAtdg&m=CGN5St_EMoPg_veV8A40jrzv5IIrj4zSvRBLDW1ZuT4&s=0guEz80GH8gJR5TaPL9XIooRdnL5xyeJ850pHayljws&e=> says:
Although contrail-specs and other Tungsten Fabric repositories are hosted on GitHub, they are managed with Gerrit. Please don’t send Pull Requests to the GitHub repositories, and go to https://review.opencontrail.org<https://urldefense.proofpoint.com/v2/url?u=https-3A__review.opencontrail.org&d=DwMFaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=HBNonG828PGilNRNwXAtdg&m=CGN5St_EMoPg_veV8A40jrzv5IIrj4zSvRBLDW1ZuT4&s=2HsoleP6D3oIWMuDRMFag7pkQOIhwNM3iiZX3Lnae0A&e=> instead. Blueprint specs are submitted this way as well.
After getting Gerrit access as described below, clone the specs repository with the command: git clone https://review.opencontrail.org/Juniper/contrail-specs<https://urldefense.proofpoint.com/v2/url?u=https-3A__review.opencontrail.org_Juniper_contrail-2Dspecs&d=DwMFaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=HBNonG828PGilNRNwXAtdg&m=CGN5St_EMoPg_veV8A40jrzv5IIrj4zSvRBLDW1ZuT4&s=z-Tqx9eKlszm2Q-xzFtyScQXq-y85GhZm7DNfM5clz4&e=>
Based on my discussions in the mailing list, it sounds like this information is out of date. My attempts to push to to that have been rejected. The new Gerrit instance is gerrit.tungsten.io, and I think we're supposed to push contributions there.
Is that correct? I would make a pull request correcting this, but I'm not sure what the truth of the matter is.
I am unable to even pull from the new gerrit instance, and on the mailing list Will said that no components have moved to the new instance. So it sounds like it's not currently possible to contribute code to either location.
Whatever the current state of the project is, the website should be updated to reflect it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_tungstenfabric_website_issues_34-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DAA4MRDY5M5OCEEAEADIX5CTPVTDGZA5CNFSM4HNIPBCKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GUCASXQ&d=DwMFaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=HBNonG828PGilNRNwXAtdg&m=CGN5St_EMoPg_veV8A40jrzv5IIrj4zSvRBLDW1ZuT4&s=xg7pkMPvtuNMXrJKaEQb0VifntfqqmGD3pLsK4FrA9Q&e=>, or mute the thread<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AA4MRD7XPGN2GTK76VCYPNLPVTDGZANCNFSM4HNIPBCA&d=DwMFaQ&c=LFYZ-o9_HUMeMTSQicvjIg&r=HBNonG828PGilNRNwXAtdg&m=CGN5St_EMoPg_veV8A40jrzv5IIrj4zSvRBLDW1ZuT4&s=clxrbBma8P04ss4CjktV87uHhHYfP-3diKNiCvxnipA&e=>.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This page says:
Based on my discussions in the mailing list, it sounds like this information is out of date. My attempts to push to to that have been rejected. The new Gerrit instance is gerrit.tungsten.io, and I think we're supposed to push contributions there.
Is that correct? I would make a pull request correcting this, but I'm not sure what the truth of the matter is.
I am unable to even pull from the new gerrit instance, and on the mailing list Will said that no components have moved to the new instance. So it sounds like it's not currently possible to contribute code to either location.
Whatever the current state of the project is, the website should be updated to reflect it.
The text was updated successfully, but these errors were encountered: