Skip to content

Conversation

@vinay79n
Copy link
Contributor

feat:

  • updated "perimeter" tag, now we directly fetch the flows' perimeter from outerbounds' env vars [os.environ.get('OB_CURRENT_PERIMETER') or os.environ.get('OBP_PERIMETER')]
  • and added "is_production" flag in get_select_dev_query_tags() function tags dict

- updated "perimeter" tag ,now we directly fetch the flows' perimeter  from outerbounds' env vars
- and added "is_production" flag in get_select_dev_query_tags() function tags dict
@vinay79n vinay79n requested review from Copilot and phitoduck October 23, 2025 19:06
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the tagging mechanism for Metaflow audit logs by changing how the perimeter is determined and adding production status tracking. Instead of deriving perimeter from the is_production flag, it now reads directly from Outerbounds environment variables, and explicitly includes the is_production flag as a separate tag.

Key Changes:

  • Perimeter tag now reads from OB_CURRENT_PERIMETER or OBP_PERIMETER environment variables
  • Added explicit is_production tag to track deployment production status

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@wiz-55ccc8b716
Copy link

wiz-55ccc8b716 bot commented Oct 23, 2025

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities -
Data Finding Sensitive Data -
SAST Finding SAST Findings 2 Medium
Total 2 Medium

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@phitoduck phitoduck merged commit 9052ef1 into main Oct 23, 2025
10 checks passed
@phitoduck phitoduck deleted the feat/add-is-prod-tag branch October 23, 2025 23:23
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.

3 participants