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.

Sign up for GitHub

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

Jump to bottom

Allow the operator to fetch images tags from CR #6330

Closed
lgarciaaco opened this issue Aug 7, 2019 · 1 comment · Fixed by #6346
Closed

Allow the operator to fetch images tags from CR #6330

lgarciaaco opened this issue Aug 7, 2019 · 1 comment · Fixed by #6346
Assignees
@lgarciaaco
Labels
cat/enhancement Enhancements to existing features and capabilities

Comments

@lgarciaaco
Copy link
Contributor

lgarciaaco commented Aug 7, 2019 •
edited

The current install process as in fuse online install does the following:

  • Create necessary cluster resources like CRD, eventually grant some cluster permissions to a specified user
  • Create syndesis operator resources (operator Role, RoleBinding and operator DeploymentConfig)
  • Create the ImageStreams with the tags released with release

Tasks

  • Allow the syndesis operator to fetch the ImageStreams specifications (repository, tags ...) from the CR. Currently this comes from within the operator and can't be used for productized images because at the moment when the syndesis operator image is being build, there is no information yet about the tags of the other syndesis images (like meta, server or ui)

    syndesis/install/operator/build/conf/config.yaml

    Lines 32 to 39 in 06c932c

    Tags:
    Komodo: latest
    OAuthProxy: v4.0.0
    PostgresExporter: v0.4.7
    Postgresql: "9.5"
    Prometheus: v2.1.0
    Syndesis: latest
    Upgrade: latest

    The workflow would try to fetch the specifications from CR, then from global secret and finally take the config values if none of the previous methods succeeded. Also, use

    syndesis/install/operator/build/conf/config.yaml

    Line 31 in 06c932c

    Tag: ""

    to validate that the ImageStream tag are all part of the same major version.
@pure-bot pure-bot bot added the notif/triage The issue needs triage. Applied automatically to all new issues. label Aug 7, 2019
@lgarciaaco lgarciaaco changed the title Allow operator to fets ImageStreams Tags from CR Allow operator to fetch ImageStreams Tags from CR Aug 7, 2019
@lgarciaaco lgarciaaco changed the title Allow operator to fetch ImageStreams Tags from CR Migrate the install process from fuse online install to the operator Aug 7, 2019
@lgarciaaco lgarciaaco self-assigned this Aug 7, 2019
@lgarciaaco lgarciaaco mentioned this issue Aug 7, 2019
Closed
@heiko-braun heiko-braun added cat/enhancement Enhancements to existing features and capabilities and removed notif/triage The issue needs triage. Applied automatically to all new issues. labels Aug 8, 2019
@heiko-braun heiko-braun mentioned this issue Aug 8, 2019
Closed
7 tasks
@lgarciaaco lgarciaaco mentioned this issue Aug 9, 2019
Merged
@pure-bot pure-bot bot closed this as completed in #6346 Aug 9, 2019
@pure-bot pure-bot bot locked as resolved and limited conversation to collaborators Aug 9, 2019
@pure-bot pure-bot bot reopened this Aug 9, 2019
@pure-bot pure-bot bot unlocked this conversation Aug 9, 2019
@lgarciaaco
Copy link
Contributor Author

Solved with #6346

@lgarciaaco lgarciaaco changed the title Migrate the install process from fuse online install to the operator Allow the operator to fetch images tags from CR Aug 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lgarciaaco lgarciaaco

Labels
cat/enhancement Enhancements to existing features and capabilities
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fetch ImageStreams tag from CR lgarciaaco/syndesis
2 participants
@heiko-braun @lgarciaaco