Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build for oem partners #462

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Conversation

grwilson
Copy link
Contributor

This PR is a draft for an external build we intend to share with our partners. This PR is part of the external-minimal variant (see delphix/appliance-build#735)

Copy link
Contributor

@sebroy sebroy left a comment

Choose a reason for hiding this comment

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

Do we need to create a special external variant (e.g. external-oem) that, along with having CRA disabled like you have here, has a minimal set of packages (e.g. only those included in minimal-common)?

@grwilson
Copy link
Contributor Author

Do we need to create a special external variant (e.g. external-oem) that, along with having CRA disabled like you have here, has a minimal set of packages (e.g. only those included in minimal-common)?

I created an external-minimal variant and linked the PR in the description. Here it is: delphix/appliance-build#735

- --enable challenge-response
- --remove unix
when:
- variant is regex("external-.*")
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe change this to:

when:
 - variant != "external-minimal"
 - variant is regex("external-.*")

as-is, this will disable CRA for the product, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I do not intend to merge this PR. It was designed to create a one-time build and I posted the PR to make sure that we're not exposing anything of concern to anybody that uses this build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants