Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Bug/1252005 #30

Closed
wants to merge 3 commits into
from

Conversation

4 participants
Contributor

babitha31 commented Dec 2, 2013

No description provided.

@babitha31 babitha31 closed this Dec 2, 2013

@babitha31 babitha31 reopened this Dec 2, 2013

Contributor

mrunge commented Dec 2, 2013

thank you for the patch. horizon uses gerrit for code contributions and launchpad for bug tracking.

Contributor

babitha31 commented Dec 2, 2013

could you please help me in setting up with gerrit to contribute this patch

Contributor

Ladas commented Dec 2, 2013

I think following this is the best option: https://wiki.openstack.org/wiki/GerritWorkflow ,thank you for your contribution.

Contributor

babitha31 commented Dec 2, 2013

thank you ladas.even i followed the same procedure.It works fine upto "git commit -a".But when i enter the command "git review",an exception raises as
ssh: could not resolve hostname ssh: Name or service not known
fatal: The remote end hung up unexpectedly

error: could not fetch gerrit

I got stuck here since few weeks.could you pls help me in sorting this out.

Contributor

Ladas commented Dec 2, 2013

Do you have 'Change-Id:' in your commit message?

Also have you tried this? git remote add gerrit ssh://@review.openstack.org:29418/openstack/nova.git

The user have to exist, and you have to add your RSA public key here https://review.openstack.org/#/settings/ssh-keys

Contributor

babitha31 commented Dec 2, 2013

the second and third steps have been done without any error.But I didn't get any Change-Id when i commit my code

Thank you for contributing to openstack/horizon!

openstack/horizon uses Gerrit for code review.

Please visit http://wiki.openstack.org/GerritWorkflow and follow the
instructions there to upload your change to Gerrit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment