| 5 months agoJoel Brown committed change 30948 into Add more logging to troubleshoot "wipe workspace" failure in processWorkspaceBeforeDeletion() | |
|
|
| 5 months agoJoel Brown committed change 30947 into Compiled with target='1.8' | |
|
|
| 5 months agoJoel Brown committed change 30942 into This build is superceeded by the P4DTG 2024.1 release.
Please use that. | |
|
|
| 5 months agoJoel Brown committed change 30941 into Add WipeTest.jar: runs p4 client -o and prints the map returned. | |
|
|
| 12 months agoJoel Brown committed change 30280 into Update for additional bug fixes coming in the next release of P4DTG.
See the Readme for fixes included. | |
|
|
| about a year agoAdam Morriss committed change 29877 into Correcting the P4PORT address - was 'workshop' now 'public' | |
|
|
| 2 years agoJoel Brown committed change 29776 into Updated instructions to mention unzipping. | |
|
|
| 2 years agoJoel Brown committed change 29671 into perforce-software-p4java:mainSet the IOptionsServer instance's User before a connect().
The connect() does a 'p4 info' and if doesn't have a user defined then it uses the default... of 'nouser'. « | |
|
|
| 3 years agoAdam Morriss created SDP-800 forStore trigger scripts in the depot.
Currently, the trigger scripts are stored in the file system. Submitting them to the depot means they're more like...ly to be backed up with the other versioned files; the files are replicated; they are versioned so we can identify changes made to them, those changes are replicated.
Currently we see triggers failing because the file hasn't been manually copied to the replica, or that the files have been modified and now do slightly different things on a replica compared with the corresponding master. « | |
|
|
| 3 years agoAdam Morriss created SDP-799 forCapture SDP version in the output of scripts.
When reviewing the results of the backup and recovery, validation and others scripts, the reader will kn...ow which version is currently in use.
This could be of particular use when sending that output on to the Support team. « | |
|
|
| 3 years agoJoel Brown edited a comment on change 28762 (ClientHelper.java, line 1237) for p4-jenkins:mainI only see this method sig called from CheckoutTask.java for changelist calculations
Check for maxChanges == 0 here:
if (getMaxChangeLimit() <= 0) ...I only see this method sig called from CheckoutTask.java for changelist calculations
Check for maxChanges == 0 here:
if (getMaxChangeLimit() <= 0) {
return new ArrayList<P4Ref>();
} « | |
|
|
| 3 years agoJoel Brown commented on change 28762 (ClientHelper.java, line 1237) for p4-jenkins:mainI onlysee this method sig called from CheckoutTask.java for changelist calculations
Check for maxChanges == 0 here:
if (getMaxChangeLimit() <= 0) ...I onlysee this method sig called from CheckoutTask.java for changelist calculations
Check for maxChanges == 0 here:
if (getMaxChangeLimit() <= 0) {
return new ArrayList<P4Ref>();
} « | |
|
|
| 3 years agoJoel Brown commented on change 28762 (ClientHelper.java, line 1287) for p4-jenkins:main | |
|
|
| 3 years agoJoel Brown commented on change 28762 (ClientHelper.java, line 1237) for p4-jenkins:mainadd check for skipping calculation of changelists here.
if (getMaxChangeLimit() <= 0) {
return new ArrayList<P4Ref>()
...add check for skipping calculation of changelists here.
if (getMaxChangeLimit() <= 0) {
return new ArrayList<P4Ref>();
}
« | |
|
|
| 3 years agoJoel Brown commented on change 28762 (ClientHelper.java, line 1287) for p4-jenkins:mainChecking getMaxChangeLimit() here stopped polling from working.
I moved the check. | |
|
|
| 3 years agoJoel Brown committed change 28709 into Allow setting character encoding (charset) with syncs with
the Java System Property -Dcharset=<value> | |
|
|
| 3 years agod_benedict voted up review 28607 for perforce-software-sdp:devAdded SDP Health Checks appendix to UNIX/Linux SDP Guide.
#review @d_benedict | |
|
|
| 3 years agoAdam Morriss commented on SDP-720 forA server needs only to know P4ROOT to find the database, including all the remaining configuration. Override with command-line flags only if needed (s ...A server needs only to know P4ROOT to find the database, including all the remaining configuration. Override with command-line flags only if needed (such as with a sidetrack server) but the default shouldn't require you to override. The 'ps' output is cosmetic at best. « | |
|
|
| 3 years agoAdam Morriss commented on SDP-720 forYou are correct: override the default settings you have put in place, then things don't work correctly.
'p4d -J correctJournal -c"set P4JOURNAL=correc ...You are correct: override the default settings you have put in place, then things don't work correctly.
'p4d -J correctJournal -c"set P4JOURNAL=correctJournal" works fine. And then 'p4d -jc' also works fine.
If you don't have P4JOURNAL set in the database and you run 'p4d -jc' it fails. « | |
|
|
| 3 years agoAdam Morriss commented on SDP-720 forI am aware of other discussions on this topic, and there is a distinct difference between SDP best practice and non-SDP best practice.
Setting P4JOUR ...I am aware of other discussions on this topic, and there is a distinct difference between SDP best practice and non-SDP best practice.
Setting P4JOURNAL (and P4LOG, and the server spec containing P4PORT) in the database means you can't forget it, you can only override it. Overriding it incorrectly, just like forgetting to set the override, has caused problems, and potentially data loss. « | |
|
|
| 3 years agoAdam Morriss created SDP-728 forNew SDP doesn't like old SDP configuration.
Up to 2018.1, the "p4d_base" script explicitly set P4JOURNAL in the database using 'p4d -cset'.
...; echo "Setting P4JOURNAL in the database to $P4JOURNAL." | tee -a $LOGFILE
echo $P4DBIN -r $P4ROOT -cset P4JOURNAL=$P4JOURNAL | tee -a $LOGFILE
$P4DBIN -r $P4ROOT "-cset P4JOURNAL=$P4JOURNAL" 2>&1 | tee -a $LOGFILE
SDP 2018.1.23504 (2018/01/19)
https://workshop.perforce.com/files/guest/perforce_software/sdp/main/Server/Unix/p4/common/bin/p4d_base?v=19
From revision 7 of the 'verify_sdp.sh' script, there is an explicit checks for P4JOURNAL in the db.config file, and rejects this as invalid.
check_configurable "$SDP_INSTANCE" P4JOURNAL ALL UNDEF
msg "Note: P4JOURNAL must be defined as a shell environment variable and must not set in db.config.\\n"
https://workshop.perforce.com/files/guest/perforce_software/sdp/dev/Server/Unix/p4/common/bin/verify_sdp.sh?v=7
This causes problems as call to 'verify_sdp' will report an error. Crops up when upgrading an existing SDP environment; calls to 'verify_sdp' are made from the upgrade and daily_checkpoint scripts, and it's also called from functions in 'backup_functions'.
NB: also a problem for Admins moving to SDP-control from a non-SDP environment where P4JOURNAL is often set in the database.
See also SDP-720. « | |
|
|
| 3 years agoAdam Morriss created SDP-725 forTypo in the instructions
SDP Guide (Unix); section 4.2, Manual Install; item 7, option 2.
The first 'ln' command should be
ln -s /P4metadata hx...metadata
Docs contain this:
Option 2, different mounted root folders, e.g. /P4metadata, /P4logs, /P4depots:
sudo chown -R perforce:perforce /P4metadata /P4logs /P4depots
cd /
ln -s /P4Medata hxmetadata
ln -s /P4logs hxlogs
ln -s /P4depots hxdepots
sudo chown -h perforce:perforce /hx* « | |
|
|
| 3 years agoJoel Brown committed change 28442 into Additional clarifications for running the tests. | |
|
|
| 3 years agoAdam Morriss committed change 28433 into perforce-software-p4convert:main Typo corrected | |
|
|
| 3 years agoAdam Morriss committed change 28432 into perforce-software-p4convert:main Minor tweaks to the language, corrected spelling errors | |
|
|
| 3 years agoAdam Morriss committed change 28431 into perforce-software-p4convert:main Updating the project readme; supported status is not correct. | |
|
|
| 3 years agoAdam Morriss created SDP-720 forAllow the setting of server configuration in the db.config table.
At present, the SDP stores settings in the environment 'outside' the database, and... scripts like 'verify_sdp' rejects any in-database settings, stating things like this:
Note: P4JOURNAL must be defined as a shell environment variable and must not set in db.config.
It is increasingly common to define the configuration in the database rather than in the environment or with 'override' flags like '-J' in this case, as this helps ensure the server is started with the same parameters each time.
Having to reconfigure a working Helix Core Server settings to allow the switch to SDP is a problem. « | |
|
|
| 3 years agoAdam Morriss committed change 28425 into Updating trigger script, swapping '' for ''.
It did work, but something may have changed in this more recent OS | |
|
|
| 3 years agoAdam Morriss created SDP-719 forTypo in the 'p4d -Gc' command.
SDP guide, section 4.2.1.1. Use of SSL
This states you should run a command like this:
/p4/common/bin/p4master_ru...n <instance> /p4/<instance>/p4d_<instance> -Gc
This is missing the 'bin' folder in the path to 'p4d_<instance>', and a corresponding command would be
/p4/common/bin/p4master_run 1 /p4/1/p4d_1 -Gc
instead of the example documented below:
/p4/common/bin/p4master_run 1 /p4/1/bin/p4d_1 -Gc « | |
|
|
| 3 years agoAdam Morriss created SDP-718 forMisleading statement in 'mkrep' script.
The mkrep script states that P4D requires service users to be 'perforce-authenticated' (AuthMethod=perforce')... and not ldap-authenticated (AuthMethod=ldap). This is incorrect: service users *can* be LDAP-authenticated.
From the script:
Service users also have an 'AuthMethod' of 'perforce' (not 'ldap') as is required by 'p4d' for 'service' users. « | |
|
|
| 3 years agoAdam Morriss modified SDP-692 forAllow configuration of a mix of ssl- and non-ssl- enabled entities in the SDP configuration.
Currently, mkdirs.sh and p4_vars have a built-in var...iable for SSL prefix, working as an all-or-nothing option. Would like the means to pick and choose the components that are to be ssl-enabled and those not: for example, install and configure a Broker that supports SSL connections, while retaining a non-ssl Helix Core Server, standardising on 'One sided SSL certificates'.
Related: https://community.perforce.com/s/article/2596 « | |
|
|
| 3 years agoAdam Morriss modified SDP-324 forFlaw in remove_empty_pending_changes.py.
This script checks the output of 'p4 -Ztag describe -s NNN' for the string "user swarm", to ensure empty pen...ding changelists relating to a Swarm instance are not removed.
The string is hard-coded, and does not allow for users named anything other than 'swarm'. This script must not be run as-is, unless the Swarm instance is and has always been running under user 'swarm'. « | |
|
|
| 4 years agoAdam Morriss created SDP-692 forAllow configuration of a mix of ssl- and non-ssl- enabled entities in the SDP configuration.
Currently, mkdirs.sh and p4_vars have a built-in var...iable for SSL prefix, working as an all-or-nothing option. Would like the means to pick and choose the components that are to be ssl-enabled and those not: for example, install and configure a Broker that supports SSL connections, while retaining a non-ssl Helix Core Server, standardising on 'One sided SSL certificates'.
Related: https://community.perforce.com/s/article/2596 « | |
|
|
| 4 years agoJoel Brown committed change 28029 into New: for testing SSL handshakes to p4d enabled servers. | |
|
|
| 4 years agod_benedict commented on review 27896 for perforce-software-sdp:devExternaAddress should be "ExternalAddress" | |
|
|
| 4 years agod_benedict voted down review 27896 for perforce-software-sdp:devClarified intent of '-r' flag value in docs.
This is a doc-only change.
#review @d_benedict | |
|
|
| 4 years agod_benedict voted up review 27898 for perforce-software-sdp:dev | |
|
|
| 4 years agoAdam Morriss approved review 27893 for perforce-software-sdp:devFixed typo and updated Version identifier.
Thanks, Adam!
#review @amo | |
|
|
| 4 years agoAdam Morriss commented on review 27891 (get_helix_binaries.sh, line 117) for perforce-software-sdp:devTypo: use 'r20.2 to denote the 2020.2 release (not 2020.1) | |
|
|
| 4 years agoJoel Brown committed change 27754 into Allow to specify number of parallel threads. | |
|
|
| 4 years agogiles created SDP-637 for Typo in SDP_Guide.Windows.pdf #6.
20/06/2020 15:05 <SYMLINKD> root [e:\p4\M1\root] - should be [e:\p4\1\root] | |
|
|
| 4 years agoJoel Brown committed change 27671 into Revise to use serverBypass | |
|
|
| 4 years agoJoel Brown committed change 27670 into Add to Sync and Streaming Sync the ability to not populate the have table
java -Dhave=false -cp JenkinsOps.jar .... | |
|
|
| 4 years agoJoel Brown committed change 27438 intoNew jiratest2020.php for Swarm 2020 release.
The different scripts correspond to the Zend framework used by Swarm.
<=2018: zend 2
2019: ze...nd 3
2020: laminas
Update README to match the script names. « | |
|
|
| 4 years agoJoel Brown committed change 27292 into Upgrade P4Java lib to 2020.1.1999383 | |
|
|
| 4 years agovkanczes modified SDP-577 forWhat would be the preferred way to get SDP to filter out the
purge-related errors when it reports the results of a verify? Could
such filtering be ma...de a standard (or optional) part of SDP?
Requested: elliot.wilen@aechelon.com
Case# 00663167
Case# 00714647 « | |
|
|
| 4 years agovkanczes created SDP-577 forWhat would be the preferred way to get SDP to filter out the
purge-related errors when it reports the results of a verify? Could
such filtering be ma...de a standard (or optional) part of SDP? « | |
|
|
| 4 years agojkovisto committed change 27029 into Comment for journalPrefix in p4dstate.sh.
Correct path to pdb.lbr in p4pstate.sh. | |
|
|
