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

chore(update): gen-manifests update #52

Merged
merged 411 commits into from
Nov 11, 2020

Conversation

cristhian-castaneda
Copy link

No description provided.

spinnakerbot and others added 30 commits March 11, 2020 01:12
Co-authored-by: Travis CI User <travis@example.org>
* chore(containers): tell curl to fail if it gets an HTTP error

This will prevent errors from going unnoticed as happened with the
missing AWS client.

* chore(containers): Upgrade the bundled tools
…1565)

* feat(stats): Rename telemetry to stats, enable by default
* fix(stats): Rename TelemetryWrapper to StatsWrapper
Co-authored-by: Cameron Fieber <cfieber@netflix.com>
Co-authored-by: Travis CI User <travis@example.org>
…aker#1572)

* feat(k8s/deploy): Add custom health check to service-settings.

Need the ability to override the default commands created and populated
in the Deployments readinessprobe. In our use case mTLS has been enabled
and we need a way to modify the readiness probe in each service so that
you can pass in the client certificate.

By adding this functionality Halyard can support a much wider array of potential
configurations for health checks for cases where a simple check of /health might not
be desirable.

* Update var to the correct generic type

* fix(k8s/deploy): Update var to the correct generic type

* chore(fix/style): ran spotlessJava to fix style issues
Co-authored-by: runner <runner@fv-az51>
Co-authored-by: runner <runner@fv-az50>
Co-authored-by: runner <runner@fv-az117.internal.cloudapp.net>
Co-authored-by: runner <runner@fv-az76>
Co-authored-by: runner <runner@fv-az51>
Slack is littered with people having this issue. It looks like this was added in halyard 1.23 on 8/27/2019. So anyone with a config after that will have this issue.
…pinnaker#1584)

The mergify config currently doesn't distinguish between merging to master and
merging to a release branch.  It currently never merges to release branches
because mergify doesn't have permission to merge to these branches (due to
branch protection rules).

I don't want to give mergify that permission without updating the config file
here, as then it would merge to release branches after any approval, whereas we
only want to merge to release branches if a PR is approved by a release manager.

This updates the mergify config to scope the existing rules to only apply to
the master branch, and adds a new rule to merge to release branches upon
approval by a release manager.

Because the 1.17 branch (and likely the 1.18 branch in some repos) is still
exclusively using Travis CI instead of github actions, also add a rule the
merges if Travis CI passes to account for these branches. This rule can be
deleted once all of the active branches in the repo are using Github actions.

Co-authored-by: Michael Plump <plumpy@google.com>
Co-authored-by: runner <runner@fv-az33>
Co-authored-by: runner <runner@fv-az51.internal.cloudapp.net>
* fix(plugins): changing plugin-manifest.js to .json file type

* fix(plugins): remove comments from json file, handle multiple plugins
* feat(plugins): add plugins to all services

* refactor(plugins): remove introspection

* refactor(plugins): change Boolean to boolean
Co-authored-by: runner <runner@fv-az118.internal.cloudapp.net>
Co-authored-by: runner <runner@fv-az51>
…innaker#1590)

This change breaks deploying any version of Spinnaker <= 1.18
using Halyard, so is being reverted.

This reverts commit 4ed4d26.
Co-authored-by: runner <runner@fv-az51>
Co-authored-by: runner <runner@fv-az51>
Co-authored-by: runner <runner@fv-az51>
Co-authored-by: runner <runner@fv-az51>
* feat(plugins): add plugins to all services

* fix(regression): update version check to handle non-semver releases

* fix(plugins): do not deploy to kayenta until 1.20.0
plumpy and others added 27 commits September 22, 2020 10:09
Co-authored-by: root <root@bcaea77c9d18>
Co-authored-by: root <root@28bd9d7e8d49>
…ct (spinnaker#1806)

The `kubernetes-spinnaker` project seemingly exists solely to host this one image. I have no idea why. Let's move it into its own Artifact Registry repository in the CDF-owned `spinnaker-community` project.

Also, the version of Redis in this image is 3.2.11, which is three years old, but I suppose that's a problem for another day.
…ing roles. (spinnaker#1805)

* feat(aws): adding externalId command and field for aws account config

* chore(aws): better definition for the new property external Id

Co-authored-by: Justin Field <justin.field@armory.io>
Co-authored-by: root <root@b272e1bb32f7>
Required a few manual changes because of spinnaker/kork#783
Co-authored-by: root <root@2897678eb22e>
Co-authored-by: root <root@c51049d641bc>
Co-authored-by: root <root@893456aa42c8>
Co-authored-by: root <root@e9b0b38e4e5f>
Co-authored-by: root <root@9c1f95bcaee1>
Co-authored-by: root <root@73fd689d0395>
…nnaker#1820)

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
# Conflicts:
#	Dockerfile.slim
#	Dockerfile.ubuntu
#	README.md
#	build.gradle
#	docs/commands.md
#	gradle.properties
#	gradle/wrapper/gradle-wrapper.properties
#	halyard-cli/src/main/java/com/netflix/spinnaker/halyard/cli/command/v1/ConfigCommand.java
#	halyard-cli/src/main/java/com/netflix/spinnaker/halyard/cli/services/v1/Daemon.java
#	halyard-config/halyard-config.gradle
#	halyard-config/src/main/java/com/netflix/spinnaker/halyard/config/config/v1/ArtifactSourcesConfig.java
#	halyard-config/src/main/java/com/netflix/spinnaker/halyard/config/config/v1/HalconfigDirectoryStructure.java
#	halyard-config/src/main/java/com/netflix/spinnaker/halyard/config/services/v1/FileService.java
#	halyard-config/src/main/java/com/netflix/spinnaker/halyard/config/validate/v1/providers/cloudfoundry/CloudFoundryAccountValidator.java
#	halyard-deploy/src/main/java/com/netflix/spinnaker/halyard/deploy/config/v1/secrets/DecryptingObjectMapper.java
#	halyard-deploy/src/main/java/com/netflix/spinnaker/halyard/deploy/services/v1/DeployService.java
#	halyard-deploy/src/main/java/com/netflix/spinnaker/halyard/deploy/services/v1/GenerateService.java
#	halyard-deploy/src/main/java/com/netflix/spinnaker/halyard/deploy/spinnaker/v1/profile/KubernetesV2ClouddriverProfileFactory.java
#	halyard-deploy/src/main/java/com/netflix/spinnaker/halyard/deploy/spinnaker/v1/service/distributed/kubernetes/v2/KubernetesV2RedisService.java
#	halyard-web/config/halyard.yml
#	install/Installer.sh
#	release/docker.sh
@cristhian-castaneda cristhian-castaneda merged commit 8e0406f into gen-manifests Nov 11, 2020
@cristhian-castaneda cristhian-castaneda deleted the chore/gen-manifests-update branch November 11, 2020 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.