-
Notifications
You must be signed in to change notification settings - Fork 611
bake: add unixtimestampparse function #3286
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -205,3 +205,29 @@ func TestHomedir(t *testing.T) { | |
| require.NotEmpty(t, home.AsString()) | ||
| require.True(t, filepath.IsAbs(home.AsString())) | ||
| } | ||
|
|
||
| func TestUnixTimestampParseFunc(t *testing.T) { | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You might want to add tests for |
||
| fn := unixtimestampParseFunc() | ||
| input := cty.NumberIntVal(1690328596) | ||
| got, err := fn.Call([]cty.Value{input}) | ||
| require.NoError(t, err) | ||
|
|
||
| expected := map[string]cty.Value{ | ||
| "year": cty.NumberIntVal(2023), | ||
| "year_day": cty.NumberIntVal(206), | ||
| "day": cty.NumberIntVal(25), | ||
| "month": cty.NumberIntVal(7), | ||
| "month_name": cty.StringVal("July"), | ||
| "weekday": cty.NumberIntVal(2), | ||
| "weekday_name": cty.StringVal("Tuesday"), | ||
| "hour": cty.NumberIntVal(23), | ||
| "minute": cty.NumberIntVal(43), | ||
| "second": cty.NumberIntVal(16), | ||
| "rfc3339": cty.StringVal("2023-07-25T23:43:16Z"), | ||
| "iso_year": cty.NumberIntVal(2023), | ||
| "iso_week": cty.NumberIntVal(30), | ||
| } | ||
| for k, v := range expected { | ||
| require.True(t, got.GetAttr(k).RawEquals(v), "field %s: got %v, want %v", k, got.GetAttr(k), v) | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -103,6 +103,7 @@ title: Bake standard library functions | |
| | [`trimspace`](#trimspace) | Removes any consecutive space characters (as defined by Unicode) from the start and end of the given string. | | ||
| | [`trimsuffix`](#trimsuffix) | Removes the given suffix from the start of the given string, if present. | | ||
| | [`try`](#try) | Variadic function that tries to evaluate all of is arguments in sequence until one succeeds, in which case it returns that result, or returns an error if none of them succeed. | | ||
| | [`unixtimestampparse`](#unixtimestampparse) | Given a unix timestamp integer, will parse and return an object representation of that date and time. A unix timestamp is the number of seconds elapsed since January 1, 1970 UTC. | | ||
| | [`upper`](#upper) | Returns the given string with all Unicode letters translated to their uppercase equivalents. | | ||
| | [`urlencode`](#urlencode) | Applies URL encoding to a given string. | | ||
| | [`uuidv4`](#uuidv4) | Generates and returns a Type-4 UUID in the standard hexadecimal string format. | | ||
|
|
@@ -1385,6 +1386,39 @@ target "webapp-dev" { | |
| } | ||
| ``` | ||
|
|
||
| ### <a name="unixtimestampparse"></a> `unixtimestampparse` | ||
|
|
||
| The returned object has the following attributes: | ||
| * `year` (Number) The year for the unix timestamp. | ||
| * `year_day` (Number) The day of the year for the unix timestamp, in the range 1-365 for non-leap years, and 1-366 in leap years. | ||
| * `day` (Number) The day of the month for the unix timestamp. | ||
| * `month` (Number) The month of the year for the unix timestamp. | ||
| * `month_name` (String) The name of the month for the unix timestamp (ex. "January"). | ||
| * `weekday` (Number) The day of the week for the unix timestamp. | ||
| * `weekday_name` (String) The name of the day for the unix timestamp (ex. "Sunday"). | ||
| * `hour` (Number) The hour within the day for the unix timestamp, in the range 0-23. | ||
| * `minute` (Number) The minute offset within the hour for the unix timestamp, in the range 0-59. | ||
| * `second` (Number) The second offset within the minute for the unix timestamp, in the range 0-59. | ||
| * `rfc3339` (String) The RFC3339 format string. | ||
| * `iso_year` (Number) The ISO 8601 year number. | ||
| * `iso_week` (Number) The ISO 8601 week number. | ||
|
|
||
| ```hcl | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. how about I'm also ok for special formatting for rfc3399 for less verbosity.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I looked at forking |
||
| # docker-bake.hcl | ||
| variable "SOURCE_DATE_EPOCH" { | ||
| type = number | ||
| default = 1690328596 | ||
| } | ||
| target "default" { | ||
| args = { | ||
| SOURCE_DATE_EPOCH = SOURCE_DATE_EPOCH | ||
| } | ||
| labels = { | ||
| "org.opencontainers.image.created" = unixtimestampparse(SOURCE_DATE_EPOCH).rfc3339 | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| ### <a name="upper"></a> `upper` | ||
|
|
||
| ```hcl | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see above
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You might want to also handle the error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.Int64()does not return an error as second value but accuracy: https://pkg.go.dev/math/big#Float.Int64There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my note for the tests … someone could conceivably supply the following values
-1,0,1.2, and"0"