Helix SwarmHelix Swarm
Loading...

karl_wirth

karl_wirth (Karl Wirth)
  • 1
    Followers
  • 0
    Following
  • 5
    Projects
Full Name
Karl Wirth
Email address
kwirth@perforce.com
Followers
douglasawh (Doug Whitfield)
Projects
  • P4Nagios
    • Owner, Member
  • P4Convert
    • Member
  • P4Jenkins
    • Member, Moderator
  • SDP
    • Member
  • P4Benchmark
    • Following
  • Activity
  • Shelves
  • Settings
  • Notifications

  • All Activity
  • Followed Activity
  • Reviews
  • Commits
  • Comments
  • Jobs
4 years agoKarl Wirth created job000751 for perforce-software-p4convert:
SVN: Not handling specific delete and branch over in same revision

Customers SVN dump causes errors and converter does not create the revisions. For... example:

117.0 A:F - trunk/US/test.txt (UTF-8)
mapping: r117 => @5

118.0 R:F - trunk/APAC/test.txt (UNKNOWN)
CASE-SENSITIVITY ISSUE:
    From node is missing from dataset; skipping!
    Please check case options and platform types.
      com.p4convert.p4.caseMode = NONE
      com.p4convert.p4.lowerCase = false

I can reproduce this with customers SVN dump on backdoor and front door methods. It's a delete and copy file over top (revision 118 in their dump file):

  Revision-number: 118
  Prop-content-length: 133
  Content-length: 133
  K 10
  svn:author
  V 5
  admin
  K 8
  svn:date
  V 27
  2021-01-15T01:13:31.790068Z
  K 7
  svn:log
  V 33
  cp US/test.txt over APAC/test.txt
  PROPS-END

  Node-path: trunk/APAC/test.txt
  Node-kind: file
  Node-action: delete

  Node-path: trunk/APAC/test.txt
  Node-kind: file
  Node-action: add
  Node-copyfrom-rev: 117
  Node-copyfrom-path: trunk/US/test.txt
  Text-copy-source-md5: 9584b1f15ac103ba6abea161944577a2
  Text-copy-source-sha1: de349faaf6b8aed497a3c751dcc6cf7baf440cd1

This revision was generated by:

  svn rm  trunk/APAC/test.txt
  svn copy  trunk/US/test.txt trunk/APAC/test.txt
  svn commit

I have tried simple and more complicated reproductions of my own and I end up with what looks like the same revision history but it works in my case. For example:

  Node-path: release/test.txt
  Node-action: delete

  Node-path: release/test.txt
  Node-kind: file
  Node-action: add
  Node-copyfrom-rev: 5
  Node-copyfrom-path: main/test.txt
  Text-copy-source-md5: 9b70a99fe8660947dcbc5db7890038ab
  Text-copy-source-sha1: 4d9d27ce60952fc3906a6b1319461b71b8892819

Ref: 00720106 - For SVN dump. «
Add a comment
6 years agoKarl Wirth created job000742 for perforce_software-p4nagios:
Make an SDP friendly version of P4Nagios.

From consulting:
The SDP has knowledge of connection information for Perforce. It seems an obvious next st...ep for P4Nagios that it picks up the connection information from the SDP environment. «
Add a comment
6 years agoKarl Wirth liked a comment on review 24959 for perforce-software-sdp:main

Good catch! Thanks for the fix!

1 comment
6 years agoKarl Wirth requested review 24959 for perforce-software-sdp:main
Fix for SDP-362.
Move testing of variable till after the
environment has been sourced.
1 comment
6 years agoKarl Wirth modified SDP-362 for
p4verify.sh -remote ignores SDP_RECENT_CHANGES_TO_VERIFY.

p4verify.sh looks for SDP_RECENT_CHANGES_TO_VERIFY at the top of the script before the envi...ronment of the instance is sourced. The following line should be moved to line 220 in the script to fix the code.

declare RecentChangesToVerify=${SDP_RECENT_CHANGES_TO_VERIFY:-2500} «
Add a comment
6 years agoKarl Wirth created SDP-362 for
p4verify.sh -remote ignores SDP_RECENT_CHANGES_TO_VERIFY.

p4verify.sh looks for SDP_RECENT_CHANGES_TO_VERIFY at the top of the script before the envi...ronment of the instance is sourced. The following line should be moved to line 220 in the script to fix the code.

declare RecentChangesToVerify=${SDP_RECENT_CHANGES_TO_VERIFY:-2500} «
Add a comment
7 years agoKarl Wirth commented on SDP-255 for

Hit this with a customer a few weeks ago. As SSL is almost mandatory these days would be great to include it in the documentation.

1 comment
7 years agoKarl Wirth archived review 23697 for p4-jenkins:main
Ignore review.
1 comment
7 years agoKarl Wirth requested review 23697 for p4-jenkins:main
Document changelist reporting behavior when using multiple p4sync steps.
JENKINS-48854
1 comment
7 years agoKarl Wirth committed change 23696 into p4-jenkins:main
Document changelist reporting behavior when using multiple p4sync steps.
JENKINS-48854
Add a comment
7 years agoKarl Wirth committed change 23488 into p4-jenkins:main
Fixing typos in the example Jenkins trigger
Add a comment
7 years agoKarl Wirth modified job000718 for
Add empty directory support to bbi.sh.
Add a comment
7 years agoKarl Wirth created job000718 for
Add empty directory support to p4bbi.sh.
Add a comment
8 years agoKarl Wirth created job000716 for perforce-software-p4convert:
Update documentation to include more accurate dump filter

When dumping an SVN repo the dump sometimes incorrectly marks a merge of multiple revisions... to a target branch as only a merge of the first revision to target. When converted this causes inaccurate contents on the target branch.

To avoid this take the SVN dump in this specific way:

   svnadmin dump /path/to/repo | svndumpfilter include /path--drop-empty-revs --renumber-revs > my.dmp «
Add a comment
8 years agoKarl Wirth created job000714 for perforce_software-p4polarion:
P4Polarion does not work with Polarion ALM 17.1.

2017-06-28 14:33:43,867 [main | u:p] ERROR com.polarion.platform.repository.internal.config.Reposito...ryConfigService$ConfigProblemCatcher  - Failed to work with configuration:
Unable to construct service com.polarion.platform.repository.repositoriesDataHandler: Error building service com.polarion.platform.repository.repositoriesDataHandler: Failure invoking constructor for class com.polarion.platform.repository.external.internal.RepositoriesDataHandler: com/polarion/platform/repository/external/IExternalRepositoryProvider$IExternalRepositoryConfiguration
[configId=externalRepositories, locations=[Location[repository default, path /.polarion/repositories/repositories.xml]]]
org.apache.hivemind.ApplicationRuntimeException: Unable to construct service com.polarion.platform.repository.repositoriesDataHandler: Error building service com.polarion.platform.repository.repositoriesDataHandler: Failure invoking constructor for class com.polarion.platform.repository.external.internal.RepositoriesDataHandler: com/polarion/platform/repository/external/IExternalRepositoryProvider$IExternalRepositoryConfiguration [bundleresource://31/META-INF/hivemodule.xml, line 102, column 19]
    at org.apache.hivemind.impl.servicemodel.AbstractServiceModelImpl.constructNewServiceImplementation(AbstractServiceModelImpl.java:169) «
1 comment
8 years agoKarl Wirth updated files in review 22351 for p4-jenkins:main
Example usage patterns for P4jenkins.
A work in progress and not ready for public consumption yet.
Add a comment
8 years agoKarl Wirth requested review 22351 for p4-jenkins:main
Example usage patterns for P4jenkins.
A work in progress and not ready for public consumption yet.
Add a comment
8 years agoKarl Wirth committed change 22061 into p4-jenkins:main
Documentation for P4Groovy (Beta)
Add a comment
8 years agoKarl Wirth approved review 20926 for p4-jenkins:main
Documentation for P4Groovy (Beta)
Add a comment
8 years agoKarl Wirth updated files in review 20926 for p4-jenkins:main
Documentation for P4Groovy (Beta)
Add a comment
8 years agoKarl Wirth updated files in review 20926 for p4-jenkins:main
Documentation for P4Groovy (Beta)
Add a comment
8 years agoKarl Wirth updated files in review 20926 for p4-jenkins:main
Documentation for P4Groovy (Beta)
Add a comment
8 years agoKarl Wirth created job000693 for perforce-software-p4maven:
Plugin seems to always use '-f' for syncing.
Would be good if it could reuse have list for existing workspaces.
Add a comment
8 years agoKarl Wirth commented on change 21583 for

Your first changelist - welcome to the workshop :-).

1 comment
8 years agoKarl Wirth committed change 21552 into p4-jenkins:main
Adding a test MD file
Add a comment
8 years agoKarl Wirth committed change 21509 into
Simple example Groovy formatter for formatting Jenkins snippets.
Expects data passed through stdin. Outputs results to stdout.
Add a comment
8 years agoKarl Wirth committed change 21155 into p4-jenkins:main
Update documentation for new P4_CHANGELIST behavior - JENKINS-37584
Add a comment
8 years agoKarl Wirth updated files in review 20961 for perforce_software-p4bbi:main
Minor readme fix - correct incorrect URL.
Add a comment
8 years agoKarl Wirth requested review 20961 for perforce_software-p4bbi:main
Minor readme fix - correct incorrect URL.
Add a comment
8 years agoKarl Wirth modified job000625 for perforce_software-p4python:
pip install fails on Windows 7 64 bit Python 2.7.9+

