Python函数返回的结果不一致

时间:2019-12-20 15:12:47

标签: python amazon-ec2

我编写了一个脚本,该脚本列出了Amazon Web Services中的EC2实例。它将结果写到汇合处。但是它的表现很奇怪。

我在Windows 10上。第一次打开powershell终端并运行它时,它将报告AWS账户中正确的服务器数量。

下一次我运行它(根本不做任何更改)时,结果加倍。并且每次您使用相同的参数再次运行它时,它都会报告相同的错误(加倍)金额。

这是列出实例的函数,我认为这是麻烦所在,但是我找不到它:

def list_instances(aws_account, aws_account_number, interactive, regions, fieldnames, show_details):
    today, aws_env_list, output_file, output_file_name, fieldnames = initialize(interactive, aws_account)     options = arguments()     instance_list = ''     session = ''     ec2 = ''     account_found = ''     PrivateDNS = None     block_device_list = None     instance_count = 0     account_type_message = ''     profile_missing_message = ''     region = ''
    # Set the ec2 dictionary
    ec2info = {}
    # Write the file headers
    if interactive == 1:
        with open(output_file, mode='w+') as csv_file:
            writer = csv.DictWriter(csv_file, fieldnames=fieldnames, delimiter=',', lineterminator='\n')
            writer.writeheader()
            if 'gov' in aws_account and not 'admin' in aws_account:
                try:
                    session = boto3.Session(profile_name=aws_account,region_name=region)
                    account_found = 'yes'
                except botocore.exceptions.ProfileNotFound as e:
                    message = f"An exception has occurred: {e}"
                    account_found = 'no'
                    banner(message)
            else:
                try:
                    session = boto3.Session(profile_name=aws_account,region_name=region)
                    account_found = 'yes'
                except botocore.exceptions.ProfileNotFound as e:
                    message = f"An exception has occurred: {e}"
                    account_found = 'no'
                    banner(message)
    print(Fore.CYAN)
    report_gov_or_comm(aws_account, account_found)
    print(Fore.RESET)
    for region in regions:
        if 'gov' in aws_account and not 'admin' in aws_account:
            try:
                session = boto3.Session(profile_name=aws_account,region_name=region)
            except botocore.exceptions.ProfileNotFound as e:
                profile_missing_message = f"An exception has occurred: {e}"                 account_found = 'no' pass else: try:                 session = boto3.Session(profile_name=aws_account,region_name=region)                 account_found = 'yes' except botocore.exceptions.ProfileNotFound as e:                 profile_missing_message = f"An exception has occurred: {e}" pass try:             ec2 = session.client("ec2") except Exception as e: pass
    # Loop through the instances
    try:
        instance_list = ec2.describe_instances()
    except Exception as e:
        pass
    try:
        for reservation in instance_list["Reservations"]:
            for instance in reservation.get("Instances", []):
                instance_count = instance_count + 1
                launch_time = instance["LaunchTime"]
                launch_time_friendly = launch_time.strftime("%B %d %Y")
                tree = objectpath.Tree(block_devices = set(tree.execute('$..BlockDeviceMappings['Ebs']['VolumeId']'))
                if block_devices:
                    block_devices = list(block_devices)
                    block_devices = str(block_devices).replace('[','').replace(']','').replace("'",'')
                else:
                    block_devices = None
                    private_ips =  set(tree.execute('$..PrivateIpAddress'))
                    if private_ips:
                        private_ips_list = list(private_ips)                         private_ips_list = str(private_ips_list).replace('[','').replace(']','').replace(''','')
                    else:
                        private_ips_list = None
                        type(private_ips_list)
                        public_ips =  set(tree.execute('$..PublicIp'))
                        if len(public_ips) == 0:
                            public_ips = None
                        if public_ips:
                            public_ips_list = list(public_ips)
                            public_ips_list = str(public_ips_list).replace('[','').replace(']','').replace("'",'')
                        else:
                            public_ips_list = None
                        if 'KeyName' in instance:
                            key_name = instance['KeyName']
                        else:
                            key_name = None
                            name = None
                        if 'Tags' in instance:
                            try:
                                tags = instance['Tags']
                                name = None
                                for tag in tags:
                                    if tag["Key"] == "Name":
                                        name = tag["Value"]
                                if tag["Key"] == "Engagement" or tag["Key"] == "Engagement Code":
                                    engagement = tag["Value"]
                            except ValueError:
                                print("Instance: %s has no tags" % instance_id
                                pass
                    if 'VpcId' in instance:
    vpc_id = instance['VpcId'] else:                         vpc_id = None if 'PrivateDnsName' in instance:                         private_dns = instance['PrivateDnsName'] else:                         private_dns = None if 'Platform' in instance:                         platform = instance['Platform'] else:                         platform = None print(f"Platform: {platform}")                     ec2info[instance['InstanceId']] = { 'AWS Account': aws_account, 'Account Number': aws_account_number, 'Name': name, 'Instance ID': instance['InstanceId'], 'Volumes': block_devices, 'Private IP': private_ips_list, 'Public IP': public_ips_list, 'Private DNS': private_dns, 'Availability Zone': instance['Placement']['AvailabilityZone'], 'VPC ID': vpc_id, 'Type': instance['InstanceType'], 'Platform': platform, 'Key Pair Name': key_name, 'State': instance['State']['Name'], 'Launch Date': launch_time_friendly                     } with open(output_file,'a') as csv_file:                         writer = csv.DictWriter(csv_file, fieldnames=fieldnames, delimiter=',', lineterminator='\n')                         writer.writerow({'AWS Account': aws_account, "Account Number": aws_account_number, 'Name': name, 'Instance ID': instance["InstanceId"], 'Volumes': block_devices,  'Private IP': private_ips_list, 'Public IP': public_ips_list, 'Private DNS': private_dns, 'Availability Zone': instance['Placement']['AvailabilityZone'], 'VPC ID': vpc_id, 'Type': instance["InstanceType"], 'Platform': platform, 'Key Pair Name': key_name, 'State': instance["State"]["Name"], 'Launch Date': launch_time_friendly})
    if show_details == 'y' or show_details == 'yes': for instance_id, instance in ec2info.items(): if account_found == 'yes': print(Fore.RESET + "-------------------------------------") for key in [ 'AWS Account', 'Account Number', 'Name', 'Instance ID', 'Volumes', 'Private IP', 'Public IP', 'Private DNS', 'Availability Zone', 'VPC ID', 'Type', 'Platform', 'Key Pair Name', 'State', 'Launch Date'                                 ]: print(Fore.GREEN + f"{key}: {instance.get(key)}") print(Fore.RESET + "-------------------------------------") else: pass                     ec2info = {} with open(output_file,'a') as csv_file:                         csv_file.close() except Exception as e: pass if profile_missing_message == '*':         banner(profile_missing_message) print(Fore.GREEN)     report_instance_stats(instance_count, aws_account, account_found) print(Fore.RESET + '\n') return output_file

这是整个代码的上下文粘贴:aws_ec2_list_instances.py

首次从命令行运行它时,它将报告EC2中的服务器总数正确(可以在AWS控制台中进行验证):

----------------------------------------------------------
There are: 51 EC2 instances in AWS Account: company-lab.
----------------------------------------------------------

下一次以绝对没有更改的方式运行时,它将报告总计:

----------------------------------------------------------
There are: 102 EC2 instances in AWS Account: company-lab.
----------------------------------------------------------

您实际上只是向上箭头命令,它会使结果加倍。当它写到融合时,您会看到列出的重复服务器。每次您按向上箭头再次运行它时,它的总数相同(102个服务器)。

如果关闭powershell命令行并再次打开,则返回报告正确结果(51台服务器)的结果,该结果与您在AWS控制台中看到的相对应。 这到底是怎么回事?为什么这样做,我该如何解决问题?

1 个答案:

答案 0 :(得分:1)

这真是神秘!我认为如果不访问您的环境,我将无法对其进行调试。我的建议是,您使用pdb来找出instance_count是如何增加到51以上的。我建议在第210行,即紧接着{{1 }}。然后,您可以在整个循环中每次检查import pdb; pdb.set_trace()的值,查看for reservation in instance_list["Reservations"]:是否实际上以某种方式具有重复数据,等等。