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

kinesis_producer can't refresh AssumeRole Credential. #108

Closed
tatsu-yam opened this issue Feb 23, 2017 · 2 comments
Closed

kinesis_producer can't refresh AssumeRole Credential. #108

tatsu-yam opened this issue Feb 23, 2017 · 2 comments

Comments

@tatsu-yam
Copy link

The kinesis_producer plugin got into the state which can't refresh AssumeRole Credential in our environment. (#58)
Even though, we use fluent-plugin-kinesis 1.0.1 (or later version).

Our setting is below

  • version
    • td-agent 2.3
    • fluent-plugin-kinesis 1.0.1 (or later version)
  • td-agent.conf
<match hoge>
  @type kinesis_producer
  stream_name  xxx
  region ap-northeast-1
  <assume_role_credentials>
    role_arn    xxxxxxxxxx
    external_id xxxxxxxxxx
  </assume_role_credentials>
  time_format %s
  include_time_key true
  buffer_type file
  buffer_path /var/log/td-agent/buffer/kinesis
  buffer_chunk_limit 500k
  buffer_queue_limit 16384
  flush_interval 1s
  try_flush_interval 0.1s
  queued_chunk_flush_interval 0.01s
  num_threads 4
</match>

When an exeception occurs in the KinesisProducer::Daemon#updaet_credentials, the thread which calls #update_credentials seems to die. (

add(make_set_credentials_message)
)

I tried deleting a policy from an IAM Role which the EC2 Instance assumes.
As a result, KinesisProducer::Daemon#update_credentials was not called every credentials_refresh_delay second.

When I added the code which rescues a Exception add(make_set_credentials_message) throws, even if its policy was changed, #update_credentials was called regularly.

But I don't know why that method is failed in our environment.

@riywo
Copy link
Contributor

riywo commented Mar 6, 2017

Hi @tatsu-yam ,

Thank you for reporting. I'll look into this.

@riywo
Copy link
Contributor

riywo commented Sep 22, 2017

I'm sorry for no response for a long time. We're are going to release v2.0.0 which supports record aggregation by a different way without KPL. So, this problem never happen and I'm closing.

Please try v2.0.0 first. If you need more help, feel free to add comment.

@riywo riywo closed this as completed Sep 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants