Skip to content
This repository has been archived by the owner on Oct 21, 2023. It is now read-only.

Missing New-VSEC2VPCEndpointService in "VaporShell/Public/Resource Types" #62

Closed
ScriptAutomate opened this issue Sep 4, 2019 · 9 comments
Assignees
Labels

Comments

@ScriptAutomate
Copy link

This function is missing because it is missing from the us-east-1 cfn spec json file, so automation reliant on it being present there isn't including it.

I reference the issue here with the CloudFormation documentation folks: Missing AWS::EC2::VPCEndpointService ResourceType in CloudFormationResourceSpecification.json for Most Regions

The automation you are using should automatically fix this once they introduce it to the proper CFN spec files, but I'm not sure how long that will take. Thought I'd give a heads up since my tooling recently discovered it missing from the spec files.

@scrthq
Copy link
Member

scrthq commented Sep 8, 2019

Hey @ScriptAutomate - apologies on slowness in ack'ing this. I'm working on rolling your suggestions for the documentation updates in now, which includes parsing the other spec files outside of us-east-1's for any missing resources and properties, which should catch odd ones out like this. I appreciate digging into it and raising awareness! I'll close this out once the new version is pushed, should be today if time allows!

@scrthq
Copy link
Member

scrthq commented Sep 8, 2019

Got it sorted to do the initial pull on us-east-1, then check the remaining spec sheets for anything missing. Working well so far:

image

@scrthq
Copy link
Member

scrthq commented Sep 8, 2019

Had to make a few adjustments to the CFNHelp module gist you shared to integrate with the build process, but this is getting rolled out shortly. Snippet including the updated help content of Get-Help New-VSEC2VPCEndpointService -Full for reference:

image

scrthq added a commit that referenced this issue Sep 8, 2019
@scrthq scrthq mentioned this issue Sep 8, 2019
scrthq added a commit that referenced this issue Sep 8, 2019
## 2.9.0 - 2019-09-08

* [Issue #61](#61) -- _Thanks, [@ScriptAutomate](https://github.com/ScriptAutomate)!_
  * Updated Parameter help for all Resource Type and Property Type functions to include the Parameter descriptions.
* [Issue #62](#62) -- _Thanks, [@ScriptAutomate](https://github.com/ScriptAutomate)!_
  * Updated the CI build process to check all spec sheets for any Resource Types and Property Types that may be missing from `us-east-1`.
* Miscellaneous
  * Brought Resource Type and Property Type functions up to current spec sheet.
scrthq added a commit that referenced this issue Sep 8, 2019
## 2.9.0 - 2019-09-08

* [Issue #61](#61) -- _Thanks, [@ScriptAutomate](https://github.com/ScriptAutomate)!_
  * Updated Parameter help for all Resource Type and Property Type functions to include the Parameter descriptions.
* [Issue #62](#62) -- _Thanks, [@ScriptAutomate](https://github.com/ScriptAutomate)!_
  * Updated the CI build process to check all spec sheets for any Resource Types and Property Types that may be missing from `us-east-1`.
* Miscellaneous
  * Brought Resource Type and Property Type functions up to current spec sheet.
@ScriptAutomate
Copy link
Author

Awesome! Looks like you're picking up stragglers just fine (including two new ones!), and the expansion of the property descriptions looks great.

A side note, more related to the feature request of comment-base help doc expansion in #61 is that I noticed that the Description (and optionally, the Synopsis) is still the previous default. The CFNHelp comes up with additional values for those (though it looks like my poc parsing may have left in some open parentheses where it didn't need to):

PS > $HelpDocs = New-CFNHelpDoc
PS > $HelpDocs | Where-Object -Property FunctionName -like "New-VS*VPCEndpointService"                                                                                     FunctionName : New-VSAWSEC2VPCEndpointService
FunctionName : New-VSAWSEC2VPCEndpointService
Description  : Specifies a VPC endpoint service configuration to 
               which service consumers (AWS accounts, IAM users, and 
               IAM roles can connect. Service consumers can create 
               an interface VPC endpoint to connect to your service.
               To create an endpoint service configuration, you must 
               first create a Network Load Balancer for your service.
Links        : {http://docs.aws.amazon.com/AWSCloudFormation/latest/U
               serGuide/aws-resource-ec2-vpcendpointservice.html}
Parameters   : {@{ParameterName=AcceptanceRequired; 
               ParameterDescription=Indicates whether requests from 
               service consumers to create an endpoint to your 
               service must be accepted.}, 
               @{ParameterName=NetworkLoadBalancerArns; 
               ParameterDescription=The Amazon Resource Names ARNs 
               of one or more Network Load Balancers for your 
               service.}}
Synopsis     : Specifies a VPC endpoint service configuration to 
               which service consumers (AWS accounts, IAM users, and 
               IAM roles can connect. Service consumers can create 
               an interface VPC endpoint to connect to your service.

@scrthq
Copy link
Member

scrthq commented Sep 9, 2019

Ahhhhhhhhhh I completely missed that and it would be easy to add! Thank you for the reminder! I'll get it added in as well, easy add since I already have the info as noted :D

@scrthq
Copy link
Member

scrthq commented Sep 9, 2019

Testing the addition now, then pushing out 2.9.1 with the updated info.

@scrthq
Copy link
Member

scrthq commented Sep 9, 2019

Looking nice, pushing momentarily:

image

@scrthq
Copy link
Member

scrthq commented Sep 9, 2019

v2.9.1 pushed with the added help bits. Thanks again!

PS - You can see the updated functions in the ci folder of the repo: https://github.com/scrthq/VaporShell/tree/master/ci

They're called from the Convert-SpecToFunction function during the CI's Build stage

@scrthq scrthq closed this as completed Sep 9, 2019
@PatMyron
Copy link

AWS::EC2::VPCEndpointService is now in the us-east-1 resource specification itself as well

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants