Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [ptp-dev] Process for commmitting PTP fix
Hi greg, Dave,
 
It has been a very long time I don't receive any mail from this group. Good to get some messages again. I have left for a very long time, but I'm recently using and working on SCI and committed something here: https://github.ibm.com/cloudland/eclipse.ptp.sci, I added it complete failover and recovery mechanism, as I'm not working on PTP anymore, I don't know if anyone needs it and might be able to merge it, I can assist if needed.
 
Best Regards

Catherine (涂宏峻)
------------------------------------------------
Tivoli China Development, IBM China Software
------------------------------------------------
Email: tuhongj@xxxxxxxxxx
Tel: 8610-82454249
 
 
----- Original message -----
From: Greg Watson <g.watson@xxxxxxxxxxxx>
Sent by: ptp-dev-bounces@xxxxxxxxxxx
To: Parallel Tools Platform general developers <ptp-dev@xxxxxxxxxxx>
Cc:
Subject: Re: [ptp-dev] Process for commmitting PTP fix
Date: Thu, Feb 22, 2018 8:42 AM
 
Great. It looks like you committed something from the master branch to ptp_9_1 because the pom.xml has the wrong parent version. The easiest thing to do would be:
 
1. Reset to the previous commit in your workspace (use Reset to: HEAD^ and Reset type: mixed)
2. Modify the pom.xml so the parent version is 9.1.4
3. Commit again using the exact commit message (including the full change-id and signed-off-by). You can get this information from the change in Gerrit.
4. Push to gerrit.
 
This should show up as a second patchset which should hopefully build.
 
Regards,
Greg
 
On Feb 21, 2018, at 5:57 PM, David Wootton <dwootton@xxxxxxxxxx> wrote:
 

Greg
I finally got Gerrit to accept this I think. Just changing signed off by didn't work, which kind of makes sense. I had to change the committer and author at the bottom of the commit window

<1D849911.gif>

Dave

<graycol.gif>Greg Watson ---02/21/2018 04:42:02 PM---It needs to be able to link your email address to your gerrit account, so I think recommitting with

From: Greg Watson <g.watson@xxxxxxxxxxxx>
To: Parallel Tools Platform general developers <ptp-dev@xxxxxxxxxxx>
Date: 02/21/2018 04:42 PM
Subject: Re: [ptp-dev] Process for commmitting PTP fix
Sent by: ptp-dev-bounces@xxxxxxxxxxx




It needs to be able to link your email address to your gerrit account, so I think recommitting with the known email address would be easiest.