C:\Python27\Scripts>.\pip install p4python
Collecting p4python
  Downloading p4python...-2016.1.1447008.tar.gz (84kB)
�[K    100% |################################| 86kB 422kB/s ta 0:00:01
    Complete output from command python setup.py egg_info:
    Attempting to load API from ftp.perforce.com
    Loaded API into None
    ****************************************************
    Cannot determine the version of openssl
    Building without SSL
    ****************************************************
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python\setup.py", line 639, in <module>
        do_setup(p4_api_dir, ssl)
      File "c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python\setup.py", line 460, in do_setup
        apiVersion = VersionInfo(p4_api_dir)
      File "c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python\setup.py", line 217, in __init__
        verFile = os.path.join(p4ApiDir, "sample", "Version")
      File "c:\python27\lib\ntpath.py", line 65, in join
        result_drive, result_path = splitdrive(path)
      File "c:\python27\lib\ntpath.py", line 115, in splitdrive
        if len(p) > 1:
    TypeError: object of type 'NoneType' has no len()

    ----------------------------------------
�[31mCommand "python setup.py egg_info" failed with error code 1 in c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python�[0m «
Add a comment
8 years agoKarl Wirth modified job000625 for perforce_software-p4python:
pip install fails on Windows Python 2.7.9+

C:\Python27\Scripts>.\pip install p4python
Collecting p4python
  Downloading p4python-2016.1.1...447008.tar.gz (84kB)
�[K    100% |################################| 86kB 422kB/s ta 0:00:01
    Complete output from command python setup.py egg_info:
    Attempting to load API from ftp.perforce.com
    Loaded API into None
    ****************************************************
    Cannot determine the version of openssl
    Building without SSL
    ****************************************************
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python\setup.py", line 639, in <module>
        do_setup(p4_api_dir, ssl)
      File "c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python\setup.py", line 460, in do_setup
        apiVersion = VersionInfo(p4_api_dir)
      File "c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python\setup.py", line 217, in __init__
        verFile = os.path.join(p4ApiDir, "sample", "Version")
      File "c:\python27\lib\ntpath.py", line 65, in join
        result_drive, result_path = splitdrive(path)
      File "c:\python27\lib\ntpath.py", line 115, in splitdrive
        if len(p) > 1:
    TypeError: object of type 'NoneType' has no len()

    ----------------------------------------
�[31mCommand "python setup.py egg_info" failed with error code 1 in c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python�[0m «
Add a comment
8 years agoKarl Wirth created job000625 for perforce_software-p4python:
pip install fails on Python 2.7.9+

C:\Python27\Scripts>.\pip install p4python
Collecting p4python
  Downloading p4python-2016.1.1447008.t...ar.gz (84kB)
    100% |################################| 86kB 422kB/s ta 0:00:01
    Complete output from command python setup.py egg_info:
    Attempting to load API from ftp.perforce.com
    Loaded API into None
    ****************************************************
    Cannot determine the version of openssl
    Building without SSL
    ****************************************************
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python\setup.py", line 639, in <module>
        do_setup(p4_api_dir, ssl)
      File "c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python\setup.py", line 460, in do_setup
        apiVersion = VersionInfo(p4_api_dir)
      File "c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python\setup.py", line 217, in __init__
        verFile = os.path.join(p4ApiDir, "sample", "Version")
      File "c:\python27\lib\ntpath.py", line 65, in join
        result_drive, result_path = splitdrive(path)
      File "c:\python27\lib\ntpath.py", line 115, in splitdrive
        if len(p) > 1:
    TypeError: object of type 'NoneType' has no len()

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in c:\users\testUser\appdata\local\temp\pip-build-e1dtqx\p4python «
Add a comment
8 years agoKarl Wirth updated files in review 20926 for p4-jenkins:main
Documentation for P4Groovy (Beta)
Add a comment
8 years agoKarl Wirth requested review 20926 for p4-jenkins:main
Documentation for P4Groovy (Beta)
Add a comment
8 years agoKarl Wirth created job000622 for
Jira: Custom labels are not replicated.
1 comment
8 years agoKarl Wirth committed change 20893 into p4-jenkins:main
Update trigger documentation to include using a CRUMB for security.
Add a comment
8 years agoKarl Wirth approved review 20891 for p4-jenkins:main
Update trigger documentation to include using a CRUMB for security.
Add a comment
8 years agoKarl Wirth updated description of review 20891 for p4-jenkins:main
Update trigger documentation to include using a CRUMB for security.
Add a comment
8 years agoKarl Wirth requested review 20891 for p4-jenkins:main
Update trigger documentation to include using a CRUMB for security.#review
Add a comment
8 years agoKarl Wirth committed change 20889 into p4-jenkins:main
Document downstream changelists.
Workaround for JENKINS-33163.
Add a comment
8 years agoKarl Wirth approved review 20887 for p4-jenkins:main
Document downstream changelists.
Workaround for JENKINS-33163.
Add a comment
8 years agoKarl Wirth requested review 20887 for p4-jenkins:main
Document downstream changelists.
Workaround for JENKINS-33163.
Add a comment
9 years agoKarl Wirth created job000563 for perforce_software-p4python:
Display where a P4 environment variable comes from

When using the P4 binary running 'p4 set' now shows the source of the currently used variable. For... example:

$ p4 set
P4CLIENT=test_ws (config '/p4/ws/.p4config')
P4PORT=20161 (config '/p4/ws/.p4config')
P4USER=test (config '/p4/ws/.p4config')

Would be good if P4Python had a similar ability. «
Add a comment
9 years agoKarl Wirth updated description of review 20177 for p4-jenkins:main
Expose HUDSON_CHANGELOG_FILE environment variable.
JENKINS-37442

#p4karl
Add a comment
9 years agoKarl Wirth updated files in review 20177 for p4-jenkins:main
Expose HUDSON_CHANGELOG_FILE environment variable.
JENKINS-37442
Add a comment
9 years agoKarl Wirth updated files in review 20177 for p4-jenkins:main
Expose HUDSON_CHANGELOG_FILE environment variable.
JENKINS-37442
Add a comment
9 years agoKarl Wirth voted up review 20131 for p4-jenkins:main
Added implementation for getAffectedFiles()
1 comment
9 years agoKarl Wirth commented on review 20131 for p4-jenkins:main

Tested with following Jelly script and looks OK:

https://github.com/jenkinsci/email-ext-plugin/blob/master/src/main/resources/hudson/plugins/emailext/

...

Tested with following Jelly script and looks OK:

https://github.com/jenkinsci/email-ext-plugin/blob/master/src/main/resources/hudson/plugins/emailext/templates/html.jelly

Example output:

CHANGES
Revision 159 by test:(edis)
edit //depot/src/f1
edit //depot/src/f2
edit //depot/src/f3

Revision 158 by test:(P4:Automated Submit Message)
edit //depot/src/f3

Revision 157 by test:(P4:Automated Submit Message)
edit //depot/src/f2

 «
1 comment
9 years agoKarl Wirth requested review 20177 for p4-jenkins:main
Expose HUDSON_CHANGELOG_FILE environment variable.
JENKINS-37442
Add a comment
9 years agoKarl Wirth created job000559 for p4perl:
Update the P4PERL API to work with Perl version 5.22 and later.
1 comment
Change User Description Created
24958 Karl Wirth #review-24959 Fix for SDP-362.
Move testing of variable till after the
environment has... been sourced. «		 6 years ago View Review
22442 Karl Wirth Example usage patterns for P4jenkins.
A work in progress and not ready for public consump...tion yet. «		 8 years ago View Review
22350 Karl Wirth Example usage patterns for P4jenkins.
A work in progress and not ready for public consump...tion yet.
#review-22351 «		 8 years ago View Review
20960 Karl Wirth Minor readme fix - correct incorrect URL.
#review-20961 		8 years ago View Review
20925 Karl Wirth #review-20926 Documentation for P4Groovy (Beta) 8 years ago View Review
20890 Karl Wirth Update trigger documentation to include using a CRUMB for security.#review 8 years ago View Review
20886 Karl Wirth Document downstream changelists.
Workaround for JENKINS-33163. #review-20887 		8 years ago View Review
20176 Karl Wirth Expose HUDSON_CHANGELOG_FILE environment variable.
JENKINS-37442 		9 years ago View Review
15861 Karl Wirth #review-15862 - Incorrect version shelved in update.
This version includes Typo fix. 		10 years ago View Review

Profile Settings

Reset to default

Reviews

Sets default values for how diffs are displayed on all reviews.

Show comments in files
View diffs side-by-side
Show space and newline characters
Ignore whitespace when calculating differences

Time Display

Sets how you would like to display time across Swarm.

Display the time in

Adjust when notifications are sent to you about reviews that you're associated with (as an author, reviewer, project member or moderator).

Email me when:

Reset to default
Check all
I change the state of a review
I am the author, and
a review is requested
files in the review are updated
tests on the review have finished
a vote is cast on a review
the state of the review changes
a review or change is committed
a comment is made on the review or change
a comment on the review or change is updated
Someone likes one of my comments
I am a member, and
a review is requested
a review or change is committed
I am a reviewer, and
files in the review are updated
tests on the review have finished
a vote is cast on a review
the state of the review changes
someone joins or leaves the review
a review or change is committed
a comment is made on the review
a comment on the review is updated
I am a moderator, and
files in the review are updated
tests on the review have finished
a review or change is committed