jaegertracing / legacy-client-java

Legacy com.uber.jaeger java client
Apache License 2.0
5 stars 6 forks source link

Add encrypted env vars to travis #1

Closed pavolloffay closed 6 years ago

pavolloffay commented 6 years ago

As this is a new repository it requires to add env vars from https://travis-ci.org/jaegertracing/jaeger-client-java/settings

pavolloffay commented 6 years ago

@jpkrohling IIRC you have configured these secrets, could you please add it to this repository too?

pavolloffay commented 6 years ago

codecov has been enabled but the token has to be probably added to travis

pavolloffay commented 6 years ago

It seems that codecov works without adding a token https://github.com/jaegertracing/legacy-client-java/pull/3#issuecomment-379820094

jpkrohling commented 6 years ago

SIGNING_KEY_ID and SIGNING_KEY_PASSPHRASE added to jaegertracing/legacy-client-java. I'm closing this issue as I think those are the only ones needed.

pavolloffay commented 6 years ago

@jpkrohling thanks

pavolloffay commented 6 years ago

@jpkrohling travis.yml also contains - ./travis/prepare-signing.sh $encrypted_677f232983c0_key $encrypted_677f232983c0_iv in before_install

the jaeger-client-java defines these vars. Shouldn't be also added here?

jpkrohling commented 6 years ago

No, those are created automatically by Travis. I think those are created when you use the travis encrypt command for the first time.

pavolloffay commented 6 years ago

ok thanks, I think the last comment shouldn't be envrypted env in travis.yml re-encrypted? https://github.com/jaegertracing/legacy-client-java/blob/master/.travis.yml#L19

jpkrohling commented 6 years ago

That can be done via travis encrypt. If you add --save (or something like that), it will change the .travis.yml with the encrypted value.

pavolloffay commented 6 years ago

That is clear to me.

The question is if those values should be re-encrypted

jpkrohling commented 6 years ago

Of course. The keys are tied to specific repositories. A value encrypted with the key for jaeger-client-java can't be decrypted by a key from legacy-client-java.

pavolloffay commented 6 years ago

Then I will reopen this as the encrypted values should be also added there. Do you want to submit a PR?