Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Release Flux v1.12.3 #2071

Merged
merged 20 commits into from
May 22, 2019
Merged

Release Flux v1.12.3 #2071

merged 20 commits into from
May 22, 2019

Conversation

2opremio
Copy link
Contributor

No description provided.

hiddeco and others added 20 commits May 22, 2019 12:54
Adding an explicit warning to users that Flux ignores JSON files.
If we are unable to match the image of the workload to a tag from our
cache, we still list the name of the tag and a question mark where we
would normally list the created date, to indicate we have no record of
it in our cache.
Before this change the `fluxctl list-workloads` command would only list
the status of a workload if it had containers. But because of how we
integrated HelmReleases into the Flux daemon, a HelmRelease only has
'containers' if we are able to detect images in the .spec.values of the
resource.

After this change it will always list the status of the workload (and
any policies), regardless of the amount of containers it (does not)
have.
We log when a resource in a manifest is dropped from sync because it's
marked as ignored; would also help troubleshooting if we log when a
resource is dropped because it's not in the allowed namespaces.
Logging all excluded resources may be quite noisy. Instead, collect
the IDs of excluded resources and log once (or no times, if there
aren't any).
Should resolve the trigger of several CVE warnings (which did not pose
a threat): CVE-2019-3829, CVE-2019-3836, CVE-2018-1000654
As a side effect, the tracing of commands also prints the full output
(instead of separating stderr and stdout), which I think is more useful.
In order for the initialization to retry if there is a netwroking glitch
Using master broke connectivity with the external world
(at least when used CircleCI)
* Run sequentially after build and tests in CircleCI to reuse artifacts
* Support running it locally (both in Linux and Darwin systems)
  through `make e2e`. An existing Kubernetes cluster will be used if available,
  otherwise a cluster will be created with Kind.
@2opremio 2opremio requested review from squaremo and hiddeco May 22, 2019 11:27
@2opremio 2opremio changed the base branch from master to release/1.12.x May 22, 2019 11:28
@hiddeco
Copy link
Member

hiddeco commented May 22, 2019

2c83246 originates from #1791 but is probably included because of 6466e32?

@2opremio
Copy link
Contributor Author

2c83246 originates from #1791 but is probably included because of 6466e32?

Yep

CHANGELOG.md Show resolved Hide resolved
Copy link
Member

@squaremo squaremo left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for compiling this Fons

@2opremio 2opremio merged commit d7c9955 into release/1.12.x May 22, 2019
@2opremio 2opremio deleted the release/1.12.3 branch May 22, 2019 12:45
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.

4 participants