openc3 5.0.6

data/lib/openc3/logs/packet_log_reader.rb

@@ -0,0 +1,345 @@
+# encoding: ascii-8bit
+
+# Copyright 2022 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU Affero General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU Affero General Public License for more details.
+
+# Modified by OpenC3, Inc.
+# All changes Copyright 2022, OpenC3, Inc.
+# All Rights Reserved
+
+require 'openc3/core_ext/io'
+require 'openc3/packets/packet'
+require 'openc3/packets/json_packet'
+require 'openc3/io/buffered_file'
+require 'openc3/logs/packet_log_constants'
+
+module OpenC3
+  # Reads a packet log of either commands or telemetry.
+  class PacketLogReader
+    include PacketLogConstants
+
+    attr_reader :redis_offset
+
+    MAX_READ_SIZE = 1000000000
+
+    # Create a new log file reader
+    def initialize
+      reset()
+    end
+
+    # Yields back each packet as it is found in the log file.
+    #
+    # @param filename [String] The log file to read
+    # @param identify_and_define [Boolean] Once the packet has been read from
+    #   the log file, whether to both identify the packet by setting the target
+    #   and packet name, and define the packet by populating all the items.
+    # @param start_time [Time|nil] Time at which to start returning packets.
+    #   Packets found with a timestamp before this time are ignored. Pass nil
+    #   to return all packets.
+    # @param end_time [Time|nil] Time at which to stop returning packets.
+    #   Packets found with a timestamp after this time are ignored. Pass nil
+    #   to return all packets.
+    # @yieldparam packet [Packet]
+    # @return [Boolean] Whether we reached the end_time while reading
+    def each(filename, identify_and_define = true, start_time = nil, end_time = nil)
+      reached_end_time = false
+      open(filename)
+
+      # seek_to_time(start_time) if start_time
+
+      while true
+        packet = read(identify_and_define)
+        break unless packet
+
+        time = packet.packet_time
+        if time
+          next if start_time and time < start_time
+          # If we reach the end_time that means we found all the packets we asked for
+          # This can be used by callers to know they are done reading
+          if end_time and time > end_time
+            reached_end_time = true
+            break
+          end
+        end
+        yield packet
+      end
+      reached_end_time
+    ensure # No implicit return value in the ensure block
+      close()
+    end
+
+    # @param filename [String] The log filename to open
+    # @return [Boolean, Exception] Returns true if successfully changed to configuration specified in log,
+    #    otherwise returns false and potentially an Exception class if an error occurred.  If no error occurred
+    #    false indicates that the requested configuration was simply not found.
+    def open(filename)
+      close()
+      reset()
+      @filename = filename
+      @file = BufferedFile.open(@filename, 'rb')
+      @max_read_size = @file.size
+      @max_read_size = MAX_READ_SIZE if @max_read_size > MAX_READ_SIZE
+      return read_file_header()
+    rescue => err
+      close()
+      raise err
+    end
+
+    # Closes the current log file
+    def close
+      @file.close if @file and !@file.closed?
+    end
+
+    # Read a packet from the log file
+    #
+    # @param identify_and_define (see #each)
+    # @return [Packet]
+    def read(identify_and_define = true)
+      # Read entry length
+      length = @file.read(4)
+      return nil if !length or length.length <= 0
+
+      length = length.unpack('N')[0]
+      entry = @file.read(length)
+      flags = entry[0..1].unpack('n')[0]
+
+      cmd_or_tlm = :TLM
+      cmd_or_tlm = :CMD if flags & OPENC3_CMD_FLAG_MASK == OPENC3_CMD_FLAG_MASK
+      stored = false
+      stored = true if flags & OPENC3_STORED_FLAG_MASK == OPENC3_STORED_FLAG_MASK
+      id = false
+      id = true if flags & OPENC3_ID_FLAG_MASK == OPENC3_ID_FLAG_MASK
+
+      if flags & OPENC3_ENTRY_TYPE_MASK == OPENC3_JSON_PACKET_ENTRY_TYPE_MASK
+        packet_index, time_nsec_since_epoch = entry[2..11].unpack('nQ>')
+        json_data = entry[12..-1]
+        lookup_cmd_or_tlm, target_name, packet_name, id = @packets[packet_index]
+        if cmd_or_tlm != lookup_cmd_or_tlm
+          raise "Packet type mismatch, packet:#{cmd_or_tlm}, lookup:#{lookup_cmd_or_tlm}"
+        end
+
+        return JsonPacket.new(cmd_or_tlm, target_name, packet_name, time_nsec_since_epoch, stored, json_data)
+      elsif flags & OPENC3_ENTRY_TYPE_MASK == OPENC3_RAW_PACKET_ENTRY_TYPE_MASK
+        packet_index, time_nsec_since_epoch = entry[2..11].unpack('nQ>')
+        packet_data = entry[12..-1]
+        lookup_cmd_or_tlm, target_name, packet_name, id = @packets[packet_index]
+        if cmd_or_tlm != lookup_cmd_or_tlm
+          raise "Packet type mismatch, packet:#{cmd_or_tlm}, lookup:#{lookup_cmd_or_tlm}"
+        end
+
+        received_time = Time.from_nsec_from_epoch(time_nsec_since_epoch)
+        if identify_and_define
+          packet = identify_and_define_packet_data(cmd_or_tlm, target_name, packet_name, received_time, packet_data)
+        else
+          # Build Packet
+          packet = Packet.new(target_name, packet_name, :BIG_ENDIAN, nil, packet_data)
+        end
+        packet.set_received_time_fast(received_time)
+        packet.cmd_or_tlm = cmd_or_tlm
+        packet.stored = stored
+        packet.received_count += 1
+        return packet
+      elsif flags & OPENC3_ENTRY_TYPE_MASK == OPENC3_TARGET_DECLARATION_ENTRY_TYPE_MASK
+        target_name_length = length - OPENC3_PRIMARY_FIXED_SIZE - OPENC3_TARGET_DECLARATION_SECONDARY_FIXED_SIZE
+        target_name_length -= OPENC3_ID_FIXED_SIZE if id
+        target_name = entry[2..(target_name_length + 1)]
+        if id
+          id = entry[(target_name_length + 3)..(target_name_length + 34)]
+          @target_ids << id
+        end
+        @target_names << target_name
+        return read(identify_and_define)
+      elsif flags & OPENC3_ENTRY_TYPE_MASK == OPENC3_PACKET_DECLARATION_ENTRY_TYPE_MASK
+        target_index = entry[2..3].unpack('n')[0]
+        target_name = @target_names[target_index]
+        packet_name_length = length - OPENC3_PRIMARY_FIXED_SIZE - OPENC3_PACKET_DECLARATION_SECONDARY_FIXED_SIZE
+        packet_name_length -= OPENC3_ID_FIXED_SIZE if id
+        packet_name = entry[4..(packet_name_length + 3)]
+        if id
+          id = entry[(packet_name_length + 4)..-1]
+          @packet_ids << id
+        end
+        @packets << [cmd_or_tlm, target_name, packet_name, id]
+        return read(identify_and_define)
+      elsif flags & OPENC3_ENTRY_TYPE_MASK == OPENC3_OFFSET_MARKER_ENTRY_TYPE_MASK
+        @redis_offset = entry[2..-1]
+        return read(identify_and_define)
+      else
+        raise "Invalid Entry Flags: #{flags}"
+      end
+    rescue => err
+      close()
+      raise err
+    end
+
+    # TODO: Currently not used
+    # Returns an analysis of the log file by reading all the packets and
+    # returning information about each packet. This information maps directly
+    # to the parameters need by the {#read_at_offset} method and thus should be
+    # called before using {#read_at_offset}.
+    #
+    # @param filename [String] The filename to analyze
+    # @param progress_callback [Proc] Callback that should receive a single
+    #   floating point parameter which is the percentage done
+    # @return [Array<Array<Integer, Integer, String, String, Time, Time>] Array
+    #   of arrays for each packet found in the log file consisting of:
+    #   [File position, length, target name, packet name, time formatted,
+    #   received time].
+    # def packet_offsets(filename, progress_callback = nil)
+    #   open(filename)
+    #   offsets = []
+    #   filesize = size().to_f
+
+    #   while true
+    #     current_pos = @file.pos
+    #     packet = read(false)
+    #     break unless packet
+    #     offsets << current_pos
+    #     if progress_callback
+    #       break if progress_callback.call(current_pos / filesize)
+    #     end
+    #   end
+
+    #   return offsets
+    # ensure
+    #   close()
+    # end
+
+    # TODO: Currently not used
+    # Reads a packet from the opened log file. Should only be used in
+    # conjunction with {#packet_offsets}.
+    #
+    # @param file_offset [Integer] Byte offset into the log file to start
+    #   reading
+    # @param identify_and_define (see #each)
+    # @return [Packet]
+    # def read_at_offset(file_offset, identify_and_define = true)
+    #   @file.seek(file_offset, IO::SEEK_SET)
+    #   return read(identify_and_define)
+    # rescue => err
+    #   close()
+    #   raise err
+    # end
+
+    # TODO: Currently not used
+    # Read the first packet from the log file and reset the file position back
+    # to the current position. This allows the client to call read multiple
+    # times to return packets, call first, and continue calling read which will
+    # return the next packet in the file.
+    #
+    # @return [Packet]
+    # def first
+    #   original_position = @file.pos
+    #   @file.seek(0, IO::SEEK_SET)
+    #   read_file_header()
+    #   packet = read()
+    #   raise "No first packet found" unless packet
+    #   @file.seek(original_position, IO::SEEK_SET)
+    #   packet.clone
+    # rescue => err
+    #   close()
+    #   raise err
+    # end
+
+    # TODO: Currently not used
+    # Read the last packet from the log file and reset the file position back
+    # to the current position. This allows the client to call read multiple
+    # times to return packets, call last, and continue calling read which will
+    # return the next packet in the file.
+    #
+    # @return [Packet]
+    # def last
+    #   raise "TODO: Implement me - Need to add end of file entry to support"
+    #   original_position = @file.pos
+    #   @file.seek(-1, IO::SEEK_END)
+    #   packet = search(-1)
+    #   raise "No last packet found" unless packet
+    #   @file.seek(original_position, IO::SEEK_SET)
+    #   packet.clone
+    # rescue => err
+    #   close()
+    #   raise err
+    # end
+
+    # @return [Integer] The size of the log file being processed
+    def size
+      @file.stat.size
+    end
+
+    # @return [Integer] The current file position in the log file
+    def bytes_read
+      @file.pos
+    end
+
+    protected
+
+    def reset
+      @file = nil
+      @filename = nil
+      @max_read_size = MAX_READ_SIZE
+      @target_names = []
+      @target_ids = []
+      @packets = []
+      @packet_ids = []
+      @redis_offset = nil
+    end
+
+    # This is best effort. May return unidentified/undefined packets
+    def identify_and_define_packet_data(cmd_or_tlm, target_name, packet_name, received_time, packet_data)
+      packet = nil
+      unless target_name and packet_name
+        if cmd_or_tlm == :CMD
+          packet = System.commands.identify(packet_data)
+        else
+          packet = System.telemetry.identify!(packet_data)
+        end
+      else
+        begin
+          if cmd_or_tlm == :CMD
+            packet = System.commands.packet(target_name, packet_name)
+          else
+            packet = System.telemetry.packet(target_name, packet_name)
+          end
+          packet.buffer = packet_data
+        rescue
+          # Could not find a definition for this packet
+          Logger.instance.error "Unknown packet #{target_name} #{packet_name}"
+          packet = Packet.new(target_name, packet_name, :BIG_ENDIAN, nil, packet_data)
+        end
+      end
+      packet
+    end
+
+    # Should return if successfully switched to requested configuration
+    def read_file_header
+      header = @file.read(OPENC3_HEADER_LENGTH)
+      if header and header.length == OPENC3_HEADER_LENGTH
+        if header == OPENC3_FILE_HEADER
+          # Found OpenC3 5 File Header - That's all we need to do
+        elsif header == COSMOS4_FILE_HEADER
+          raise "COSMOS 4 log file must be converted to OpenC3 5"
+        elsif header == COSMOS2_FILE_HEADER
+          raise "COSMOS 2 log file must be converted to OpenC3 5"
+        else
+          raise "OpenC3 file header not found"
+        end
+      else
+        raise "Failed to read at least #{OPENC3_HEADER_LENGTH} bytes from packet log"
+      end
+    end
+
+    def seek_to_time(time)
+      raise "TODO: Implement me - Use index file or offsets"
+    end
+  end
+end

