cron job to use batch/v1beta1

Bug #1753524 reported by Hemanth Nakkina
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
Fix Committed
Undecided
Unassigned
Trunk
Fix Committed
Undecided
Unassigned
openstack-helm
Fix Released
Undecided
Unassigned
openstack-helm-infra
In Progress
Undecided
Unassigned

Bug Description

In Elasticsearch helm chart, Cron job Curator expects batch/v2alpha1 to be enabled in k8s api versions

https://github.com/openstack/openstack-helm-infra/blob/master/elasticsearch/templates/cron-job-curator.yaml#L18

As per documentation @ https://v1-8.docs.kubernetes.io/docs/concepts/workloads/controllers/cron-jobs/

Note: CronJob resource in batch/v2alpha1 API group has been deprecated starting from cluster version 1.8. You should switch to using batch/v1beta1, instead, which is enabled by default in the API server.

This bug is an enhancement to modify batch/v2alpha1 to batch/v1beta1

Revision history for this message
Hemanth Nakkina (hemanth-n) wrote :

This bug is applicable for CronJobs in

openStack-helm-infra
  Elasticsearch/templates/cron-job-curator

openstack-helm
  cinder/templates/cron-job-cinder-volume-usage-audit.yaml
  heat/templates/cron-job-engine-cleaner.yaml
  keystone/templates/cron-job-fernet-rotate.yaml
  keystone/templates/cron-job-credential-rotate.yaml

summary: - cron job curator to use batch/v1beta1
+ cron job to use batch/v1beta1
Changed in openstack-helm-infra:
assignee: nobody → Hemanth Nakkina (hemanth-n)
Revision history for this message
Pete Birley (portdirect) wrote :

The -ve to this change is that it breaks backward compatibility - though many of our charts are 1.8 onwards only - so its swings and roundabouts.

Changed in openstack-helm-infra:
status: New → In Progress
Changed in openstack-helm:
assignee: nobody → Hemanth Nakkina (hemanth-n)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-helm (master)

Reviewed: https://review.openstack.org/549923
Committed: https://git.openstack.org/cgit/openstack/openstack-helm/commit/?id=8c9ac9f5df6a5d832667dc940e7e367377019b98
Submitter: Zuul
Branch: master

commit 8c9ac9f5df6a5d832667dc940e7e367377019b98
Author: Hemanth Nakkina <email address hidden>
Date: Tue Mar 6 08:43:54 2018 +0530

    Remove Api version constraint for CronJob

    Cronjob resource is deprecated in batch/v2alpha1 from
    k8s 1.8 and batch/v1beta1 is enabled by default. All the
    CronJobs are already using batch/v1beta1 but there is condition
    to check if api version have batch/v2alpha1.

    Remove the api version constraint on batch/v2alpha1

    Partial-Bug: #1753524
    Change-Id: I7eeb7d6cc2630311ec5d613b9e059824daae0620

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-helm-infra (master)

Reviewed: https://review.openstack.org/549795
Committed: https://git.openstack.org/cgit/openstack/openstack-helm-infra/commit/?id=5392ac810b3b46a0ef28d345013ba36eff766f59
Submitter: Zuul
Branch: master

commit 5392ac810b3b46a0ef28d345013ba36eff766f59
Author: Hemanth Nakkina <email address hidden>
Date: Mon Mar 5 21:23:49 2018 +0530

    Remove Api version requirement for CronJob Curator

    Curator job in Elasticsearch helm chart has a condition on
    api version batch/v2alpha1. Cronjob resource is deprecated
    in batch/v2alpha1 from k8s 1.8 and batch/v1beta1 is enabled
    by default.

    Remove the condition on API version as it is no more required.

    Closes-Bug: #1753524
    Change-Id: Ia296b3742e655fae508e5d4402e7f3881db31688

