Project

General

Profile

Actions

Bug #7908

closed

mac address toggle between lower case and upper case in audit logs

Added by Ohad Levy over 9 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Network
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

I've seen in my audit logs the following:

Audit.where(:auditable_id => 194, :auditable_type => 'Host').limit(20).reorder("created_at DESC").map(&:audited_changes)
  Audited::Adapters::ActiveRecord::Audit Load (1.3ms)  SELECT `audits`.* FROM `audits` WHERE `audits`.`auditable_id` = 194 AND `audits`.`auditable_type` = 'Host' ORDER BY created_at DESC LIMIT 20
=> [{"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"]}, {"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"]}, {"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"], "ip"=>["10.35.27.6", "127.0.0.1"]}, {"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"]}, {"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"]}, {"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"]}, {"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"]}, {"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"]}, {"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"]}, {"mac"=>["E4:1F:13:CC:36:C0", "e4:1f:13:cc:36:c0"]}, {"mac"=>["e4:1f:13:cc:36:c0", "E4:1F:13:CC:36:C0"]}]

Related issues 2 (0 open2 closed)

Related to Foreman - Feature #7401: Add support for bond interfacesClosedMarek Hulán09/10/2014Actions
Related to Foreman - Bug #8456: undefined local variable or method `normalize_addresses' for #<Host::Discovered:0x000000102dbdc8>ClosedMarek Hulán11/20/2014Actions
Actions #1

Updated by Ohad Levy over 9 years ago

  • Related to Feature #7401: Add support for bond interfaces added
Actions #2

Updated by Marek Hulán over 9 years ago

  • Status changed from New to Feedback
I'm unable to reproduce this from tests. I suspect the host save without validation that skips before_validation callbacks (which normalizes mac address), but whatever I do, I always get mac addresses downcased. Could you please:
  • upload several facter outputs from the host (to see which macaddresses changes the case and so I could write test based on this)
  • track audits per one host update
  • make sure it started to happen after bond PR was merged or find first such Audit record
Actions #3

Updated by The Foreman Bot over 9 years ago

  • Status changed from Feedback to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/1965 added
  • Pull request deleted ()
Actions #4

Updated by Dominic Cleal over 9 years ago

  • Assignee set to Marek Hulán
Actions #5

Updated by Dominic Cleal over 9 years ago

  • translation missing: en.field_release set to 21
Actions #6

Updated by Marek Hulán over 9 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #7

Updated by Dominic Cleal over 9 years ago

  • Related to Bug #8456: undefined local variable or method `normalize_addresses' for #<Host::Discovered:0x000000102dbdc8> added
Actions

Also available in: Atom PDF