Skip to content

Replace real user/host identifiers with fake placeholders in zelta_test_policy.conf#65

Merged
rlogwood merged 3 commits intofeature/new-testsfrom
copilot/sub-pr-64
Mar 15, 2026
Merged

Replace real user/host identifiers with fake placeholders in zelta_test_policy.conf#65
rlogwood merged 3 commits intofeature/new-testsfrom
copilot/sub-pr-64

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 15, 2026

The committed test fixture zelta_test_policy.conf contained real usernames and hostnames (dever@lusvr01, dever@uvm1), raising a privacy concern even though the file is regenerated at runtime.

Changes

  • test/runners/test_generation/config/zelta_test_policy.conf: Replace real identifiers with clearly-fake placeholders and a neutral timestamp
# Before
BACKUP_SITE:
  dever@lusvr01:
    datasets:
    - apool/treetop: dever@uvm1:bpool/backups

# After
BACKUP_SITE:
  user@source-host:
    datasets:
    - srcpool/data: user@backup-host:dstpool/backups

The runtime behavior is unchanged — generate_zelta_policy_config.sh overwrites this file with sandbox environment values before tests run.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…st_policy.conf

Co-authored-by: rlogwood <3377419+rlogwood@users.noreply.github.com>
Copilot AI changed the title [WIP] [WIP] Address feedback from review on new tests feature Replace real user/host identifiers with fake placeholders in zelta_test_policy.conf Mar 15, 2026
Copilot AI requested a review from rlogwood March 15, 2026 20:57
Copy link
Copy Markdown
Collaborator

@rlogwood rlogwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

file replaced with generic information to show it's structure, as it automatically generated

@rlogwood rlogwood marked this pull request as ready for review March 15, 2026 21:04
@rlogwood rlogwood merged commit 17a87c6 into feature/new-tests Mar 15, 2026
@rlogwood rlogwood deleted the copilot/sub-pr-64 branch March 15, 2026 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants