Skip to content

Conversation

@VinozzZ
Copy link
Contributor

@VinozzZ VinozzZ commented Sep 7, 2022

  • update porter to v1.0.0-rc.1 and set it as the default porter-agent version
  • use kubernetes-plugin v1.0.0-rc.1 in integration test
  • update getporter/magefiles to v0.3.2
    After this PR is merged, we will do release v0.7.0 for the porter-operator

@VinozzZ VinozzZ changed the title update porter to v1.0.0-rc.1 WIP update porter to v1.0.0-rc.1 Sep 7, 2022
@VinozzZ VinozzZ force-pushed the update-porter-to-rc.1 branch 4 times, most recently from 34fd628 to 6ff15d1 Compare September 7, 2022 22:55
feed string
version string
}{
{name: "kubernetes", feed: "https://cdn.porter.sh/plugins/atom.xml", version: "latest"},
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It doesn't look like we are publishing to the latest tag anymore

Copy link
Member

Choose a reason for hiding this comment

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

When you run porter plugin install and use the latest version, it downloads the atom.xml file and selects the highest version of that plugin, excluding pre-releases.

@VinozzZ VinozzZ changed the title WIP update porter to v1.0.0-rc.1 update porter to v1.0.0-rc.1 Sep 7, 2022
@VinozzZ
Copy link
Contributor Author

VinozzZ commented Sep 7, 2022

This PR needs to wait for the ghcr.io/getporter/dev/porter-agent-kubernetes:v1.0.0-rc.1 to be published first

@codecov
Copy link

codecov bot commented Sep 8, 2022

Codecov Report

Merging #116 (832e16d) into main (8419997) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #116   +/-   ##
=======================================
  Coverage   78.07%   78.07%           
=======================================
  Files          12       12           
  Lines         976      976           
=======================================
  Hits          762      762           
  Misses        136      136           
  Partials       78       78           
Flag Coverage Δ
unit-tests 78.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Signed-off-by: Yingrong Zhao <[email protected]>
@VinozzZ VinozzZ force-pushed the update-porter-to-rc.1 branch from 6ff15d1 to 832e16d Compare September 8, 2022 21:06
Copy link
Member

@carolynvs carolynvs left a comment

Choose a reason for hiding this comment

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

LGTM!

feed string
version string
}{
{name: "kubernetes", feed: "https://cdn.porter.sh/plugins/atom.xml", version: "latest"},
Copy link
Member

Choose a reason for hiding this comment

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

When you run porter plugin install and use the latest version, it downloads the atom.xml file and selects the highest version of that plugin, excluding pre-releases.

@carolynvs carolynvs merged commit 3c83e41 into getporter:main Sep 8, 2022
@VinozzZ VinozzZ deleted the update-porter-to-rc.1 branch September 8, 2022 21:22
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.

2 participants