data/lib/openc3/logs/packet_log_writer.rb

@@ -0,0 +1,299 @@
+# encoding: ascii-8bit
+
+# Copyright 2022 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU Affero General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU Affero General Public License for more details.
+
+# Modified by OpenC3, Inc.
+# All changes Copyright 2022, OpenC3, Inc.
+# All Rights Reserved
+
+require 'openc3/logs/log_writer'
+require 'openc3/logs/packet_log_constants'
+
+module OpenC3
+  # Creates a packet log. Can automatically cycle the log based on an elasped
+  # time period or when the log file reaches a predefined size.
+  class PacketLogWriter < LogWriter
+    include PacketLogConstants
+
+    # @param remote_log_directory [String] The s3 path to store the log files
+    # @param label [String] Label to apply to the log filename
+    # @param logging_enabled [Boolean] Whether to start with logging enabled
+    # @param cycle_time [Integer] The amount of time in seconds before creating
+    #   a new log file. This can be combined with cycle_size but is better used
+    #   independently.
+    # @param cycle_size [Integer] The size in bytes before creating a new log
+    #   file. This can be combined with cycle_time but is better used
+    #   independently.
+    # @param cycle_hour [Integer] The time at which to cycle the log. Combined with
+    #   cycle_minute to cycle the log daily at the specified time. If nil, the log
+    #   will be cycled hourly at the specified cycle_minute.
+    # @param cycle_minute [Integer] The time at which to cycle the log. See cycle_hour
+    #   for more information.
+    # @param redis_topic [String] The key of the Redis stream to trim when files are
+    #   moved to S3
+    def initialize(
+      remote_log_directory,
+      label,
+      logging_enabled = true,
+      cycle_time = nil,
+      cycle_size = 1_000_000_000,
+      cycle_hour = nil,
+      cycle_minute = nil,
+      redis_topic: nil
+    )
+      super(
+        remote_log_directory,
+        logging_enabled,
+        cycle_time,
+        cycle_size,
+        cycle_hour,
+        cycle_minute,
+        redis_topic: redis_topic
+      )
+      @label = label
+      @index_file = nil
+      @index_filename = nil
+      @cmd_packet_table = {}
+      @tlm_packet_table = {}
+      @target_dec_entries = []
+      @packet_dec_entries = []
+      @next_packet_index = 0
+      @target_indexes = {}
+      @next_target_index = 0
+
+      # This is an optimization to avoid creating a new entry object
+      # each time we create an entry which we do a LOT!
+      @index_entry = String.new
+    end
+
+    # Write a packet to the log file.
+    #
+    # If no log file currently exists in the filesystem, a new file will be
+    # created.
+    #
+    # @param entry_type [Symbol] Type of entry to write. Must be one of
+    #   :TARGET_DECLARATION, :PACKET_DECLARATION, :RAW_PACKET, :JSON_PACKET
+    # @param cmd_or_tlm [Symbol] One of :CMD or :TLM
+    # @param target_name [String] Name of the target
+    # @param packet_name [String] Name of the packet
+    # @param time_nsec_since_epoch [Integer] 64 bit integer nsecs since EPOCH
+    # @param stored [Boolean] Whether this data is stored telemetry
+    # @param data [String] Binary string of data
+    # @param id [Integer] Target ID
+    # @param redis_offset [Integer] The offset of this packet in its Redis stream
+    def write(entry_type, cmd_or_tlm, target_name, packet_name, time_nsec_since_epoch, stored, data, id = nil, redis_offset = '0-0')
+      return if !@logging_enabled
+
+      @mutex.synchronize do
+        prepare_write(time_nsec_since_epoch, data.length, redis_offset)
+        write_entry(entry_type, cmd_or_tlm, target_name, packet_name, time_nsec_since_epoch, stored, data, id) if @file
+      end
+    rescue => err
+      Logger.instance.error "Error writing #{@filename} : #{err.formatted}"
+      OpenC3.handle_critical_exception(err)
+    end
+
+    # Starting a new index file is a critical operation so the entire method is
+    # wrapped with a rescue and handled with handle_critical_exception
+    # Assumes mutex has already been taken
+    def start_new_file
+      super
+      @file.write(OPENC3_FILE_HEADER)
+      @file_size += OPENC3_FILE_HEADER.length
+
+      # Start index log file
+      @index_filename = create_unique_filename('.idx'.freeze)
+      @index_file = File.new(@index_filename, 'wb')
+      @index_file.write(OPENC3_INDEX_HEADER)
+
+      @cmd_packet_table = {}
+      @tlm_packet_table = {}
+      @next_packet_index = 0
+      @target_indexes = {}
+      @target_dec_entries = []
+      @packet_dec_entries = []
+      Logger.debug "Index Log File Opened : #{@index_filename}"
+    rescue => err
+      Logger.error "Error starting new log file: #{err.formatted}"
+      @logging_enabled = false
+      OpenC3.handle_critical_exception(err)
+    end
+
+    # Closing a log file isn't critical so we just log an error
+    def close_file(take_mutex = true)
+      write_entry(:OFFSET_MARKER, nil, nil, nil, nil, nil, nil, nil) if @file
+      super
+
+      @mutex.lock if take_mutex
+      begin
+        if @index_file
+          begin
+            write_index_file_footer()
+            @index_file.close unless @index_file.closed?
+            Logger.debug "Index Log File Closed : #{@index_filename}"
+            date = first_timestamp[0..7] # YYYYMMDD
+            s3_key = File.join(@remote_log_directory, date, "#{first_timestamp}__#{last_timestamp}__#{@label}.idx")
+            S3Utilities.move_log_file_to_s3(@index_filename, s3_key)
+          rescue Exception => err
+            Logger.instance.error "Error closing #{@index_filename} : #{err.formatted}"
+          end
+
+          @index_file = nil
+          @index_filename = nil
+        end
+      ensure
+        @mutex.unlock if take_mutex
+      end
+    end
+
+    def get_packet_index(cmd_or_tlm, target_name, packet_name)
+      if cmd_or_tlm == :CMD
+        target_table = @cmd_packet_table[target_name]
+      else
+        target_table = @tlm_packet_table[target_name]
+      end
+      if target_table
+        packet_index = target_table[packet_name]
+        return packet_index if packet_index
+      else
+        # New packet_table entry needed
+        target_table = {}
+        if cmd_or_tlm == :CMD
+          @cmd_packet_table[target_name] = target_table
+        else
+          @tlm_packet_table[target_name] = target_table
+        end
+        id = nil
+        target = System.targets[target_name]
+        id = target.id if target
+        write_entry(:TARGET_DECLARATION, cmd_or_tlm, target_name, packet_name, nil, nil, nil, id)
+      end
+
+      # New target_table entry needed
+      packet_index = @next_packet_index
+      if packet_index > OPENC3_MAX_PACKET_INDEX
+        raise "Packet Index Overflow"
+      end
+
+      target_table[packet_name] = packet_index
+      @next_packet_index += 1
+
+      id = nil
+      begin
+        if cmd_or_tlm == :CMD
+          id = System.commands.packet(target_nam, packet_name).config_name
+        else
+          id = System.telemetry.packet(target_name, packet_name).config_name
+        end
+      rescue
+        # No packet def
+      end
+      write_entry(:PACKET_DECLARATION, cmd_or_tlm, target_name, packet_name, nil, nil, nil, id)
+      return packet_index
+    end
+
+    def write_entry(entry_type, cmd_or_tlm, target_name, packet_name, time_nsec_since_epoch, stored, data, id)
+      raise ArgumentError.new("Length of id must be 64, got #{id.length}") if id and id.length != 64 # 64 hex digits, gets packed to 32 bytes with .pack('H*')
+
+      length = OPENC3_PRIMARY_FIXED_SIZE
+      flags = 0
+      flags |= OPENC3_STORED_FLAG_MASK if stored
+      flags |= OPENC3_ID_FLAG_MASK if id
+      case entry_type
+      when :TARGET_DECLARATION
+        target_index = @next_target_index
+        @target_indexes[target_name] = target_index
+        @next_target_index += 1
+        if target_index > OPENC3_MAX_TARGET_INDEX
+          raise "Target Index Overflow"
+        end
+
+        flags |= OPENC3_TARGET_DECLARATION_ENTRY_TYPE_MASK
+        length += OPENC3_TARGET_DECLARATION_SECONDARY_FIXED_SIZE + target_name.length
+        length += OPENC3_ID_FIXED_SIZE if id
+        @entry.clear
+        @entry << [length, flags].pack(OPENC3_TARGET_DECLARATION_PACK_DIRECTIVE) << target_name
+        @entry << [id].pack('H*') if id
+        @target_dec_entries << @entry.dup
+      when :PACKET_DECLARATION
+        target_index = @target_indexes[target_name]
+        flags |= OPENC3_PACKET_DECLARATION_ENTRY_TYPE_MASK
+        if cmd_or_tlm == :CMD
+          flags |= OPENC3_CMD_FLAG_MASK
+        end
+        length += OPENC3_PACKET_DECLARATION_SECONDARY_FIXED_SIZE + packet_name.length
+        length += OPENC3_ID_FIXED_SIZE if id
+        @entry.clear
+        @entry << [length, flags, target_index].pack(OPENC3_PACKET_DECLARATION_PACK_DIRECTIVE) << packet_name
+        @entry << [id].pack('H*') if id
+        @packet_dec_entries << @entry.dup
+      when :OFFSET_MARKER
+        flags |= OPENC3_OFFSET_MARKER_ENTRY_TYPE_MASK
+        length += OPENC3_OFFSET_MARKER_SECONDARY_FIXED_SIZE + @last_offset.length
+        @entry.clear
+        @entry << [length, flags].pack(OPENC3_OFFSET_MARKER_PACK_DIRECTIVE) << @last_offset
+      when :RAW_PACKET, :JSON_PACKET
+        target_name = 'UNKNOWN'.freeze unless target_name
+        packet_name = 'UNKNOWN'.freeze unless packet_name
+        packet_index = get_packet_index(cmd_or_tlm, target_name, packet_name)
+        if entry_type == :RAW_PACKET
+          flags |= OPENC3_RAW_PACKET_ENTRY_TYPE_MASK
+        else
+          flags |= OPENC3_JSON_PACKET_ENTRY_TYPE_MASK
+        end
+        if cmd_or_tlm == :CMD
+          flags |= OPENC3_CMD_FLAG_MASK
+        end
+        length += OPENC3_PACKET_SECONDARY_FIXED_SIZE + data.length
+        @entry.clear
+        @index_entry.clear
+        @index_entry << [length, flags, packet_index, time_nsec_since_epoch].pack(OPENC3_PACKET_PACK_DIRECTIVE)
+        @entry << @index_entry << data
+        @index_entry << [@file_size].pack('Q>')
+        @index_file.write(@index_entry)
+        @first_time = time_nsec_since_epoch if !@first_time or time_nsec_since_epoch < @first_time
+        @last_time = time_nsec_since_epoch if !@last_time or time_nsec_since_epoch > @last_time
+      else
+        raise "Unknown entry_type: #{entry_type}"
+      end
+      @file.write(@entry)
+      @file_size += @entry.length
+    end
+
+    def write_index_file_footer
+      footer_length = 4 # Includes length of length field at end
+      @index_file.write([@target_dec_entries.length].pack('n'))
+      footer_length += 2
+      @target_dec_entries.each do |target_dec_entry|
+        @index_file.write(target_dec_entry)
+        footer_length += target_dec_entry.length
+      end
+      @index_file.write([@packet_dec_entries.length].pack('n'))
+      footer_length += 2
+      @packet_dec_entries.each do |packet_dec_entry|
+        @index_file.write(packet_dec_entry)
+        footer_length += packet_dec_entry.length
+      end
+      @index_file.write([footer_length].pack('N'))
+    end
+
+    def s3_filename
+      "#{first_timestamp}__#{last_timestamp}__#{@label}" + extension
+    end
+
+    def extension
+      '.bin'.freeze
+    end
+  end
+end