You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openwhisk.apache.org by OpenWhisk Team Slack <ra...@apache.org> on 2019/03/08 07:20:37 UTC
[slack-digest] [2019-03-07] #general
2019-03-07 03:49:12 UTC - keonhee Kim: @Saurab Joshi
Let’s check three things below.
- Is your apihost port 80 right?
- Is there a `@` character in the database password?
- Connect DB admin (maybe `172.17.0.1:5984/_utils`) > `whisk_alarms_alarmservice` > `Views` > `triggers_by_worker` > Check to exist alarm documents
+1 : Dominic Kim
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551930552128600
----
2019-03-07 11:11:25 UTC - Bruce Adams: I see some enhancements I’ll suggest.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551957085128800?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 11:11:55 UTC - Bruce Adams: (My day yesterday was a never-ending disaster.)
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551957115129000?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 11:46:31 UTC - Carlos Santana: @Roberto Diaz what do you think about the signature being handler(jsonValue) -> Result(jsonValue, jsonValue) { } ?
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551959191129200?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 11:48:22 UTC - Carlos Santana: @Michele Sciabarra I was going to open a Github Issue to discuss main method signature, but the repo <https://github.com/apache/incubator-openwhisk-runtime-rust> doesn't have Issues enable.
Did you open the INFRA ticket to enable Github Issues and the other things, if you give me INFRA link I can try ping infra guys cc @Rodric Rabbah
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551959302129400?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 11:49:58 UTC - Michele Sciabarra: I think I do not have the permissions to do those things @Rodric Rabbah
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551959398129600?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 11:56:24 UTC - Carlos Santana: Yes I know you don't have permission I either, but you can open a Ticket in INFRA
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551959784129900?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 11:57:38 UTC - Carlos Santana: You can open the ticket here <https://issues.apache.org/jira/projects/INFRA/issues>
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551959858130100?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 11:59:09 UTC - Carlos Santana: You can use this as template <https://issues.apache.org/jira/browse/INFRA-16770>
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551959949130300?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:01:05 UTC - Michele Sciabarra: Hmm my Apache password does not work there
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960065130500?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:02:32 UTC - Roberto Diaz: @Carlos Santana what do you mean? change `main(hashmap<String,Value>)->hashmap<String,Value>` by `handler(Value)->Result<Value,Error>`
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960152130700?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:05:27 UTC - Michele Sciabarra: The "Value" thing was a bit annoying I dot not know if there is a generic type in Rust to represent something in an HashMap that can be a String or a Number or an Array or another HashMap
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960327130900?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:05:39 UTC - Michele Sciabarra: that would be ideal
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960339131100?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:10:03 UTC - Carlos Santana: @Michele Sciabarra You need to create a new account for issues website, the Apache ID doesn't work there, that anyone even without an Apache ID can open issues
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960603131300?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:10:17 UTC - Carlos Santana: Yeah I know it sucks to have multiple accounts
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960617131500?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:10:39 UTC - Carlos Santana: @Roberto Diaz I sent an email to the dev list, we can discuss there while we get the github issue enable
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960639131700?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:11:03 UTC - Roberto Diaz: <https://stackoverflow.com/questions/39146584/how-do-i-create-a-rust-hashmap-where-the-value-can-be-one-of-multiple-types>
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960663132100?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:11:04 UTC - Carlos Santana: What I meant was like this <https://gist.github.com/csantanapr/50cae6a62b27192f32b1bd4801d8d7c4#file-rust_playground-rs-L40>
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960664132400?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:11:05 UTC - Carlos Santana:
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960665132600
----
2019-03-07 12:11:16 UTC - Carlos Santana: `fn handler_b(param: JsonValue) -> Result<JsonValue, JsonValue> {`
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960676132900?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:11:25 UTC - Roberto Diaz: this thread is talking about what @Michele Sciabarra says
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960685133100?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:11:50 UTC - Carlos Santana: For the most part all our runtimes have json in json out, or json in promise out resolve(json) or reject(json)
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960710133300?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:12:27 UTC - Carlos Santana: isn't having a jsonValue easier to unmarshall and marshall that in to an object?
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960747133500?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:12:51 UTC - Roberto Diaz: yess is an option
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960771133700?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:12:51 UTC - Carlos Santana: take into account I only have 3 days of experience with Rust :slightly_smiling_face:
100 : Roberto Diaz
rocket : Roberto Diaz
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960771133900?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:13:17 UTC - Roberto Diaz: instead of pass the hashmap pass the enum from value
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960797134100?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:13:48 UTC - Carlos Santana: It I saw it similar to what I did with Swift, where we have similar Decodable and Codable and json the runtime takes care of the serdes not the developer
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960828134300?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:15:13 UTC - Carlos Santana: like dealing with nesting jsons types in a struct
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960913134800?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:15:55 UTC - Carlos Santana: I can take the jsonValue and just create the object of a Person, which might have name and age, but also Address which is another Struct and son on
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551960955135000?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:16:55 UTC - Carlos Santana: didn't find a one liner to go from input HashMap to Person struct
```
struct Person {
name: String,
age: u8,
adress: Address,
}
struct Address {
street: String,
zipcode: String
}
```
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551961015135200?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:25:06 UTC - Roberto Diaz: yes, you are right
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551961506135500?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 12:25:42 UTC - Roberto Diaz: with hashmap that's not possible, the much that you can do is iterate over it and populate the object
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551961542135700?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 14:04:36 UTC - Saurab Joshi: @keonhee Kim Thanks! it was my DBURL that was messed up
tada : Rodric Rabbah, keonhee Kim
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551967476136200
----
2019-03-07 14:40:18 UTC - Bruce Adams: I’ve got some proposed changes here: <https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1>
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551969618136500?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 14:44:06 UTC - Bruce Adams: I didn’t do anything with actions/lib.rs. I agree that the signature might be better simplified. The simplest thing looks like this:
```pub fn main(input_data: Value) -> Value {
``` Having input_data mutable does not sound right to me.
100 : Roberto Diaz
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551969846136700?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 14:45:58 UTC - Bruce Adams: Serde has <https://docs.serde.rs/serde_json/fn.from_value.html> which makes it very easy for someone to consume the input. Similarly, <https://docs.serde.rs/serde_json/fn.to_value.html> makes it easy to take an arbitrary struct conform to the return type.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551969958136900?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 15:41:50 UTC - Bruce Adams: Hmm. I wonder how we want to deal with errors. Seems like `main` should return a `Result<Value, Error>`.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551973310137400?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 15:43:02 UTC - Bruce Adams: Reworked example that feels more _Rust-like_ to me.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551973382137600?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 15:45:20 UTC - Bruce Adams: That would require the action_loop to deal with error returns, but that seems OK and very helpful for developers.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551973520138000?thread_ts=1551890864.115200&cid=C3TPCAQG1
----
2019-03-07 15:51:36 UTC - Michele Sciabarra: @Roberto Diaz please review the PR <https://github.com/apache/incubator-openwhisk-runtime-rust/pull/1>
+1 : Roberto Diaz
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551973896138900?thread_ts=1551973896.138900&cid=C3TPCAQG1
----
2019-03-07 15:51:49 UTC - Michele Sciabarra: as that is your work I will proxy your opinon
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551973909139300
----
2019-03-07 17:22:00 UTC - Rodric Rabbah: Does rust have a src formatter a la gofmt?
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551979320140100?thread_ts=1551979320.140100&cid=C3TPCAQG1
----
2019-03-07 17:45:47 UTC - Roberto Diaz: `rustfmt`
thankyou : Rodric Rabbah
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551980747140200?thread_ts=1551979320.140100&cid=C3TPCAQG1
----
2019-03-07 17:53:34 UTC - Rodric Rabbah: <https://issues.apache.org/jira/browse/INFRA-17976> created to configure rust repo
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551981214140700
----
2019-03-07 18:21:09 UTC - Rodric Rabbah: :pray: who can review this PR <https://github.com/apache/incubator-openwhisk/pull/4327>
`Use Gauge metric instead of Histogram where applicable?`
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551982869141200?thread_ts=1551982869.141200&cid=C3TPCAQG1
----
2019-03-07 18:39:18 UTC - Carlos Santana: I thought @Sugandha had a similar PR or working towards it ^^
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551983958141900
----
2019-03-07 20:23:13 UTC - Saurab Joshi: Do we have to use `./gradlew distDocker` to build docker images? Can we just pull those images from Dockerhub?
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551990193142400
----
2019-03-07 20:23:53 UTC - Rodric Rabbah: you can skip the build and go to deploy, but change the image prefix to openwhisk
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551990233142900
----
2019-03-07 20:24:22 UTC - Rodric Rabbah: if using ansible, `-e"docker_image_prefix=openwhisk"`
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551990262143200
----
2019-03-07 20:26:14 UTC - Saurab Joshi: I am using ansible. Do i add this variable when I am running openwhisk.yml playbook?
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551990374143900
----
2019-03-07 20:26:30 UTC - Rodric Rabbah: you can do it on the command line
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551990390144400
----
2019-03-07 20:27:00 UTC - Rodric Rabbah: `wskdev deploy -e"docker_image_prefix=openwhisk"`
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551990420144700
----
2019-03-07 20:27:32 UTC - Rodric Rabbah: or `ansible-playbook -i environments/local openwhisk.yml -e 'docker_image_prefix=openwhisk'`
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551990452145100
----
2019-03-07 20:41:39 UTC - Saurab Joshi: nice im trying that right now
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991299145300
----
2019-03-07 20:41:50 UTC - Saurab Joshi: is it possible to deploy them with just docker-compose?
+1 : Rodric Rabbah
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991310145600
----
2019-03-07 20:42:13 UTC - Saurab Joshi: i know there is docker-compose just for development that i saw on github
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991333146300
----
2019-03-07 20:42:20 UTC - Rodric Rabbah: yep
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991340146500
----
2019-03-07 20:42:33 UTC - Rodric Rabbah: follow docs for `make quickstart`
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991353146900
----
2019-03-07 20:47:03 UTC - Saurab Joshi: we were thinking of deploying openwhisk using docker-compose instead of ansible. There is nothing wrong with that right?
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991623148000
----
2019-03-07 20:47:13 UTC - Rodric Rabbah: nope
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991633148200
----
2019-03-07 20:47:36 UTC - Rodric Rabbah: if you use the make quickstart target, be aware that uses the openwhisk release 0.9 images
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991656148700
----
2019-03-07 20:48:00 UTC - Rodric Rabbah: so depending on your needs you may need to adapt that makefile
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991680149200
----
2019-03-07 20:48:10 UTC - Saurab Joshi: okay makes sense
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991690149400
----
2019-03-07 20:48:13 UTC - Saurab Joshi: i'll give that a try
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991693149600
----
2019-03-07 20:48:19 UTC - Saurab Joshi: thanks
+1 : Rodric Rabbah
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551991699149800
----
2019-03-07 21:42:33 UTC - Roberto Diaz: Reviewed the PR cc @Bruce Adams
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1551994953150000?thread_ts=1551973896.138900&cid=C3TPCAQG1
----
2019-03-07 23:46:55 UTC - Roberto Diaz: here are some changes to includes the @Carlos Santana and @Bruce Adams to use `Value` as input param and `Result<Value,Error>` as returned value:
<https://github.com/bruceadams/incubator-openwhisk-runtime-rust/compare/proposal...rdiaz82:proposal>
(The changes are based on the latest Bruce PR.)
cc @Michele Sciabarra
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1552002415150200?thread_ts=1551890864.115200&cid=C3TPCAQG1
----