Regards,
Greg
    • On Feb 21, 2018, at 4:18 PM, David Wootton <dwootton@xxxxxxxxxx> wrote:

      Greg
      If I amend the commit again to change the signed of by line to my drwootton@... address, re-commit the try the push again, will that work or will that cause more problems (It looks like the bugzilla bug 530211 has my drwootton@... email, not my IBM email). I have access to both email accounts and don't care myself which one gets any review emails.

      Dave


      <graycol.gif>"David Wootton" ---02/21/2018 03:22:01 PM---Greg Why I try that URL, Firefox insists that it displays the page at.

      From: "David Wootton" <dwootton@xxxxxxxxxx>
      To: Parallel Tools Platform general developers <ptp-dev@xxxxxxxxxxx>
      Date: 02/21/2018 03:22 PM
      Subject: Re: [ptp-dev] Process for commmitting PTP fix
      Sent by: ptp-dev-bounces@xxxxxxxxxxx

       
       

      Greg
      Why I try that URL, Firefox insists that it displays the page at.      https://git.eclipse.org/r/#/dashboard/self, where I am already logged in from a previous session and the page is titled 'Mt Reviews' with no reviews listed (which is correct)

      If I manually change the URL to       https://git.eclipse.org/r Firefox changes the URL to https://git.eclipse.org/r/r and displays a page with the message
      The page you requested was not found, or you do not have permission to view this page

      I am getting closer though. When I went to my account settings page at git.eclipse.org I realized my git/Gerrit id is dwootton, not drwootton (My registered email address is drwootton@... now and the dwootton account name is left over from prior to 2013)

      Once I used the correct account name, I tried the push to gerrit again and this time it failed with a message I was using an email address which was not registered, which is correct since my current IBM email address is not registered with git/Gerrit.

      So I went to the account settings page and tried registering a new email address. That resulted in a comfirmation email with a link being set to my IBM email address. I got the email, clicked the link and got a page that sais 'Code Review - Error Invalid token' and the registration failed :-( I tried pasting the link directly into both Firefox and Chrome and both failed.

      I tried registering a new email address a second time with the same result.

      Dave

      <graycol.gif>Greg Watson ---02/21/2018 12:42:46 PM---Hi Dave, Can you sign in to Gerrit (https://urldefense.proofpoint.com/v2/url?u=https-3A__git.eclipse

      From: Greg Watson <g.watson@xxxxxxxxxxxx>
      To: Parallel Tools Platform general developers <ptp-dev@xxxxxxxxxxx>
      Date: 02/21/2018 12:42 PM
      Subject: Re: [ptp-dev] Process for commmitting PTP fix
      Sent by: ptp-dev-bounces@xxxxxxxxxxx

       



      Hi Dave,

      Can you sign in to Gerrit (https://git.eclipse.org/r)? When you do, go to account settings and HTTP password. You'll see your username, then click on the generate password button and use the username/password combination when Eclipse asks for it.

      Greg
                    • On Feb 21, 2018, at 8:06 AM, David Wootton <dwootton@xxxxxxxxxx> wrote:

                      Greg
                      I pasted the URL into this dialog
                      <1C504177.gif>
                      I hit finish and got prompted for username and password, which I entered and got the same error
                      <1C923625.gif>

                      I also tried the URL as https://drwootton@xxxxxxxxxxxxxxx/r/ptp/org.eclipse.ptp with the same results.

                      I wonder if I am really authorized to push commits to PTP.

                      Dave

                      <graycol.gif>Greg Watson ---02/21/2018 07:44:33 AM---Dave, Change the repository URL to https://urldefense.proofpoint.com/v2/url?u=https-3A__git.eclipse.

                      From: Greg Watson <g.watson@xxxxxxxxxxxx>
                      To: Parallel Tools Platform general developers <ptp-dev@xxxxxxxxxxx>
                      Date: 02/21/2018 07:44 AM
                      Subject: Re: [ptp-dev] Process for commmitting PTP fix
                      Sent by: ptp-dev-bounces@xxxxxxxxxxx

                       




                      Dave,

                      Change the repository URL to https://git.eclipse.org/r/ptp/org.eclipse.ptp

                      Regards,
                      Greg
                                                    • On Feb 21, 2018, at 7:24 AM, David Wootton <dwootton@xxxxxxxxxx> wrote:

                                                      Greg
                                                      Now I have the signed off line in the commit and I deleted the remote I configured. Then I tried Team->Remote->Push to Gerrit again and got prompted for a password. I went to                                                       https://git.eclipse.org/r/#/settings/http-password to get the password. There was nothing in the password filed of HTTP Password settings even though I have clicked 'Generate password' before. I clicked Generate password again, got a password and copied it to the password field in the dialog prompt, which resulted in the dialog
                                                      <1E192408.gif>

                                                      I also tried my regular Eclipse account password here before I realized that wasn't what I should be using and that didn't work either.

                                                      Dave
                                                      <graycol.gif>Greg Watson ---02/20/2018 08:13:14 AM---Dave, You'll have to amend the commit, or reset to the previous commit and re-commit, to add the Sig

                                                      From: Greg Watson <g.watson@xxxxxxxxxxxx>
                                                      To: Parallel Tools Platform general developers <ptp-dev@xxxxxxxxxxx>
                                                      Date: 02/20/2018 08:13 AM
                                                      Subject: Re: [ptp-dev] Process for commmitting PTP fix
                                                      Sent by: ptp-dev-bounces@xxxxxxxxxxx

                                                       
                                                       



                                                      Dave,

                                                      You'll have to amend the commit, or reset to the previous commit and re-commit, to add the Signed off line. There's a button on the "Commit Changes" dialog to add it. You also need to add a Change-Id using the button next to it.

                                                      I'm not sure why it's saying it couldn't connect to the repository. I use https://git.eclipse.org/r/ptp/org.eclipse.ptp for the origin remote and enter the credentials through the Git dialog, so you could try that. I think you already have this configured, so you could just remove the remote you have configured with the ssh URI. Use the Git perspective to manage the repo.

                                                      Regards,
                                                      Greg
                                                                                                                    • On Feb 19, 2018, at 5:28 PM, David Wootton <dwootton@xxxxxxxxxx> wrote:

                                                                                                                      Greg
                                                                                                                      Guess I've been away from this for too long and have forgotten the process.
                                                                                                                      I committed my changes, but then realized I forgot to add the 'Signed off' line so I don't know how to fix that.

                                                                                                                      Something is also wrong with my ssh setup since I get the following when I try to push to Gerrit (Team->Remote->Push to Gerrit)
                                                                                                                      <18826309.gif>

                                                                                                                      I went to https://git.eclipse.org/r/#/settings/ssh-keys where I can upload ssh public keys and added what is supposed to be my ssh public key, which I got by opening the Key management tab in Preferences->General->Network Connections->SSH2, clicking Load Existing Key and picking the id_rsa file in that directory, then pasting the resulting public key into the git.eclipse.org page. I can see the key show up in the list at the top of the page but that still resulted in the same error when I tried the push again.

                                                                                                                      Either I have done something wrong or I need to wait more than a minute or two for updates to take effect.

                                                                                                                      Dave

                                                                                                                      <graycol.gif>Greg Watson ---02/19/2018 04:22:24 PM---Hi Dave, It looks like you already have it set up to use Gerrit. Have you tried committing your chan

                                                                                                                      From: Greg Watson <g.watson@xxxxxxxxxxxx>
                                                                                                                      To: Parallel Tools Platform general developers <ptp-dev@xxxxxxxxxxx>
                                                                                                                      Date: 02/19/2018 04:22 PM
                                                                                                                      Subject: Re: [ptp-dev] Process for commmitting PTP fix
                                                                                                                      Sent by: ptp-dev-bounces@xxxxxxxxxxx

                                                                                                                       




                                                                                                                      Hi Dave,

                                                                                                                      It looks like you already have it set up to use Gerrit. Have you tried committing your changes then pushing to remote (or push to Gerrit?)

                                                                                                                      Greg
On Feb 19, 2018, at 8:56 AM, David Wootton <dwootton@xxxxxxxxxx> wrote:

Greg
I looked up the process for committing to Gerrit where https://wiki.eclipse.org/PTP/contributing tells me to follow the steps at https://wiki.eclipse.org/CDT/git#Using_Gerrit_for_CDT substituting PTP for CDT. I got to the step where I'm supposed to update the ref mapping section and can't find that in the Git repository view, which looks like

<15148656.gif>
What do I need to do to add a RefSpec?

Dave

_______________________________________________
ptp-dev mailing list
ptp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/ptp-dev


_______________________________________________
ptp-dev mailing list
ptp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://urldefense.proofpoint.com/v2/url?u=https-3A__dev.eclipse.org_mailman_listinfo_ptp-2Ddev&d=DwICAg&c=jf_iaSHvJObTbx-siA1ZOg&r=yA1Giwd7Ls577uUKQ3fQWICGHopYggQ46OvlB30WK5M&m=dLM9RKonzIHxgM0XO32IB0T9zETWSDYwclg_G2fR1JU&s=HXCOyk4fLrTayyHosjM_qJhv1XEBmE_a9DX15m6O4dc&e=

 
_______________________________________________
ptp-dev mailing list
ptp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/ptp-dev
 

Back to the top