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

#451 Added chocolatey package support for Functions. #564

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

martinpinto
Copy link
Contributor

Added a new package for chocolatey for the latest Functions version.

@seiflotfy seiflotfy self-assigned this Mar 3, 2017
@seiflotfy seiflotfy added this to the Beta 1 milestone Mar 3, 2017
<projectUrl>https://github.com/iron-io/functions/</projectUrl>
<!--<iconUrl>http://cdn.rawgit.com/__REPLACE_YOUR_REPO__/master/icons/functions.png</iconUrl>-->
<copyright>2017 Functions Iron.io</copyright>
<tags>serverless faas docker lambda</tags>
Copy link
Contributor

Choose a reason for hiding this comment

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

IFAIK we'd like to keep ourselves for being associated with Serverless (see https://serverless.com/). So it'll be better to have next tags:
functions faas docker containers (as an alternative - include COE there).

Copy link
Contributor

Choose a reason for hiding this comment

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

I'd say keep serverless and add functions. We do want to be associated with the term serverless as that's what everyone is referring to it as for the time being.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok done. Kept serverless and added functions.


<title>Iron.io Functions (Install)</title>
<authors>Iron.io Inc.</authors>
<projectUrl>https://github.com/iron-io/functions/</projectUrl>
Copy link
Contributor

Choose a reason for hiding this comment

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

trailing slash? just remove slash right after functions

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok done. Removed both the trailing slash from the <projectUrl> and <packageSourceUrl>.


$packageName= 'functions'
$toolsDir = "$(Split-Path -parent $MyInvocation.MyCommand.Definition)"
$url = 'https://github.com/iron-io/functions/releases/download/0.2.25/fn.exe'
Copy link
Contributor

Choose a reason for hiding this comment

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

Could this URL always grab the latest release? Similar to install.sh ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately this URL is bound to that version. We might need to adjust scripts to auto increase the version. Updated version to 0.2.36.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, we need to make the scripts update the version on release, same way as we do for git, docker, cli install, etc.

How does this get submitted to Chocolatey? We'll want to make sure that's automated too.


$packageName= 'functions'
$toolsDir = "$(Split-Path -parent $MyInvocation.MyCommand.Definition)"
$url = 'https://github.com/iron-io/functions/releases/download/0.2.25/fn.exe'
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, we need to make the scripts update the version on release, same way as we do for git, docker, cli install, etc.

How does this get submitted to Chocolatey? We'll want to make sure that's automated too.

seiflotfy and others added 7 commits March 29, 2017 20:52
* Fix lambda import

* more updates

* more changes
* Add inactivity_timeout to routes API

Closes: iron-io#544

* Fix failing datastore tests

* Rename inactivity_timeout to idle_timeout

* Update swagger doc

* Update hot fn doc

* Fix json tags

* Add function timeouts docs

* Rewording
* Solving postgres marshal/unmarshal issue

Postgres datastore was not marshaling the App config during its insert, that behavior was resulting in issues when fetching the App and the datastore couldn't unmarshal the config.

The same issue was probably happening with the Route's headers in some situations.

This commit's idea is to always try to marshal configs and headers when inserting/updating Apps or Routes. But in Apps and Routes get methods, if the  config/headers unmarshal fails, it returns an empty config/headers.

* fix one more unmarshal case

* returning error when unmarshaling non-empty
@CLAassistant
Copy link

CLAassistant commented Mar 29, 2017

CLA assistant check
All committers have signed the CLA.

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.

6 participants