#!/usr/bin/env python3
# -*- coding: utf-8 -*-
# ==============================================================================
# Copyright and license info is available in the LICENSE file included with
# the Server Deployment Package (SDP), and also available online:
# https://workshop.perforce.com/view/p4-sdp/main/LICENSE
# ------------------------------------------------------------------------------
# tag::includeManual[]
"""
NAME:
ValidateContentFormat.py
DESCRIPTION:
This trigger is intended for file content validation as part of change-content trigger
Works for YAML, XML - only checks files matching required file extensions - see below.
To install, add a line to your Perforce triggers table like the following:
validate-yaml change-content //....yaml "python /p4/common/bin/triggers/ValidateContentFormat.py -p %serverport% -u perforce --yaml %change% "
validate-xml change-content //....xml "python /p4/common/bin/triggers/ValidateContentFormat.py -p %serverport% -u perforce --xml %change% "
or (if server is standard SDP and has appropriate environment defaults for P4PORT and P4USER):
validate-yaml change-content //....yaml "python /p4/common/bin/triggers/ValidateContentFormat.py --yaml %change% "
validate-xml change-content //....xml "python /p4/common/bin/triggers/ValidateContentFormat.py --xml %change% "
You may need to provide the full path to python executable, or edit the path to the trigger.
Also, don't forget to make the file executable.
"""
# end::includeManual[]
# Python 2.7/3.3 compatibility.
from __future__ import print_function
import sys
import re
import os
import P4Triggers
import P4
import yaml
import tempfile
from lxml import etree
def yaml_validator(fname):
try:
with open(fname, 'r') as f:
content = yaml.load(f)
return ""
except Exception as e:
return str(e)
def xml_validator(fname):
try:
with open(fname, 'r') as f:
doc = etree.parse(f)
return ""
except Exception as e:
return str(e)
# Regex extensions to search for in filenames
file_extensions = {"yaml": ["\.yaml$", "\.yml$"],
"xml": ["\.xml$"]}
# Format validator
file_validator = {"yaml": yaml_validator,
"xml": xml_validator}
class ValidateContentFormat(P4Triggers.P4Trigger):
"""See module doc string for details"""
def __init__(self, *args, **kwargs):
P4Triggers.P4Trigger.__init__(self, **kwargs)
self.parse_args(__doc__, args)
def add_parse_args(self, parser):
"""Specific args for this trigger - also calls super class to add common trigger args"""
parser.add_argument('--yaml', default=False, action='store_true',
help="Trigger validates YAML files")
parser.add_argument('--xml', default=False, action='store_true',
help="Trigger validates XML files")
parser.add_argument('change', help="Change to process - %%change%% argument from triggers entry.")
super(ValidateContentFormat, self).add_parse_args(parser)
def run(self):
"""Runs trigger"""
error_count = 0
try:
self.logger.debug("ValidateContent trigger firing")
self.setupP4()
self.p4.connect()
exts = ""
if self.options.yaml:
exts = file_extensions["yaml"]
validator = file_validator["yaml"]
elif self.options.xml:
exts = file_extensions["xml"]
validator = file_validator["xml"]
change = self.getChange(self.options.change)
for df in change.files:
valid_ext = False
fname = df.depotFile.lower()
for ext in exts:
if re.search(ext, fname):
valid_ext = True
break
if not valid_ext:
continue
name_parts = df.depotFile.split("/")
tfile = os.path.join(tempfile.gettempdir(), name_parts[-1])
self.logger.debug("Validating %s as local: %s" % (df.depotFile, tfile))
self.p4.run_print("-o", tfile, "%s@=%s" % (df.depotFile, self.options.change))
result = validator(tfile)
self.logger.debug("Validated %s - result: %s" % (tfile, result))
if result:
error_count += 1
msg = "Invalid format for %s\n%s" % (df.depotFile, result)
self.logger.warning(msg)
print(msg)
if error_count:
print("%d files failed" % error_count)
return error_count
except Exception:
return self.reportException()
if __name__ == '__main__':
""" Main Program"""
trigger = ValidateContentFormat(*sys.argv[1:])
sys.exit(trigger.run())
| # | Change | User | Description | Committed | |
|---|---|---|---|---|---|
| #1 | 33433 | Claude (AI Agent by Anthropic) |
Copy Up from //p4-sdp/dev into //p4-sdp/main. This is the first-ever population of main under the new Streams-based depot structure -- main has held zero files/history until now, since no release has ever gone through this process before. 463 files, covering the entire 2026.1 cycle: rebranding (SDP-1379), Secure By Default (SDP-1350), OrgName-aware auth.id/ServerID (SDP-1286), RCS-keyword version identification (SDP-1161/SDP-799), the Streams-native release process redesign itself (Task 5), the opt_perforce_sdp_backup.sh false-error fix, the P4D 2026.1 test-suite targeting, refreshed P4*.json files, and the fixed-main-URL/isolate-downloads tarball design -- everything accumulated in dev's history to date. Isolated paths (ai_dev_support/, Version, doc/*.html, doc/*.pdf, doc/gen/*.man.txt, doc/gen/sdp_install.cfg, Unsupported/doc/*.html, Unsupported/doc/*.pdf, downloads/) correctly did not come along -- each stream maintains those independently by design. Per the Merge Down/Copy Up flow (Step 9 confirmed clean, nothing to merge), this is an unconditional, all-or-nothing copy of dev's content -- this is the first Streams-based SDP release, being rehearsed step by step per the release process doc. Agent: Claude Code, Model: Claude Sonnet 5 (claude-sonnet-5), operating as bot_Claude_Anthropic. |
||
| //p4-sdp/dev/Unsupported/Samples/triggers/ValidateContentFormat.py | |||||
| #2 | 33409 | Claude (AI Agent by Anthropic) |
Copy Up from //p4-sdp/dev_rebrand into //p4-sdp/dev. This is the first promotion of dev_rebrand's work into dev since dev_rebrand was created (2025-05-24) -- 303 files, covering the entire 2026.1 rebranding effort (SDP-1379), the Secure By Default adaptation (SDP-1350), OrgName-aware auth.id/ServerID (SDP-1286), RCS-keyword version identification (SDP-1161/SDP-799), and the Streams-native release process redesign (Task 5) done this session, plus everything else accumulated in dev_rebrand's history before this session. Per the Merge Down/Copy Up flow, this is intentionally a full, unconditional blast-replace of dev's content from dev_rebrand -- all selectivity/care happened in the preceding Merge Down (dev -> dev_rebrand, changes 33407-33408), which absorbed Robert Cowham's independent dev-side work first so nothing of his is lost by this Copy Up. Two files are worth calling out since they might look alarming in isolation: - tools/mdcu.sh is deleted -- intentional, retired this session in favor of the two direct Streams commands now documented in doc/ReleaseProcessOverview.md. - tools/ReleaseProcessOverview.md is deleted -- this is a stale relic of a file move dev_rebrand made back in 2025-05-24 (tools/ -> doc/) that was never previously propagated to dev; the current, fully-rewritten doc/ReleaseProcessOverview.md is added/updated correctly by this same changelist. |
||
| #1 | 31397 | C. Thomas Tyler | Populate -b SDP_Classic_to_Streams -s //guest/perforce_software/sdp/...@31368. | ||
| //guest/perforce_software/sdp/dev/Unsupported/Samples/triggers/ValidateContentFormat.py | |||||
| #2 | 26681 | Robert Cowham |
Removing Deprecated folder - if people want it they can look at past history! All functions have been replaced with standard functionality such as built in LDAP, or default change type. Documentation added for the contents of Unsupported folder. Changes to scripts/triggers are usually to insert tags for inclusion in ASCII Doctor docs. |
||
| #1 | 26652 | Robert Cowham |
This is Tom's change: Introduced new 'Unsupported' directory to clarify that some files in the SDP are not officially supported. These files are samples for illustration, to provide examples, or are deprecated but not yet ready for removal from the package. The Maintenance and many SDP triggers have been moved under here, along with other SDP scripts and triggers. Added comments to p4_vars indicating that it should not be edited directly. Added reference to an optional site_global_vars file that, if it exists, will be sourced to provide global user settings without needing to edit p4_vars. As an exception to the refactoring, the totalusers.py Maintenance script will be moved to indicate that it is supported. Removed settings to support long-sunset P4Web from supported structure. Structure under new .../Unsupported folder is: Samples/bin Sample scripts. Samples/triggers Sample trigger scripts. Samples/triggers/tests Sample trigger script tests. Samples/broker Sample broker filter scripts. Deprecated/triggers Deprecated triggers. To Do in a subsequent change: Make corresponding doc changes. |
||
| //guest/perforce_software/sdp/dev/Server/Unix/p4/common/bin/triggers/ValidateContentFormat.py | |||||
| #4 | 26544 | Robert Cowham |
Validate XML as well as YAML content for valid structure at least. Refactored to make it easier to add others in future. |
||
| #3 | 23153 | Robert Cowham | Make executable | ||
| #2 | 23152 | Robert Cowham | Refactor to push common arg_parse into P4Triggers | ||
| #1 | 23143 | Robert Cowham | Trigger to validate contents of files - in this case Yaml | ||