GoogleCloudPlatform / GoogleCloudPlatform/nodejs-docs-samples

testing: Add support for runtime environment variables to GitHub Workflows

Đang mở
#3,501 3 bình luận 0 reaction 1 người được giao Được giao cho @grayside Xem trên GitHub
priority: p2 samples type: cleanup
Ngôn ngữ chính
JavaScript
Star
3k
Fork
2k
Merge trung bình
2 ngày 19 giờ
Pull request đã merge (30 ngày)
44

Mô tả

GitHub Workflows is the preferred test driver for tests in this repository, but the current workflow implementation has several limitations. One of them is the ability to set runtime environment variables that are specific to a Workflow.

## Approach

Options:
1. Hard code a central map of environment variables, and refactor code to ensure clean namespacing
2. Create a mechanism to pass environment variables to our reusable test workflow

Option 2 is the most versatile with the least need to refactor existing code.

Option 1 has the further downside of placing test-specific parameters in a central file. I'm not sure if any changes to `test.yml` are triggering wide-scale test execution currently, but with this approach we'd likely need to trigger more tests from any change, which would include adding environment variables for unrelated tests.

@pattishin has asked that as part of the PR fixing this issue, we look for existing hard-coded values and shift them to environment variable configuration.

This issue is a blocker on implementation of GHA-based test workflows for a number of samples, with #3149 as the driver for research today.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.