cloud-images

Merge ~davidkrauser/cloud-images/+git/aws-eks-website:sack-v2 into ~cloud-images-release-managers/cloud-images/+git/aws-eks-website:master

  1. Git
  2. lp:~davidkrauser/cloud-images/+git/aws-eks-website
  3. sack-v2
  4. Merge into master
Proposed by David Krauser
Status: Merged
Merged at revision: 21e376e8d477d9e328bab9439da7f819dc12fa2b
Proposed branch: ~davidkrauser/cloud-images/+git/aws-eks-website:sack-v2
Merge into: ~cloud-images-release-managers/cloud-images/+git/aws-eks-website:master
Diff against target: 295 lines (+56/-54)
1 file modified
index.html (+56/-54)
Reviewer Review Type Date Requested Status
Robert C Jennings (community) Approve
Tobias Koch (community) Approve
Review via email: mp+369602@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Tobias Koch (tobijk) wrote :

LGTM, though I don't know how to verify the ami ids.

review: Approve
Revision history for this message
Robert C Jennings (rcj) :
review: Needs Fixing
Revision history for this message
Robert C Jennings (rcj) wrote :

The diff looks good (i.e. I have verified you're only changing AMI IDs beyond the changelog). Just looking for more detail in the changelog diff.

Revision history for this message
Robert C Jennings (rcj) :
review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches