Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

jobs for Planner renamed and fixed #912

Merged
merged 1 commit into from
Dec 3, 2018

Conversation

ppitonak
Copy link
Contributor

@ppitonak ppitonak commented Dec 3, 2018

No description provided.

@ppitonak
Copy link
Contributor Author

ppitonak commented Dec 3, 2018

Please remove the following two jobs:

  • devtools-test-e2e-openshift.io-planner
  • devtools-test-e2e-prod-preview.openshift.io-planner

@riuvshin
Copy link
Contributor

riuvshin commented Dec 3, 2018

[test]

@skryzhny
Copy link
Contributor

skryzhny commented Dec 3, 2018

It looks like credential names doesn't correspond to urls, like mixed -preview and prod URLs or secrets

ee_test_start_time: '35 */2 * * *'
cluster: us-east-2a
feature_level: released
osio_creds: osio-ci-planner-002
Copy link
Contributor

Choose a reason for hiding this comment

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

I do not see this secret in vault and some others from this PR
https://vault.devshift.net/ui/vault/secrets/devtools-osio-ci/list/e2e/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@skryzhny you are right that they are mixed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@riuvshin
Copy link
Contributor

riuvshin commented Dec 3, 2018

[test]

@riuvshin riuvshin merged commit ccacdf1 into openshiftio:master Dec 3, 2018
@ppitonak ppitonak deleted the planner_jobs branch December 3, 2018 16:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants