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

Added chrony config via mco #247

Merged
merged 1 commit into from Mar 25, 2020
Merged

Added chrony config via mco #247

merged 1 commit into from Mar 25, 2020

Conversation

e-minguez
Copy link
Member

Description

Adds optional chrony configuration

Fixes #126

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Testing

I've locally tested it by running the playbook manually

Test Configuration:

  • Versions: ansible 2.9
  • Hardware: localhost

Checklist

  • I have performed a self-review of my own code
  • If a change is adding a feature, it should require a change to the README.md and the review should catch this.
  • A change should not being merged unless it passes CI or there is a comment/update saying what testing was passed.
  • PRs should not be merged unless positively reviewed.

@rlopez133
Copy link
Collaborator

[test-ipi]

@rlopez133 rlopez133 requested review from rlopez133 and abays and removed request for SchSeba March 25, 2020 12:39
@rlopez133 rlopez133 added approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. labels Mar 25, 2020
@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

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:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot merged commit 3ef7780 into openshift-kni:master Mar 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add chrony configs
4 participants