Changed in openstack-helm:
status: New → Fix Released
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/42040
Submitter: <email address hidden> (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/42138
Submitter: <email address hidden> (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0

Review in progress for https://review.opencontrail.org/42212
Submitter: <email address hidden> (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged
Download full text (9.0 KiB)

Reviewed: https://review.opencontrail.org/42138
Committed: http://github.com/Juniper/openstack-helm-infra/commit/c2040f36112b244d50920b91ef5fb401bc25a777
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit c2040f36112b244d50920b91ef5fb401bc25a777
Author: Madhukar Nayakbomman <email address hidden>
Date: Wed Apr 18 16:14:25 2018 -0700

Rebasing openstack-helm-infra

Changes from below commits are been added as part of rebase

OSH-Infra: Update labels for chart components

This ps adds more granular node selectors for the charts in osh
infra to match what is currently done in osh

Change-Id: I8957a95053b9fb3ea329fd37ff049cd223a7695d

Address errors with Elasticsearch and Kibana

This moves Elasticsearch and Kibana to use the latest version
(6.2.2), as the images we were using are no longer supported with
the 6.x release. There was a change in the doc reference in the
log entries that prevented the previous ES version from indexing
those entries, resulting in a busted gate. Moving Kibana to 6.2.2
was required to match major/minor versions with Elasticsearch

The Elasticsearch version change also required changing config file
locations, changing the entrypoint used for launching the service,
changing the running user for the elasticsearch service, and
updated the ES tests as some of the API responses changed between
versions

This also required updating the elasticsearch template job as the
mapping definition entries changed between versions

Change-Id: Ia4cd9a66851754a1bb8f225c7e24513c43568e93

kube-state-metrics: use endpoints section and lookups to set port

This PS moves kube-state-metrics to use the endpoints section and
lookups to set the ports it serves on.

Change-Id: Icb4757a59852e508148ca9f1e682c722e40042c9

Remove Api version requirement for CronJob Curator

Curator job in Elasticsearch helm chart has a condition on
api version batch/v2alpha1. Cronjob resource is deprecated
in batch/v2alpha1 from k8s 1.8 and batch/v1beta1 is enabled
by default.

Remove the condition on API version as it is no more required.

Closes-Bug: #1753524
Change-Id: Ia296b3742e655fae508e5d4402e7f3881db31688

KubeADM: force calico interface to match to same as kubelet

This PS forces calico to use the same interface as the kubelet
for inter-node communication.

Change-Id: I7d03beec46cccba4a57e092bf108695e8e88996f

Kibana: use endpoints section and lookups to set port

This PS moves kibana to use the endpoints section and lookups to
set the port it serves on.

Change-Id: I710428f92e80faf6ac5bb444f938447248e99217

Prometheus: use endpoints section and lookups to set port

This PS moves prometheus to use the endpoints section and lookups
to set the port it serves on.

Change-Id: Ifae665e21128dd566da5a68b9904a94a68df4018

Grafana: use endpoints section and lookups to set port

This PS moves grafana to use the endpoints section and lookups to
set the port it serves on.

Change-Id: I51d4c10297f3423569539dcbf7fbecad24d5e47f

Fluent-logging: use endpoints section and lookups to set port

This PS moves fluent-logging to use the endpoints section and
lookups to set the port it serves on.

Change-Id: I7cbbd8d6287942eb36f70ae74872405038e5...

Read more...

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :
Download full text (9.0 KiB)

Reviewed: https://review.opencontrail.org/42212
Committed: http://github.com/Juniper/openstack-helm-infra/commit/47e07754407710a373b9805cfc0336be9b017767
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 47e07754407710a373b9805cfc0336be9b017767
Author: Madhukar Nayakbomman <email address hidden>
Date: Wed Apr 18 16:14:25 2018 -0700

Rebasing openstack-helm-infra

Changes from below commits are been added as part of rebase

OSH-Infra: Update labels for chart components

This ps adds more granular node selectors for the charts in osh
infra to match what is currently done in osh

Change-Id: I8957a95053b9fb3ea329fd37ff049cd223a7695d

Address errors with Elasticsearch and Kibana

This moves Elasticsearch and Kibana to use the latest version
(6.2.2), as the images we were using are no longer supported with
the 6.x release. There was a change in the doc reference in the
log entries that prevented the previous ES version from indexing
those entries, resulting in a busted gate. Moving Kibana to 6.2.2
was required to match major/minor versions with Elasticsearch

The Elasticsearch version change also required changing config file
locations, changing the entrypoint used for launching the service,
changing the running user for the elasticsearch service, and
updated the ES tests as some of the API responses changed between
versions

This also required updating the elasticsearch template job as the
mapping definition entries changed between versions

Change-Id: Ia4cd9a66851754a1bb8f225c7e24513c43568e93

kube-state-metrics: use endpoints section and lookups to set port

This PS moves kube-state-metrics to use the endpoints section and
lookups to set the ports it serves on.

Change-Id: Icb4757a59852e508148ca9f1e682c722e40042c9

Remove Api version requirement for CronJob Curator

Curator job in Elasticsearch helm chart has a condition on
api version batch/v2alpha1. Cronjob resource is deprecated
in batch/v2alpha1 from k8s 1.8 and batch/v1beta1 is enabled
by default.

Remove the condition on API version as it is no more required.

Closes-Bug: #1753524
Change-Id: Ia296b3742e655fae508e5d4402e7f3881db31688

KubeADM: force calico interface to match to same as kubelet

This PS forces calico to use the same interface as the kubelet
for inter-node communication.

Change-Id: I7d03beec46cccba4a57e092bf108695e8e88996f

Kibana: use endpoints section and lookups to set port

This PS moves kibana to use the endpoints section and lookups to
set the port it serves on.

Change-Id: I710428f92e80faf6ac5bb444f938447248e99217

Prometheus: use endpoints section and lookups to set port

This PS moves prometheus to use the endpoints section and lookups
to set the port it serves on.

Change-Id: Ifae665e21128dd566da5a68b9904a94a68df4018

Grafana: use endpoints section and lookups to set port

This PS moves grafana to use the endpoints section and lookups to
set the port it serves on.

Change-Id: I51d4c10297f3423569539dcbf7fbecad24d5e47f

Fluent-logging: use endpoints section and lookups to set port

This PS moves fluent-logging to use the endpoints section and
lookups to set the port it serves on.

Change-Id: I7cbbd8d6287942eb36f70ae74872405038e523...

Read more...

Changed in openstack-helm-infra:
assignee: Hemanth Nakkina (hemanth-n) → nobody
Changed in openstack-helm:
assignee: Hemanth Nakkina (hemanth-n) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.