openshift-metal3 / dev-scripts

Scripts to automate development/test setup for openshift integration with https://github.com/metal3-io/
Apache License 2.0
92 stars 182 forks source link

Document METAL3_DEV_ENV variable #1627

Closed elfosardo closed 5 months ago

elfosardo commented 6 months ago

/cc @derekhiggins @tdomnesc

elfosardo commented 6 months ago

/retest

elfosardo commented 6 months ago

/retest

elfosardo commented 6 months ago

the issue with e2e-metal-ipi-serial-ipv4 is independent from this change

elfosardo commented 6 months ago

/retest giving it another chance

dtantsur commented 6 months ago

/approve

openshift-ci[bot] commented 6 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dtantsur

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openshift-metal3/dev-scripts/blob/master/OWNERS)~~ [dtantsur] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
derekhiggins commented 5 months ago

/hold I already run install requirements multiple times, when I've made local changes to metal3-dev-env, I want to get this error message, it reminds me to set METAL3_DEV_ENV (to avoid running sync_repo_and_patch)

If the error message doesn't happen my local updates would be silently zapped, without realising it I wouldn't be running the code I wanted and my changes would be lost.

elfosardo commented 5 months ago

@derekhiggins thanks, I think METAL3_DEV_ENV is not documented anywhere, not even in config_example.sh I'm going to convert this patch to a doc patch so we're all aware of that

elfosardo commented 5 months ago

@derekhiggins @dtantsur please give it another look, thanks!

derekhiggins commented 5 months ago

/unhold /lgtm thanks

elfosardo commented 5 months ago

/unhold

derekhiggins commented 5 months ago

/lgtm