Skip to content

Conversation

@camilamacedo86
Copy link
Contributor

@camilamacedo86 camilamacedo86 commented Nov 27, 2021

Description
Remove the duplicated manifests pushed in the postgresql bundles.
Seems like that the operator authors migrated their distributions to the bundle format but forgot to remove the files duplicated into the directories.

Motivation
Unable to load the bundles dir with the API (Error faced: Unable to load the bundle: error loading objs in directory: [unable to decode object: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal string into Go value of type map[string]interface {})

@openshift-ci openshift-ci bot requested review from J0zi and mvalarh November 27, 2021 16:14
@github-actions
Copy link
Contributor

@jkatz , please approve as you are original reviewer(s).

  • To add more contributors to ci.yaml follow instruction here.
  • To approve follow instruction here

@github-actions
Copy link
Contributor

Dockerfile or bundle.Dockerfile is added/changed. Note that for security reasons none of these files are going to be used when building bundle. Docker file will be generated and all label information is taken from annotations.yaml.

@github-actions github-actions bot changed the title fix: remove the duplicated manifests pushed in the postgresql bundles operators [R] postgresql (4.0.1 4.1.0 4.2.0 4.2.1 4.2.2 4.3.2 4.4.0 4.4.1 4.5.0 4.5.1 4.6.1 4.6.2 4.7.0 4.7.3 5.0.2 5.0.3 5.0.4) Nov 27, 2021
@camilamacedo86
Copy link
Contributor Author

camilamacedo86 commented Nov 27, 2021

Hi @jkatz,

Could you please check this one?

c/c @mvalarh @J0zi

@framework-automation
Copy link
Collaborator

/merge possible

@framework-automation
Copy link
Collaborator

/merge possible

@github-actions
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

1 similar comment
@github-actions
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

@framework-automation
Copy link
Collaborator

/merge possible

1 similar comment
@framework-automation
Copy link
Collaborator

/merge possible

@github-actions
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

1 similar comment
@github-actions
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

@jkatz
Copy link
Contributor

jkatz commented Nov 29, 2021

@camilamacedo86 👍

@framework-automation
Copy link
Collaborator

/merge possible

@framework-automation framework-automation merged commit cf3ad02 into k8s-operatorhub:main Dec 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

5 participants